Vielleicht möchte er nur die Götter beschwören, dass sie doch endlich wieder Ladung aus großen Vögeln abwerfendeets hat geschrieben:Und das while True ist warum da?
Und warum glaubst du, dass das gut sein soll? Hier ist die Schleife doch offensichtlich total unnötig.Nobuddy hat geschrieben:Vielleicht habe ich da mal was falsch verstanden. Dachte es wäre immer gut, eine bestimmte Situation mit break zu beenden. Ist das falsch?
Man benutzt break, wenn man es braucht, um aus einer Schleife auszubrechen. Und eine Schleife braucht man, wenn man Code mehr als einmal laufen lassen will. Brauchst du in deinem gezeigten Code eine Schleife? Nein. Also brauchst du auch kein break. Du kannst natuerlich auch so progammieren:Nobuddy hat geschrieben:Vielleicht habe ich da mal was falsch verstanden. Dachte es wäre immer gut, eine bestimmte Situation mit break zu beenden. Ist das falsch?deets hat geschrieben:Und das while True ist warum da? Wo du doch eh dauernd raus-breakst? Und warum printed eine solche Funktion eine Fehlermeldung anstatt eine Exception zu schmeissen, mit der du im umgebenden Code etwas anfangen kannst, statt einfach so zu tun, als ob alles ok gewesen waere?
Das mit der Exception, ist eine gute Idee, werde das ändern.
Code: Alles auswählen
while True:
while True:
while True:
print "einmal ausgefuehrt"
break
break
break
Code: Alles auswählen
def sysprog(filename):
# Betriebssytem ermitteln und Datei mit Standardprogramm öffnen.
linux = sys.platform.startswith('lin')
osx = sys.platform.startswith('os')
windows = sys.platform.startswith('win')
text = ('Datei %s wird mit dem Standardprogramm\nIhres Betriebsystem zum Editieren geöffnet!' % filename)
if linux:
standardedit = 'xdg-open'
print(text)
elif osx:
standardedit = 'open'
print(text)
elif windows:
# standardedit = 'os.startfile'
standardedit = 'start'
print(text)
try:
process = subprocess.Popen([standardedit, filename])
process.wait()
except UnboundLocalError:
print('Sie arbeiten mit dem Betriebssystem %s.' % sys.platform)
print('Das Standardprogramm zum Starten von %s, ist nicht bekannt.\n' % filename)
print('Öffnen Sie die Datei %s zum Editieren mit einem Texteditor.' % filename)
print('Die Datei %s befindet sich unter\n%s.' % (filename, os.path.realpath(filename)))
).
Code: Alles auswählen
def sysprog():
# Betriebssytem ermitteln und Datei mit Standardprogramm öffnen.
linux = sys.platform.startswith('lin')
osx = sys.platform.startswith('os')
windows = sys.platform.startswith('win')
if linux:
standardedit = 'xdg-open'
elif osx:
standardedit = 'open'
elif windows:
standardedit = 'start'
else:
raise RuntimeError("Konnte Betriebsystem nicht herraus finden")
return standardedit
def fileopen(filename):
try:
default_program = sysprog()
except RuntimeError, e:
print(e)
else:
print('Datei %s wird mit dem Standardprogramm\nIhres Betriebsystem zum Editieren geöffnet!' % filename)
process = subprocess.Popen([default_program, filename])
process.wait()Code: Alles auswählen
except RuntimeError, e:Code: Alles auswählen
except RuntimeError as e:Code: Alles auswählen
if os.platform.startswith('win'):
startcmd = 'start' # standardedit trifft hier ja nicht mehr zu…
elif os.platform.startswith('darwin'): # und nicht 'os' ;)
startcmd = 'open'
else:
startcmd = 'xdg-open'
Code: Alles auswählen
def sysprog():
# Betriebssytem ermitteln und Datei mit Standardprogramm öffnen.
if sys.platform.startswith('win'):
startcmd = 'start'
elif sys.platform.startswith('darwin'):
startcmd = 'open'
else:
try:
startcmd = 'xdg-open'
except:
raise RuntimeError("Konnte Betriebsystem nicht herraus finden")
return startcmdCode: Alles auswählen
def sysprog():
# Betriebssytem ermitteln und Datei mit Standardprogramm öffnen.
if sys.platform.startswith('lin'):
startcmd = 'xdg-open'
elif sys.platform.startswith('darwin'):
startcmd = 'open'
elif sys.platform.startswith('win'):
startcmd = 'start'
else:
raise RuntimeError("Konnte Betriebsystem nicht herraus finden")
return startcmdCode: Alles auswählen
class UnsupportedOSError(Exception): pass
def sysprog():
...
else:
raise UnsupportedOSError("Konnte ...")Code: Alles auswählen
def sysprog():
"""
BENUTZE DOC-STRINGS ZUR BESCHREIBUNG VON FUNKTIONEN
Betriebssytem ermitteln und Datei mit Standardprogramm öffnen.
"""
if sys.platform.startswith('lin'):
return 'xdg-open'
elif sys.platform.startswith('darwin'):
return 'open'
elif sys.platform.startswith('win'):
return 'start'
raise UnsupportedOSError("Konnte Betriebsystem nicht herraus finden")Code: Alles auswählen
def get_starter():
try:
return {'Windows': 'start',
'Linux': 'xdg-open',
'Darwin': 'open'
}[platform.system()]
except KeyError:
raise RuntimeError(
'Passender Starter konnte nicht ermittelt werden')
+1snafu hat geschrieben: In Python tendiert man aber generell eher zu Wörterbüchern als zu einem Haufen von `if`/`elif`-Konstrukten. Wobei sich die Anhäufung hier natürlich noch in Grenzen hält.
Code: Alles auswählen
linux = sys.platform.startswith('lin')
osx = sys.platform.startswith('os')
windows = sys.platform.startswith('win')
if linux:
standardedit = 'xdg-open'
elif osx:
standardedit = 'open'
elif windows:
standardedit = 'start'
Das wäre dann wohl 'UnsupportedOSError'. Da muß ich mich aber noch mehr damit auseinandersetzen.snafu hat geschrieben:Früher oder später würde ich anstatt `RuntimeError` wohl eine eigene Exception bevorzugen (sprach EyDu ja schon an). Der neue Exception-Typ kann aber IMHO durchaus von `RuntimeError` abgeleitet werden.
Code: Alles auswählen
import sys
CMD_INFO = {
'win': 'start',
'linux': 'xdg-open',
'darwin': 'open' # Mac OS X
}
def get_cmd():
for indetifier, cmd in CMD_INFO.items():
if sys.platform.startswith(indetifier):
return cmd
raise UnsupportedOSError("Konnte Betriebsystem nicht herraus finden")