mySQL Update mit $_POST und Konstanten funktioniert einfach nicht

hpuettma

Hallo, ich bin es schon wieder.
Es ist mir schon fast peinlich, dass ich Euch schon wiederholt mit meinen Problemen belästigt habe.
Aber leider habe ich ein Problem, dass ich alleine nicht lösen kann.

Über eine $_POST Eingabe übergebe ich ein Datum und eine Anzahl, für die ich Datensätze mit geringfügig unterschiedlichem Inhalt in die DB schreibe (generiere). Dann zeige ich die geschriebenen Datensätze an, selektiere EINEN, um weitere Daten hinzuzufügen. Zunächst wird die $_POST Eingabe der Anzahl übernommen, dann sollen noch zwei weitere Konstanten (Nachname und Vorname) in den selektierten Datensatz geschrieben werden.
Die $_POST Eingabe wird in den Datensatz geschrieben, die beiden anderen Felder aber nicht.
Über Google habe ich einen Hinweis auf den Zeitpunkt gefunden, zu dem die Daten verfügbar sind. Leider konnte ich damit nicht sehr viel anfangen.
Ich schaffe es einfach nicht, die beiden Update-Befehle zu kombinieren und alle drei Informationen gemeinsam wegzuschreiben. Ein Versuch die Daten über zwei Update-Befehle zu schreiben – man muss sich zu helfen wissen - scheitert genauso. Vielleicht kann mir jemand einen FUNKTIONIERENDEN Update-Befehl zuschicken, der diese Problematik löst. Der Code ist nur ein Ausschnitt!

Danke für Eure Hilfe.

PHP

<html>
<?php
   header('Content-Type: text/html; charset=ISO-8859-1');
?>
<head>
  <link rel="stylesheet" type="text/css" href="tische_buchen.css">


<script type="text/javascript">
function send(ak,id)
{

   if(ak==1)
            document.f.ak.value = "up";
      else
            return;

   document.f.id.value = id;
   document.f.submit();
}
</script>
</head>


<body>
<h2>Tisch belegen</h2>
<p> </p>
 
<?php


   $nach = "Nachname";
   $vor = "Vorname";

   SESSION_START();


   $con = mysqli_connect("","root");
   mysqli_select_db($con, "firma");


   /* Aktion ausführen */

   if(isset($_POST['ak']))

   {
      /* ändern */
      if($_POST['ak']=="up")
      {

         $id = $_POST['id'];
/*
         Der Befehl funktioniert vorne und hinten nicht - habe gefühlte tausend Varianten versucht!
         $sql = "update reservierungen set "
           . "resanzpers =     '"     . $_POST['pe'][$id] . "'"
           . "reskundenach = '$nach'"                    
           . "reskundevor  = '$vor'"
           . " where res_id = $id";




         Der folgende Befehl funktioniert     
*/           
         $sql = "update reservierungen set "
           . "resanzpers =     '"     . $_POST['pe'][$id] . "'"
           . " where res_id = $id";


         mysqli_query($con, $sql);
         echo "Reservierung wurde aktualisiert";
      }
   }


   /* Formular-Beginn */

Alles anzeigen

Bandit

Richtig debuggen

1. Man bemerkt, dass ein Skript nicht das tut, was es soll.
2. Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(-1);
3. Man verwendet ini_set('display_errors', true); damit die Fehler auch angezeigt werden.
4. Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
5. 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.
6. Schritt 5 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
7. Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
8. Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
9. Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

hpuettma

Hallo Bandit, ich werde versuchen diese wissenschaftliche Abhandlung zu verstehen und umzusetzen. Dazu werde ich sicherlich einige Tage benötigen. Danke für die für mich sehr komplizierten Hinweise. Dann melde ich mich wieder.

hpuettma

Hallo nochmal,

ich habe den Quellcode mal aktiviert und den nicht funktionierenden Teil laufen lassen. Ich bekomme keinen Syntax-Fehler und auch über das Error-Reporting keinerlei Hinweise. Der Code läuft einfach durch, das Ergebnis ist aber in der DB nicht zu sehen.
Über andere Foren habe ich gesehen, dass dieses Problem viele User in der Vergangenheit hatten. Eine zufriedenstellende Lösung habe ich aber nicht gefunden. Vielleicht kann Arne sich das mal ansehen, denn die wissenschaftlichen Abhandlungen von Bandit bringen mich immer weiter ab von meiner Lösung. Ich brauche nämlich einen Tipp und keine Zurechtweisung. Danke.

HTML

