Das deutsche Python-Forum

Guten Morgen Forum,


mein Problem ist, dass ein
nicht das gewünschte Ergebniss (nämlich vom offset bis EOF in den buffer zu schreiben) bewirkt.
Vielleicht mache ich einen erheblichen Denkfehler?

Liebe Grüße
Keks

EDIT:

Die Ausgabe des Scriptes. Ich erwarte eigentlich dass die Bufferlen immer um $bytes kleiner wird, was sie offensichtlich nicht macht!:

Mal ins Blaue geraten (hab kein Windows hier): Was passiert, wenn du die `calc.exe` im Modus `rb` (für Binärdatei) öffnest?

Danke snafu, dass war das Problem!

EDIT:

hier fehlte auch der binary mode:

try:
NFILE = open(nfilename,"wb")
print nfilename + " created!\n"
except all:

kekschaot hat geschrieben:Danke snafu, dass war das Problem!

Ich hätte da noch zwei Hinweise bezüglich der Namensgebung für Bezeichner.

Du solltest eigene Bezeichner nicht so nennen wie eingebaute Funktionen. file ist so ein Fall. Du überschreibst damit die eingebaute Funktion.

Bezeichner in Großbuchstaben stehen laut Konvention für Konstanten. NFILE wird aber nicht als Konstante verwendet.

Schau dir einfach mal den Style Guide for Python Code an.

@kekschaot: Noch ein paar Anmerkungen zum Programm:

Namen komplett in Grossbuchstaben sind per Konvention für Konstanten gedacht, `FILE` ist aber keine Konstante.

Das Argument `file` bei `split_file()` ist keine Datei sondern ein Dateiname, sollte deshalb auch besser `filename` heissen. Die Vorbelegung des Arguments mit einer ``calc.exe`` die im Hauptverzeichnis von Laufwerk C liegt, ist IMHO ein wenig *sehr* spezifisch.

Dateien sollte man mit der ``with``-Anweisung öffnen, dann muss man sich keine Gedanken darum machen, dass die Datei wieder geschlossen wird.

``except all:`` macht nicht das was Du denkst. `all()` ist eine Funktion. Die wird sicher nirgends als Ausnahme verwendet, also ist es sinnfrei damit ein ``except`` zu schreiben. Der Fall wird einfach niemals eintreten.

Funktionsnamen einfach nur hin zu schreiben reicht nicht wenn man sie auch *aufgerufen* haben möchte. Siehe ``exit`` im ersten sinnfreien ``except all:``-Zweig.

``while FILE:`` ist irreführend, weil ein Dateiobjekt in dem Kontext *immer* wahr ist. Dann sollte man aber auch direkt ``while True:`` schreiben um den Leser nicht unnötig zu verwirren.

``+`` und `str()` um Zeichenketten zusammen zu setzen ist eher BASIC als Python. Bei ``print`` kann man sich das sparen wenn man mehrere Ausdrücke durch Kommas getrennt angibt, und sonst ist eher Zeichenkettenformatierung mittels ``%``-Operator oder `format()`-Methode auf Zeichenketten üblich.

Das zweite `FILE.seek()` ist unnötig, denn danach wird auf die Datei nicht mehr zugegriffen bevor am Anfang des nächsten Schleifendurchlaufs *nochmal* die Methode mit einem anderen Versatz aufgerufen wird.

`quit()` sollte man nicht verwenden. Die Funktion existiert für die interaktive Shell sollte aber nicht in Programmen verwendet werden, da das vorhandensein nicht garantiert ist. Die Dokumentation dazu:

http://docs.python.org/library/constants.html#constants-added-by-the-site-module hat geschrieben:The site module (which is imported automatically during startup, except if the -S command-line option is given) adds several constants to the built-in namespace. They are useful for the interactive interpreter shell and should not be used in programs.

Ansonsten gibt es auf einer höheren Ebene zwei Punkte:

1. Die Quelldatei wird unnötig oft gelesen. Beim ersten Lesen wird doch schon die komplette Datei in den Speicher gelesen — warum nicht von den Daten dann immer kleiner werdende Teile speichern?

2. Das Programm funktioniert nur so lange vernünftig, wie die komplette Datei in den Arbeitsspeicher passt.