Update von Daten mittel Formular

Pilo · 13.11.2021

Mit untenstehendem Programm kann ich mittels eines Formulars Variablen mit einem neuen Wert belegen. Nun möchte ich die Variablen in einer MySQL-Datenbank unterbringen, etwa in einer Tabelle mit dieser Struktur:
id INT AUTO_INCREMENT PRIMARY KEY,
er varchar (15),
sie varchar (15)
Nach der Kommentarzeile CONNECT sollte dann die Verbindung zu der Tabelle gemacht werden. Die Zeile nach UPDATE müßte die Variablen in der Tabelle ändern (updaten) und die Zeile nach SELECT die aktuellen Daten einer vorgegebenen ID aus der Tabelle auslesen.
Mein Problem: Ich komme mit der MySQL-Syntax nicht zurecht und kriege bei all meinen Versuchen leediglich Fehlermeldungen...
Kann mir jemand helfen?

<?php
// CONNECT
if (!empty($_POST)) {
// UPDATE
$er=$_POST['er']; $sie=$_POST['sie'];}
else {
// SELECT
$er="Emil"; $sie="Lisa"; }
echo "<h2>".$er." liebt ".$sie."</h2>";
?>
<form method="post">
Er:<br>
<input type='text' name='er' value='<?php echo $er ?>'><br>
Sie:<br>
<input type='text' name='sie' value='<?php echo $sie?>'><br><br>
<input type="submit" value=" OK ">
</form>

scatello · 13.11.2021

Pilo schrieb:
Ich komme mit der MySQL-Syntax nicht zurecht

Warum nicht? Was hast du bisher versucht?

Pilo schrieb:
kriege bei all meinen Versuchen leediglich Fehlermeldungen

Die wie lauten?

Pilo schrieb:
Kann mir jemand helfen?

Klar, wenn du uns deinen Versuch zeigst, können wir den bestimmt korrigieren.

Und wenn du Quellcode postest, benutze bitte die Formatierungmöglichkeiten den Forums:

Pilo · 13.11.2021

Mein Versuch mit MySgl beginnt so:
1 <?php
2 mysql_connect("localhost", "root","usbw") or die ("Verbindung nicht möglich");
3 mysql_select_db("test") or die ("Datenbank existiert nicht");
4 if (!empty($_POST)) {
5 $er = $_POST['er'];
6 $sie = $_POST['sie'];
7 $sql= "UPDATE 'Members' SET er='$er' sie='$sie' WHERE id='1'"; }
und liefert bereits in Zeile 5 folgende Fehlermeldung:
Notice: Undefined index: er in D:\! Web\root\test\dbase.php on line 5

scatello · 13.11.2021

Das hat ja erst einmal nichts mit MySQL zu tun. Allerdings solltest du mal den roten Kasten auf https://www.php.net/manual/de/function.mysql-connect.php beachten. Ich weiß ja nicht, welche PHP-Version du benutzt, allerdings ist die definitv veraltet und du solltest dringend updaten.

Und was hast du daran nicht verstanden:

scatello schrieb:
Und wenn du Quellcode postest, benutze bitte die Formatierungmöglichkeiten den Forums:

Pilo · 13.11.2021

Nicht meine PHP-Version ist veraltet, sondern die Bücher, aus denen ich meine Kenntnis bezogen habe. Vielen Dank für den Hinweis.
Inzwischen habe ich mein Progrämmchen angepaßt (und auch noch von Schreibfehlern bereinigt).
Alles funktioniert - mit Ausnahme des wichtigsten Teils: UPDATE (Zeile 6).
Ich wäre Dir sehr dankbar, wenn Du mir sagen könntest, was da stehen muß?

Code:

<?php
$mysqli = new mysqli("localhost", "root", "usbw", "test");
if (!empty($_POST)) {
   $er  = $_POST['er'];
   $sie = $_POST['sie'];
   $sql="UPDATE members SET er='$er'  sie='$sie' WHERE id='1'";}
$result = $mysqli->query("SELECT * FROM `Members` WHERE id='1'");
$row = $result->fetch_assoc();
echo "<h2>".$row['er']." liebt ".$row['sie']."</h2>";
?>
<form method="post">
   Er:<br>
   <input type='text' size='20' name='er' value='<?php echo $row["er"]?>'><br>
   Sie:<br>
   <input type='text' size='20' name='sie' value='<?php echo $row["sie"]?>'><br><br>
   <input type="submit" value=" OK ">
</form>

PS: Wie Du siehst, habe ich nun begriffen, wie man Code eingibt.

