Verbesserung zum Formular? Brauche mal jemand der drüberschaut...

Ragnaroek · 10.04.2009, 19:08

Hallo allerseits...

habe folgendes Problem gehabt:

Bilder für eine Slidewshow sollten per DB online/offline geschaltet werden (also per Statuswert: 1=online; 0=offline)

Die Bilder sollten mittles eines einfachen Formular geschaltet werden können (bei mir per Radiobutton). Die neuen Daten sollten dann in die DB geschrieben werden.

So. Das Formular funktioniert auch. Nur möchte ich ja ständig dazulernen und frage mich, ob ich das ganze nicht doch über ein paar Ecken zuviel gelöst habe.

Hier mal das Script für das Formular:

PHP-Code:

  <?php

include ('../inc/db.php');

mysql_query("SET NAMES 'utf8'");  

$mysql_abfrage  = "SELECT * FROM bilder_index ORDER BY bilder_index_id";

$mysql_ergebnis=mysql_query($mysql_abfrage);
$num = mysql_num_rows($mysql_ergebnis);

while($datensatz = mysql_fetch_assoc($mysql_ergebnis))
{
    if ($datensatz["bilder_index_status"] == 1)
    {
        echo '<tr>';
        echo '<td class="rahmen"><img src="../images/index/' . $datensatz["bilder_index_bild"] . '.jpg" </td>
        <input type="radio" name="index_status_' . $datensatz["bilder_index_id"] . '" value="1" checked /></td><input type="radio" name="index_status_' . $datensatz["bilder_index_id"] . '" value="0" /></td>';                                        
        echo '</tr>';
    }
    
    else
    {
        echo '<tr>';
        echo '<td class="rahmen"><img src="../images/index/' . $datensatz["bilder_index_bild"] . '.jpg" </td>
        <input type="radio" name="index_status_' . $datensatz["bilder_index_id"] . '" value="1" /></td><td class="rahmen"><input type="radio" name="index_status_' . $datensatz["bilder_index_id"] . '" value="0" checked /></td>';                                        
        echo '</tr>';
    }
}

echo '</table>';

echo '<input class="senden_admin" name="submit" type="submit" value="Senden" />&nbsp;';

?>

Zuerst wird die DB abgefragt und der Inhalt nach Status ausgegeben. Dazu wird über die IF-Anweisung abgefragt, wie der Status ist, damit "checked" richtig gesetzt wird. Um den input-name eindeutig gesetzt bekomme, hänge ich am Namen "index_status_" noch die ID des Datensatzes an. Im Value wird dann der Status übermittelt. Dann wird das Formular per send abgeschickt...

Auf der nächsten Seite wird das nächste Script aufgerufen:

PHP-Code:

  <?php
                    
include ('../inc/db.php');

mysql_query("SET NAMES 'utf8'");  

$mysql_abfrage  = "SELECT * FROM bilder_index ORDER BY bilder_index_id";

$mysql_ergebnis=mysql_query($mysql_abfrage);
$num = mysql_num_rows($mysql_ergebnis);

while($datensatz = mysql_fetch_assoc($mysql_ergebnis))
{
    echo $_POST["index_status_" . $datensatz["bilder_index_id"] . ""] . '<br />';

    mysql_query("SET NAMES 'utf8'");     
    $mysql_aendern = "UPDATE bilder_index SET bilder_index_status = '" . $_POST["index_status_" . $datensatz["bilder_index_id"] . ""] . "' ";
    $mysql_aendern .= "WHERE bilder_index_id = '" . $datensatz["bilder_index_id"] . "' ";
    
    mysql_query($mysql_aendern);
    
    $num = mysql_affected_rows();
}

Hier wird erst mal wieder der Index der DB abgefragt (ist ja genau der gleiche wie auf der anderen Seite). Anhand dessen haben wir wieder die ID für die exakte Namenszuweisung (z.B. index_status_3). Damit kann auch der Wert der übermittelten Variable abgefragt und in der DB aktualisiert werden.

Ich hoffe ihr konntet folgen.

Ich bin mir sicher dass das über ein Feld noch einfacher gelöst werden kann, aber das hat zum verr.... bei mir nicht funktioniert. Das PRoblem war für mich zudem dass die IDs aus der DB ja nicht immer kontinuierlich sein werden.

Wenn man es einfacher machen könnte, wäre ich für einen Hinweis sehr dankbar...

besten dank

Lapje

litterauspirna · 10.04.2009, 19:15

Also auf alle Fälle solltest du Select * in deiner Abfrage nicht machen, sondern anstatt des * die Felder die bracuhst dur Komma getrennt angeben.

Dein checked dürfte so auch nicht funktionieren, denn das heist dann checked="checked".

Und mit dem Thema SQL-Injections solltest du dich auseinander setzen.

Ansonsten fäält mir so nichts auf.

[OT]

Dein Nick gefällt mir, hat der was mit der blackmetalband zu tun?

Gruß litter

Ragnaroek · 10.04.2009, 19:34

Hai,

nein, eher mit einem Kinderbuch... Lapje kommt von Lapje Luum, kommt aus dem niederländischen und bedeutet soviel wie "kleine Insel"...

