search

VPatrick / PerlArchiver

Perl-Anwendung zur Archivierung von Unterbäumen in Dateisystemen
0 stars 0 forks source link

Delete #10

Closed VPatrick closed 9 years ago

VPatrick commented 9 years ago

Hi, kannst du in deiner Klasse Delete die Methode verbose(...) aus der Klasse Verbosity verwenden? Dadurch können wir verbose vereinheitlichen. Danke!

KasikciMu commented 9 years ago

@mia8287 Die Klasse Delete funktioniert auch auf meinem PC perfekt nur hätte ich eine Bitte bei der Frage ob man wirklich die Löschen will steht "file" obwohl ich ein ganzes Archiv lösche. Könntest du das bitte ändern. Ist nur ein Schönheitsfleck, der mir so aufgefallen ist :D

Ramunno commented 9 years ago

Hi hab gesehen das das alle Methoden bei dir public sind! Soll das so sein?

Ramunno commented 9 years ago

Destruktor fehlt und Ressourcen werden nicht freigeben!

Ramunno commented 9 years ago

Haben sich deine Methoden geändert? Brauche die aktuellen für das Klassendiagramm!

Ramunno commented 9 years ago

Deine Methoden sind nicht sehr einheitlich vom Namen! Einmal schreibest du Link und einmal Lnk!

mia8287 commented 9 years ago

Destruktor eingefügt und Lnk durch Link in den Methoden ersetzt

Ramunno commented 9 years ago

Delete funktioniert nicht richtig. Wenn ich das letzte Archive (neuste) lösche, dann werden zwar die Dateien in das vorhergehenden kopiert, allerdings werden die links in den anderen Archiven davor nicht aktualisiert.

Ramunno commented 9 years ago

Wenn eine Archiv gelöscht wird das eine Datei Änderungen zum Nachfolger hat, dann werden die Links in den Vorgänger Archiven nicht aktualisiert!

Ramunno commented 9 years ago

Ist es für dich überhaupt wichtig wie das Archiv wirklich heißt heißt? Du bekommst doch nur einen Archivpfad übergeben.

KasikciMu commented 9 years ago

@mia8287 Kannst du dein Delete so anpassen, dass falls es das letzte Archiv ist der Eintrag in der Hashtabelle gelöscht wird.

mia8287 commented 9 years ago

Erledigt