Sample-Sourcen ?

[routeconverter]

Das wäre eine Variante. Allerdings würde das ein wenig den Test einschränken. In dem Test sollen eigentlich mögliche Eingaben vom Benutzer geprüft werden, auch wenn sie nicht 100%ig der Formatierung entsprechen. D.h. z.B. fehlende Nullen.
Wenn ich das erst im Test reformatiere, dann habe ich das im Testcode gemacht, aber ob das der Laufzeitcode korrekt macht, wird dann nicht geprüft.

Das verstehe ich nicht so ganz. Es ging doch darum im Test die für die Zeitzone UTC ein DateTime Format zu bekommen, oder?

Genau. Ich habe das jetzt mal etwas angepasst, so dass der Aufruf und die Prüfung nur noch intern wäre (ähnlich zu den Zeitzonen).
Was denkst du dazu ?

Das ist erstmal gut genug, um weiterzumachen.

[lundefugl]

Hallo Christian

Das verstehe ich nicht so ganz. Es ging doch darum im Test die für die Zeitzone UTC ein DateTime Format zu bekommen, oder?

Der Test testet die Edit-Logik vom Tablemodell. D.h. die Strings sind die simulierten Eingaben des Benutzers bzw. die Callbacks von Swing dazu.
Und der Test soll prüfen, ob das korrekt geparsed und dann korrekt unter Berücksichtigung der Eingabezeitzone auf der zugrundeliegenden Position gesetzt wird. Und dabei gibt es dann auch die Sonderfälle, dass man nur das Datum oder nur die Zeit alleine editiert.

Und damit soll der Test die Parser-Logik davon auch testen. Und da der Benutzer auch Eingaben machen kann, die korrekt sind, aber sich nicht 100%ig an die Vorgaben halten, wollte ich das mit testen.

