Quote:
Originally Posted by xCPx
Warum bin ich mir bei dem py script nur 1000% sicher das es aus dem arius2 Clienten ist ?
Alsodas ist was ich damals geschrieben hab ?
|
Dann eben die Kritik an dir:
Das Script ist sehr schlecht geschrieben und widerspricht jedem Styleguide den du zu Python finden wirst.
Mehrfache if bzw. elif Abfragen sind unnötige Prozeduren, die das ganze zwar nicht merkbar inperformanter machen, nichtsdestotrotz ist die Gesamtperformance von dem Script verdammt schlecht und unnötige Speichervariablen werden verwendet.
Ich würde hierbei mehr auf ein Dict zurückgreifen und auch die "if name.find" garantiert rausnehmen, sie stören, machen den Code unübersichtlich und werden nicht empfohlen.
Mal ein Codebeispiel das ich dazu geschrieben habe, was auch wesentlich schneller abläuft, muss halt noch minimal abgeändert werden, da ich es nur lokal geschrieben habe:
PHP Code:
import re
name = "[Dev]BaumKuchen"
legendDict = {
"[Dev]": "123%s321",
"[SA]": "321%s123"
}
praefix = None
try:
praefix = "[%s]" % (re.findall("\[(.*)\].*", name)[0])
except (IndexError):
pass
if praefix in legendDict:
print(legendDict[praefix] % ("Baum"))
einen guten Abend noch
Edit:
@[Patrick]:
Vielleicht verwirrt das restliche Code, aber nur noch eintragen zu müssen "[GM]": "Whatever%s", ist denke ich mindestens genauso übersichtlich wie eine komplette if/elif Abfrage.