<html>
<?php
   header('Content-Type: text/html; charset=ISO-8859-1');
?>
<head>
  <link rel="stylesheet" type="text/css" href="buchen.css">


<script type="text/javascript">
function send(ak,id)
{

   if(ak==1)
            document.f.ak.value = "up";
      else
            return;

   document.f.id.value = id;
   document.f.submit();
}
</script>
</head>


<body>
<h2>belegen</h2>
<p> </p>
 
<?php


   $nach = "Nachname";
   $vor = "Vorname";

   SESSION_START();


   $con = mysqli_connect("","root");
   mysqli_select_db($con, "firma");


   /* Aktion ausführen */

   if(isset($_POST['ak']))

   {
      /* ändern */
      if($_POST['ak']=="up")
      {

         $id = $_POST['id'];


         $sql = "update reservierungen set "
           . "reskundenach = '$nach'"                    
           . "reskundevor  = '$vor'"
           . " where res_id = $id";
/*
         $sql = "update reservierungen set "
           . "resanzpers =     '"     . $_POST['pe'][$id]         . "'"
           . " where res_id = $id";
*/
         mysqli_query($con, $sql)or die(mysql_error());
         echo "Daten wurde aktualisiert";
      }
   }


   /* Formular-Beginn */

Alles anzeigen

- - - Aktualisiert - - -

Das Feld im Kommentarteil "resanzpers" wird in die DB geschrieben, wenn kein weiteres Feld angegeben wird. Sind alle 3 Felder angegeben wird KEIN Feld in die DB geschrieben.

- - - Aktualisiert - - -

Sorry, ich hatte den falschen Quellcode gesendet, hier der richtige. Die Aussage von mir trifft aber weiterhin zu.

HTML

