global variable in for Schleife

[DasIch]

Hm... dann müßte "file" aber fast immer überschrieben werden.

``file`` sollte man sowieso nicht benutzen es sei den man erbt davon.

Welche builtins überschreibst du denn üblicherweise?

``id`` und ``file`` als einfache Namen. ``all`` überschreibe ich manchmal mit einer Funktion die False zurück gibt wenn das iterable leer ist, wenn ich mich dabei erwische mehrmals ``all(foo) and foo`` zu schreiben.

EDIT: Fehler korrigiert, danke BlackJack

[EyDu]

Welche builtins überschreibst du denn üblicherweise?

``id`` und ``file`` als einfache Namen. ``all`` überschreibe ich manchmal mit einer Funktion die False zurück gibt wenn das iterable leer ist, wenn ich mich dabei erwische mehrmals ``all(foo) or foo`` zu schreiben.

"file" finde ich einen äußerst unglücklichen Namen aus. Dieser sagt genau nichts aus. Da würde ich lieber einen Namen wählen, welcher Aussagt, was in der Datei drin ist, oder was rein soll.
"all" mit einem anderen Verhalten zu überschreiben ist auch sehr unglücklich. Ich würde nicht erwarten, dass "all" auf einmal bei leeren Listen ein anderes Ergebnis liefert.

[BlackJack]

@DasIch: Bist Du sicher, dass Du nicht ``foo and all(foo)`` meintest!? Denn ``all(foo) or foo`` ergibt bei leerem `foo` auch `True`.

[DasIch]

"file" finde ich einen äußerst unglücklichen Namen aus. Dieser sagt genau nichts aus. Da würde ich lieber einen Namen wählen, welcher Aussagt, was in der Datei drin ist, oder was rein soll.

Wenn ich in ``Config.__init__`` eine Datei öffne und parse muss ich die nicht auch noch ``config_file`` nennen, dass ist redundant.

@Blackjack: Jetzt nicht mehr

[sma]

Um mal nach all diesem Gerede über Namen auf die eigentliche Frage zurückzukommen:

Designfehler?

Ja.

Das ganze sollte so aussehen:

Code: Alles auswählen

hauptspeise, brutto, netto = lies_hauptspeise("whatever")
...
def lies_hauptspeise(whatever):
    ...
    return row.hauptspeise, row.brutto, row.netto

Noch besser wäre, ein Objekt mit passenden Attribute zu benutzen.

Stefan

[str1442]

Für Dateiobjekte benutze ich (sofern durch den Kontext ersichtlich ist, was für ein Dateiobjekt das ist) "fp". Nicht nur fand ich das öfters in anderem Python Code, aber auch in C benutzt man diesen Namen wohl oft - das macht ihn recht eindeutig. Steht für "file pointer".