Wert um eins erhöhen funktioniert nicht

[Sucker]

Habe folgenden Code um die visits meiner seiten zu zählen

  if(isset($_GET['go'])){
      $site = mysql_real_escape_string($_GET['go']);
      if( !mysql_num_rows(mysql_query("SELECT * FROM views WHERE site = '$site'") ) ) {
        mysql_query("INSERT INTO views (site, clicks) VALUES ('$site', '1')");
      }                 
      else{
        mysql_query("UPDATE views SET clicks = clicks+1 WHERE site = '$site'");      
      }
  } 

Jetzt wird der Wert von Clicks aber um 2 erhöht (manchmal auch nur um eins..) jemand ne ahnung an was das liegt?

danke schonmal

[Nikefly]

heisst das nicht "clicks++1" ansonsten überschreibt er doch den Wert, oder wie war das noch?

[axo]

Original geschrieben von Nikefly
heisst das nicht "clicks++1" ansonsten überschreibt er doch den Wert, oder wie war das noch?

wie war das noch? keine ahnung, etwas anderes halten?

1. setze auf die 'site' spalte einen UNIQUE index.
2. setze den default-wert der site-spalte auf 1 (und auf INT falls nicht schon geschehen).
3.

INSERT INTO views (site) VALUES ('$site')
   ON DUPLICATE KEY UPDATE clicks=clicks+1

4. mysql_num_rows(mysql_query("SELECT *")) will ich nie wieder sehen. denk nach warum.

[firstlord18]

on duplicate geht leider nur mit mysql > 4.1 oder so ...

[Abraxax]

@firstlord18
machst du bitte dein banner aus dein signatur raus. oder deutlich kleiner! (88x31)

[axo]

Original geschrieben von firstlord18
on duplicate geht leider nur mit mysql > 4.1 oder so ...

stimmt. >= 4.1

in dem fall eben:
1. DEFAULT auf 0
2.

INSERT IGNORE INTO ...
UPDATE clicks=clicks+1 WHERE ...

spart immer noch eine abfrage.

ps. die welt ist so verdammt klein, was?

[Sucker]

danke schonmal so weit, allerdings funktioniert es immer noch nicht

habs bis jetzt so:

  if(isset($_GET['go'])){
      $site = mysql_real_escape_string($_GET['go']);
      if(mysql_num_rows(mysql_query("SELECT id FROM views WHERE site = '$site'") ) ) {
        mysql_query("INSERT INTO views (site) VALUES ('$site')
 ON DUPLICATE KEY UPDATE clicks=clicks+1");
      }                 
  } 

warum eigentlich nicht "mysql_num_rows(mysql_query("SELECT *"))", sorry, haber noch ziemlich keine ahnung von der materie

benutze MySQL 4.1.10a

[axo]

Original geschrieben von Sucker
[B]danke schonmal so weit, allerdings funktioniert es immer noch nicht

habs bis jetzt so:

natürlich funktioniert's nicht. das IF muss weg. es bleibt nur noch:

function updateClicks($site) {
    $site = mysql_real_escape_string($site);
    $qry  = "INSERT INTO views (site) 
                VALUES ('%s') 
                ON DUPLICATE KEY 
                UPDATE clicks=clicks+1";
    $sql  = sprintf($qry, $site    );
    mysql_query($sql) or die(mysql_error());
}

if( isset($_GET['go']) ) {
    updateClicks($_GET['go']);
} 

warum eigentlich nicht "mysql_num_rows(mysql_query("SELECT *"))", sorry, haber noch ziemlich keine ahnung von der materie

denk mal darüber nach, was die datenbank machen muss, wenn du so die anzahl an zeilen abfragst.

[Sucker]

wow danke!

aber wird leider immer noch um 2 erhöht und als anfangswert wird 3 gesetzt oO?

[axo]

Original geschrieben von Sucker
wow danke!

aber wird leider immer noch um 2 erhöht und als anfangswert wird 3 gesetzt oO?

dann rufst du das ding anscheinend zu oft auf oder du hast die datenbank falsch eingerichtet.
1. rest des codes zeigen
2. 'exportiere' die tabelle über phpmyadmin - nicht die daten, nur die struktur.