Aaaalso: Da Ihr ja um Feedback gebeten habt...
Das ist ja ein ganz schön umfangreiches Projekt. Da habt Ihr Euch was vorgenommen. Leider habe ich von MySQL und PHP keine Ahnung. Ich habe es mit xampp unter Mandriva Linux 2008.0 getestet und will Euch kurz schildern, was mir aufgefallen ist.
Fangen wir bei der
README.TXT an:
Alle Dateien in ein beliebiges Verzeichnis Deiner Wahl kopieren (z.B "cocktails/).
Das Verzeichnis muss vom Dokumentverzeichnis des Webservers aus erreichbar sein! Es muss also entweder im Dokumentverzeichnis liegen oder im Dokumentverzeichnis muss eine Verknüpfung auf das Verzeichnis liegen (getestet mit symlink unter Linux).
Es wäre auch schön, wenn kurz gesagt würde, was die Ordner beinhalten und wo die Daten liegen.
Nun zum
Installer:
Bitte Zugangsdaten für die Datenbank angeben. Wenn die Datenbank noch nicht besteht, kannst Du sie auch unten erstellen lassen.
Das klingt nach zwei Optionen, nämlich für bestehende und nicht bestehende DB. Es gibt aber nur einen Knopf "Datenbank erstellen". Was soll bei bestehender DB passieren?
Die Eingabefelder bedürfen auch einer besseren Erklärung, z.B. ob DB-Username und DB-Passwort an dieser Stelle vergeben werden oder bestehende Daten abgefragt werden sollen.
Die Passwörter sollten auch nicht im Klartext angezeigt werden. Kann es sein, dass das PW kein Leerzeichen enthalten darf?
Doch jetzt zur eigentlichen
Anwendung:
Die
Zutatentypen (Alkohol/Liköre/Sirup/Säfte/sonstiges) sind nicht konsistent, da mal Singular, mal Plural, und Liköre auch Alkohol sind. Vorschlag: Brände/Liköre/Weine/Sirupe/Säfte/Limonaden/sonstiges.
In der
Zutaten-Liste werden Alkoholgehalt und EK-Preis werden nicht angezeigt.
Bei der
Gläsern gibt es den Knopf "Cocktail editieren". Dass man die Gläser selbst verwalten kann, finde ich super, auch, dass die Standardgläser bereits angelegt sind. Hier würde ich nur den Tumbler noch als Standard mit dazunehmen und die Schreibweisen vereinheitlichen.
Bei den
Kategorien würde ich mir wünschen, sie wie die Gläser bearbeiten zu können. Richtig gut wäre die Möglichkeit, verschiedene Sätze von Kategorien anlegen zu können, um je nach Fall eine andere Einteilung nutzen zu können, ohne alles komplett über den Haufen zu werfen. Die jetzigen Kategorien finde ich nicht so arg überzeugend, da die Digestifs fehlen (wenn schon Aperitifs aufgeführt werden) und insbesondere zwischen "Aperitif" und "Stark" eine große Überlappung zu erwarten ist.
Auf der Seite
Cocktail hinzufügen ist ein Schreibfehler in der Überschrift. Was ich mir hier noch wünsche, ist ein optionales Feld für die Angabe des
Rezept-Autors. Bei der
Zubereitung sind zum Teil mehrere Einträge gleichzeitig zutreffend, z.B. "auf Eis schütteln" und "abseihen". Mögliche Lösungen: 1. Mehrfachauswahl zulassen 2. alle Fälle/Kombinationen durch entsprechende Einträge abdecken 3. Standard-Zubereitungen bearbeitbar machen. Ich wünsche mir letzteres.
Die Einträge in der
Cocktail-Liste sollten Links zu den Rezepten enthalten! Auch fehlt bei Alkoholgehalt und EK-Preis die Einheit.
Die
Cocktail-Anzeige sollte Preis, Promille und Kalorien nur dann anzeigen, wenn für alle Zutaten des Rezepts diese Angaben jeweils auch vorhanden sind - sonst kommt nämlich Blödsinn dabei heraus.
Die
Zutat-Karte würde ich um Zubereitungsart (Schlagworte) und Glastyp ergänzen und in Mixer-Karte umbenennen.
In der
Zutatensuche finde ich es sehr gut, dass nach Eingabe von Buchstaben eine Pulldown-Liste der mit diesen Buchstaben möglichen Zutaten angezeigt wird! Die Suchergebnis-Liste würde ich aber nicht dort anzeigen, sondern auf der Seite, und die Sucheingabe im Eingabefeld stehen lassen. Dringender Wunsch: Suche nach mehreren Zutaten!
Aufgetretene Fehlermeldungen:
Cocktail-Anzeige:
Warning: arsort() expects parameter 1 to be array, null given in /opt/lampp/htdocs/cocktails/classes/dao/cocktails.php on line 379
Warning: Variable passed to each() is not an array or object in /opt/lampp/htdocs/cocktails/classes/dao/cocktails.php on line 380
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/classes/dao/cocktails.php on line 393
Zutatensuche, wenn kein Cocktail mit der Zutat vorhanden ist:
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/classes/dao/cocktails.php on line 343
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/classes/dao/cocktails.php on line 365
Warning: arsort() expects parameter 1 to be array, null given in /opt/lampp/htdocs/cocktails/classes/dao/cocktails.php on line 379
Warning: Variable passed to each() is not an array or object in /opt/lampp/htdocs/cocktails/classes/dao/cocktails.php on line 380
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/classes/dao/cocktails.php on line 393
Error: SQL Error in Query
SQL:SELECT * from cocktail_glaeser where id=
SQL ERROR: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1
Cocktail-Liste: Sortieren nach Menge/Alkoholgehalt/Preis:
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/cocktail_list.php on line 134
Warning: uksort() : The argument should be an array in /opt/lampp/htdocs/cocktails/array_sort.php on line 37
Warning: Variable passed to each() is not an array or object in /opt/lampp/htdocs/cocktails/array_sort.php on line 41
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/cocktail_list.php on line 134
Warning: uksort() : The argument should be an array in /opt/lampp/htdocs/cocktails/array_sort.php on line 37
Warning: Variable passed to each() is not an array or object in /opt/lampp/htdocs/cocktails/array_sort.php on line 41
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/cocktail_list.php on line 134
Warning: uksort() : The argument should be an array in /opt/lampp/htdocs/cocktails/array_sort.php on line 37
Warning: Variable passed to each() is not an array or object in /opt/lampp/htdocs/cocktails/array_sort.php on line 41
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/cocktail_list.php on line 134
Warning: uksort() : The argument should be an array in /opt/lampp/htdocs/cocktails/array_sort.php on line 37
Warning: Variable passed to each() is not an array or object in /opt/lampp/htdocs/cocktails/array_sort.php on line 41
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/cocktail_list.php on line 134
Warning: uksort() : The argument should be an array in /opt/lampp/htdocs/cocktails/array_sort.php on line 37
Warning: Variable passed to each() is not an array or object in /opt/lampp/htdocs/cocktails/array_sort.php on line 41
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/cocktail_list.php on line 134
Warning: uksort() : The argument should be an array in /opt/lampp/htdocs/cocktails/array_sort.php on line 37
Warning: Variable passed to each() is not an array or object in /opt/lampp/htdocs/cocktails/array_sort.php on line 41
Warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/cocktails/cocktail_list.php on line 134
Warning: uksort() : The argument should be an array in /opt/lampp/htdocs/cocktails/array_sort.php on line 37
Warning: Variable passed to each() is not an array or object in /opt/lampp/htdocs/cocktails/array_sort.php on line 41