fastix®: Was ist an dem Counter falsch?

Beitrag lesen

Moin!

$v = "SELECT * FROM counter WHERE IP = $ip";
$sql = mysql_query($v,$verbindung);
if (mysql_num_rows($sql) <> "") {

Also: es funktioniert, aber ich würde die variablen anders bennenn. Dir mag es egal sein, aber auch Du musst Dein Projekt mal pflegen:

$sql wäre die bessere Variable für den SQL- String.
$result wäre der bessere Vorschlag für das zrückgelieferte Ergebnis.
mysql_num_rows($result) liefert ein numerisches Ergebnis, Du kannst als direkt nachschauen, ob gleich Null ist.

Also:

$sql = "SELECT IP FROM counter WHERE IP = $ip";
$result = mysql_query($sql,$verbindung) OR die("<hr>Datenbankabfrage hat nicht geklappt:<br>$sql<br>mysql_error()");
if (mysql_num_rows($result) == 0)
{
    $sql = "INSERT INTO counter (IP) VALUES ('$ip')";
    $result2 = mysql_query($eintrag,$verbindung) or die(<hr>Datenbankeintrag hat nicht geklappt:<br>$sql<br>mysql_error()");
    eins();
    mysql_free_result($result2);
}
mysql_free_result($result); # Immer tapfer den Speicher wieder freigeben...

MFFG (Mit freundlich- friedfertigem Grinsen)

fastix®

--
Meinereinerselbst ist auf der Suche nach Aufträgen