Aber eigentlich war das sogar ein Nebenkriegsschauplatz, der mir nur wieder aufgefallen ist (auch wenn er für die meisten Anwender den größten Vorteil bringt). Den Pullrequest zum Flightradar24-Format und die Samples dazu (https://forum.routeconverter.com/attachm...p?aid=2870) hast du noch auf dem Schirm ?

Gruß
Thomas

[lundefugl]

Hallo Christian

da man scheinbar auf Github keine direkte Message an dich schreiben kann, der weg hier rüber.
Ich habe zwar eine Mail von Github bekommen, dass du da einen Kommentar gemacht hast, aber den sehe ich jetzt irgendwie nicht. Hast du den wieder gelöscht oder gibt es dazu noch Klärungsbedarf ?

Gruß
Thomas

[routeconverter]

Aber eigentlich war das sogar ein Nebenkriegsschauplatz, der mir nur wieder aufgefallen ist (auch wenn er für die meisten Anwender den größten Vorteil bringt). Den Pullrequest zum Flightradar24-Format und die Samples dazu (https://forum.routeconverter.com/attachm...p?aid=2870) hast du noch auf dem Schirm ?

Ja, den habe ich angeschaut und Kommentare hinzugefügt.

[routeconverter]

Der Kern ist: ich finde, Flightradar24-spezifische Logik sollte nicht in der CsvPosition enthalten sein, sondern in einem FlightRadar24Format so wie ExcelFormat oder CsvFormat. Eine Subklasse von CsvFormat wäre gut und dann sind eventuell ein paar kleine Änderungen an CsvPosition oder CsvFormat nötig.

Leider habe ich Deine Testdateien nicht, um zu verstehen, was bei dem Format anders ist.

Vielleicht sendest Du mir die und reduzierst den Pull Request erstmal auf den Test?

[lundefugl]

Hallo Christian

Leider habe ich Deine Testdateien nicht, um zu verstehen, was bei dem Format anders ist.

eigentlich hatte ich die Samples hier im Chat gepostet. Klappt der Link bei dir nicht: https://forum.routeconverter.com/attachm...p?aid=2870 ?

Aus irgendeinem Grund sehe ich deine Kommentare auf Github nicht. Weisst du, ob man da was spezielles für machen muss ? Angemeldet bin ich ja.
Mit Github fehlt mir etwas die Erfahrung, da ich da nur selten Pullrequests stelle.

Bezüglich der Integration selbst war ich auch nicht ganz happy. Ich schaue am Wochenende nochmals, ob mir da noch was besseres zu einfällt. Ich wollte die ganze Parserlogik vom CSV-Format eigentlich nicht dublizieren. Aber evtl. kann man da was über eine Art Specializer machen, um ein Subformat davon zu erstellen.

Da waren auch 2 Änderungen mit drin, die auch das allgemeine Format betrafen. Die kann ich mal versuchen als einen eigenen Pullrequest zu machen und auch einen Unittest dazu. Dann verstehst du evtl. besser, was das soll und wo aktuell das Problem ist.

Evtl. siehst du auch meine beiden Kommentare nicht (wie ich deine nicht sehe), die ich am Anfang schon mit dazu gemacht habe.
   

Gruß
Thomas

[routeconverter]

eigentlich hatte ich die Samples hier im Chat gepostet. Klappt der Link bei dir nicht: https://forum.routeconverter.com/attachm...p?aid=2870 ?

Die habe ich übersehen, aber gerade heruntergeladen.

Aber evtl. kann man da was über eine Art Specializer machen, um ein Subformat davon zu erstellen.

Das ist mein Vorgehen bei anderen Formaten

Evtl. siehst du auch meine beiden Kommentare nicht (wie ich deine nicht sehe), die ich am Anfang schon mit dazu gemacht habe.

Doch, ich sehe Deine Kommentare. Habe aber die Intention nicht verstanden.

[routeconverter]

Ich versuche mal, ein Flightradar24Format zu bauen.

https://github.com/cpesch/RouteConverter...690b13dbbb

[routeconverter]

Bitte probiere mal den master Branch aus. Ich habe jetzt mal die Annahme gemacht, dass das Format beim Schreiben mit genau den Spalten in genau der Reihenfolge herauskommt, die in den Beispieldateien zu sehen ist.

[lundefugl]

Hallo Christian

ich versuche dir mal meine Gedanken zu den beiden kommentierten Änderungen zu erklären.

Problem mit dem Editieren von CSVs.

Wenn man ein Ausgangs-CSV in folgender Form hat (im Zip anbei Test-1.csv).

Code:

Description,Latitude,Longitude,Altitude,Speed,Time
Nordkap,71.1709797,25.7831594,263.0,10,"01.01.26 00:01:00"
Westkap,62.1866052,5.1259121,481.0,20,"02.01.26 00:02:00"
Südkap,57.982407,7.0477149,21.0,30,"03.01.26 00:03:00"

Und dann dieses per RouteConverter editiert. Z.B. setze ich beim zweiten Wert die Höhe auf 500m.
Wenn man nun speichert, dann kommt das raus (im Zip anbei Test-2.csv).

Code:

Altitude,Description,Elevation,Latitude,Longitude,Speed,Time
263.0,Nordkap,,71.1709797,25.7831594,10,"01.01.26 00:01:00"
481.0,Westkap,500.0,62.1866052,5.1259121,20,"02.01.26 00:02:00"
21.0,Südkap,,57.982407,7.0477149,30,"03.01.26 00:03:00"

Hier habe ich jetzt nun eine "Altitude" und "Elevation" Spalte. Ausserdem sind beim zweiten Eintrag zwei unterschiedliche Werte da drin.

Und wenn man nun das Test-2-File wieder liest, dann hat man zwar seine modifizierte Höhe, aber die anderen Höhen werden nicht angezeigt.

In meinen Augen sollte beim Speichern immer in der Spalte gespeichert werden, die zuvor im File bereits drin war. Und damit hat man beim erneuten Laden auch wieder seine zuletzt bearbeiteten Werte.

Gruß
Thomas


  CSV-Testfiles.zip (Size: 688 bytes / Downloads: 7)