Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource

Darkxor

Hallo,
ich wollte mit einem Script + Cronjob testen, ob 24 Stunden seit
IP Eintragung bereits rum sind, ist dies der Fall soll die IP aus der DB gelöscht werden.

Nun habe ich den timestamp für meine IP um 86400 Sek (= 1 Tag) zurückgesetzt um das Script testen zu können.

Dies funktionierte auch, jedoch kommt trotzdem die Fehlermeldung:

Zitat

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource

Ich habe nun schon versucht mit mysql_error() mehr rauszubekommen, dies schlug aber fehl und es kommt kein weiterer Hinweis.

Wenn der Eintrag dann gelöscht ist, kommt die Fehlermeldung nicht mehr. Ist mysql_fetch_arry() nicht dafür geeignet einzelne IPs zu löschen, wenn die anderen noch nicht ihre 24h abgesessen haben?

--- Hier das Cronjobscript ---

PHP

<?php


$zeitnow = time();
$tagsek = 86400;


include 'dbconnect.php';
$sql = "SELECT * FROM nutzer";
$z = mysql_query($sql);
while($row = mysql_fetch_array($z))
{
    echo mysql_error();
    $timedifferenz = 0;
    $timedifferenz = $zeitnow - $row['zeit'];
    if ($timedifferenz > $tagsek)
    {
        $id = $row['id'];
        $sql = "DELETE FROM nutzer WHERE id = '$id'";
        $z = mysql_query($sql);
    }
}


?>

Alles anzeigen

Ich hoffe damit kann mir wer helfen. Danke !

SinnlosS

Wo da der Fehler in deiner Abfrage sein soll seh ich grad auch nicht, es sei denn du baust vorher gar keine Verbindung zur DB auf oder die Tabelle "nutzer" existiert nicht.
Ansonsten, meinst du nicht es wäre so wesentlich kürzer und einfacher (ungetestet):

PHP

$one_day_ago = time()-86400;
$sql = "
DELETE FROM
    nutzer
WHERE
    zeit<".$one_day_ago;
mysql_query($sql);

SinnlosS

Achja, was mir grad noch einfällt:

PHP

echo mysql_error();

Das macht in der Schleife absolut keinen Sinn, setz das davor. Wenn du nämlich einen Mysql-Error hast wirst du auch kein brauchbares Ergebnis haben das von mysql_fetch_array() abgeholt werden kann, also springt dein Script gar nicht erst in die Schleife.

Bandit

16.14. Warum soll ich nicht SELECT * schreiben?
http://www.php-faq.de/q/q-sql-select.html

Darkxor

Hallo,
mysql_error(); hatte ich auch an mehren Stellen zu Probezwecken
eingebaut, aber brachte alles keine genaueren Fehlermeldungen.

Ich habe nun die MySQL Anweisungen in der Schleife einfach andere
Namen vergeben und bisher habe ich keine Fehlermeldung mehr auch
nach einigen Testen, jedoch kann ich jetzt nicht wirklich
ausschließen, dass dies grad nur Zufall ist oder so.

Wenn nochwas sein sollte melde ich mich dann nomma.

Hier der momentane Code:

PHP

<?php


$zeitnow = time();
$tagsek = 86400;


include 'dbconnect.php';
$sql = "SELECT * FROM nutzer";
$z = mysql_query($sql);
while($row = mysql_fetch_array($z))
{
    $timedifferenz = 0;
    $timedifferenz = $zeitnow - $row['zeit'];
    if ($timedifferenz > $tagsek)
    {
        $id = $row['id'];
        $sql1 = "DELETE FROM nutzer WHERE id = '$id'";
        $iwas = mysql_query($sql1);
    }
}


?>

Alles anzeigen

Bandit

Deine Schleife ist absolut nicht nötig und verjubelt unnötig Resourcen. Sieh' dir das Beispiel von "SinnlosS" nochmal an, das ist deutlich besser als deine Schleife.

Darkxor

Ok nutze dann jetzt die Lösung von SinnlosS.
Danke SinnlosS dafür.
Damit tritt nach Tests das Problem nicht auf.

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource

Tags

Benutzer online in diesem Thema