Uwagi

[MarcinCiura]

1. Katalogi Jakdojade i __pycache__ do wyrzucenia (1. rozdział Poradnika).
2. Niektóre pliki wydają się dublować funkcjonalność innych plików. Nie wiem, którym mam się przyglądać. Proszę wyrzucić wszystko, co nie jest częścią ostatecznej wersji.
3. Pylint pomoże Panu ogarnąć używanie białych znaków (2. rozdział).
4. Nazwy zmiennych od a do z nie są właściwe. Czytelnik powinien móc dzięki nazwom zmiennych zrozumieć, co się dzieje.
5. Funkcje powinny być w snake_case (2. rozdział).
6. W while (True) nie powinno być nawiasów.
7. Przypuszczam, że w obecnej wersji trzeba dokładnie wpisać nazwę przystanku, co jest delikatnie mówiąc niewygodne dla użytkownika. Może Pan odgapić klasę Completer i wywołania funkcji modułu readline stąd (pod Windows trzeba doinstalować pyreadline; mam nadzieję, że importy zadziałają — nie mam jak ich sprawdzić pod Windows — jakby coś szwankowało, proszę dać znać). Żeby przetestować odgapiony kod, proszę wpisać dwie pierwsze litery nazwy upatrzonego przystanku i dwa razy nacisnąć Tab. Zamiast normalize() wystarczy samo .lower(), bo nazwy przystanków nie zawierają apostrofów w żadnej formie.

[MarcinCiura]

Mam dwa pomysły, które mogą rzucić światło na niedziałanie Pana programu:

1. Proszę zakomentować wszystkie 4 wiersze importujące readline (całą resztę zostawić jak jest), znowu uruchomić program i zobaczyć, czy cokolwiek drgnęło.
2. Jeśli nie, to sprawdzić, czy uruchomienie gołego python3 wypisuje coś w poniższym stylu:

Python 3.6.9 (default, Apr 18 2020, 01:56:04) 
[GCC 8.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> 

Jeśli wykonanie któregoś z powyższych punktów da zaskakujący wynik, proszę dać znać.

[Dydek123]

Problem z uruchamianiem programów przez wiersz poleceń udało mi rozwiązać, pozostałe moduły uruchamiają się normalnie. Natomiast teraz pojawił się problem z importowaniem biblioteki sklearn, który będę próbował rozwiązać. Jednak uruchomiłem program pod Linuxem i zarówno program jak i funkcja uzupełniania przystanków przy wprowadzaniu działa poprawnie.

[MarcinCiura]

Znakomicie. Ale sklearn jest ciut do czegoś innego. Ścieżki w grafach znajduje tylko dlatego, że dodali do niego wszystko, co im wpadło do głowy "razem ze zlewozmywakiem", jak mówią Angole. Najlepszy dowód ich ignorancji to chwalenie się kopcami Fibonacciego, które są takie mądre, że tylko głupcy ich używają (gdyby Pan kiedyś potrzebował porządnych kopców, proszę wybrać parujące [pairing heaps]). Zamiast sklearn polecam networkx albo python-igraph.

[Dydek123]

Dobry wieczór,
po ostatnich konsultacjach dodałem dodatkowe funkcje testujące. Proszę o sprawdzenie, czy powinienem jeszcze coś poprawić.

[MarcinCiura]

Dziękuję za wiadomość. Oto kolejne uwagi.

1. Zmienne globalne są złe, a jednoznakowe bardzo złe. Proszę zrobić z nich atrybuty w odpowiednich klasach, inicjalizowane w konstruktorach (z funkcji get_value() zrobić metodę).
2. Obawiam się, że to nie działa tak, jak Pan myśli. Chyba powinien Pan zewnętrzne i przemianować na cokolwiek innego, bo teraz jest miszmasz.
3. To przerobić na for ... in self.trasa_linii[1:]:.
4. Analogicznie zmiany w szukaj.py. Dodatkowo proszę tam dłuższy fragment jest po if __name__ == '__main__': przeobrazić w funkcję main().
5. Tu i dalej chyba chodzi Panu o assertCountEqual().

[Dydek123]

1. Poprawione zarówno dla GUI.py oraz szukaj.py
2. Poprawione
3. Poprawione we wskazanym miejscu oraz kilku innych
4. Poprawione
5. Zmienione

[MarcinCiura]

LGTM, jest Pan trzydziesty szósty. Gratuluję uporu przy realizacji projektu.

[Dydek123]

Dziękuję,
raport z projektu oraz pliki umieściłem na elfie.