und das mit dem checked funzt aber wunderbar...ist das jetzt dann nur Zufall?

Bei Selfhtml wirds zumindest auch so geschrieben:

SELFHTML: HTML/XHTML / Formulare / Auswahllisten, Radio-Buttons und Checkboxen

das mit dem Select habe ich schon geändert, mit injections befasse ich mich auch mal...gibts die bei MySQL auch?

Squall · 10.04.2009, 19:38

Zitat:

Zitat von Ragnaroek

und das mit dem checked funzt aber wunderbar...ist das jetzt dann nur Zufall?

Kein zufall, nur veraltete schreibweise.

agundur · 10.04.2009, 19:46

so spontan würde ich sagen Deine DB-Abfragen könnten noch geprüft werden ..

PHP-Code:

   
$result = mysql_query(....);

if (!$result) {

    die('Ungültige Abfrage: ' . mysql_error());

}

usw.

Ragnaroek · 10.04.2009, 19:58

Zitat:

Zitat von Squall

Kein zufall, nur veraltete schreibweise.

Nicht dass ich das, was Du schreibst in Frage stellen will...aber warum wird das bei SelfHTML noch immer so geschrieben? Haben die nicht abgedatet? Oder wieder ein Unterschied zwischen HTML und (X)HTML?

Nachtrag: Ja, ist XHTML...habs gesehen...und geändert

Squall · 10.04.2009, 20:12

Zitat:

Zitat von Ragnaroek

Nicht dass ich das, was Du schreibst in Frage stellen will...aber warum wird das bei SelfHTML noch immer so geschrieben? Haben die nicht abgedatet? Oder wieder ein Unterschied zwischen HTML und (X)HTML?

Ich bin kein Gott ^^ fehler können auch mir passieren, also kritisier mich ruhig

Was deine Frage angeht, wenn du auf klick hier schaust, steht unten im kleingedruckten

Zitat:

Wenn Sie XHTML-Standard-konform arbeiten wollen, müssen Sie dieses Attribut in der Form checked="checked" notieren.

Ragnaroek · 10.04.2009, 20:15

Da haben sich unsere Posts überschnitten..habs auch gerade gelesen und schon geändert...wieder was gelernt..

Danke

Manko10 · 12.04.2009, 12:39

Dein Thread-Titel ist schlecht bzw. unglücklich gewählt. Bitte ändere diesen und beachte dabei Bitte aussagekräftige Threadtitel verwenden

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
PHP: Formular Generator	Zergling-new	Tutorials	16	06.07.2010 03:14
Formular; ich brauche mal euren Rat	maeck	PHP Tipps 2008	8	17.04.2008 23:58
Wert aus Textfeld in neues Formular übernehmen (kein Submit)	PsychoEagle	HTML, Usability und Barrierefreiheit	9	02.01.2007 14:51
[Erledigt] Formular in Formular		HTML, Usability und Barrierefreiheit	4	19.10.2005 12:18
[Erledigt] PHP Formular ~~wichtig~~		PHP Tipps 2005	27	29.05.2005 19:38
Brauche PHP Formular		Beitragsarchiv	9	16.08.2004 16:23

