Niedawno temu (niecały tydzień) dostałem komentarz pod wstawką z poradą odnośnie kodu źródłowego. Treśc możecie podejrzeć tutaj. Dodatkowo, moja Pierwsza Naczelna Testerka uznała, że trzeba zmniejszyć, poprawić i ujednolicić zdjęcia.
Const w getterach
„Gettery i Settery to metody, które służą do pobierania oraz ustawiania wartości składowych, zazwyczaj prywatnych bądź chronionych w klasach. Gettery, jak sama nazwa wskazuje służą do pobierania wartości zmiennych. Zazwyczaj są to metody const, które nie modyfikują danych tylko je zwracają.”
~ https://forum.pasja-informatyki.pl/user/Buby
Dlatego też…
„gettery suffixuj słowkiem const”
Co też zostało uczynione. Przeszukałem wszystkie klasy i gettery zostały „suffixowane”.
Item::CountLines(cont,t=0); <- nie najlepsza konstrukcja
I to się zgadza. Słabo wygląda. Zostało skorygowane wstawieniem resetowania zmiennej przed zliczaniem linii.
Redukcja rozmiarów zdjęć
Popracowałem niecałe pół godziny w Gimpie i rozmiar paczki zmniejszył się ze 190Mb do 26 Mb. Dodatkowo redukcja obciążenia pamięci jest na poziomie 7-krotności poprzedniego obciążenia. Okno kategorie zużywało 140Mb, tymczasem jest 24-27Mb. Niektóre zdjęcia zostały wymienione, jednak jeszcze kilka zdjęć jest do wymiany. Są ładne, ale nieproporcjonalne, przez co wygląda to jeszcze trochę pokracznie. Taki urok MVP ;p
Linki do pobrania paczki, którą można wypakować na pulpicie – Say Hear See.rar
Link do repozytorium – Github SHS
Link do ostatniego commitu – 06a7f91
Podziękowania:
Kamil Klimek – dzięki za radę!