Tja, das ist jetzt die Aufgabe, das funktionierend zu lösen ;-)

Ist das immer noch Spagetti-Code? def finddoublefiles(): run_again = False for filetype in allfiles: for to_check_file in allfiles[filetype]: for file in allfiles[filetype]: if not to_check_file == file and to_check_file[0] == file[0]: if filecmp.cmp(to_check_file[1], file[1]): doublefiles.append(f...

Wie so ist für ein OS *.jpg nicht gleich *.JPG? Bei meiner Webseite war das auch so, der Link zu einer Datei war richtig geschrieben und nur wegen der Endung wurde das Bild nicht geladen. Ich wollte zwar eh es so umschreiben das z. B. *.jpg und *.jpeg gleich verarbeitet werden. Weil nicht alle Betr...

Py-Prog hat geschrieben:Ich weiß nicht was dass bringen soll, was ist falsch an den globalen Variablen.

Ist einfach schlechter Stil. Produziert Spagehetticode. Die Forensuche ist dein Freund :-)

snoer hat geschrieben:weisst du wie das datenbankintern so abläuft?

was?

einfacher ist ja nicht immer pauschal besser :)

beim Programmieren fast immer.

Also ich würde da schon den Ansatz mit dem Status verwenden. Ist viel einfacher und unkomplizierter als Daten-Rumschieben zwischen Tabellen.

Das hört sich alles ziemlich arg komisch an... wieso willst du das überhaupt tun? Ich bin mir sicher, dass es einen eleganteren Lösungsansatz für dein Problem gibt :-)

Und was ist hiermit? Schau dir außerdem `collections.defaultdict` an. `finddoublefiles`: Viel zu viele Einrückungsebenen. Versuche, den Code in mehrere Teilfunktionen aufzuteilen und/oder Einrückungstiefe durch geschicktes Verwenden von `break` und `continue` zu reduzieren. Außerdem kannst du anstat...

Probier's doch einfach aus?! `PyKey` darf nicht gelöscht werden, weil `PyList_GetItem` den Refcount NICHT erhöht.

>>> # Measure some strings: ... a = ['cat', 'window', 'defenestrate'] >>> for x in a: ... print x, len(x) ... cat 3 window 6 defenestrate 12 Ist x denn keine Variable die vorher definiert werden muss? nö. in Python gibt's sowas nicht. Du musst niemals Variablen deklarieren, bevor du ihnen eine Wert...

Darii hat geschrieben:Bedenke aber auch den Fall, dass der Interpreter crashed und finally dann nicht ausgeführt wird.

Jagut, soo wichtig ist das Aufräumen dann auch nicht. ;-)

nö.

`scandirectory`: Das `global` ist unnötig. Du solltest `os.walk` verwenden. Schau dir außerdem `collections.defaultdict` an. `finddoublefiles`: Viel zu viele Einrückungsebenen. Versuche, den Code in mehrere Teilfunktionen aufzuteilen und/oder Einrückungstiefe durch geschicktes Verwenden von `break` ...

@Dauerbaustelle: Ich kenne mich im Detail mit dem Code nicht aus, aber wo wird denn diese temporäre Collection erzeugt? In der Generatorfunktion? Dann wird sie dort doch nur erzeugt wenn Du das erste Element von dem Generator holst. Davor wird von der Funktion *nichts ausgeführt*! Ups. Peinlicher D...

Warum willst du etwas aufräumen, wenn du noch gar nichts getan hast? Extra für dich hol' ich ein wenig weiter aus. :-P Der Generator iteriert über das Resultat einer Map/Reduce-Operation auf MongoDB, das in einer temporären Collection (das MongoDB-Äquivalent zu Tabellen, falls du dich damit nicht a...

Ich verstehe das Problem nicht. Dieses Beispiel zeigt "1", "2" und "Cleanup": def g(): yield 1 yield 2 print "Cleanup" for i in g(): print i def g(): yield 1 yield 2 print "Cleanup" g() zeigt aber kein "Cleanup". Ich suche eine Möglichkeit...

Tach! Mit try: ... yield ... finally: aufräum() kann man ja Aufräumarbeiten machen, wenn der Generator zu Ende ist, garbage-collected wird oder eine Exception auftrat. Wird der aus dem Funktionsaufruf resultierende Generator aber nie angefasst, wird der `finally`-Block natürlich nicht ausgeführt -- ...

@Dauerbaustelle: Natürlich können in der Praxis auch zwei Dateien mit unterschiedlichem Inhalt den gleichen Hash-Wert ergeben. Wenn man sicher gehen will, muss man die Dateien dann auch tatsächlich vergleichen und kann einen Hash-Wert auch nur zum Ausschliessen von Vergleichskandidaten verwenden. Ä...

1) ja *.jpg und *.txt könnten den Gleichen inhalt haben, ... (denk einfach mal nach wieso) Warum vergleichst du dann nur Dateien der gleichen Endung? 2)Wie willst du denn wissen welche Dateien gleich sind oder nicht Ich nehme mal an, dass du von dem Größenvergleich sprichst? Wenn die beide untersch...

Warum sortierst du das nach Dateiendung? "foo.jpg" und "bar.txt" können ja auch den selben Inhalt haben :-) Außerdem würde es Sinn machen, erstmal die Dateigrößen zu vergleichen, weil man dazu die Dateien nicht auslesen muss. Du könntest auch, anstatt Dateiinhalte zu vergleichen,...