scatello · 13.11.2021

PHP:

$sql="UPDATE members SET er='$er'  sie='$sie' WHERE id='1'";

In dieser Zeile setzt du eine Variable, aber ich denke, es wäre nicht die allerdümmste Idee, wenn man dazu auch ein $mysqli->query absetzt.

Und:

16.14. Warum soll ich nicht SELECT * schreiben?
http://net.co.at/doc/howto/docs/dclp-faq/q/q-sql-select.html

Das solltest du dir auch ansehen: https://www.php.net/manual/de/mysqli.real-escape-string.php oder das: https://www.w3schools.com/php/php_mysql_prepared_statements.asp

Pilo · 14.11.2021

Zeile 6 lautet nun

Code:

$mysqli->query("UPDATE members SET er='$er' sie='$sie' WHERE id='1'"); }

Aber die Daten werden noch immer nicht geändert.

scatello · 14.11.2021

Dann wird es Zeit für:

Richtig debuggen

Man bemerkt, dass ein Skript nicht das tut, was es soll.
Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(-1);
Man verwendet ini_set('display_errors', true); damit die Fehler auch angezeigt werden.
Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde. Wichtig bei MySQL Fehlern (...not a valid MySQL result resource...): mysqli_error() verwenden oder Abfrage ausgeben und zb mit phpmyadmin testen.
Schritt 5 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

basti1012 · 23.11.2021

Ich musste 2 Wochen Zwangsurlaub machen und habe auch wieder viel verlernt.
Ich meine aber auch das bei mehreren Spalten Updates ein Komma zwischen den einzelnen Spalten befehle kommen.
so vieleicht

PHP:

$mysqli->query("UPDATE `members` SET `er`='$er',`sie`='$sie' WHERE `id`='1'");

Kann aber auch sein das mein Gehirn ein Schaden bekommen hat

Pilo · 26.11.2021

Dein Gehirn, Basti1012, tut es noch prima. Und so habe ich mit zwei Kommas (und besser Aufpassen beim Groß- und Kleinschreiben von Variablen) mein Progrämmchen ans Laufen gebracht. Mit Dank an Dich und Scatello hier die endgültige Fassung:

Code:

<?php
$mysqli = new mysqli("localhost", "root", "usbw", "test");
if (!empty($_POST)) {
   $er  = $_POST['er'];
   $sie = $_POST['sie'];
   $mysqli->query("UPDATE member SET Er='$er', Sie='$sie' WHERE id='1'"); }
$result = $mysqli->query("SELECT * FROM member WHERE id='1'");
$row = $result->fetch_assoc();
echo "<h2>".$row['Er']." liebt ".$row['Sie']."</h2>";
?>
<form method="post">
   Er:<br>
   <input type='text' size='20' name='er' value='<?php echo $row["Er"]?>'><br>
   Sie:<br>
   <input type='text' size='20' name='sie' value='<?php echo $row["Sie"]?>'><br><br>
   <input type="submit" value=" OK ">
</form>

scatello · 26.11.2021

Pilo schrieb:
hier die endgültige Fassung

Die aber grausam ist. Informiere dich zum Thema SQL-Injektion. Niemals einfach Daten aus einem Formular direkt in die Datenbank schreiben!

Pilo · 27.11.2021

Danke für den Hinweis. Folgende Änderung sollte - denke ich - genügen:

Code:

$er  = mysql_real_escape_string($_POST['er']);
$sie = mysql_real_escape_string($_POST['sie']);

scatello · 27.11.2021

Pilo schrieb:
Folgende Änderung sollte - denke ich - genügen:

Och büdde, du nutzt mysqli objektorientiert, also kann mysql_real_esacpe_string doch nicht richtig sein!

Siehe https://www.php.net/manual/de/mysqli.real-escape-string.php

Und das solltest du dir auch ansehen: https://www.php.net/manual/de/mysqli.error.php

Pilo · 27.11.2021

Ok, dann noch dies improvement (um dem "i" in mysqli gerecht zu werden):

Code:

   $er  = $mysqli->real_escape_string($_POST['er']);
   $sie = $mysqli->real_escape_string($_POST['sie']);

Update von Daten mittel Formular

Pilo

Member

scatello

Well-known member

Pilo

Member

scatello

Well-known member

Pilo

Member

scatello

Well-known member

Pilo

Member

scatello

Well-known member

basti1012

Well-known member

Pilo

Member

scatello

Well-known member

Pilo

Member

scatello

Well-known member

Pilo

Member

Update von Daten mittel Formular