Форум программистов, компьютерный форум, киберфорум
Наши страницы
C++
Войти
Регистрация
Восстановить пароль
 
Рейтинг 5.00/3: Рейтинг темы: голосов - 3, средняя оценка - 5.00
os80
0 / 0 / 0
Регистрация: 14.10.2015
Сообщений: 2
1

Как оптимизировать обращение к элементам вектора?

14.10.2015, 06:18. Просмотров 553. Ответов 4
Метки нет (Все метки)

Добрый день.

Подскажите, пожалуйста, где я не прав. Есть класс, в нем координаты и другие параметры, описывающие существ (овцы и волки). Эти существа содержатся в векторе и в цикле проверяются между собой на разные условия: близость волков, поиск травы на карте, столкновение двух овец для размножения и т.д.
Вот пример кода из цикла проверок. В данном случае, овца убегает от волка, если видит его.

C
1
2
3
4
5
if((*it)->name=="Sheep" && (*it2)->name=="Wolf")
if(abs((*it)->x-(*it2)->x)<(*it)->SeeSheepRadius && abs((*it)->y-(*it2)->y)<(*it)->SeeSheepRadius){
   if((*it)->x<(*it2)->x) (*it)->dx = -0.01; else (*it)->dx = 0.01;
   if((*it)->y<(*it2)->y) (*it)->dy = -0.01; else (*it)->dy = 0.01;
}
Проблема в том, что если условий таких несколько и существ больше нескольких сотен, то всё это очень тормозит.
Подозреваю, что много времени занимает именно обращение к переменным класса в элементе вектора: (*it)->x.
Что с этим можно сделать? Читал про списки и вектора, для этих задач лучше подходит вектор.

Весь код на гитхабе: github.com/os80/Glass-Grass
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
14.10.2015, 06:18
Ответы с готовыми решениями:

Как можно еще оптимизировать код?
Как еще можно оптимизировать данный код? Если вкратце, то он выводит значение...

Код потребляет очень много памяти. Подскажите как оптимизировать?
Есть код на C++ (компилируется под GCC) - подсчёт коэффициентов осцилляторной...

Как получить доступ к элементам формы из созданного класса?
Всем доброго времени суток!:help: Скажите, пожалуйста, как мне обратиться к...

Обращение к элементам массива
Здрасте народ. У меня проблема в наличии двухмерный масив 25х25. Нужно чтобы...

Обращение к элементам формы в цикле
Всем привет. Только начал освоение C++, работаю в Embarcadero XE7. Возникла...

4
nmcf
6274 / 5577 / 2537
Регистрация: 14.04.2014
Сообщений: 23,468
14.10.2015, 10:31 2
Для начала избавься от строк, вместо Sheep и Wolf сделай числа 0/1 или bool.
2
nonedark2008
1051 / 785 / 220
Регистрация: 28.07.2012
Сообщений: 2,189
14.10.2015, 10:35 3
Цитата Сообщение от os80 Посмотреть сообщение
Подозреваю, что
Возьми анализатор производительности и проверь самостоятельно что у тебя не так.
Под Windows лично я предпочитаю Intel VTune.
1
AlexVRud
478 / 190 / 72
Регистрация: 04.07.2014
Сообщений: 537
14.10.2015, 16:36 4
1. Замени name на перечисление. Сравнивать числа быстрее чем строки.
2. Судя по (*it)->x у тебя указатель на указатель. Это может быть очень медленно.
3. У тебя классы, так пользуйся ими по полной, реализованные функции внутри объявления являются inline, т.е. их код будет вставлен в места использования. Это упрощает читабельность кода, например:
C++
1
2
    if (it->species() == Animal::Sheep && it2->species() == Animal::Wolf && it->see(*it2) )
      it->escape(*it2);
читается и пишется легче.
Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
#include <iostream>
#include <vector>
#include <cmath>
 
/*
if((*it)->name=="Sheep" && (*it2)->name=="Wolf")
if(abs((*it)->x-(*it2)->x)<(*it)->SeeSheepRadius && abs((*it)->y-(*it2)->y)<(*it)->SeeSheepRadius){
   if((*it)->x<(*it2)->x) (*it)->dx = -0.01; else (*it)->dx = 0.01;
   if((*it)->y<(*it2)->y) (*it)->dy = -0.01; else (*it)->dy = 0.01;
}
*/
 
 
class Animal {
public:
  enum Species {
    Wolf,
    Sheep
  };
 
  Animal(Species species, double x, double y, double dx, double dy,
         double speed, double review_radius)
    : species_(species),
      x_(x), y_(y), dx_(dx), dy_(dy),
      speed_(speed), review_radius_(review_radius)
  {}
 
  Species species() const { return species_; }
  
  void setPosition(double x, double y)
  {
    x_ = x; y_ = y;
  }  
  double x() const { return x_; };
  double y() const { return y_; };
 
  void setOffset(double dx, double dy)
  {
    dx_ = dx; dy_ = dy;
  }  
 
  double dx() const { return dx_; };
  double dy() const { return dy_; };
 
  void setReviewRadius(double review_radius)
  {
    review_radius_ = review_radius;
  }
 
  bool see(const Animal& animal) {
      return (fabs(x_ - animal.x_) < review_radius_) &&
             (fabs(x_ - animal.x_) < review_radius_);
  }
  
  void escape( const Animal& animal) {
      dx_ = (x_ < animal.x_) ? -speed_ : speed_;
      dy_ = (y_ < animal.y_) ? -speed_ : speed_;
  }
 
private:
  Species species_;
  double x_;
  double y_;
  double dx_;
  double dy_;
  double speed_;
  double review_radius_;
};
 
int main() {
    std::vector<Animal> animals;
    animals.push_back(Animal(Animal::Sheep, 0.2, 0.1, 0.0, 0.0, 0.01, 1.0));
    animals.push_back(Animal(Animal::Wolf, 0.0, 0.0, 0.0, 0.0, 0.1, 1.0));
    Animal *it = &animals[0];
    Animal *it2 = &animals[1];
    
    if (it->species() == Animal::Sheep && it2->species() == Animal::Wolf && it->see(*it2) )
      it->escape(*it2);
    
    std::cout << animals[0].dx() << std::endl;
    return 0;
}
1
os80
0 / 0 / 0
Регистрация: 14.10.2015
Сообщений: 2
14.10.2015, 18:01  [ТС] 5
Огромное спасибо, народ, за ответы. Мне нужно время, что б их переварить и оформить в свой код.

Добавлено через 34 минуты
Цитата Сообщение от AlexVRud Посмотреть сообщение
У тебя классы, так пользуйся ими по полной
Отдельное спасибо за ответ. Как я понял, ты предельно точно решил мою проблему. К сожалению, опыта не хватает оперативно поправить код и посмотреть, будет тормозить или нет. Не знаю, сколько времени уйдет на коммит, так что пропаду пока Еще раз спасибо
0
14.10.2015, 18:01
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
14.10.2015, 18:01

sFrameBar - Items Обращение к элементам
Здраствуйте форумчане у меня вопрос как обратится к одному из элементов в...

Как можно оптимизировать?
Подскажите пожалуйста. Имеется форма на которой находятся 20 edit'ов, как можно...

Посоветуйте как оптимизировать данный код
Есть такой код. Подскажите как его оптимизировать: т.е. сделать не таким...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
5
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2018, vBulletin Solutions, Inc.
Рейтинг@Mail.ru