[kuhwa]

#programowanie #cplusplus

Chcialby ktos przejzec moj kod i wytknac ewentualne bledy/rzeczy ktore moglbym zrobic lepiej?

To jest moje zadanie do szkoly juz skonczone i dziala jak trzeba etc, ale jestem ciekaw czy mocno przekombinowalem ( ͡° ͜ʖ ͡°)

[wytrzzeszcz]

@kuhwa: pw

[jestem_bordo]

@kuhwa: pewnie

[sosnnaa]

@kuhwa: Wrzuć kod na jakimś pastebinie/github i myślę że parę osób obejrzy.
Na przyszłość taguj #codereview, trochę osób obserwuje i łatwiej trafić będzie na wpis.

[kuhwa]

http://pastebin.com/bgq1ijDr

includy tez moge dac, ale dostarczyl je nam wykladowca i nie kazal zmieniac.

[gumis112211]

@kuhwa: Jakiej szkoły? jak średnia to dawaj pan, na tyle jeszcze C++ kojarzę ;]

[kuhwa]

@gumis112211: Uniwersytet

Generalnie to sa stworki na 30x30 mapie, kazdy ma 0-9 hp i co sekunde zrzuca sie jedzenie w losowych miejscach, jak stworek ma jedzenie w okolicy (5 kratek) to dostaje hp, jak nie to traci. Przy 0 umiera. To tyle w sumie.

[sosnnaa]

@kuhwa:
1. Kontrolę nad grą możesz zamknąć w klasie.
2. Zauważ że wyodrębniłeś pewne grupy kodu komentarzami ("Dead creatures by food", "Food eating" itd). Można to wyodrębnić do funkcji/metod.
3. Nazewnictwo - "Console c", "RandomNumber r" - wysil się trochę, na pewno da się to ująć jakoś.

[sosnnaa]

4. Ten potrójny if przy poszukiwaniu jedzenia - spróbuj jakoś nazwać warunek tego sprawdzenia i go wyodrębnić.
W tym momencie patrząc na te ify ni #!$%@? nie wiadomo o co chodzi na pierwszy rzut oka.
5. Na sztywno wpisane w kodzie rozmiary (30). Co jak trzeba będzie zrobić planszę 40x40?

[kuhwa]

@sosnnaa: obserwuje i notuje, jutro bede poprawial, dzieki wielkie :3

[diwi]

@kuhwa: Polecam bardzo książkę http://helion.pl/ksiazki/czysty-kod-podrecznik-dobrego-programisty-robert-c-martin,czykov.htm pozycja obowiązkowa dla każdego programisty :)

[kuhwa]

@diwi: musze przeczytac, bo jeszcze nic z takich rzeczy nie ogarnialem, jedyne co to zabawa samemu + info z wykladow, a wypadaloby cos ciekawego przeczytac na te tematy :p

[kalectwo]

@kuhwa: 1. raczej nie ma sensu tworzyć getterów, setterów, et cetera jeżeli nie ma w nich żadnej logiki. Trochę javą zalatuje. Prościej znaczy lepiej.
2. ZeroMemory jest majkrosoftowskie, nie przyzwyczajaj się. Lepiej już memset czy coś (to i tak to samo). A najlepiej ręcznie pętlą, niekoniecznie najszybsze, ale bedzie działać jak zamienisz na klasy.
3. mapę najlepiej za pomocą std::array> zakodować, jeżeli to w twoich możliwościach. Fajnie dzięki temu się