Die häufigsten Fehler (vor dem Eröffnen eines neuen Beitrags lesen => Regeln!)

[RFC822]

Fehler, bei dem man mitunter lange sucht: Verwechslung zwischen == und ===
if (strpos($a, $b) === 0) { ... }
Wenn man wissen will, ob ein Zeichen an Position 0 steht, muss man === verwenden, weil (strpos() == 0) sowohl für strpos() === 0 als auch für strpos() === FALSE (wenn das Zeichen nicht gefunden wurde) WAHR sein würde!

[oPerILaZ]

variablen die im script benutzt werden, muessen erst definiert werden, auch wenn sie nur abgefragt werden...

// das erzeugt einen fehler:
for( $x = 0; $x < 3; $x++ )
{
$str .= 'bla';
}
echo $str;
// ....

// so ists richtig
$str = '';
for( $x = 0; $x < 3; $x++ )
{
$str .= 'bla';
}
echo $str;


moeglichkeit nicht definierte variablen trotzdem ohne fehlermeldung abzufragen ist isSet


if( !isset($foo) )
{
echo 'Unbekannte Variable : $foo '
}


blabla, man sowas in verstaendliche worte zu fassen ist gar nich mal so einfach...

[nimrod]

> // das erzeugt einen fehler:

kein fehler, just a notice

[Killsystem]

Das mit dem .= ist vielleicht verwirrend.
Mit einem = sieht es so aus

// das erzeugt einen fehler:
for( $x = 0; $x < 3; $x++ )
{
$str = $str+'bla';
}
echo $str;
// Ausgabe: Notice: Undefined variable: str in test.php on line 4

// so ists richtig
$str = '';
for( $x = 0; $x < 3; $x++ )
{
$str .= 'bla';
}
echo $str;
//Ausgabe "blablabla" 

Nur eine Ergänzung

[ta_sys]

fehlende varibalen in funktionen?
Viele vergessen die verwendeten Varibalen nicht "global" zu setzen.
Hier ein Bespiel:

	function generate_log($string) { 
		global $us_haendler_logs;
		global $REMOTE_ADDR;
		$time = mktime();
		mysql_query("INSERT INTO $us_haendler_logs ( id , ip , time , action ) VALUES ( '' , '$REMOTE_ADDR' , '$time' , '$string' )");	
	}

Die Remote Adresse (also IP des Nutzers) muss global gemacht werden um sie zu nutzen. Ebenso wie meine Sql-Tabelle welche in der config festgelegt wurde ($us_haendler_logs).

[ta_sys]

Komplexe Formulare erstellen / auswerten.
Problem: dynamische Varbiblennamen bei Formularen, aber wie?
Lösung:

<form action="index.php" method="post">
<?
$result = mysql_query("SELECT * FROM tabelle_mit_produkten");
while($out = mysql_fetch_array($result)) {
 ?>
 <input type="text" size="9" name="menge_<? echo $out[produkt_id]; ?>">
 <?
}
?>
<input type="hidden" name="berechnen" value="1">
<input type="submit" value="Bestellung berechnen">
</form>
<?

if($_POST[berechnen] == 1) {

 $result = mysql_query("SELECT * FROM tabelle_mit_produkten");
 while($out = mysql_fetch_array($result)) {
  $formular_inhalt = "menge_" . $out[produkt_id];
  echo $_GET[$formular_inhalt];
  // oder in array schreiben: $cr8_array .= $_GET[$formular_inhalt] . "<br>";

 }

 //echo $cr8_array;
}

[morpheus]

Ich moechte dir nicht zu nahe treten aber deine Tipps und Code-Beispeile sind schrecklich. Du solltest noch viel lesen und programmieren bevor du Tipps an andere weitergibst, weil vor allem Anfaenger deine Tipps ernst nehmen und verwenden koennten.

So hat man PHP-Code vielleicht vor Urzeiten geschrieben aber heute ist das schon lange nicht mehr der Fall.

Die Remote Adresse (also IP des Nutzers) muss global gemacht werden um sie zu nutzen. Ebenso wie meine Sql-Tabelle welche in der config festgelegt wurde ($us_haendler_logs).

Seit PHP 4.1 unterstuetzt PHP die sogenannten Superglobals, welche sich von jedem Scope aus nutzen lassen koennen und nicht erst per 'global' sichtbar gemacht werden muessen.

Several predefined variables in PHP are "superglobals", which means they are available in all scopes throughout a script. There is no need to do global $variable; to access them within functions or methods.

http://at2.php.net/manual/en/languag...perglobals.php

Diese solltest du auch verwenden, da $REMOTE_ADDR etc. schon lange obsolet ist.

Wenn du schon alle Tabellennamen in einem File stehen haben willst dann verwenden Konstanten und keine Variablen dafuer. Dafuer sind diese auch gedacht, da diese in Funktionen auch sichtbar sind und nicht erst per global eingebunden werden muessen.

Optimalerweise befinden sich alle SQL-Queries aber in einer Datei und lassen sich sehr leicht per Search&Replace anpassen falls das notig ist und Tabellennamen werden nicht durch konstanten ersetzt. Das was du machst (den SQL-Code in allen Dateien zu verstreuen) ist typischer Spaghetti-Code und absolut nicht zu empfehlen. So hat man Web-Anwendung vor vielen Jahren geschrieben und daher kommt auch der schlechte Ruf, den PHP hat. Leider gibt es auch heute noch viele Leute, welche das so machen.

Sieh dir mal das MVC-Pattern an. Das Symfony Book hat eine gute allgeimeine Erklaerung dazu und zeigt beispielshaft wie man dies in PHP umsetzen kann:
http://www.symfony-project.org/book/...Symfony-s-Code

[morpheus]

Komplexe Formulare erstellen / auswerten.
Problem: dynamische Varbiblennamen bei Formularen, aber wie?

Was sind denn dynamische Variablennamen bzw. was ist der Sinn davon.

Dieses Code-Snippet ist IMHO uebrigens auch bedenklich. Du mischt HTML und SQL-Queries und gibst Inhalt, welchen du vom Client erhalten hast ungefiltert aus. Das hat Sicherheitsprobleme zur Folge, da so jeder HTML und JavaScript-Code in deiner Website einbinden kann. Das ganze nennt sich XSS [1].

[1] http://www.cgisecurity.com/articles/xss-faq.shtml