[legolass]

@morsik: Wydajniej chyba będzie string2 = ''.join((string1, 'b')). Jak testowałem to tuple były odrobinę szybszę ( ͡° ͜ʖ ͡°)

[Fitoplankton]

@legolass: 8 rzeczywiście, zapomniałem, że dwie osoby o tym samym imieniu mogą mieć różne preferencje

[Fitoplankton]

@legolass: O.o trochę xd, bo nie spodziewałem się takiego rozwiązania ( ͡° ͜ʖ ͡°)

[morsik]

@legolass: no to ostro zupełnie jak tak ;o
@Fitoplankton: Problem z + przy stringach jest taki, że jak dodajesz 2 stringi, to oryginalny ma robiony resize pamięci (realloc w C), a to jest kosztowne. W przypadku ''.join((a, b, c,)) tworzony alokowana pamięć jest od zera od razu o znanym rozmiarze.

[legolass]

@Fitoplankton: @morsik: Jak już tak dbamy o wydajność to przypisanie do atrybutu klasy jest cięższe od przypisania do lokalnej zmiennej. :P

[morsik]

@Fitoplankton: a co do powyższego jeszcze dodam, że jak faktycznie przy 2 stringach zysk może być minimalny, to przy 10 stringach daje to 10 realloców, a to już sporo.

[Fitoplankton]

@morsik: @legolass: jestem trochę zdziwiony, że nie wytknęliście jeszcze sortowania przed pobraniem kolejnej osoby do listy ( ͡° ͜ʖ ͡°). Co prawda - łącznie w jednej generacji nastąpi ok. 300 pobrań osoby do listy, więc aż taka optymalizacja nie jest raczej potrzebna. Trudno mi wyczuć tę granicę między dążeniem w optymalizację, a czytelnością kodu (bo bardziej skomplikowanego kodu to raczej ładnie nie napiszę)

i

[dzien_dobry]

@Fitoplankton: To nie jest Pull request.

[morsik]

Trudno mi wyczuć tę granicę między dążeniem w optymalizację, a czytelnością kodu (bo bardziej skomplikowanego kodu to raczej ładnie nie napiszę)

@Fitoplankton: Ostatnio w pracy miałem ten problem i trzeba sobie odpowiedzieć co ten kod ma właściwie robić i jakie mogą być przypadki zastosowania (najlepiej przykłady ekstremalne).
U siebie wybrałem czytelniejszy kod mimo wszystko, bo bardziej zoptymalizowana i potencjalnie szybsza opcja była strasznie zagmatwana… szczerze mówiąc to czytelniej byłoby tę implementację już

[morsik]

@dzien_dobry: ale w commitach też możesz komentować linijki :P

[Fitoplankton]

@morsik: @legolass: ok, jutro przysiądę połatać i przeformować to wszystko, dzięki :) A zadam już siłą rozpędu małe pytanko - jak myślicie, czy przy takim poziomie kodu (mam w planach także ze 2 jeszcze projekty do października zrobić) dostałbym się na jakiś bezpłatny staż? Czy to nadal daleko poza moim zasięgiem?

[morsik]

@Fitoplankton: Jak na 200 linii, 11 rzeczy które wylistował Ci @legolass to dość mało. Na staż zawsze możesz próbować — w końcu to tylko staż ;)

Serio, widziałem skrypty które miały 50 linijek i było w nich prawie tyle samo standardowo popełnianych błędów (było nawet po kilka błędów w jednej linijce).

[Fitoplankton]

@morsik: to jest tak naprawdę pierwszy tak poważny program napisany od zera w pythonie. Staram się też qt dorzucać powoli (choćby mój ostatni wpis). Ok, dzięki jeszcze raz, motywacja zauważalnie wzrosła ( ͡° ͜ʖ ͡°) Pozdrawiam!

[legolass]

@Fitoplankton: Jeśli Ty się czujesz na siłach to próbuj. Ogólnie jak na początek jest bardzo dobrze.

@morsik: Mógłbym więcej wylistować ( ͡° ͜ʖ ͡°)

[morsik]

Mógłbym więcej wylistować ( ͡° ͜ʖ ͡°)

@legolass: Ja się lubię dopieprzać do wcięć i formatowania ogólnie, ale tu widzę dość ładnie jest :P

Ja akurat jestem niedzielnym programistą. W pracy czasem coś piszę, ale jestem administratorem, więc ten Python to raczej dodatek do pracy ;)

[Fitoplankton]

Ja się lubię dopieprzać do wcięć i formatowania ogólnie

@morsik: od początku wiedziałem, że na raz pepa do głowy nie wcisnę. PyCharm wybitnie wręcz pomaga w kontrolowaniu konwencji, każdy błąd poprawiam od razu i jest gici, teraz nawet w vimie byłbym w stanie jako tako utrzymać składnię ( ͡° ͜ʖ ͡°)

[legolass]

@Fitoplankton: PyCharm ze skrótami z vima jest lepszy :P

[Fitoplankton]

@legolass: o! nawet nie wpadłem na takie połączenie ( ͡° ͜ʖ ͡°). Kurczę, na samą myśl, że na studia będę musiał znowu VS instalować, to mi się słabo robi :P moje 120 na ssd pęknie jak nic ( ͡° ʖ̯ ͡°)