Форум программистов, компьютерный форум CyberForum.ru

C++

Войти
Регистрация
Восстановить пароль
 
os80
0 / 0 / 0
Регистрация: 14.10.2015
Сообщений: 2
#1

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

14.10.2015, 06:18. Просмотров 500. Ответов 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
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
14.10.2015, 06:18     Как оптимизировать обращение к элементам вектора?
Посмотрите здесь:

C++ Обращение к элементам вектора
Обращение по адресу к элементам структуры C++
sFrameBar - Items Обращение к элементам C++ Builder
Обращение к элементам массива C++ Builder
C++ Обращение к элементам массива структур
Обращение к элементам класса через [] C++
Обращение к элементам массива используя указатель C++
C++ Как получить доступ к элементам вектора
C++ Сортировка эл-ов списка, обращение к элементам
Обращение к элементам матрицы C++
C++ Обращение к элементам вектора
Осуществить обращение к элементам массива по указателю C++

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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
nmcf
4944 / 4282 / 1431
Регистрация: 14.04.2014
Сообщений: 16,806
14.10.2015, 10:31     Как оптимизировать обращение к элементам вектора? #2
Для начала избавься от строк, вместо Sheep и Wolf сделай числа 0/1 или bool.
nonedark2008
854 / 593 / 116
Регистрация: 28.07.2012
Сообщений: 1,601
14.10.2015, 10:35     Как оптимизировать обращение к элементам вектора? #3
Цитата Сообщение от os80 Посмотреть сообщение
Подозреваю, что
Возьми анализатор производительности и проверь самостоятельно что у тебя не так.
Под Windows лично я предпочитаю Intel VTune.
AlexVRud
442 / 152 / 38
Регистрация: 04.07.2014
Сообщений: 426
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;
}
os80
0 / 0 / 0
Регистрация: 14.10.2015
Сообщений: 2
14.10.2015, 18:01  [ТС]     Как оптимизировать обращение к элементам вектора? #5
Огромное спасибо, народ, за ответы. Мне нужно время, что б их переварить и оформить в свой код.

Добавлено через 34 минуты
Цитата Сообщение от AlexVRud Посмотреть сообщение
У тебя классы, так пользуйся ими по полной
Отдельное спасибо за ответ. Как я понял, ты предельно точно решил мою проблему. К сожалению, опыта не хватает оперативно поправить код и посмотреть, будет тормозить или нет. Не знаю, сколько времени уйдет на коммит, так что пропаду пока Еще раз спасибо
Yandex
Объявления
14.10.2015, 18:01     Как оптимизировать обращение к элементам вектора?
Ответ Создать тему
Опции темы

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