[wytrzzeszcz]

#programowanie #python
ten kod działa i robi co ma robić ale czuje ze jest brzydki (przerabia pliki ze strony ZTM-u w #gdanks na jakiś format do analizy)
wyśmiejcie go czy coś bym wiedział co poprawić
http://pastebin.com/mXMehTcZ

[l.....e]

polecam test pep8 https://www.python.org/dev/peps/pep-0008/

Ładnie ci wyrzuci błedy składniowe (choc jezeli dziala to pewnie ich nie ma) oraz podpowie jak kod moze wygladac ładniej

[J.....n]

@wytrzzeszcz:
nie znam się na Pythonie ale jeśli już używasz angielskich nazw zmiennych i funkcji to używaj ich cały czas, bo widzę np. funkcje "make_kurs"
no i nie hardkoduj ścieżek do plików txt, niech je wczytuje z configa albo niech ją funkcja przyjmuje w argumencie wtedy to będzie bardziej użyteczne do wielokrotnego użytku

[wytrzzeszcz]

@FedoraTyrone: if _name == "main_":czo to?

[G.....n]

@wytrzzeszcz: make_calendar powinno używać CSV.
time_formatter na pewno jest w bibliotece standardowej.
Rozdziel parsowanie kodu zawartości i pisanie do pliku.

Używaj Pythona 3 jeśli nie nastawiasz się na pracę w korpo.

[k.....l]

@wytrzzeszcz:

- Zastosuj jakieś standardy formatowania, bo teraz się czytać nie da.
- Nie używaj niepotrzebnie nawiasów w if'ach.
- Między metodami raz masz jedną pustą linię, raz dwie.
- Nie używaj spacji po nawiasach, używaj po przecinkach,

[wytrzzeszcz]

- Nie dodawaj stringów w pętli.

- Nie dodawaj stringów w pętli.

@kisi3l: dlaczego

[G.....n]

@kisi3l:

nie dodawaj stringów w pętli

http://stackoverflow.com/a/13345099/2446102

[k.....l]

@wytrzzeszcz: Gdyż stringi są immutable, i przez to tworzysz niepotrzebne obiekty.

[wytrzzeszcz]

@kisi3l: czyli zapis "a"+"b"+"c" zrobi obiekt "ab" i doda do niego "c" tworząc "abc"?

[k.....l]

@wytrzzeszcz: Tak jest.

[wytrzzeszcz]

@kisi3l: i "a"+"b"+"c"+"d" nie da "ab"+"cd" tylko ab+c+d jop?

[k.....l]

@wytrzzeszcz: Generalnie każda konkatenacja (+) utworzy nowego stringa (nie może 'zmienić' istniejącego, bo ten jest immutable). Zatem będzie to coś w stylu: a + b + c + d = ab + c + d = abc + d = abcd.

[wytrzzeszcz]

@kisi3l: i każdy string pośreni istnieje w pamieci tak długo jak go używamy ?

[k.....l]

@wytrzzeszcz:
Moja wiedza o Pythonie nie pozawala mi niestety odpowiedzieć na to pytanie.

Te obiekty są 'tymczasowe', więc zakładam, że nie istnieją dłużej, niż muszą.

[wytrzzeszcz]

@kisi3l: luźno :D w C to się samemu o tym myślałeo wiec było jasne

[G.....n]

@kisi3l: Przeczytaj kod C CPythona najpierw. Koszt takich operacji jest bardzo mały, bo Python używa pod maską dość skomplikowanych struktur danych i operacje na stringach są względnie tanie ze względu na to, że rzeczywiście spawnują nowe obiekty, ale kod pamięci nie jest O(n^2) jak w JVM tylko bliżej O(log n). Nowe obiekty zawierają głównie wskaźniki na fragmenty stringa, nie jego kopie.
Kosztem jest większy koszt czytania stringa.

[k.....l]

@Ginden: Ja zdaję sobie sprawę, że trzeba by było być paranoikiem, żeby przejmować się złożonością czy 'zasobożernością' tak trywialnych operacji, ale są pewne standardy, których powinno się trzymać, i pewne idiomy, które już na dobre zagościły w pamięci programistów.

[G.....n]

@kisi3l: Programistów Javy. Python jak widać korzysta z innych struktur danych i przenoszenie doświadczeń z jednego języka na drugi jest pozbawione większego sensu.
To nie przypadek, że w Pythona nie ma klasy StringBuilder - nie ma takiej potrzeby.