Kod w pythonie nie kompiluje się

Question

 

def poczta_polska(lista_klientow):
    kolejka = []
    lista_wyjscia = []

    for klient in lista_klientow:
        kolejka.append(klient)

    while kolejka:
        osoba = kolejka.pop(0)

        if osoba[1] == True:
            kolejka.append(osoba)
        else:
            lista_wyjscia.append(osoba[0])

    return lista_wyjscia

lista_klientow = [("Anna", False), ("Bartek", True), ("Cezary", False), ("Dorota", True), ("Ewa", False)]

wynik = poczta_polska(lista_klientow)
print(wynik)

 

Dlaczego wynik się nie wyświetla po kompilacji?

Answer 1

Nie rozumiem co ten kod ma robić, ale no - masz pętlę która nigdy się nie kończy. Pętla czeka aż kolejka będzie pusta, ale ciągle wyciągasz i wstawiasz z powrotem te same osoby (z True).

EDIT: btw, tutaj nie ma co mówić o kompilacji, po prostu mówisz o odpaleniu programu/skryptu ;)

Answer 2

Python "zawiera" dużo "fajnych" i bardzo przydatnych funkcji w swojej składni, może rozważ użycie jednej z nich jak np. map i jak drugi parametr użyj filter, wtedy możemy zapisać np. tak:

[ działający kod on-line ]

def poczta_polska(lista_klientow):
    # Używamy filter, aby wybrać klientów z drugim parametrem równym False
    lista_wyjscia = list(map(lambda x: x[0], filter(lambda x: not x[1], lista_klientow)))
    
    return lista_wyjscia

lista_klientow = [("Anna", False), ("Bartek", True), ("Cezary", False), ("Dorota", True), ("Ewa", False)]

wynik = poczta_polska(lista_klientow)
print(wynik)

 

BTW, 

Dlaczego wynik się nie wyświetla po kompilacji?

python jest językiem interpretowanym, więc nie ma mowy o kompilacji jako takiej, choćby znanej z np. C++, przy uruchamianiu kodu. 

Comments

Ja wciąż nie wiem co ten oryginalny kod nawet miał robić, więc nie chciałem proponować przepisywania na cokolwiek. Twój strzał jest... może i dobry, ale wciąż strzał ;)

A inna sprawa ze to

lista_wyjscia = list(map(lambda x: x[0], filter(lambda x: not x[1], lista_klientow)))

to jest jakiś potworek który prędzej by ktoś napisał specjalnie żeby sparodiować Pythona i zniechęcić ludzi do używania map/filter :P Normalnie by się to zapisało jako po prostu

lista_wyjscia = [klient[0] for klient in lista_klientow if not klient[1]]

 

 

---

IMO, Ja, osobiście wolę w dużej większości przypadków używać map.

When should I use a Map instead of a For Loop?

Może i ten przykład był za prosty, i masz rację, że może "nadużyłem" map, i z przyzwyczajenia "pojechałem" z map. , ale zawsze mam "z tyłu głowy", że python to język interpretowany, więc wbudowane funkcje są skompilowane i powinny działać szybciej z definicji, a pętla for w kodzie jest interpretowana, ale nie będę się upierał. 

---

When should I use a Map instead of a For Loop?

Pierwsza odpowiedź: "map is useful when you want to apply the function to every item of an iterable". Ale tutaj nie miałeś z góry gotowej funkcji jak `int`, tylko musiałeś zadeklarować lambdę, nawet dwie.

Druga odpowiedź: benchmark, comprehension najszybszy (a od 3.12 jeszcze szybszy, bo dodali comprehension inlining, PEP 709)

Trzecia odpowiedź: "Just use list comprehensions: they're more pythonic". No i ten punkt że można trywialnie je konwertować między listami i generatorami też słuszny.

;)

PS:

więc wbudowane funkcje są skompilowane i powinny działać szybciej z definicji, a pętla for w kodzie jest interpretowana, ale nie będę się upierał

Różnica polega na tym, że z map() masz lambdę `lambda x: x[0]`, którą trzeba wywołać hipotetycznie 100000 razy - a overhead wołania Pythonowych funkcji jest nietrywialny. "x[0]" to w końcu wciąż kod, który trzeba zinterpretować. (byłoby odrobinę lżej z itemgetter(0)) A przy comprehension kompilator od razu tworzy pętlę z ciałem wykonującą append x[0] 100000 razy, bez wołania żadnych pośrednich funkcji.