Wo liegt der Fehler in den paar Codezeilen?

$link = mysql_connect("adresse", "name", "passwort")
or die("Keine Verbindung möglich!");
//Verbindung erfolgreich
mysql_select_db("counter")
or die("Auswahl der Datenbank fehlgeschlagen");
//Datenbankauswahl war erfolgreich

$result=mysql_query("DELETE FROM counter WHERE id=$id and zeit=time-14400")
if (!$result) {//IP gelöscht
++$counterstand
}
mysql_query("INSERT INTO counter (ip,zeit) VALUES ('$ip','time')");

// schliessen der Verbinung
mysql_close($link);

Ist das ein Quiz?

$result=mysql_query("DELETE FROM counter WHERE id=$id and zeit=$time-14400")
mysql_query("INSERT INTO counter (ip,zeit) VALUES ('$ip','$time')");

$time anstatt time nehme ich an.

Wieviele Punkte bekomme ich dafür?

Bitte das nächste mal etwas konkreter,
Danke

Sorry, bin absoluter Anfänger.

Damit will ich mir die aktuelle Zeit und das aktuelle Datum holen.

So ist das richtig?

Kommt drauf an wie Du das Datenfeld Zeit definiert hast.

now() liefert Dir Zeitwerte entweder mit oder ohne Trennzeichen

mysql> select now(), now()-0;
+---------------------+----------------+
| now() | now()-0 |
+---------------------+----------------+
| 2002-08-02 20:55:30 | 20020802205530 |
+---------------------+----------------+
1 row in set (0.00 sec)


Vielleicht klappt bei Dir

$result=mysql_query("DELETE FROM counter WHERE id=$id and zeit=now()-14400")
mysql_query("INSERT INTO counter (ip,zeit) VALUES ('$ip','now()-0')");


Wobei ich mir nicht sicher bin, ob Du nicht alle Records die älter als jetzt - 14400 sSekunden sind löschen möchtest, alao

$result=mysql_query("DELETE FROM counter WHERE id=$id and zeit<now()-14400")

Ob das mit dem now() paßt ist wie gesagt (geschrieben) abhängig davon wie das Feld zeit definiert ist.

zeit ist als datetime definiert.

Ich möchte ansonsten damit die Zeile löschen wo die IP übereinstimmt und die zeit innerhalt der letzten 4 Stunden liegt.

<?php
$link = mysql_connect("adresse", "name", "passwort")
or die("Keine Verbindung möglich!");

//Verbindung erfolgreich
mysql_select_db("counter")
or die("Auswahl der Datenbank fehlgeschlagen");
//Datenbankauswahl war erfolgreich

$result=mysql_query("DELETE FROM counter WHERE ip='$ip' and zeit < now() - INTERVAL 4 HOUR");
if (!$result) {
$counterstand++;
}
mysql_query("INSERT INTO counter (ip,zeit) VALUES ('$ip',now())");

// schliessen der Verbinung
mysql_close($link);
?>

Hi, hier mal der komplette Code, leider funktioniert es immer noch nicht.

<?php

$ip=getenv("REMOTE_ADDR");// Aktuelle IP Adresse von Besucher
$counterfile="counter.txt";

//counterzahl laden
if (!file_exists($counterfile)) {
$datei=fopen($counterfile,"r");
fgets($counterzahl);
fclose($datei);
}

// Verbindung aufbauen, auswählen einer Datenbank
$link = mysql_connect("adresse", "name", "passwort")
or die("Keine Verbindung möglich!");
//Verbindung erfolgreich
mysql_select_db("counter")
or die("Auswahl der Datenbank fehlgeschlagen");
//Datenbankauswahl war erfolgreich

$result=mysql_query("DELETE FROM counter WHERE ip='$ip' and zeit < now() - INTERVAL 4 HOUR");
if (!$result) {
++$counterstand;
}
mysql_query("INSERT INTO counter (ip,zeit) VALUES ('$ip',now())");

// schliessen der Verbinung
mysql_close($link);

//counterzahl speichern
if (!file_exists($counterfile)) {
$datei=fopen($counterfile,"w");
fputs($datei,$counterzahl);
fclose($datei);
}

//counter anzigen
$anzahlstellen=4;
// evtl. Auff&uuml;llen von Nullen auf anzahlstellen
$counterzahl=str_pad($counterzahl, $anzahlstellen, "0", STR_PAD_LEFT);
//$groesse=GetImageSize("0.png");//Gr&ouml;&szlig;e eines Bildes bestimmen, da alle gleich gro&szlig;
//$counter=ImageCreate(($anzahlstellen*$groesse[0]),$groesse[1]);
//Counterbild erstellen
For($i=0; $i<$anzahlstellen; $i++)
{
$src_img=ImageCreateFromPNG("$counterzahl[$i].png");
//entsprechendes Ziffernbild laden
ImageCopy($counter, $src_img, ($groesse[0] * $i), 0, 0, 0, $groesse[0], $groesse[1]);
//Bild an die entsprechende Stelle im Counterbild kopieren
ImageDestroy($src_img);}

Header("Content-type: image/png");
ImagePNG($counter);
ImageDestroy($counter);

?>

$counterstand++;
anstatt
++$counterstand;

Was funktioniert nicht? Bitte etwas konkreter. Beim Zahnarzt sagst Du ja auch welcher Zahn Dir weh tut oder sagst Du dort nur "Weh tut"?

Wenn Du es ihm nicht sagst riskierst Du damit, daß er Dir alle Zähne extrahiert. :D

Gibt es Fehlermeldungen? Wenn ja welche?

Vielleicht solltest Du beim Connect was konkretes eingeben?

$link = mysql_connect("adresse", "name", "passwort")

vielleicht

$link = mysql_connect("localhost", "root", "")

ich weiß es nicht, es kann ja adresse, name, passwort auch korrekt sein, who knows?

Da müßte doch eine Fehlermeldung kommen, aber vermutlich kommt da keine, sonst hättest Du es ja bekanntgegeben ....
:rolleyes:

Auswahl der Datenbank fehlgeschlagen!

Die Tabelle counter ist natürlich erzeugt:
ip varchar(15) default=0.0.0.0 not null
zeit datetime default=0000-00-00 00:00:00 not null

Auswahl der Datenbank fehlgeschlagen?

Na dann trag da bitte die korrekten Anmeldeparameter ein - die kannst nur Du in Erfahrung bringen - vielleicht passt das aber, es sei denn esw muß mit einem anderen User connectet werden oder mit einem anderen Passwort.

Sorry ich kann Dir nicht sagen wie User/Passwort für Deine Datenbank aussehen.

$link = mysql_connect("localhost", "root", "")

Die Anmeldeparameter sind aber richtig, der Fehler liegt in der Zeile:
mysql_select_db("counter")
Und die Tabelle die ich erstellt habe, heißt counter oder kommt da nicht der Tabellenname hin?

Danke, ich habs! Da kommt nicht der Tabellen, sondern der Datenbankname hin.

Gruß

Wenn keine Zeile mit den Bedingungen existiert, soll die Zahl von counterstand um eins erhöht werden. Wo liegt da der Fehler im Code, damit die Zahl erhöht wird?
$result=mysql_query("DELETE FROM counter WHERE ip='$ip' and zeit < now() - INTERVAL 4 HOUR");
if (!$result) {
$counterstand++;
}