Ganze Tabelle wird immer überschrieben :/

[AltSim]

Hallo!

Ich habe ein nerviges Problem!

Und zwar habe ich ein Formular erstellt das die eingegeben Daten an ein PHP Script übergibt und dieses dann die Daten auf meine Datenbank überträgt.

Der Code im Script sieht so aus:

PHP-Code:

<?php

...

// Variablen aus dem Forumlar übernehmen
  $nachname=$_POST['nachname'];
  $vorname=$_POST["vorname"]; 
  $adresse=$_POST["adresse"];
  $steuernummer=$_POST["steuernummer"];
  $zaehlerstand=$_POST["zaehlerstand"];
  $auswahl=$_POST["auswahl"];


// Überpruefen ob der Kunde schon vorhanden ist
  $query1 = "SELECT steuernummer FROM wasser_2012 WHERE steuernummer LIKE '$steuernummer' and nachname LIKE '$nachname'"; 
  $result = mysql_query($query1); // Fehlermeldung ausgeben, bzw. Kunden einfuegen
  $num=mysql_NumRows($result); 
  if ($num)
  {
    if ($auswahl == 1)
        {
            $query2="UPDATE wasser_2012
            SET nachname='$nachname',
            vorname='$vorname',
              adresse='$adresse',
            steuernummer='$steuernummer',
            zaehlerstand_quartal_1='$zaehlerstand';";
            
               $quartalausgabe = "Quartal I";            
            }
            
        elseif ($auswahl == 2)
        {
            $query2="UPDATE wasser_2012
            SET nachname='$nachname',
            vorname='$vorname',
              adresse='$adresse',
            steuernummer='$steuernummer',
            zaehlerstand_quartal_2='$zaehlerstand';";
            
               $quartalausgabe = "Quartal II";        
            }
            
        elseif ($auswahl == 3)
        {
            $query2="UPDATE wasser_2012
            SET nachname='$nachname',
            vorname='$vorname',
              adresse='$adresse',
            steuernummer='$steuernummer',
            zaehlerstand_quartal_3='$zaehlerstand';";
            
               $quartalausgabe = "Quartal III";        
            }
            
        else
        {
            $query2="UPDATE wasser_2012
            SET nachname='$nachname',
            vorname='$vorname',
              adresse='$adresse',
            steuernummer='$steuernummer',
            zaehlerstand_quartal_4='$zaehlerstand';";
            
               $quartalausgabe = "Quartal IV";             
            }
            
            
            $result2 = mysql_query($query2); 
            
    if ($result2)
    { 
      print"<p>Ihr Zählerstand in Höhe von <b>$zaehlerstand</b> für den <b>$quartalausgabe</b> wurde erfolgreich in die Datenbank aufgenommen.</p>";
     
      
    }
    else
    { 
       print"Datenbankfehler! Kunde konnte nicht aufgenommen werden!";
    }  
            
  } 
  
      else 
          { 
            echo "Ihre Steuernummer <b>$steuernummer</b> wurde nicht gefunden!";
          }  
      
    
  
  
mysql_close(); 
?>

Die Daten werden zuerst übernommen und auf Variablen gespeichert.
Dann werden in mySQL alle Datensätze ausgewählt wo die Steuernummer und der Nachname gleich sind wie sie im Formular eingegeben wurden.

Dannach wird geprüft ob die auswahl (das ist im Formular in Listenfeld) entweder 1, 2, 3 oder 4 ist. Dazu hab ich verschiedene IF Verzweigungen angelegt.

Der einzige Unterschied bei diesen Verzweigungen ist das der Zählerstand in unterschiedliche Spalten gespeichert wird.

Soweit so gut ... nur wenn ich im Formular nun einen Datenstatz eingebe überspielt er mir die ganze Tabelle ... jede einzelne Spalte .. so wie es im Formular eingegeben wurde.

Ich finde einfach nicht heraus warum ... komischereweise hat es schon perfekt geklappt .. ich weiß leider nicht mehr was ich zerissen haben könnte dass es nun nicht mehr funktioniert.
Ich hab den Code auch schon komplett neu geschrieben ... Tabelle neu angelegt ... ohne Erfolg.