10.04.2009, 19:08
Ragnaroek Neuer Benutzer Registriert seit: 13.01.2008 Beiträge: 11	Verbesserung zum Formular? Brauche mal jemand der drüberschaut... Hallo allerseits... habe folgendes Problem gehabt: Bilder für eine Slidewshow sollten per DB online/offline geschaltet werden (also per Statuswert: 1=online; 0=offline) Die Bilder sollten mittles eines einfachen Formular geschaltet werden können (bei mir per Radiobutton). Die neuen Daten sollten dann in die DB geschrieben werden. So. Das Formular funktioniert auch. Nur möchte ich ja ständig dazulernen und frage mich, ob ich das ganze nicht doch über ein paar Ecken zuviel gelöst habe. Hier mal das Script für das Formular: PHP-Code: <?php include ('../inc/db.php'); mysql_query("SET NAMES 'utf8'"); $mysql_abfrage = "SELECT * FROM bilder_index ORDER BY bilder_index_id"; $mysql_ergebnis=mysql_query($mysql_abfrage); $num = mysql_num_rows($mysql_ergebnis); while($datensatz = mysql_fetch_assoc($mysql_ergebnis)) { if ($datensatz["bilder_index_status"] == 1) { echo '<tr>'; echo '<td class="rahmen"><img src="../images/index/' . $datensatz["bilder_index_bild"] . '.jpg" </td> <input type="radio" name="index_status_' . $datensatz["bilder_index_id"] . '" value="1" checked /></td><input type="radio" name="index_status_' . $datensatz["bilder_index_id"] . '" value="0" /></td>'; echo '</tr>'; } else { echo '<tr>'; echo '<td class="rahmen"><img src="../images/index/' . $datensatz["bilder_index_bild"] . '.jpg" </td> <input type="radio" name="index_status_' . $datensatz["bilder_index_id"] . '" value="1" /></td><td class="rahmen"><input type="radio" name="index_status_' . $datensatz["bilder_index_id"] . '" value="0" checked /></td>'; echo '</tr>'; } } echo '</table>'; echo '<input class="senden_admin" name="submit" type="submit" value="Senden" /> '; ?> Zuerst wird die DB abgefragt und der Inhalt nach Status ausgegeben. Dazu wird über die IF-Anweisung abgefragt, wie der Status ist, damit "checked" richtig gesetzt wird. Um den input-name eindeutig gesetzt bekomme, hänge ich am Namen "index_status_" noch die ID des Datensatzes an. Im Value wird dann der Status übermittelt. Dann wird das Formular per send abgeschickt... Auf der nächsten Seite wird das nächste Script aufgerufen: PHP-Code: <?php include ('../inc/db.php'); mysql_query("SET NAMES 'utf8'"); $mysql_abfrage = "SELECT * FROM bilder_index ORDER BY bilder_index_id"; $mysql_ergebnis=mysql_query($mysql_abfrage); $num = mysql_num_rows($mysql_ergebnis); while($datensatz = mysql_fetch_assoc($mysql_ergebnis)) { echo $_POST["index_status_" . $datensatz["bilder_index_id"] . ""] . '<br />'; mysql_query("SET NAMES 'utf8'"); $mysql_aendern = "UPDATE bilder_index SET bilder_index_status = '" . $_POST["index_status_" . $datensatz["bilder_index_id"] . ""] . "' "; $mysql_aendern .= "WHERE bilder_index_id = '" . $datensatz["bilder_index_id"] . "' "; mysql_query($mysql_aendern); $num = mysql_affected_rows(); } Hier wird erst mal wieder der Index der DB abgefragt (ist ja genau der gleiche wie auf der anderen Seite). Anhand dessen haben wir wieder die ID für die exakte Namenszuweisung (z.B. index_status_3). Damit kann auch der Wert der übermittelten Variable abgefragt und in der DB aktualisiert werden. Ich hoffe ihr konntet folgen. Ich bin mir sicher dass das über ein Feld noch einfacher gelöst werden kann, aber das hat zum verr.... bei mir nicht funktioniert. Das PRoblem war für mich zudem dass die IDs aus der DB ja nicht immer kontinuierlich sein werden. Wenn man es einfacher machen könnte, wäre ich für einen Hinweis sehr dankbar... besten dank Lapje


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

10.04.2009, 19:15
litterauspirna Erfahrener Benutzer Registriert seit: 24.04.2008 Beiträge: 3.172 PHP-Kenntnisse: Anfänger	Also auf alle Fälle solltest du Select * in deiner Abfrage nicht machen, sondern anstatt des * die Felder die bracuhst dur Komma getrennt angeben. Dein checked dürfte so auch nicht funktionieren, denn das heist dann checked="checked". Und mit dem Thema SQL-Injections solltest du dich auseinander setzen. Ansonsten fäält mir so nichts auf. [OT] Dein Nick gefällt mir, hat der was mit der blackmetalband zu tun? Gruß litter __________________ Aus dem Dynamo Lande kommen wir. Trinken immer reichlich kühles Bier. Und dann sind wir alle voll, die Stimmung ist so toll. Aus dem Dynamo Lande kommen wir. http://www.lit-web.de

10.04.2009, 19:34
Ragnaroek Neuer Benutzer Registriert seit: 13.01.2008 Beiträge: 11	Hai, nein, eher mit einem Kinderbuch... Lapje kommt von Lapje Luum, kommt aus dem niederländischen und bedeutet soviel wie "kleine Insel"... und das mit dem checked funzt aber wunderbar...ist das jetzt dann nur Zufall? Bei Selfhtml wirds zumindest auch so geschrieben: SELFHTML: HTML/XHTML / Formulare / Auswahllisten, Radio-Buttons und Checkboxen das mit dem Select habe ich schon geändert, mit injections befasse ich mich auch mal...gibts die bei MySQL auch?

10.04.2009, 19:46
agundur Erfahrener Benutzer Registriert seit: 04.12.2008 Beiträge: 132	so spontan würde ich sagen Deine DB-Abfragen könnten noch geprüft werden .. PHP-Code: `$result = mysql_query(....); if (!$result) { die('Ungültige Abfrage: ' . mysql_error()); }` usw. __________________ AFFE ist nur im Dezimalsystem ein Palindrom.

10.04.2009, 20:15
Ragnaroek Neuer Benutzer Registriert seit: 13.01.2008 Beiträge: 11	Da haben sich unsere Posts überschnitten..habs auch gerade gelesen und schon geändert...wieder was gelernt.. Danke

12.04.2009, 12:39
Manko10 Supermoderator HD Registriert seit: 16.03.2008 Beiträge: 8.709 PHP-Kenntnisse: Fortgeschritten	Dein Thread-Titel ist schlecht bzw. unglücklich gewählt. Bitte ändere diesen und beachte dabei Bitte aussagekräftige Threadtitel verwenden __________________ Refining Linux Advent Calendar series “24 Outstanding ZSH Gems”