Counter schreiben... Hilfe!

[SNCIS]

Hi,

ich bin gerade dabei mir ein eigens PHP Script für eine Counter zu schreiben, der eine ID Sperre enthält. Nur bekomme ich andauernd eine Fehlermeldung, obwohl alles richtig sein müsste. Wer kann mir helfen?


<?php

// Connect zur Datenbank
$server = "mysql.xxxxxxx.de";
$db = "xxxxxx";
$user = "xxxxxx";
$pw = "xxxxxx";

$conn = mysql_connect($server, $db, $pw);

// Fehlerausgabe bei nicht vorhandender Verbindung
if($conn) {
echo "Die Verbindung zur Datenbank $db wurde hergestellt";
} else {
echo "Es konnte keine Verbindung zur Datenbank $db hergestellt werden";
}

//Auswahl der Tabelle
$select = mysql_select_db("counter", $conn);

// Auslesen der Datenbank
$sql = "SELECT * FROM counter";
$result = mysql_query($sql, $conn);
$numrow = mysql_num_rows($result);
echo $numrow;

while ($zeile = mysql_fetch_row($result))
{
echo "Besucher seit dem 01.01.2002";
implode($zeile);
echo $count;
echo $ip;
}

// Erhöhung des Counters nur bei ungleicher IP
if ($ip != $REMOTE_ADDR)
{
$count = $count++;
}

// Überschreiben des Datensatzes
"Update counter SET counter = '$counter', ip = '$REMOTE_ADDR' WHERE ip = '$ip'";
$result = mysql_query($sql, $conn);
}
mysql_close($conn);

?>

[Stuck Mojo]

Hi !!

Also auf die Schnelle....

-> implode($zeile)

Flascher Syntax für diese Funktion...

-> implode(";",$zeile);
...aber warum benutzt du das überhaupt??

Und wo deklarierst du die Variablen für $count und $ip ??

Isn bissel schwer da so auf die Schnelle durchzusteigen...weil mir einiges merkwürdig vorkommt...
Soll für jeden Besucher ein neuer Datensatz in der DB angelegt werden??
...mmhhh... Wieso ließt du alle Datensätze nacheinander aus, und erhöhst nur den letzten um 1??
mmhh... erklär mal ganz genau, wie du dir das vorgestellt hast. So wie's jetzt aussieht muß dort einiges umgestellt werden. Gut wäre auch noch die DB-Struktur...

Gruß Jan

[Lars]

Ersetze doch mal

PHP-Code:

// Fehlerausgabe bei nicht vorhandender Verbindung 
if($conn) { 
echo "Die Verbindung zur Datenbank $db wurde hergestellt"; 
} else { 
echo "Es konnte keine Verbindung zur Datenbank $db hergestellt werden"; 
} 


durch

PHP-Code:

mysql_connect($server, $db, $pw) or die (mysql_error()); 


Dann bekommst Du die SQL-Fehlermeldungen, welche Du mitsamt den eventuellen PHP-Fehlermeldungen hier posten könntest.

Ausserdem verstehe ich nicht so ganz, warum Du mitten im Skript die DB auswählst, obwohl das schon beim mysql_connect geschehen ist!?
Eine Tabelle muss nicht gewählt werden, da das ja beim SELECT * FROM TABLE passiert.
Aber genau das versuchst Du augenscheinlich (ich beziehe mich da auf den Kommentar).

[SNCIS]

Also,

mein Counter soll nur die Gesamtzahl der User zählen die meine Seite besucht haben und wie viele User gerade Online sind. Ich habe hier momentan eine Tabelle die sich Counter nennt mit den Feldern Counter und IP. Solange der User mit der gleichen IP auf meiner Seite ist, soll der Counter nicht höher zählen. Das soll schon das ganze Script sein.

[Lars]

Wie wäre es denn mit einer Session?

[SNCIS]

@schmobi

Mit deiner Variante funtzt es leider auch nicht. Trotdem danke für deine Hilfe.

[Lars]

Ich bleib am Ball ...