[Felonious_Gru]

ile ja sie naszukałem tego błędu xD

#python zdradlwa bestio ( ͡° ʖ̯ ͡°)

[Felonious_Gru]

@h3xxx nie zdążyłem

[Cogres]

@Felonious_Gru: masakra pisać tak nieczytelny kod, u mnie cr by nie przeszło za żadne skarby

[Felonious_Gru]

@Cogres: a jak byś napisał taką funkcję?

[Ilythiiri]

@Felonious_Gru: Rozbij to na jakieś kroki. Tak ,żeby dało się sprawdzić co się dzieje logicznie po drodze bez przeklejania kawałka linijki. Python ma preferencje i możliwości do pisania zwięzłego 1-linijkowego kodu, ale nie ma zaleceń do brandzlowania się w tym kierunku.

[Felonious_Gru]

@Ilythiiri: dynamiczne typowanie mnie zjadło, zwięzłość kodu nie ma nic do rzeczy :P

[Felonious_Gru]

@Ilythiiri: z resztą rozbiłem:
1. posortuj
2. sprawdź długość
3. w zależnośc od długości podaj lub oblicz wynik

[Cogres]

@Felonious_Gru: słuchaj kolegi, dobrze prawi.
Jednolinijkowce mają to do siebie, że po tygodniu nikt nie wie co robią.

[Felonious_Gru]

@Cogres: czyli idealny łup dla testera kodu (｡◕‿‿◕｡)

[Felonious_Gru]

@Cogres: poświęć mi 5 minut i napisz to po swojemu ( ͡º ͜ʖ͡º)

[wytrzzeszcz]

@Felonious_Gru to wygląda jak Perl

[Felonious_Gru]

@wytrzzeszcz: nie wiem, podobno uczę się pythona xD

[Felonious_Gru]

@wytrzzeszcz: ale ewidentnie nie jest to brainfuck ( ͡° ͜ʖ ͡°)

[meohaw]

@Felonious_Gru: Na przykład wiele razy wołasz len(inp), zupełnie niepotrzebnie, tak samo jak obliczanie tego środkowego indeksu.

Proponuję coś w tym stylu:
http://dpaste.com/28NZW83

[pshemeq]

@Felonious_Gru z takich ogólnych uwag:
1. Długość listy możesz wyciągnąć do zmiennej, w przeciwnym wypadku wywołuje ta funkcje parę razy.
2. Jeśli masz dwa przypadki w których funkcja się kończy i zwracasz coś to na dobra sprawę else jest niepotrzebne - wydłuża zapis, a bez niego i tak by tak samo działało.
3. PEP8!
4. Staraj się nadawać nazwy mówiące czym jest dana zmienna. Sort nie jest dobrym wyborem.

[piotrb]

2. Jeśli masz dwa przypadki w których funkcja się kończy i zwracasz coś to na dobra sprawę else jest niepotrzebne

@pshemeq: @Felonious_Gru: Przy czym to ma jeszcze większą moc w w przypadku:

if warunek:
___return cos
else:
___duzo kodu
___duzo kodu
___duzo kodu
___return cos

I oczywiście nie tylko w pythonie.

[pshemeq]

@piotrb oczywiście, e 100% się zgadzam. Jednak w powyższym przypadku jest zbędne

[Felonious_Gru]

@piotrb:
@pshemeq:

Już się kiedyś przejechałem na tam pomijaniu else, to dla mnie zwiększa czytelność kodu, a nie zwiększa mocno ilości obliczeń