[zygzagZ]

Mirki, jak z jakością mojego kodu?
Mogę aplikować na programistę 15k/mc? ( ͡° ͜ʖ ͡°)
#humorinformatykow #programowanie

[poszlo_wyszlo]

@zygzagZ: brak konsekwencji w formatowaniu #usunkonto

[m.....i]

@zygzagZ: Za brak enterów i obecność magic numbers i magic strings #usunkonto

[hctkko]

@zygzagZ: brak string buildera, kto jeszcze używa +=

[KrzaQ2]

@zygzagZ: Kod na 3+ lub 3-, zaleƶy czym jest exist. Jeśli członkiem klasy to dobrze, inaczej źle.

Nie rezerwujesz pamięci stringa ret, więc będziesz miał kilka zbędnych realokacji. Dodatkowo nazwany jest ret, mimo, ƶe funkcja zwraca void.

Nie uƶywasz args, jednocześnie przekazując ten argument przez

[Huczek]

@zygzagZ: Open Peja
Kto jeszcze tworzy OT ?

[KrzaQ2]

@hctkko: Nie ma nic złego w += jako takim.

[hctkko]

@KrzaQ2: ale tutaj, przy takiej ilości iteracji jest bez sensu.

[KrzaQ2]

@hctkko: Ale to nie += jest problemem, tylko brak odpowiedniej prealokacji. Polecam http://en.cppreference.com/w/cpp/string/basic_string/reserve

[Cesarz_Polski]

@zygzagZ: za brak komentarzy do kodu #usunkonto xD

[ly000]

ret += "usuń";
ret += "konto";
ret += "( ͡° ͜ʖ ͡°)";

[villager]

@zygzagZ: solidne 2/10

[zygzagZ]

Znaczy się 3-, ale już przeniosłem do klasy.
@maciej-jantarski: @KrzaQ2: @Cesarz_Polski:
Fix'd wklej.to

@poszlo_wyszlo:
Staram się używać camelCase, jednak nie ja pisałem biblioteki std i stdext. A że istniejące nazwy są używane w wielu miejscach w kodzie, w wielu plikach,

[m.....i]

@zygzagZ: Komentarze nie załatwiają magic numbers. Te rzeczy powinny być jako stałe, na początku pliku lub w innym pliku (nie ma reguły, ważne by nie były gdzieś pośrodku). Dlaczego? Bo jeśli kilka razy używasz 65k jako warunku zakończenia pętli, i kiedyś zmienisz to na 128k, to zmieniasz jedną zmienną a nie 10. DRY.

[tata_marcina]

@zygzagZ: Mozesz jeszcze tagowac #codereview tak na przyszlosc, przydaloby sie rozkrecic ten tag :)

[l.....l]

@zygzagZ:
Czytałem poprawioną wersję stąd: http://wklej.to/gWBuZ
1. Za długa funkcja. Powinna być podzielona na kilka mniejszych, z których każda ma tylko jedno zadanie - ta po prostu robi kilka rzeczy. Pamiętaj przy tym o odpowiednim nazewnictwie.
2. Zależnie od konwencji, jedna deklaracja zmiennej, jedna linijka. Nie ma sensu deklarować przy tym zmiennej tam gdzie jeszcze nie jest potrzebna - to z resztą wyjdzie jak podzielisz tą funkcję na podfunkcje.
3. Niekonsekwentnie używane klamry - raz otwierasz je w nowej linijce, raz na końcu linijki. Sam poza tym jestem przeciwnikiem dodawania klamer w if'ie jeśli jest trywialny i ma tylko jedną linijkę.
4. Deklarowanie std::string przez przypisanie pustego ciągu znaków nie ma

[gkw_]

@zygzagZ: W kod nie patrzylem ale gratuluje używania najlepszego softu do kodu ST3 ;) Pozdrawiam

[k.....r]

15. Zamiast functionLog powinieneś użyć std::stringstream. Operatory += na string'u są bardziej kosztowne, a dodatkowo nie będziesz musiał robić konwersji do string'a ręcznie.

@losiul:

brak string buildera, kto jeszcze używa

[l.....l]

@kuhar:
Potrafię podać kilka powodów:
- Brak konwersji do strumienia (musisz to robić ręcznie). Dodatkowo, przy konwersji powstaje tymczasowy string, który istnieje tylko na chwilę, tylko po to żeby go dołączyć do reszty string'a.
- Brak kontroli nad konwersją (chcesz zaokrąglić liczbę zmiennoprzecinkową? chcesz komórkę szerokości 7 znaków? itd, itp)
- nieoptymalne zarządzanie pamięcią z prealokacją
Coś bym jeszcze wymyślił, ale późno jest.

[ponton]

@zygzagZ: przede wszystkich sama funkcja jest za długa, rozbij to na kilka mniejszych funkcji

[k.....r]

@losiul: na dobrą sprawę wystarczy pilnować, żeby nie było alokacji po prawej stronie += i wydajność jest porównywalna