Mój pierwszy "większy" projekt C++ SFML

pytanie zadane 27 maja 2020 w Nasze projekty przez nanautzin Obywatel (1,510 p.)

Witam
Chciałbym pochwalić się moim pierwszym "większym" projektem, większym w cudzysłowie ponieważ pewnie dla części z Was wcale nie taki duży
Proszę o konstruktywną krytykę na temat wykonania oraz na temat kodu, co mógłbym zmienić w przyszłości, jakie mam złe nawyki w kodowaniu

https://github.com/lukasz-kukulka/Magic_board

PLIK INSTALACYJNY: https://drive.google.com/file/d/1zvSctT-U9XQMUmfjuJWX6MROYyJ75DqL/view?fbclid=IwAR16KY-lggdyJZq8jppJhtp2wnvnbkk_LDlHtesErTeeTVy2bc7QClTFW4A

Z góry dziękuję za odpowiedzi

1	komentarz 27 maja 2020 przez tkz Nałogowiec (42,000 p.) Używasz branchy jak commitow? I ten chaos w plikach, fajnie jakbyś je pogrupował do folderów. Domyślam się, że pisałeś w jakimś IDE, jak mamy zbudować ten projekt?

komentarz 27 maja 2020 przez nanautzin Obywatel (1,510 p.)

Dzięki za odpowiedź
Z gitem nie za bardzo miałem styczność i na początku go olałem, tak szczerze używałem go tylko przez GitHub Desktop, dopiero dzisiaj przy nim przysiadłem i bawiłem się konsolą. Używałem CodeBlocks i biblioteki SFML

1 odpowiedź

odpowiedź 27 maja 2020 przez VirtualMember Pasjonat (15,790 p.)
wybrane 15 czerwca 2020 przez nanautzin

Najlepsza

Tak jak wspomniał @tkz, przydałby się jakiś CMake albo makefile co można by zbudować ten projekt. Jedną z rzeczy, która mi się nie podoba to to, że definiujesz explicite pusty destruktor. Jeśli chcesz sforsować wygenerowanie destruktora przez kompilator i mieć go explicite w klasie użyj słowa kluczowego default. Wiele stałych masz jako magic numbers. Pliki powinny być pogrupowane. Jeśli twój kod nie będzie używany jako `plain c` to nie używaj nazewnictwa .h dla headerów C++. Zamiast tego nazywaj je .hpp. Twoje formatowanie jest niespójne, użyj jakiegoś formatera, na przykład clang format. Niektóre pliki masz puste. Brak readme w repozytorium. Plik z logami nie powinien znaleźć się w repo. Zaimplementuj albo ją sam, albo użyj gotowca do pętli stałokrokowej. Preferuj używania intów z określoną ilością bitów: int32_t int64_t int16_t int8_t itp zamiast int. Int ma tylko powiedziane według standardu, że ma >= 16 bitów. Co prawda na większości współczesnych architektur ma 32 bity, nie mniej jednak złą praktyką jest nadużywanie intów. Popracuj nad nazwami commitów: test5 nie dużo mówi. Usuń pliki z twojego ide takie jak .cbp i inne.