• 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
Możesz ukryć, zamknąć lub zmodyfikować swoje pytanie, za pomocą przycisków znajdujących się pod nim. Nie krępuj się poprawić pochopnie opublikowanego pytania czy zamknąć go po uzyskaniu satysfakcjonującej odpowiedzi. Umożliwi to zachowanie porządku na forum.Przyciski pytania

60,207 zapytań

105,878 odpowiedzi

219,906 komentarzy

32,422 pasjonatów

Przeglądających: 183
Pasjonatów: 6 Gości: 177

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.

...