Cześć,
w ramach ćwiczeń C++ napisałem prostą księgarnię i chciałbym się dowiedzieć (po raz pierwszy) co ludzie z zewnątrz sądzą o moim kodzie.
Liczę na krytykę i ciekawe uwagi, sugestie ;)
Kod na Pastebin'ie
0
2
Mało tu do oceniania, tak szczerze powiedziawszy
- Konstruktor - zapoznaj się z "initialization list"
- Książkę jako taką naprawdę interesuje ilość książek (sztuk)? Chyba skopciłeś tutaj coś
- Książka może mieć wielu autorów
- Skoro już brniemy w coś takiego... po wypożyczeniu książki ich ilość to równo "-1", ale po oddaniu zwiększa się o jeden?
- Pokazanie książek z księgarni modyfikuje jakoś stan księgarni (może coś w książkach robi)? Jeśli nie, to wypada odpowiednio taką metodę oznaczyć
- Postinkrementacja... @_13th_Dragon Ci powie :P
- Księgarnia nie jest odporna na wykraczanie poza zakresy, jeśli poprosisz ją o nieistniejącą książkę to leży i kwiczy
- Przestrzeń nazw std... byłoby okej, gdyby to był szablon - którego nie rozdziela się na deklarację i definicję
- Czy po wypożyczeniu książki nie powinniśmy mieć do niej... no nie wiem, dostęp?
-
dodaj_ksiazke(ksiazka &nowa_ksiazka)
Powinieneś to przesłać jako stałą referencję - ^ to samo w przypadku stringów
- Co jeśli chciałbyś uzyskać informacje o książce zapisując je w stringu? Albo w innym pliku?
- Testy?
2
Nie powinieneś trzymać ilości przy książce.
Użyj std::map<ksiazka,unsigned> ksiazki;
0
Nazwy klas piszemy wielką literą :)
0
w takim razie odszczekuje.
P.S nie moge edytowac postow, pozdro
Zarejestruj się i dołącz do największej społeczności programistów w Polsce.
Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.