• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Ocena kodu wstępnego do Snake

0 głosów
183 wizyt
pytanie zadane 8 listopada 2018 w C i C++ przez Ichilnia Początkujący (260 p.)
edycja 8 listopada 2018 przez Ichilnia
To oczywiscie wstęp. Zależy mi czy dobrze operuje klasami, obiektami i tablicą dynamiczna.

https://github.com/Ichilnia/snake?files=1
1
komentarz 8 listopada 2018 przez RafalS VIP (102,670 p.)
Wrzuć to na githuba.
komentarz 8 listopada 2018 przez Ichilnia Początkujący (260 p.)

3 odpowiedzi

+1 głos
odpowiedź 8 listopada 2018 przez j23 Szeryf (89,240 p.)
wybrane 8 listopada 2018 przez Ichilnia
 
Najlepsza
  • 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...

komentarz 8 listopada 2018 przez Ichilnia Początkujący (260 p.)
Dziękuję :)
1
komentarz 8 listopada 2018 przez 10kw10 Pasjonat (23,020 p.)
ja bym jeszcze dodal metode draw() do Board zamiast w main
0 głosów
odpowiedź 8 listopada 2018 przez RafalS VIP (102,670 p.)
edycja 8 listopada 2018 przez RafalS

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?

komentarz 8 listopada 2018 przez Ichilnia Początkujący (260 p.)
To zostalo zmienione natomiast mechanizm kontroli błędów dodam. Dziękuję :)
0 głosów
odpowiedź 8 listopada 2018 przez Ichilnia Początkujący (260 p.)
Dziękuję za porady :)

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

Podobne pytania

+2 głosów
1 odpowiedź 532 wizyt
pytanie zadane 2 czerwca 2016 w C i C++ przez Pixel040 Gaduła (3,020 p.)
+1 głos
3 odpowiedzi 397 wizyt
pytanie zadane 30 maja 2016 w C i C++ przez Rolnik Nowicjusz (220 p.)
+1 głos
2 odpowiedzi 437 wizyt
pytanie zadane 1 maja 2016 w C i C++ przez Curiosity Nowicjusz (130 p.)
Porady nie od parady
Odznacz odpowiedź zieloną fajką, jeśli uważasz, że jest ona najlepsza ze wszystkich i umożliwiła ci rozwiązanie problemu.Najlepsza odpowiedź

60,207 zapytań

105,878 odpowiedzi

219,906 komentarzy

32,422 pasjonatów

Przeglądających: 177
Pasjonatów: 3 Gości: 174

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...