search

hhu-propra1 / time-tracker

Track your study time!
MIT License
9 stars 5 forks source link

--add akzeptiert auch falschen Input #35

Closed sakla106 closed 2 years ago

sakla106 commented 2 years ago

Describe the bug Man kann bei --add jeden String als Datum wählen und er wird so im Event abgespeichert. Außerdem wird ein nicht-Zahlen String als Argument nicht abgefangen.

To Reproduce Steps to reproduce the behavior: java -jar tt.jar --add hallo,90,Test,"Test" für das Datum java -jar tt.jar --add today,drei,Test,"Test" für das Argument

Expected behavior Ich habe eigentlich erwartet, dass bei ungültigem Datum (und ungültigem Datumsformat) und bei einem String, der keine gültige Zahl ist, ein ERROR Status zurückgegeben wird.

Screenshots Dieser Screenshot zeigt die Ausgabe der Tabelle mit ungültigen Daten. bug

dragonempire96 commented 2 years ago

java -jar tt.jar --add today,drei,Test,"Test" wirft bei mir direkt ne exception

sakla106 commented 2 years ago

java -jar tt.jar --add today,drei,Test,"Test" wirft bei mir direkt ne exception

Ja genau. Aber soll das Programm denn ne Exception werfen, die ein normaler User möglicherweise nicht versteht, oder lieber eine handgemachte Fehlermeldung à la "Das ist keine Zahl"?

Also ernstgemeinte Frage. Wenn die Exception ok ist, dann muss man da auch nichts dran machen :D

dragonempire96 commented 2 years ago

die kann man ja abfangen in dem das, wie mit dem Datum, via try catch abfängt