Fehler beim erhöhen eines Datenbankeintrages

[BL_OfDragons]

Hi,

ich habe ein Problem mit mysql. Folgender Code schreibt Daten in 2 Tabellen, einmal erhöht es den Tagewert, und einmal schreibt es einen Eintrag in eine Tabelle, die, die letzten 250 User inc. Ip und Referer speichert. Doch komischerweise ist die Anzahl der Einträge in der Tabelle mit den Informationen größer als die Zahl der User/pro tag in der onlineuser_day-tabelle.

$if_ip_exist=mysql_query("SELECT IP,Time FROM onlineuser WHERE IP='$ip'",$link); 
if(mysql_num_rows($if_ip_exist)) 
   mysql_query("UPDATE onlineuser SET Time='$current_time' WHERE ip='$ip'",$link); 
else 
{ 
   mysql_query("INSERT INTO onlineuser (ip, time, Referrer) VALUES('$ip', '$current_time', '$url')"); 

   $day=date("Y-m-d",$current_time); 

   mysql_query("LOCK TABLES onlineuser_day WRITE,onlineuser_day READ",$link); 
   if(mysql_num_rows(mysql_query("SELECT User FROM onlineuser_day WHERE Date='$day'",$link))) 
   { 
        mysql_query("UPDATE onlineuser_day SET User=User+1 WHERE Date='$day'",$link); 
   } 
   else 
   {    
   mysql_query("INSERT INTO onlineuser_day (User, Date) VALUES('1', '$day')",$link); 
   } 
} 

Woran liegt das?

[BL_OfDragons]

Kann mir niemand helfen?

[mrhappiness]

ich vermisse da ein UNLOCK...

geht es nicht auch so?

mysql_query("UPDATE onlineuser SET Time='$current_time' WHERE ip='$ip'",$link);
if (mysql_affected_rows()==0)
{
  mysql_query("INSERT INTO onlineuser 
    (ip, time, Referrer) 
    VALUES('$ip', '$current_time', '$url')");
  $day=date("Y-m-d",$current_time);
  mysql_query("UPDATE onlineuser_day SET User=User+1 WHERE  Date='$day'",$link);
  if (mysql_affected_rows()==0)
    mysql_query("INSERT INTO onlineuser_day 
      (User, Date) 
      VALUES('1', '$day')",$link); 
} 

oder noch viel besser:

INSERT INTO onlineuser
  SET 
    ip=$ip,
    referrer=$url,
    time=NOW()

dann hast du nur eine tabelle und kannst mit lustigen sql-funktion das machen, was du momentan in zwei tabellen machst

[BL_OfDragons]

Das unlock habe ich vergessen zu posten. Ich teste das jetzt mal einen Tag mit deiner Variante. Naja, die Tabelle wird mit deiner 2. Variante schnell ziemlich groß.

[mrhappiness]

Original geschrieben von BL_OfDragons
Naja, die Tabelle wird mit deiner 2. Variante schnell ziemlich groß.

nicht größer als deine zwei tabellen

aber bei der 2. variante is der overhead geringer und du hast die ganze lock/unlock problematik nicht

[BL_OfDragons]

Naja, die eine Tabelle hat nur die eintäge des aktuellen Tages (der rest wird gelöscht)

[MelloPie]

kannst Du doch auch so machen.
mach einen cron der jeden morgen die daten von gestern konsolidiert und in deine andere tabelle schreibt und dann löscht

[BL_OfDragons]

Es funktioniert immer noch nicht

[mrhappiness]

Original geschrieben von BL_OfDragons
Es funktioniert immer noch nicht

Es ist immer noch was falsch