Überseh ich da im Code etwas? Ist im mySQL im Hintergrund etwa am laufen das mir immer alle Datensätze bei einem Update auswählt?

Hoffe ihr könnt mir helfen!

mfg AltSim

[vt1816]

1. Übernimm niemals ungeprüft Daten in Deine Datenbank.
2. Wieso arbeitets Du mit LIKE aber ohne Platzhalter?
3. Bei einem UPDATE auf eine Datenbanktabelle ohne WHERE werden immer alle Datensätze aktualisiert.
4. ...

[AltSim]

Ok danke für deine Hinweise!

Mit dem UPDATE und dem WHERE hattest du natürlich Recht ... das war ein Typischer Anfängerfehler der nicht passieren hätte dürfen!

Ich hatte mir eingebildet dass ja eh nur diese Datensätze geändert werden die gändert werden müssen da ich ja oben bereits ein WHERE drinnen habe.

Jetzt sieht mein Code so aus:

PHP-Code:

<?PHP

// Variablen aus dem Forumlar übernehmen
  $nachname=$_POST['nachname'];
  $vorname=$_POST["vorname"]; 
  $adresse=$_POST["adresse"];
  $steuernummer=$_POST["steuernummer"];
  $zaehlerstand=$_POST["zaehlerstand"];
  $auswahl=$_POST["auswahl"];


// Überpruefen ob der Kunde schon vorhanden ist
  $query1 = "SELECT steuernummer FROM wasser_2012 WHERE steuernummer LIKE '$steuernummer' and nachname LIKE '$nachname'"; 
  $result = mysql_query($query1); // Fehlermeldung ausgeben, bzw. Kunden einfuegen
  $num=mysql_NumRows($result); 
  if ($num)
  {
    if ($auswahl == 1)
        {
            $query2="UPDATE wasser_2012
            SET zaehlerstand_quartal_1='$zaehlerstand'
            WHERE steuernummer='$steuernummer' AND nachname='$nachname'";
            
               $quartalausgabe = "Quartal I";            
            }
            
        elseif ($auswahl == 2)
        {
            $query2="UPDATE wasser_2012
            SET zaehlerstand_quartal_2='$zaehlerstand'
            WHERE steuernummer='$steuernummer' AND nachname='$nachname'";
            
               $quartalausgabe = "Quartal II";        
            }
            
        elseif ($auswahl == 3)
        {
            $query2="UPDATE wasser_2012
            SET zaehlerstand_quartal_3='$zaehlerstand'
            WHERE steuernummer='$steuernummer' AND nachname='$nachname'";
            
               $quartalausgabe = "Quartal III";        
            }
            
        else
        {
            $query2="UPDATE wasser_2012
            SET zaehlerstand_quartal_4='$zaehlerstand'
            WHERE steuernummer='$steuernummer' AND nachname='$nachname'";
            
               $quartalausgabe = "Quartal IV";             
            }
            
            
            $result2 = mysql_query($query2); 
            
    if ($result2)
    { 
      print"<p>Ihr Zählerstand in Höhe von <b>$zaehlerstand</b> für den <b>$quartalausgabe</b> wurde erfolgreich in die Datenbank aufgenommen.</p>";
     
      
    }
    else
    { 
       print"Datenbankfehler! Kunde konnte nicht aufgenommen werden!";
    }  
            
  } 
  
      else 
          { 
            echo "Ihre Steuernummer <b>$steuernummer</b> oder ihr Name <b>$nachname</b> wurde nicht gefunden!<br>Bitte überprüfen Sie Ihre Eingaben!";
          }  
      
    
  
  
mysql_close(); 
?>

Meinst du mit dem prüfen dass ich die übergebenen Werte auf ihren Typ prüfen soll?
-> isnumeric usw.?

[vt1816]

Zitat von AltSim

[..]
Meinst du mit dem prüfen dass ich die übergebenen Werte auf ihren Typ prüfen soll?
-> isnumeric usw.?

Zum Beispiel.

Stichwort SQL Injektion