[Wujek_Jaskra]

Wrzucam jeszcze raz ( ͡° ͜ʖ ͡°)

Cześć,
miesiąc temu zacząłem naukę #python i popełniłem poniższy mini program.
Czy ktoś mógłby zerknąć i wytknąć wszystkie błędy ?

https://pastebin.pl/view/ee29c9e3

#python #pracait #naukaprogramowania #codereview #programowanie

[qbek]

@Wujek_Jaskra: czasami używasz polskiego, czasami angielskiego, ujednolić to
o zmiennych nazwanych aa już nic nie powiem ;-)

korzystaj z jakiegoś IDE które będzie Cię strzelać po łapach za kiepskie formatowanie (np.

self.tekst1 = Entry(okno, textvariable = self.var, width = 70)

raczej nie daje się spacji wokoło = przy przekazywaniu argumentu

pamiętaj żeby ograniczać długość linii kodu (np. 80 znaków)

[n.....e]

@Wujek_Jaskra: Tak jak wyżej, nazwy zmiennych na angielski, nazwy zmiennych powinny informować co się w nich znajduje, popraw formatowanie. PyCharm ładnie krzyczy na rzeczy typu spacje między znakiem równości w przekazywaniu argumentów). Możesz też w nim ustawić długość linijki (80 to dla mnie zdecydowanie za mało, dałbym raczej 120-160, ma się mieścić na jednym ekranie).
Pętle typu

for k, v in alfabet.items():
 morse_inv[v] = k

Zamieniłbym na dict/list comprehension.
Nie odpalałem,

[Przegrywek123]

@Wujek_Jaskra: przeczytaj Czysty Kod, rozdział o nazywaniu zmiennych i funkcji.

[Wujek_Jaskra]

Dzięki za zerknięcie i poświęcony czas :)