Wywoływanie funkcji spowalnia program

pytanie zadane 9 lutego 2016 w C i C++ przez Paweł Głomski Obywatel (1,650 p.)

Pisząc grę ala Mario zrobiłem funkcję odpowiadającą za rysowanie mojej planszy. W argumentach funkcji znajdowała się tablica o wymiarach [20][300], oraz jeszcze kilka pojedynczych zmiennych o różnych typach. Wywoływanie tej funkcji w main'ie, sprawiało, że pętla główna gry nie dochodziła do 15 powtórzeń na sekundę. Przerzucenie całego kodu z funkcji do main'a sprawiło, że wszystko zaczęło chodzić kilkadziesiąt razy szybciej... Rodzi się więc i pytanie: Dlaczego? Czy ciągłe tworzenie nowych zmiennych przez funkcję zajmuje tyle czasu, czy może jest jakiś inny powód?

Z góry dzięki za odpowiedzi.

3 odpowiedzi

odpowiedź 12 lutego 2016 przez adrian17 Ekspert (345,160 p.)
wybrane 12 lutego 2016 przez Paweł Głomski

Najlepsza

Musiałbyś pokazać kod. Na razie jedynym moim strzałem jest że przypadkiem kopiujesz jakieś ciężkie obiekty przy wchodzeniu do funkcji.

komentarz 12 lutego 2016 przez Paweł Głomski Obywatel (1,650 p.)
edycja 12 lutego 2016 przez Paweł Głomski

struct Closest		
{
	Sprite closestLX ;	
	Sprite closestRX ;	
	Sprite bellow[2];	
};
struct Block
{
	bool collision = false;	
	bool isEmpty = true ;	
};
class Character
{
	int dmg;
	int drawFrameCount;	
	Texture rightTexture, leftTexture;	
	RectangleShape shape;
	float jumpFrameCount;	
	float walkFrameCount;
	float walkCounter;	
	int number;	
	Block map[mapYsize][mapXsize];
public:	
	Vector2f charSpawn;
	int hp;
	float speed;	
	int x;
	int y;
	int height;
	int width;
};

Closest drawMap(RenderWindow& window, Block map[][mapXsize], Texture texture, Sprite sprite, Character main)
{
       // cialo funkcji
}

komentarz 12 lutego 2016 przez adrian17 Ekspert (345,160 p.)

No i dobrze zgadłem - kopiujesz argumenty do funkcji, podczas gdy powinieneś je przekazywać przez referencję.

komentarz 12 lutego 2016 przez Paweł Głomski Obywatel (1,650 p.)

Czyli w takich funkcjach (często wywoływanych) najlepiej jest przekazywać argumenty przez referencję? W jakich przypadkach przekazuje się je wtedy "normalnie", skoro traci się wtedy czas na kopiowanie?

komentarz 12 lutego 2016 przez adrian17 Ekspert (345,160 p.)

Bardzo lekki obiekt (typ prosty lub mała struktura z dwoma-trzema typami prostymi) - kopia. Obiekt ciężki lub z kosztownymi konstruktorami kopiującymi -> const ref.

Dokładniej zapisane zalecenia:

https://raw.githubusercontent.com/isocpp/CppCoreGuidelines/master/param-passing-normal.png

odpowiedź 9 lutego 2016 przez RegularLemon Stary wyjadacz (12,280 p.)

odpowiedź 11 lutego 2016 przez Paweł Głomski Obywatel (1,650 p.)

Odświeżam, oraz liczę na bardziej konkretne odpowiedzi.

komentarz 11 lutego 2016 przez draghan VIP (106,230 p.)

W weekend (albo niedługo po), jak nie zapomnę, to postaram się odpowiedzieć. Teraz mam jeszcze egzaminy, nie mam czasu ani ochoty na pisanie porządnych odpowiedzi.

A w międzyczasie możesz wejść na IRC i podpytać o to adriana17, jeśli będzie miał chwilę czasu, na pewno Ci wyjaśni, o co chodzi. :)