Problem mit .zip-Sicherungsdateien

[Cucum]

Hi leute

Ich habe vor kurzem angefangen mich in Python einzuarbeiten...Hab ansonsten noch keine Programmiererfahrung und bin jetzt dabei ein erstes Programm zu schreiben.(nach Anleitung von dem Buch bytesofpython, wenn das jmd kennt...)

Aufgabe war es nun ein Programm zu schreiben, mit dessen Hilfe bestimmte Dateien als ZIP-Datei gesichert werden sollten...
Die Lösungen stehen in dem Buch und nachdems bei mir nicht geklappt hat habe ich es einfach abgetippt (darum die ganzen Kommentare darin^^)...Nur leider funktioniert es immernoch nicht .Vielleicht weiß ja jmd was falsch ist^^
Hier ist das Programm:

Code: Alles auswählen

import os
import time

#1. Die Dateien und Verzeichnisse, die gesichert werden sollen werden in
#der folgenden Liste angegeben

quellen = ['G:\\Schule', 'G:\\Sicherheit']

#2. Die Sicherung muss in das folgende Verzeichnis gespeichert werden:
ziel_verzeichnis = 'C:\Users\Tamara\Desktop\Sicherungen'

#3. Die Dateien werden in einer ZIP-Datei gesichert.

#4. Der Name der ZIP-Datei setzt sich aus dem aktuellen Datum
# und der Uhrzeit wie folgt zusammen:
ziel = ziel_verzeichnis + time.strftime('%Y%m%d%H%M%S') + '.zip'

#5.
zip_befehl = 'zip -qr %s %s' % (ziel, '  '.join(quellen))

#Sicherung starten
if os.system(zip_befehl) == 0:
    print 'Erfolgreiche Sicherung nach', ziel
else:
    print 'Sicherung fehlgeschlagen'

...und dann kommt immer "Sicherung fehlgeschlagen"
...achja ich benutze Windows und hab für zip-Dateien "7-zip" drauf

Danke schonmal!
lg
cucum

[Hyperion]

- Du willst subprocess.Popen benutzen!;-)
- Pfade stets als raw-String angeben: r"G:\Schule" (oder sogar so: r"G:/Schule"), damit vermeidest Du das Escapen wo evtl. auch der Fehler bei Dir liegt (der Pfad nach der Liste!)
- Pfade setzt man mit os.path.join() zusammen

Das so auf die Schnelle.

Welches Buch nutzt Du denn?

Es gibt in der Standard-Lib doch Funktionen fürs zippen. Wieso da ein externes Programm bemühen? Da würde ich am Anfang doch zur Übung auf die eingebauten Dinge zurückgreifen.

[BlackJack]

@Cucum: Ist das `zip`-Programm denn überhaupt installiert und im %PATH% von Windows erreichbar!?

[Cucum]

danke erstma für die antwort
Ich hab bytesofpython gedownloaded und da ich im Moment noch nich grad viel kann bei python dacht ich, ich mach erstmal was da steht und experimentier danach rum^^
was ist subprocess.Popen?...vlt sollt ichs wissen wenn ichs benutzt hab aber ich weiß es jedenfalls nicht xD
naja das mit den Pfaden hab ich geändert:

Code: Alles auswählen

import os
import time

#1. Die Dateien und Verzeichnisse, die gesichert werden sollen werden in
#der folgenden Liste angegeben

quellen = [r'G:/Schule', r'G:/Sicherheit']

#2. Die Sicherung muss in das folgende Verzeichnis gespeichert werden:
ziel_verzeichnis = r'C:/Users/Tamara/Desktop/Sicherungen'

#3. Die Dateien werden in einer ZIP-Datei gesichert.

#4. Der Name der ZIP-Datei setzt sich aus dem aktuellen Datum
# und der Uhrzeit wie folgt zusammen:
ziel = ziel_verzeichnis + time.strftime('%Y%m%d%H%M%S') + '.zip'

#5.
zip_befehl = 'zip -qr %s %s' % (ziel, ' '.join(quellen))

#Sicherung starten
if os.system(zip_befehl) == 0:
    print 'Erfolgreiche Sicherung nach' , ziel
else:
    print 'Sicherung fehlgeschlagen'

...aber jetzt zeigt er nichmal an "Sicherung fehlgeschlagen" sondern "invalid syntax" xD

[Cucum]

@BlackJack

ja es ist installiert...

[ms4py]

Statt os.system solltest du subprocess.POpen nutzen.
Genaueres steht in der Modulbeschreibung:
http://docs.python.org/library/subprocess.html

In der Dokumentation findet sich auch das Kapitel:
"Data Compression and Archiving" Vielleicht willst du ja auch davon was nutzen und nicht das externe zip-Programm.
http://docs.python.org/library/archiving.html

Verwende os.path.join für die Pfadangaben.

Diese Dinge wurden alle schon mal erwähnt und von dir ignoriert?!

Dann weiter:
Wenn du einen Syntax-Fehler bekommst, solltest du vielleicht sagen WO dieser auftritt.

Zu jeder Zeile ein Kommentar ist sehr schlechter Stil (auf jeden Fall in einem Python-Skript, bei Assembler sieht es da anders aus ). Ist doch logisch was z.B. in einem Backup-Skript die Variable ``ziel_verzeichnis`` darstellen soll...

[Cucum]

danke
jaa ich weiß das das mit den kommentaren nicht gerade sinnvoll ist...hab ich aber oben auch geschrieben^^...abgetippt aus dem buch der vollständigkeit halber nachdem ichs nicht hinbekommen hab...