Keine Fehlermeldung, falls der Parameter kein gültiges Verzeichnis darstellt

Question

Keine Fehlermeldung, falls der Parameter kein gültiges Verzeichnis darstellt

Closed this issue 8 years ago · 12 comments

Übergibt man dem Programm einen Parameter, der kein gültiges Verzeichnis darstellt (so wie ich gerade z.B. "C;\test" (Semikolon statt Doppelpunkt), dann gibt es eine leere Ergebnismenge ohne Hinweis auf die Falscheingabe.

Answer 1 · 2017-01-13T20:10:41.000Z

Wenn das Programm ohne Pfadangabe startet, würde ich direkt zu Maske mit der Pfadeingabe wechseln.
Mit Pfadangabe sollte der Suchvorgang direkt starten, hierbei muss er spätestens vom DuplicateLengthFinder sauber validiert werden.
Ich schaue mir das mal an.

Answer 2 · 2017-01-14T13:27:11.000Z

Die Fehlermeldungen aus dem DuplicateLengthFinder sollten jetzt helfen, das Programm korrekt zu benutzen. @niklaspolke bitte reviewen, ob der Punkt schon gelöst ist.

Answer 3 · 2017-01-15T13:52:47.000Z

Aus meiner Sicht ausreichend gelöst.

Answer 4 · 2017-01-15T15:25:03.000Z

Ich finde einen Stacktrace zwar nicht schön als Fehlermeldung für den Anwender, aber das genaue Problem ist ja enthalten:

Exception in thread "main" java.lang.IllegalArgumentException: Not existing path: asdf

Aus meiner Sicht ausreichend gelöst.

Answer 5 · 2017-01-15T15:48:07.000Z

Beim mir erscheint es vernünftig ohne Stacktrace

huluvu424242@stachel:~/git/DupFinder$ java -jar target/DupFinder-0.0.1-SNAPSHOT.jar 
FEHLER: Parameter <Verzeichnis> fehlt
 usage: DupFinder <Verzeichnis>
<Verzeichnis> = Verzeichnis in dem rekursiv nach Duplikaten gesucht wird
huluvu424242@stachel:~/git/DupFinder$

@niklaspolke Wie rufst Du das Programm auf? Hast die aktuellste Version?

Answer 6 · 2017-01-15T15:51:15.000Z

Ah ok, als Parameter einen Folder genutzt den es nicht gibt. Ja das könnte schöner sein ;)

huluvu424242@stachel:~/git/DupFinder$ java -jar target/DupFinder-0.0.1-SNAPSHOT.jar 
FEHLER: Parameter <Verzeichnis> fehlt
 usage: DupFinder <Verzeichnis>
<Verzeichnis> = Verzeichnis in dem rekursiv nach Duplikaten gesucht wird
huluvu424242@stachel:~/git/DupFinder$ java -jar target/DupFinder-0.0.1-SNAPSHOT.jar asdd
Exception in thread "main" java.lang.IllegalArgumentException: Not existing path: /home/huluvu424242/git/DupFinder/asdd
	at de.b0n.dir.processor.DuplicateLengthFinder.<init>(DuplicateLengthFinder.java:33)
	at de.b0n.dir.processor.DuplicateLengthFinder.getResult(DuplicateLengthFinder.java:169)
	at de.b0n.dir.processor.DuplicateLengthFinder.getResult(DuplicateLengthFinder.java:148)
	at de.b0n.dir.DupFinder.main(DupFinder.java:41)
huluvu424242@stachel:~/git/DupFinder$

Answer 7 · 2017-01-15T15:51:36.000Z

@FunThomas424242 Meine Fehlermeldung kommt nur, wenn Du ein Verzeichnis eingibst, welches aber nicht existiert (wie in meinem Beispiel der String asdf)

Answer 8 · 2017-01-15T16:07:15.000Z

Aus meiner Sicht kann mit den bisherigen Korrekturen die Main-Klasse implementiert werden, um sinnvoll auf die Parameter zu reagieren.
Ich überlege mir dazu etwas.

Answer 9 · 2017-01-15T16:28:33.000Z

Hab da mal einen Vorschlag erarbeitet. Recht einfach aber evtl. ausreichend.
siehe #13

Answer 10 · 2017-01-18T06:40:21.000Z

Ach herje ich hab das @ignore noch im DupFinderTest vergessen.
@mkymikky Bitte mal rausnehmen, dann sollte die Coverage steigen.
Also das ganz oben an der Klasse.

Answer 11 · 2017-01-18T19:10:00.000Z

So fertig - jetzt kann gemergelt werden ;)
siehe #14

Answer 12 · 2017-01-19T18:48:15.000Z

@mkymikky Jippi kann geschlossen werden.