Open geraldo opened 11 years ago
Gefällt mir gut. Ich würde aber auch die Bewertung, die Anzahl der Kommentare und die Anzahl der Fotos als sortierbare Spalten aufnehmen. Bei der tabellarischen Anordnung fällt besonders stark auf, dass die Gattung groß- und die Art kleingeschrieben ist. Kann man das vereinheitlichen? (mache dazu ein eigenes Issue auf).
Das mit Bewertung, Anzahl Kommentare und Anzahl Fotos als sortierbare Spalten ist vor allem eine Performancefrage. Alle drei Werte kommen vom Backend, dh. aus der Datenbank und verlangen eine Abfrage über PHP. Das hat zur Folge dass sich die Ladezeit zumindest um ein paar Sekunden erhöht.
Eine andere performanceverträgliche Alternative wäre diese Daten zB. einmal stündlich per cronjob zu exportieren, mit der Gefahr, dass sie nicht immer ganz aktuell sind.
Wenn der Cronjob so gemacht ist, dass während des Exports keine instabile Zustände eintreten können (zB der Export geht schief und die JSON-Datei ist weg oder inkonsistent), wäre ein stündlicher Export aus meiner Sicht fürs Erste akzeptabel.
Hab Bewertung, Anzahl Kommentare und Anzahl Fotos der Tabelle als Spalten beigefügt. Frag sie nicht dynamisch über PHP vom Backend ab sonder schreib sie über eine Exportfunktion direkt in die JSON Datei. Schaut euch das mal an.
Vorschlag wie wir das Exportieren über einen Cronjob hinkriegen ohne dass ein instabiler Zustand auftreten kann? Exportieren dauert 5-10 Sekunden, dh. wenn jemand in diesem Moment zugreift, bekommt er wahrscheinlich eine kaputte Datei, dh. eine leere Tabelle geliefert. Vielleicht in eine temporäre Datei schreiben und sie dann kopieren? Das dürfte zumindest das Risiko auf einen Bruchteil senken da das Kopieren einer 200 kB Datei nicht lange dauern kann.
sieht gut aus und ist sehr schnell! kopieren der temporären dateit ist vermutlich gar nicht notwendig. es sollte ein unbenennen der temporären datei mit dem kommando mv und der -f option genügen. wenn das nicht funktioniert, dann löschen der alten datei und umbenennen der neuen datei. das geht auch sehr schnell.
Suche ist freigeschaltet! Daten werden jede halbe Stunde exportiert und nachträglich über verwendete JSON Datei kopiert.
Am Handy seh ich ein Platzproblem: Tabelle ist breiter als meine Auflösung und es erscheint ein horizontaler Scrollbalken. Wir bräuchten gute Abkürzungen für Bewertung, Kommentare und Fotos, ist es verständlich wenn wir Bew., Komm. und Fot. schreiben?
Bis jetzt gibt es keinen zentralen Überblick über alle Bäume auf Linz Pflückt. Ausserdem überlegen wir schon länger eine Suche zu implementieren. Ich glaube eine Tabllenansicht mit implementierten Suchfeld ist ein gutes Feature das beide Anforderungen erfüllt.
Hier die Testversion: http://linz.pflueckt.at/suche/
Die Seite verwendet DataTables da dieses jQuery Plugin eine effiziente und featurereiche Tabellenansicht mit implementierter Suche anbietet:
Diese Seite verwendet eine statische JSON Datei da eine Datenbankabfrage über alle Datensätze mit JSON-Ausgabe zu lange dauert. Diese JSON Datei sollte aber periodisch mit cron von der Datenbank exportiert werden damit die Daten konsistent bleiben.
Um die Ladezeit erträglich zu halten, habe ich wenig wichtigere Daten wie Grössenangaben und Reifezeiten nicht exportiert.
Testberichte und Verbessserungsvorschläge bitte hier posten.