✔ Fehler: Undefined variable ...

[JamesT]

Kurze Frage,

bei meinem Script erhalte ich den Fehler

Notice: Undefined variable: id in /home/.../index.php on line 5

Ich poste mal Zeile 5-10 des Quellcodes:

	if($id AND $do=="delete"){
		mysql_query("DELETE FROM
							drucker
						WHERE
							id=".$_GET['id']);
	}

Ich habe gelesen das ich da mit "f(!isset($wert))" was machen kann.

Nur weiss ich nicht wie.

Kann mir jemand abhilfe schaffen?

Vielen Dank und Grüße aus Bielefeld

JamesT

 

[Hroudtwolf]

Ich habe gelesen das ich da mit "f(!isset($wert))" was machen kann.

isset($id) gibt True zurück wenn die Variable gesetzt ist, also existiert.
Also was du brauchst.

If (isset($id))
{ 
 Echo "Es komme was wolle";
}

 

[5noOPY]

	if($id AND $do=="delete"){
		mysql_query("DELETE FROM
							drucker
						WHERE
							id=".$_GET['id']);
	}

Ersetzte doch mal dein $id mit $_GET['id'] in der Bedingung deiner IF-Abfrage.
Also so:

	if($_GET['id'] && $do=="delete"){
		mysql_query("DELETE FROM
							drucker
						WHERE
							id=".$_GET['id']);
	}

mfg 5noOPY

 

[JamesT]

Ich hab das jetzt nach 5noopys tipp angepasst, und erhalte nun den Fehler "undefined index" anstatt "undefined variable"

BTW: Wenn ich error_reporting in der php ini ausmache, läuft das script. Aber ich möchte die Fehler gern weghaben, auch wenn Sie eigentlich nicht stören.

Grüße

JamesT

 

[rexxitall]

Ihr habt immer das Problem das ihr scheinbar immer $_GET und $_POST unreflektiert einsetzt. Immer gleich in den Aufruf rein und dann wundern wo es kracht ...


Was spricht eigentlich gegen :

$id=$_GET["id"]; 

//$id ist nun garantiert da wenn auch ggf als leere Variable

if (($do=="delete") and ($id <>"")){
    echo "machwas";
}

 

[SnEaKy]

Also wenn man den Teil davor richtig programmiert hat, dann kann man $_GET und $_POST auch direkt in Aufrufe oder Sonstiges reinschreiben. Natürlich muss man sich dann sicher sein, dass die Variablen auch existieren!

Die Variante

$id=$_GET["id"];  

//$id ist nun garantiert da wenn auch ggf als leere Variable 

if (($do=="delete") and ($id <>"")){ 
    echo "machwas"; 
}

ist natürlich auch möglich, aber ich persönlich bin kein Freund die $_GET und $_POST erst nochma einer anderen Variable zuzuordnen, wenn ich mir doch sicher bin, dass die aufjedenfall da sind. In meinen Augen doppelte Arbeit. Deswegen empfehle ich die Variante

If (isset($_GET["id"]))
{ 
 Echo "Es komme was wolle";
}else{
 Echo "Keine Variable vorhanden";
}

Es wird überprüft ob es die Variable gibt und wenn ja - auch wenn sie einen leeren Wert enthält - kann ich sie benutzen.

Und um nochmal auf das Problem zurückzukommen, ich nehme stark an, dass entweder $id oder $do nicht existieren Lass dir doch mal vor der Abfrage beide Variablen ausgeben mit

echo $id."<br>".$do;

Dann kannst du sehn, wie die Werte sind und ggf bekommst du dann einen Fehler in Line 4 weil wirklich eine nicht existiert

 

[rexxitall]

Geschmackssache ich bezeichne sowas als klingonischen Programmierstil
Wenn du nachher Fehler suchst machst du dir mit so etwas nur das Leben schwer.
Da tipp ich lieber eine Zeile mehr und verbrate temporär ein paar Bytes Speicherplatz
Und lesbarer / nachvollziehbarer werden solche Konstrukte auch nicht.

 

[JamesT]

Vielen Dank für eure Tipps, ich habe es in den Griff bekommen. ;-)

Grüße

Jamest

 

[Gumbo]

Ihr habt immer das Problem das ihr scheinbar immer $_GET und $_POST unreflektiert einsetzt. Immer gleich in den Aufruf rein und dann wundern wo es kracht ...

Wozu den Variablenwert zusätzlich in einer Variable speichern? Dadurch erhöht sich doch bloß der Speicherbedarf des Skriptes.

Ich versuche wenn möglich direkt mit den vordefinierten Variablen zu arbeiten. Und falls nicht, speichere ich nur den gefilterten/validierten Wert in einer zusätzlichen Variable, beispielsweise:

if( isset($_GET['id']) && intval($_GET['id']) <= 0 ) {
	echo 'Falscher ID-Wert';
	$id = null;
} else {
	$id = intval($_GET['id']);
}

// …

if( !empty($id) ) {
	// gültiger ID-Wert
}

Dass in diesem Fall die ID nicht-numerisch ist, kann auf einen Angriff hindeuten und sollte deswegen sogar protokolliert werden.