Ocena kodu wstępnego do Snake

Question

To oczywiscie wstęp. Zależy mi czy dobrze operuje klasami, obiektami i tablicą dynamiczna.

https://github.com/Ichilnia/snake?files=1

Comments

Wrzuć to na githuba.

---

Zrobione

https://github.com/Ichilnia/snake?files=1

Answer 1

• W Board masz wyciek - tworzysz dwuwymiarową tablicę boardGame i nigdzie jej nie zwalniasz. Dla tej klasy powinieneś zastosować rule of three. I zamiast bawić się z new użyj std::vector.
• Dlaczego w konstruktorze Snake obiekt Board przekazujesz przez wartość? To samo w funkcji write. Przekazuj przed (const) referencję.

Na razie tyle...

Comments

Dziękuję :)

---

ja bym jeszcze dodal metode draw() do Board zamiast w main

Answer 2

Nie wiem jak to zrobiłeś, ale część spacji po sklonowaniu repo była symbolem o numerze 20 a nie spacja, przez co program wykrzaczal sie w dziwny sposob, bo edytor wyswietlal ten znak jako spacje.

void Board::changeValue(bool a, int x, int y)
{
	boardGame[x][y] = a;
}

Może jakieś sprawdzenie czy nie wychodzisz poza tablice?

	for (int j = 0; j < 100; j++)
	{
		for (int i = 0; i < 50; i++)

masz zmienne boardWidth i height. Czemu więc są tu zahardkodowane 100 i 50?

Comments

To zostalo zmienione natomiast mechanizm kontroli błędów dodam. Dziękuję :)

Answer 3

Dziękuję za porady :)

Obecnie wdrazam to co mi napisaliście i opuscilam konsole i koduje juz gierkę za pomocą SFML-a.