Counter mit Mysql und IP Sperre

[Greenleon]

hallo...

Ich habe mal angefangen mir einen Counter zu scripten. Da mein Anbieter Datenbanken zur verfuegung stellt habe ich den Counter mit Mysql angefangen...

Der normale Counter funktioniert schon aber die IP Sperre will nicht so ganz^^

Hier mal der Quellcode

<?php

	#Auf die Datenbank zugreifen
	
	require('dblogin.inc.php');
	$db= mysql_connect(DB_HOST, DB_USER, DB_PASS);
	mysql_select_db(DB_NAME_1, $db);
	
	
	#Gesamt auslesen und updaten
	
	$gesamt_obj = mysql_fetch_object( mysql_query("SELECT gesamt FROM gs_counter WHERE id='1'",$db));

	
	$datum_heute = date("ymd");
	$user_ip = $_SERVER['REMOTE_ADDR'];
	
	
	#Felder Loeschen, welche nicht vom heutigen Tag stammen

	mysql_query("DELETE FROM gs_ips WHERE date < '".$datum_heute."'");
	
		
	#Abfragen ob die IP des aktuellen Clienten schon gespeichert ist
	
	$treffer = mysql_num_rows("SELECT * FROM counter_ip WHERE ip == $user_ip");
	
	if($treffer == 0)
		{
		mysql_query("INSERT INTO gs_ips (ip) VALUES ('$user_ip')"); #IP des aktuellen Clienten speichern
		$gesamt_neu = ++$gesamt_obj->gesamt;
		mysql_query("UPDATE gs_counter SET gesamt='$gesamt_neu' WHERE id='1'",$db);

		}else{
		$gesamt_neu = $gesamt_obj->gesamt;
		};
	


	#Verbindung trennen
	
	mysql_close($db);
?>

Mit dem Code meckert der aber immer und zwar

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in ...URL... .php on line 26

[Private_pAuLa]

Zitat von Greenleon

[..]

#Abfragen ob die IP des aktuellen Clienten schon gespeichert ist

$treffer = mysql_num_rows("SELECT * FROM counter_ip WHERE ip == $user_ip");

if($treffer == 0)
{
mysql_query("INSERT INTO gs_ips (ip) VALUES ('$user_ip')");

[..]

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in ...URL... .php on line 26

ich mag mich irren, aber ich glaube du musst das ganze so formulieren:

PHP-Code:

$RS = mysql_query("SELECT * FROM counter_ip WHERE ip == $user_ip");
$treffer = mysql_num_rows($RS); 


und dann if( !$treffer ) und so weiter

also erst ne abfrage und dann schauen wieviele zeilen die abfrage gefunden hat...

wenn ich falsch liegen sollte verbessert mich bitte!!

[MatMel]

Zitat von Private_pAuLa

und dann if( !$treffer ) und so weiter

Das mit dem == 0 kann er beibehalten, da mysql_num_rows 0 zurückgibt, wenn keine Zeile vorhanden ist.
Ansonsten stimmts was du geschreiben hast;)

[Greenleon]

hat leider nix genuetzt....

[MatMel]

Kommt noch der gleiche Fehler oder was kommt jetzt ?

[Greenleon]

genau das^^

[Greenleon]

achja ich hatte eben 400 mal meine ip in der datenbank..

[chris17]

PHP-Code:

$RS = mysql_query("SELECT * FROM counter_ip WHERE ip == $user_ip");
$treffer = mysql_num_rows($RS); 


Doppelte Gleichheitszeichen sind hier falsch.
Du musst single-quoten, da die IP kein Integer ist.
Währen der Entwicklung immer "or die mysql_error()" verwenden, das bringt wichtige Hinweise auf mögl. Fehler.

PHP-Code:

$RS = mysql_query("SELECT * FROM counter_ip WHERE ip = '$user_ip'") or die(mysql_error());
$treffer = mysql_num_rows($RS); 


[Greenleon]

danke fuer den tipp.... ich bemerkte naemlich gerade dass ich mich zusaetzlich noch beim tabellennamen verschrieben hab^^


so... waer cool wenn mir noch jemand helfen koennte eine, gesamt, heute und gestern funktion einzubauen!

[MatMel]

Speichere die Zeit am besten als Timestamp.
Dann kannst du mit folgenden Bedingungen arbeiten:

$gespeicherte_zeit > time() - 3600 => heute
$gespeicherte_zeit < time() - 3600 AND $gespeicherte_zeit > time() - 7200 =>gestern
keine Bediungung => gesamt