/* ändern */
      if($_POST['ak']=="up")
      {

         $id = $_POST['id'];


         $sql = "update reservierungen set "
           . "resanzpers =     '"     . $_POST['pe'][$id]         . "'"
           . "reskundenach = '$nach'"                    
           . "reskundevor  = '$vor'"
           . " where res_id = $id";
/*
         $sql = "update reservierungen set "
           . "resanzpers =     '"     . $_POST['pe'][$id]         . "'"
           . " where res_id = $id";
*/
         mysqli_query($con, $sql)or die(mysql_error());
         echo "Daten wurde aktualisiert";

Alles anzeigen

Bandit

Zitat von hpuettma

Ich brauche nämlich einen Tipp und keine Zurechtweisung. Danke.

Zurechtweisung? Geht's noch? Was hat das damit zu tun? Das sind Hinweise, wie man einem Fehler auf die Spur kommen kann. Und unter "Richtig debuggen" steht mehr als nur das Erhöhen des Error-Reportings! Und das Ende vom Lied ist, dass du in dem zuletzt geposteten Script kein Error-Reporting hochgedreht hast und keine Ausgabe von Variablen drin hast. Weiterhin überprüfst du auf DB-Fehler falsch. Von SQL-Injektions will ich erst garnicht anfangen, da wir ja sonst wieder bei wissenschaftlichen Abhandlungen sind.

Aber ich sage jetzt besser nichts mehr, denn sonst weise ich dich ja zurecht, und das möchte ich nicht.

hpuettma

Hallo Bandit, der Kommentar war nicht persönlich gemeint. Das Debugging ist für mich ein Buch mit sieben Siegeln und ich habe keine Ahnung wie man es sinvoll codiert. Eigentlich wollte ich einen Tipp bekommen und nicht tagelang ein neues Kapitel von PHP und MySQL aufmachen. Dann habe ich vielleicht später mein eigentliches Problem ganz vergessen. Ein Beispiel-Code mit eingebautem Debugging wäre toll gewesen.

Bandit

Zitat von hpuettma

Ein Beispiel-Code mit eingebautem Debugging wäre toll gewesen.

Also, wenn du es nicht schaffst, ein

PHP

echo "Vor mysqli_query, Query = $sql<br>";

selber einzubauen, dann suche dir ein anderes Hobby, programmieren ist dann nämlich nichts für dich, was ja auch nicht schlimm ist.

PHP

mysqli_query($con, $sql) 
   or die("MySQL-Error: " . mysqli_error($con));

hpuettma

Übrigens - mit ECHO habe ich die beiden Variablen ausgegeben. Die Inhalte sind vor dem Update o.k.

- - - Aktualisiert - - -

Das ist die Fehlerausgabe mit der Debugging Unterstützung:

Code

Nachname: Nachname Vorname: VornameVor mysqli_query, Query = update  reservierungen set resanzpers = 	'6'reskundenach = 'Nachname'reskundevor   = 'Vorname' where res_id = 1004
MySQL-Error: You have an error in  your SQL syntax; check the manual that corresponds to your MariaDB  server version for the right syntax to use near 'reskundenach =  'Nachname'reskundevor  = 'Vorname' where res_id = 1004' at line 1

Jetzt weiß ich also, dass ich einen Systax-Error habe ...

Zu meinem Hobby: Ich bin 67 Jahre alt, habe bereits vor mehr als 40 Jahren beruflich Batch-Programme in Assembler und Cobol programmiert und hatte immer viel Freude an Programmieren. Die Dialog-Programmierung ist mir allerdings noch sehr fremd. Da ich nicht einrosten möchte, habe ich gedacht, man könnte sich durch PHP und mySQL geistig fit halten.

Bandit

Wenn du schon programmiert hast, dann verstehe ich nicht, wieso du mit einer einfachen Variablenausgabe Probleme hast und dir das nicht selber auffällt:

update reservierungen set resanzpers = '6'reskundenach = 'Nachname'reskundevor = 'Vorname' where res_id = 1004

Ein Komma an der richtigen Stelle soll manchmal Wunder bewirken :-D:-D

update reservierungen set resanzpers = '6', reskundenach = 'Nachname', reskundevor = 'Vorname' where res_id = 1004

hpuettma

In meinem ersten Post hatte ich schon gesagt, dass ich tausend Varianten des Update-Befehls ausprobiert habe.
Mit Komma im Befehl erhalte ich einen Systax-Fehler

HTML

$sql = "update reservierungen set "
           . "resanzpers =     '"     . $_POST['pe'][$id] . "'",
           . "reskundenach = 'Nachname'",                    
           . "reskundevor  = 'Vorname'",
           . " where res_id = $id";

Leider muss ich jetzt kurz außer Haus! Bin gegen Mittag wieder erreichbar. Sorry.

Bandit

PHP

$sql = "update
          `reservierungen`
         set
           `resanzpers` = '" . $_POST['pe'][$id] .  "',
           `reskundenach` = 'Nachname',
           `reskundevor`  = 'Vorname'
         where
           `res_id` = $id";

hpuettma

auch dieser Code scheitert mit einem Syntax-Fehler

HTML

$sql = "update reservierungen set "
           . "resanzpers =     '"     . $_POST['pe'][$id] . "'",
           . "reskundenach = 'Nachname'",                    
           . "reskundevor  = 'Vorname'"
           . " where res_id = $id";

Arne Drews

Hallo, die Komma liegen außerhalb des String. Die gehören vor die DoubleQuotes.

Bandit

Frage: wieso ignorierst du Posting #11???

hpuettma

Hallo Bandit, dein Quellcode und mein Post mit einem weiteren Versuch hatten sich überschnitten. Nachdem ich nun Deinen Code eingebaut habe, bekomme ich leider wieder einen Fehler: undefined Index für "PE". Die beiden Variablen werden geschrieben, der POST-Eintrag aber nicht.

Code

/* ändern */
      if($_POST['ak']=="up")
      {

         $id = $_POST['id'];

        $sql = "update
          `reservierungen`
         set
           `resanzpers` = '" . $_POST['pe'][$id] .  "',
           `reskundenach` = 'Nachname',
           `reskundevor`  = 'Vorname'
         where
           `res_id` = $id";  


         echo "Vor mysqli_query, Query = $sql<br>";  
         mysqli_query($con, $sql)
            or die("MySQL-Error: " . mysqli_error($con));
         echo "Daten wurde aktualisiert";

Alles anzeigen

Ich habe im Internet einen Hinweis gefunden, dass der Zeitpunkt zu dem die Daten verfügbar sind unterschiedlich ist, ob es eine POST-Information oder eine Variable ist...

Bandit

Zitat von hpuettma

undefined Index für "PE"

Dann gibt es $_POST['pe'] nicht.

hpuettma

Doch! - Denn wenn ich den Quellcode ohne die Variablen nutze funktioniert es doch

Arne Drews

Ich würde das ganze erstmal übersichtlicher aufbauen:

PHP

if( $_POST['ak'] == 'up' ) {


    $id = $_POST['id'];


    $sql_tmp = "update `reservierungen` set `resanzpers` = '{post_id}', `reskundenach` = 'Nachname', `reskundevor`  = 'Vorname' where `res_id` = {id}";
    $sql = strtr($sql_tmp, [
            '{post_id}' => $_POST['pe'][$id],
            '{id}' => $id
        ]);


    echo 'Vor mysqli_query, Query = ' . $sql . '<br>';


    mysqli_query($con, $sql)
        or die('MySQL-Error: ' . mysqli_error($con));


    echo 'Daten wurde aktualisiert';


}

Alles anzeigen

Weiterhin lügt PHP nicht. Wenn er sagt, dass $_POST['pe'] nicht existiert, existiert es an der Stelle auch nicht!
Wird Dein $_POST davor irgendwo überschrieben vielleicht? Oder kommt diese Meldung nur beim erstmaligen Aufrufen der Seite?
Oder die Meldung kommt von einer ganz anderen Zeile?!

hpuettma

ich muss mich korrigieren, mit DEINEN Code und nur der POST Information hatte ich noch nicht getestet. Werde ich machen und melde mich ein wenig später.

Hallo Arne, ich werde den neuen Code in eine eigene Datei übernehmen und neu testen, dauert ein wenig.

- - - Aktualisiert - - -

Hallo zusammen, ich habe den korrigierten Quellcode von Arne in meinen Ablauf kopiert und getestet. Der Syntaxfehler ist weg! Das Update funktioniert trotzdem nicht, siehe Ausgabeergebnis.

HTML

<html>


<?php


       header('Content-Type: text/html; charset=ISO-8859-1');

 ?>
<head>
  <link rel="stylesheet" type="text/css" href="tische_buchen.css">


<script type="text/javascript">
function send(ak,id)
{

   if(ak==1)
            document.f.ak.value = "up";
      else
            return;

   document.f.id.value = id;
   document.f.submit();
}
</script>
</head>


<body>
<h2>Tisch belegen</h2>
<p> </p>
 
<?php
    ini_set('display_errors', 1);
    error_reporting(E_ALL);
   // error_reporting(-1);


   
   $nach = "Nachname";
   $vor = "Vorname";

   SESSION_START();


   $con = mysqli_connect("","root");
   mysqli_select_db($con, "firma");


   /* Aktion ausführen */

   if(isset($_POST['ak']))

   {
      /* ändern */
      if( $_POST['ak'] == 'up' ) {


        $id = $_POST['id'];


         $sql_tmp = "update `reservierungen` set `resanzpers` =  '{post_id}', `reskundenach` = 'Nachname', `reskundevor`  = 'Vorname'  where `res_id` = {id}";
        $sql = strtr($sql_tmp, [
            '{post_id}' => $_POST['pe'][$id],
            '{id}' => $id
        ]);


    echo 'Vor mysqli_query, Query = ' . $sql . '<br>';


    mysqli_query($con, $sql)
        or die('MySQL-Error: ' . mysqli_error($con));


    echo 'Reservierungen wurde aktualisiert';

      }
   }


   /* Formular-Beginn */
   echo "<form name='f' action='3tische_buchen.php' method='post'>";
   echo "<input name='ak' type='hidden' />";
   echo "<input name='id' type='hidden' />";


   /* Tabellen-Beginn */
   echo "\n\n<table border>"
    . "<tr>"
    . "<td>ResDatum</td>"
    . "<td>ResZeit</td>"
    . "<td>Tisch</td>"
    . "<td>MinPers</td>"
    . "<td>MaxPers</td>"
    . "<td>AnzPers</td>"
    . "<td>Status</td>"
    . "</tr>";


       /* SQL-Abfrage ausführen */

    $resdatum2 = $_SESSION['resdatum2'];
    $anzpers = $_SESSION['respersonen'];
    echo "Reservierung für Anzahl Personen: " . $anzpers;
    echo "<br />";
    echo "<br />";

    $sql  = "select * from reservierungen";
    $sql .= " where resdatum = $resdatum2";
    $sql .=    " and resmin <= $anzpers";
    $sql .=    " and resmax >= $anzpers";
    $sql .= " and resanzpers = 0";
    $sql .=    " order by resdatum";


    $res = mysqli_query($con, $sql);

    /* Anzahl Datensätze ermitteln und ausgeben */
       $num = mysqli_num_rows($res);


    // echo "$num Reservierungs-Datensätze gefunden<br />";
    // echo "<br />";

   /* Alle vorhandenen Datensätze */
   while ($dsatz = mysqli_fetch_assoc($res))
   {
      $id = $dsatz['res_id'];
      echo "\n\n<tr>"
       . "<td><input name='rd[$id]' value='" . $dsatz['resdatum']         . "' size='15' /></td>"
       . "<td><input name='rz[$id]' value='" . $dsatz['reszeit']         . "' size='10' /></td>"
       . "<td><input name='ti[$id]' value='" . $dsatz['restisch']         . "' size='10'  /></td>"
       . "<td><input name='mi[$id]' value='" . $dsatz['resmin']         . "' size='10'  /></td>"
       . "<td><input name='ma[$id]' value='" . $dsatz['resmax']         . "' size='10'  /></td>"
       . "<td><input name='pe[$id]' value='" . $dsatz['resanzpers']     . "' size='10'  /></td>"

       . "<td><a href='javascript:send(1,$id);'>Tisch wird jetzt belegt</a>"
       . "</tr>";
   }
   echo "</table>";
   echo "</form>";

   mysqli_close($con);


?>
</body>


</html>

Alles anzeigen

Ausgabeergebnis:

HTML

Tisch belegen  
   Vor mysqli_query, Query = update `reservierungen` set `resanzpers` =  '4', `reskundenach` = 'Nachname', `reskundevor`  = 'Vorname' where  `res_id` = 1105
Reservierungen wurde aktualisiert  Reservierung für Anzahl Personen: 2


ResDatumResZeitTischMinPersMaxPersAnzPersStatus  Tisch wird jetzt belegt  Tisch wird jetzt belegt

Ich habe die beiden Post-Felder mit einem Vorschlagswert vorbelegt (Tagesdatum und Anzahl Personen 2). Da ich keinen Syntax-Fehler bekomme, MUSS doch das Feld $_Post['pe'] vorhanden sein. Vielleicht wird es über die Vorschlagswerte überschrieben???? Noch eine Frage in diesem Zusammenhang. Die beiden Variablen habe ich in der Ausgabe nicht angelistet. Wichtig ist nur die Personenzahl aus dem Post. Kann das zu einem Fehler führen?

- - - Aktualisiert - - -

Hier ein Auszug der DB:

HTML

| Alles anzeigen|


Anzahl der Datensätze: 
Zeilen filtern: Nach Schlüssel sortieren:   


 
+ Optionen    res_id  resdatum  reszeit  restisch  resmin  resmax  resanzpers  reskundenach  reskundevor  reskundetel        Bearbeiten   Kopieren   Löschen 1105 2017-05-02 1 3 2 4 






       Bearbeiten   Kopieren   Löschen 1106 2017-05-02 1 5 1 2 






       Bearbeiten   Kopieren   Löschen 1107 2017-05-02 1 6 4 8 






     
  Bearbeiten   Kopieren   Löschen 1108 2017-05-02 1 7 2 2

Alles anzeigen

Der ausgewählte Datensatz 1105 wurden nicht geschrieben.

- - - Aktualisiert - - -

Ich habe das Ausgabeergebnis mal per Hardcopy eingefügt. Dann kann man sich den Ablauf besser vorstellen. Über einen LINK im Statusfeld "soll" die Eingabe im Feld Anzpers und die beiden Konstanten in die DB geschrieben werden.

Tisch belegen

Vor mysqli_query, Query = update `reservierungen` set `resanzpers` = '4', `reskundenach` = 'Nachname', `reskundevor` = 'Vorname' where `res_id` = 1105
Reservierungen wurde aktualisiert Reservierung für Anzahl Personen: 2

ResDatum	ResZeit	Tisch	MinPers	MaxPers	AnzPers	Status
						Tisch wird jetzt belegt
						Tisch wird jetzt belegt

- - - Aktualisiert - - -

Das Hardcopy hat nicht alle Werte gezeigt. Hier noch ein Snapshot, die Tabellenwerte sind aus den bestehenden Datensätze vorbelegt!

Bandit

Und genau deshalb ist "Richtig debuggen" so extrem wichtig!!!

PHP

echo "<input name='id' type='hidden' />";

PHP

$id = $_POST['id'];

Was soll da wohl in $id stehen?

Ich würde dir auch dringend empfehlen, erstmal ohne JavaScript zu arbeiten.

- - - Aktualisiert - - -

Ändere mal

PHP

if(isset($_POST['ak']))
{

in

PHP

if(isset($_POST['ak']))
{
   echo "<pre>"; var_dump($_POST); die ("</pre>");

und sieh' dir die Ausgabe an

mySQL Update mit $_POST und Konstanten funktioniert einfach nicht

Tags

Benutzer online in diesem Thema