[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

[wytrzzeszcz]

@Ginden: calendar.write( lines+"P,1,1,1,1,1,0,0,"+date+",20220923\n")
jak to powinnem zrobic więc (to pętla jest gdzie date i lines się zmienia reszta hardcode)

[G.....n]

@wytrzzeszcz: W module jest CSV writer.

@kisi3l:
PEP8 wyraźnie mówi:

. In performance sensitive

[k.....l]

@Ginden:
Argument o czytelności to subiektywna opinia. Przypomina mi historię, gdzie w u kogoś w firmie zakazali używania operatorów ternarnych, bo ktoś stwierdził, że są nieczytelne.

[k.....l]

@wytrzzeszcz: Btw, w metodzie makecalendar() zapomniałeś chyba o linenumber.

[wytrzzeszcz]

@kisi3l: nie no działa nie kumam błędu :(

[k.....l]

@wytrzzeszcz:
Ale co nie działa? Jakiego błędu? Mam na myśli linię nr 32.

[wytrzzeszcz]

@kisi3l: aaa kumam

[morsik]

calendar.write( lines+"P,1,1,1,1,1,0,0,"+date+",20220923\n")

@wytrzzeszcz: pewnie mniej wydajniej ale na pewno najładniej:
a) stary format z printfa (FYI, nie powinieneś już używać)

calendar.write("%sP,1,1,1,1,1,0,0,%s,20220923\n" %

[G.....n]

@morsik: Ale czemu nie csv.writer?

[morsik]

@Ginden: Ale co? Ja odpowiedziałem tylko na pytanie @wytrzzeszcz które brzmiało:

@Ginden: calendar.write( lines+"P,1,1,1,1,1,0,0,"+date+",20220923\n")

jak to powinnem zrobic więc (to pętla jest gdzie date i lines się zmienia reszta hardcode)

a rozmowa była o łączeniu stringów i tylko odniosłem się do łączenia stringów ;)
Jeśli lepsze by było csv.writer to go tam użyjcie, ja nie znam więc się na jego temat nie wypowiadam.

[askorek]

@wytrzzeszcz:
if last = 0 ??

[wytrzzeszcz]

@askorek: a to wersja przed poprawką wtedy :(
chciałem wykryć 2 zera w ciągu