newcomers second script - ASCII groping -

[5inch]

Hallo Forum,

ich habs nach langer Zeit mal wieder geschaft mir ein script zu basteln.
Ich fänds klasse, wenn Ihr mir ein paar Tipps gebt, was ich besser machen kann.
Ich fang ja gerade erst mit Python an.
Also hier kommt der code:

Code: Alles auswählen

fileSourcePts = "bl05.pts"
textSourceTif = "05.tif"
textSourceJpg = "05.jpg"
fileOutput = "NewBl"

def allFiles():
    global ZeiJpg
    global ZeiTif
    global ZeiPts
    for count in range(1,13):
        num = str(count)
        ZeiJpg = ((num.zfill(2) + ".jpg"))
        ZeiTif = ((num.zfill(2) + ".tif"))
        ZeiPts = ((num.zfill(2) + ".pts"))
        replaceText()
    

def replaceText():
    tmpFile = open(fileSourcePts, "r") 
    firstRun = tmpFile.read() 
    tmpFile.close() 
    tmpFile = open((fileOutput + ZeiPts), "w") 
    secondRun = firstRun.replace(textSourceJpg, ZeiJpg)
    tmpFile.write(secondRun.replace(textSourceTif, ZeiTif))
    tmpFile.close()
    filePrint = str(fileOutput + ZeiPts)
    print (filePrint + ":created and saved...")


#Hauptprogramm

allFiles()

[/me]

Ich fänds klasse, wenn Ihr mir ein paar Tipps gebt, was ich besser machen kann.

Eine ganz kleine Liste, da ich aktuell keine Zeit für Details habe:
- Namens- und Formatierungskonventionen von PEP-8 einhalten
- auf global() verzichten
- Beim Öffnen von Dateien mit with arbeiten
- überflüssige Klammern entfernen

[5inch]

so,
was macht das script,
es durchsucht eine ASCII.pts Datei nach:

"05.tif"
"05.jpg"

und schreibt 12 Version

von:
01.tif...12.tif
01.jpg...12.jpg

als 01.pts...12.pts

in 12 Dateien raus....

[5inch]

@:/me

wao, das ging schnell....

merci, gute tipps...

[5inch]

@:/me

wie krieg ich die "globals" weg,
da muss ich echt passen....

und warum solln die denn weg?

merci, 5inch

[EyDu]

Du solltest keine globalen Variablen verwenden, da du damit Abhängigkeiten zwischen vielen Komponenten schaffst. Abhängikeiten sind generell eine schlechte Idee, da immer mehr und mehr beachtet werden muss. Drehst du irgendwo an einer kleinen Schraube, dann ergeben sich oft Probleme in ganz anderen Bereichen. Übergebe daher Daten als Parameter in eine Funktion hinein und lass dir die Ergebnisse mit `return` zurückgeben.

Sebastian

[BlackJack]

@5inch: Neben der Schreibweise sind auch die Namen selbst nicht immer gut gewählt. Funktionen sollte, wie Methoden, Tätigkeiten beschreiben. Und Abkürzungen sollte man vermeiden wenn sie nicht allgemein bekannt sind. Ich rätsele zum Beispiel was der Präfix `Zei` wohl bedeuten mag.

Statt `str()` und `str.zfill()` würde man in Python eher Zeichenkettenformatierung mit dem ``%``-Operator oder der `format()`-Methode auf Zeichenketten verwenden.

Durchnummerierte Namen sind ein „code smell“, auch wenn es keine Ziffern sondern Zahlen als Worte sind. Das betrifft `firstRun` und `secondRun`. Die beschreiben nicht wirklich die Bedeutung der Objekte die daran gebunden sind. Den zweiten Namen kann man sich auch Sparen wenn man das Zwischenergebnis nicht an einen Namen bindet.

Der `str()`-Aufruf in der vorletzten Zeile von `replaceText()` ist überflüssig. Der Name `filePrint` erschliesst sich mir nicht. Und man hätte ihn auch wieder einsparen können, wenn man die Zeichenkette per Zeichenkettenformatierung erstellen würde, statt mit ``+``.

`str()` und ``+`` ist so BASIC, aber nicht wirklich schönes Python.

Zum Programmablauf selbst, scheinst Du die selbe Datei 12 mal zu öffnen und einzulesen, was mir 11 mal zu viel erscheint.

[5inch]

@BlackJack:

danke für Deine Tipps.
Ja wie schon erwähnt, ich arbeite dran besser zu werden... deshalb bin ja in dem Forum und nehme gerne Eure Hilfe in Anspruch.

zu Deinen Tipps, alles soweit verstanden, danke, tu mich noch schwer mit den Anwendungen,
aber da muss ich durch.

danke erst mal,

cheers 5inch