Beiträge von m.scatello

  • ID übergabe an AJAX bei Klick auf DIV

    Vielleicht noch ein Hinweis zu dem PHP-Teil. Das ist einfach nur grausam und unübersichtlich:

    Code
    echo '<div class="row"><div class="flex-item"><input type="checkbox" id="ud_radio" name="tourid"  value="'.$row['id'].'"></div><div class="flex-item">'.$row['startdatum'].'</div><div class="flex-item"><div class="start">'.$row['land_start'].'</div><div class="start">'.$row['plz_start'].'</div><div class="start">'.$row['start'].'</div></div> <div class="flex-item">'.$row['enddatum'].'</div><div class="flex-item">'.$row['ende'].'</div> <div class="flex-item">'.$row['preis'].'€</div></div>';

    Warum nicht so:

    Wenn man Fehler sucht oder einen Debugger benutzt, dann ist deine Version komplett unbrauchbar.

    Und dann noch:

    Beim Einsatz von SQL-Datenbanken sollte das Abrufen aller Spalten eines Datensatzes per SELECT * vermieden werden. Stattdessen sollten die gewünschten Spaltennamen explizit aufgelistet werden (SELECT col1, col2 FROM …). Einige Vorteile:

    Häufig werden in einer Abfrage nicht alle Spalten benötigt, die aktuell existieren oder in Zukunft existieren könnten, sodass unnötige Daten übertragen werden.

    Bei expliziter Angabe der gewünschten Spalten können Datenbanksysteme unter Umständen Indizes zur Beantwortung von Abfragen nutzen.

    Eine Auflistung der Spaltennamen macht den Code verständlicher, da die Tabellenstruktur nicht erst im Datenbankschema nachgelesen werden muss.

    Bei Änderungen am Datenbankschema (etwa bei Entfernung oder Umbenennung einer Spalte) schlägt eine Query, die kein SELECT * nutzt, sofort fehl. Das ist hilfreich, da so keine Stelle im Code übersehen werden kann, an der wahrscheinlich auch die PHP-Datenstrukturen an das neue Schema angepasst werden müssen.

  • Wie soll ich diesen Fehler beheben?

    Stimmen die Namen pid, pname, doctor und docemail?

    Vermutlich geht

    Code
    $userrow = $database->query("select * from doctor where docemail='$useremail'");

    in die Hose, also frage die Datenbank, ob es einen Fehler gibt.

    Beim Einsatz von SQL-Datenbanken sollte das Abrufen aller Spalten eines Datensatzes per SELECT * vermieden werden. Stattdessen sollten die gewünschten Spaltennamen explizit aufgelistet werden (SELECT col1, col2 FROM …). Einige Vorteile:

    Häufig werden in einer Abfrage nicht alle Spalten benötigt, die aktuell existieren oder in Zukunft existieren könnten, sodass unnötige Daten übertragen werden.

    Bei expliziter Angabe der gewünschten Spalten können Datenbanksysteme unter Umständen Indizes zur Beantwortung von Abfragen nutzen.

    Eine Auflistung der Spaltennamen macht den Code verständlicher, da die Tabellenstruktur nicht erst im Datenbankschema nachgelesen werden muss.

    Bei Änderungen am Datenbankschema (etwa bei Entfernung oder Umbenennung einer Spalte) schlägt eine Query, die kein SELECT * nutzt, sofort fehl. Das ist hilfreich, da so keine Stelle im Code übersehen werden kann, an der wahrscheinlich auch die PHP-Datenstrukturen an das neue Schema angepasst werden müssen.

  • Konfigurator

    Pascal

    Könntest du bitte etwas mehr auf deine Rechtschreibung und Interpunktion achten. Halbwegs korrekte Interpunktion und Groß-/Kleinschreibung sind in einem Forum bzw. allgemein bei schriftlicher Kommunikation auch ein Zeichen von Höflichkeit dem Gesprächspartner gegenüber.

    Ich habe den Eindruck, dass du noch sehr unerfahren in Sachen Programmierung bist, daher sage ich dir ganz ehrlich, dass dein Vorhaben nichts für Anfänger ist. Solche Konfiguratoren sind schon ein wenig komplizierter, da sollte man HTML/CSS/PHP/JavaScript halbwegs gut beherrschen.

  • Image Slider soll nach Klick auf Thumbnail öffnen

    In einem anderen Forum wurde oft dies zum Thema Crossposting geschrieben:

    Zitat

    Crossposting bedeutet, dass ein und dieselbe Frage in mehreren Foren oder Newsgroups gleichzeitig gestellt wird. Ein solches Verhalten wird von den meisten Helfern nicht toleriert, denn damit stellst Du einerseits die Kompetenz der Helfer jedes einzelnen Forums und Newsgroup in Frage und bewirkst zudem, dass ein zig-faches an Stunden aufgewendet wird, um nur ein einziges Problem zu lösen, denn überall werden sich die Helfer daran setzen, eine Lösung für Dich zu finden. Das ist ganz klar ein Missbrauch an dem kostenlosen und freiwilligen Support, der hier angeboten wird. Wenn Deine Frage klar und deutlich gestellt wurde, dann wirst Du ganz bestimmt innerhalb kürzester Zeit eine Antwort erhalten. Sollte dies auch nach mehreren Tagen nicht der Fall sein, dann versuche bitte die Frage besser zu formulieren. Falls Du dennoch in einem zweiten Forum posten solltest, erwähne bitte, dass Du die Frage schon woanders gestellt hast, und dass Du es tust, weil keine Lösung gefunden werden konnte.

  • Formularfelder sind auf kleinen Displays zu breit

    Lupus_III

    Wenn ich dir noch einen Tipp geben darf: du solltest das Formular noch einmal designtechnisch überarbeiten. Es ist nicht schön, wenn man mit den Augen immer von der Mitte nach links und dann wieder zur Mitte muss, das macht die Sache sehr unruhig und sieht auch nicht wirklich gut aus. Ich würde ein div drumherum machen, diesem eine Breite verpassen und mittig setzen.