[J.....L]

#naukaprogramowania #python napisałem prosty programik . Wpisujesz dwa kody pocztowe, np. 95-030 i 60-777 , i program sciąga baze danych kodów pocztowych (baza jest w zipie) rozpakowywuje ją, otwiera plik CSV , i wypisuje wszystkie kody wraz z adresami (ulice i miasta )kodów pomiędzy wpisanymi. Może ktoś ew. zerknąć, na kod, czy da się tam coś poprawić, może jakieś wskazówki z grubsza, co warto by do oddzielnych funkcji wydzielić a czego nie ? dzięki

https://pastebin.com/4GQaHhCx

[GRzA6969]

@JungleJamPL ja bym wydzielił zaciąganie bazy, rozpakowanie, czytanie i szukanie w bazie na osobne funkcje. Używaj open z context managerem, bo nie zamknąłeś pliku. Input z klawiatury do listy to nie jest dobry pomysł w tym przypadku. Może warto by ta funkcja zamiast zwracania listy, zwracała dwie zmienne?

[J.....L]

@GRzA6969: wiesz akurat listę tutaj celowo zwrocilem, z tego względu ze dzieki temu nie ma znaczenia ktora zmienną (mniejszą badz większą) uzytkownik wpisze jako piwrwsza, bo potem przy porownaniu poprostu porównuje z min/max(lista), bo w przeciwnym wypadku wystarczylo by ze uzytkownik wprowadzil wiekszy kod na poczatku i potem algorytm nie zadzialal by juz.

[Krasiu]

@JungleJamPL: ten plik ma nagłówek, więc DictReader chyba miałby większy sens.