Verfasst: Donnerstag 27. November 2008, 10:22
Ich verstehe dich nicht: Du hast eine Datei 'a.jpg'. Diese Datei wird in zwei Ordnern gefunden. Was soll er jetzt machen? Die Datei nur einmal kopieren, also nur den ersten Fund von 'a.jpg'? Oder soll er beide 'a.jpg'-Dateien, die er gefunden hat, kopieren? Die erste als a.jpg und die zweite als a1.jpg?
Ich dachte letzteres
Ich rede jetzt über diesen Code. Nein, deine Interpretation der Abläufe ist nicht ganz korrekt.
Es ist noch früh am Morgen, aber ich glaube der Code macht so oder so nicht das, was er soll. Gesucht wird nach einer Datei in #42. Dort kriegst du _alle_ Treffer als Liste. Kommt a.jpg zwei Mal vor, sind da also zwei Einträge drin. In #48 wird über diese Treffer iteriert. So weit, so gut. Also, erster Treffer, wir gehen in #48 in die Schleife rein. Die Print-Ausgabe "Gefunden in:" ist so nicht ganz richtig, gefunden haben wir das schon länger (nämlich in der Funktion, die wir in #42 aufgerufen haben). #50 zerlegt den Dateinamen in Dateiendung und Rest. Und ab #51 ist das denke ich jetzt falsch. Denn jetzt wird für den _ersten_ Treffer hundert(+1) (0-99 und '') Mal etwas angehängt und jedes Mal kopiert. Das stimmt so nicht. Der erste Treffer des Dateinamens sollte ohne eine Änderung kopiert werden, an den zweiten Treffer des Dateinamens müssen wir logischerweise etwas anhängen.
Mein Versuch (bestimmt nicht perfekt: es ist nicht sichergestellt, dass die fotoliste wieder geschlossen wird). Ich habe einen anderen Ansatz gewählt um an weitere Treffer der selben Datei Anhänge zu basteln.
Ich dachte letzteres
Ich rede jetzt über diesen Code. Nein, deine Interpretation der Abläufe ist nicht ganz korrekt.
Es ist noch früh am Morgen, aber ich glaube der Code macht so oder so nicht das, was er soll. Gesucht wird nach einer Datei in #42. Dort kriegst du _alle_ Treffer als Liste. Kommt a.jpg zwei Mal vor, sind da also zwei Einträge drin. In #48 wird über diese Treffer iteriert. So weit, so gut. Also, erster Treffer, wir gehen in #48 in die Schleife rein. Die Print-Ausgabe "Gefunden in:" ist so nicht ganz richtig, gefunden haben wir das schon länger (nämlich in der Funktion, die wir in #42 aufgerufen haben). #50 zerlegt den Dateinamen in Dateiendung und Rest. Und ab #51 ist das denke ich jetzt falsch. Denn jetzt wird für den _ersten_ Treffer hundert(+1) (0-99 und '') Mal etwas angehängt und jedes Mal kopiert. Das stimmt so nicht. Der erste Treffer des Dateinamens sollte ohne eine Änderung kopiert werden, an den zweiten Treffer des Dateinamens müssen wir logischerweise etwas anhängen.
Mein Versuch (bestimmt nicht perfekt: es ist nicht sichergestellt, dass die fotoliste wieder geschlossen wird). Ich habe einen anderen Ansatz gewählt um an weitere Treffer der selben Datei Anhänge zu basteln.