Skip to content

MVP Say Hear See – pobierz i sprawdź

Posted in Daj się poznać 2017, and Programowanie

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ę!

Be First to Comment

Dodaj komentarz

%d bloggers like this: