produkt zählen, aber pro user nur 1x: meine lösung korrekt?

Promaetheus · 14.11.2006, 16:35

hi leute!

ich habe eine produkttabelle mit der spalte prod_count. ich möchte nun, sobald das produkt aufgerufen wird, dass dies gezählt wird. wenn der kunde aber öfter klickt, soll der mehrmalige klick nicht gezählt werden. ich verwende hier noch keine sessions!

wäre nett wenn ihr mir sagen würdet ob meine lösung korrekt ist:
1. ich habe eine zusätzliche tabelle "count_nonreccurence" mit den feldern ip, prod_id, time erstellt, in welche ich beim produktaufruf die betreffenden daten speichern lasse. das ganze schaut schematisch so aus:

beim produktaufruf:

PHP-Code:

  <?php

// schauen ob bereits ein count vorliegt:

$sql = "SELECT prod_id

        FROM products

        WHERE prod_id = '$prod_id'

        AND   ip = '".$_SERVER['REMOTE_ADDR']."'";

$query = mysql_query($sql);

 
// wenn kein count innerhalb von 30min vorliegt -> counten:

if(mysql_num_rows($query) != 0) {

  $sql = "UPDATE products

          SET prod_count = prod_count + 1

          WHERE prod_id = '$prod_id'";

  $query = mysql_query($sql);

  

// und in die kontrolltabelle eintragen:

  $sql = "INSERT into count_nonreccurence

                (prod_id,

                 customer_ip,

                 timestamp)

          VALUES ('$prod_id',

                  '".$_SERVER['REMOTE_ADDR']"',

                  NOW)";

  $query = mysql_query($sql);

}

 
// alte einträge löschen:

$timerange = time() - 1800;

$sql = "DELETE FROM products

        WHERE prod_id = '$prod_id'

        AND   timestamp < '$timerange'";

$query = mysql_query($sql);

?>

Zergling-new · 14.11.2006, 17:35

Probiers doch aus

Promaetheus · 14.11.2006, 17:44

hab ich grad. mit einigen modifikationen und mehreren korrekturen funktioniert es.

wollte nur wissen ob das auch up2date ist was ich da produzier. funktionieren tut es so:

PHP-Code:

  <?phpfunction prod_count($prod_id, $ip, $time='1800') {

  $timeout = time() - $time;

 
  $sql = "SELECT prod_id

          FROM ".TABLE_COUNT_CONTROL."

          WHERE prod_id = '$prod_id'

          AND   user_ip = '$ip'";

  $query = pd_mysql_query($sql);

 
  if(mysql_num_rows($query) == 0) {

    $sql = "UPDATE ".TABLE_PRODUCTS."

            SET prod_view_count = prod_view_count + 1

            WHERE prod_id = '$prod_id'";

    $query = mysql_query($sql);

  

    $sql = "INSERT into ".TABLE_COUNT_CONTROL."

                  (prod_id,

                   user_ip,

                   timestamp)

            VALUES ('$prod_id',

                    '$ip',

                    '".time()."')";

    $query = pd_mysql_query($sql);

  }

  

  $sql = "DELETE FROM ".TABLE_COUNT_CONTROL."

          WHERE prod_id = '$prod_id'

          AND   timestamp < '$timeout'";

  $query = pd_mysql_query($sql);

}

?>

die konstanten und pd_mysql_query hab ich ausgelagert.

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Session: Untersch. Seiten f. eingeloggte User und norm. User		PHP Tipps 2005-2	2	20.07.2005 12:46
[Erledigt] MySQL user rausfinden		Datenbanken	4	26.05.2005 12:15
[Erledigt] PHP linux user wechseln		PHP-Fortgeschrittene	12	05.12.2004 16:11
Access denied for user: '@localhost' / SYSTEM@localhost		Datenbanken	0	10.11.2004 20:35

14.11.2006, 16:35
Promaetheus Erfahrener Benutzer Registriert seit: 07.11.2003 Beiträge: 526	produkt zählen, aber pro user nur 1x: meine lösung korrekt? hi leute! ich habe eine produkttabelle mit der spalte prod_count. ich möchte nun, sobald das produkt aufgerufen wird, dass dies gezählt wird. wenn der kunde aber öfter klickt, soll der mehrmalige klick nicht gezählt werden. ich verwende hier noch keine sessions! wäre nett wenn ihr mir sagen würdet ob meine lösung korrekt ist: 1. ich habe eine zusätzliche tabelle "count_nonreccurence" mit den feldern ip, prod_id, time erstellt, in welche ich beim produktaufruf die betreffenden daten speichern lasse. das ganze schaut schematisch so aus: beim produktaufruf: PHP-Code: <?php // schauen ob bereits ein count vorliegt: $sql = "SELECT prod_id FROM products WHERE prod_id = '$prod_id' AND ip = '".$_SERVER['REMOTE_ADDR']."'"; $query = mysql_query($sql); // wenn kein count innerhalb von 30min vorliegt -> counten: if(mysql_num_rows($query) != 0) { $sql = "UPDATE products SET prod_count = prod_count + 1 WHERE prod_id = '$prod_id'"; $query = mysql_query($sql); // und in die kontrolltabelle eintragen: $sql = "INSERT into count_nonreccurence (prod_id, customer_ip, timestamp) VALUES ('$prod_id', '".$_SERVER['REMOTE_ADDR']"', NOW)"; $query = mysql_query($sql); } // alte einträge löschen: $timerange = time() - 1800; $sql = "DELETE FROM products WHERE prod_id = '$prod_id' AND timestamp < '$timerange'"; $query = mysql_query($sql); ?> __________________ mfg Alexander Haim


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

14.11.2006, 17:35
Zergling-new Erfahrener Benutzer Registriert seit: 21.05.2008 Beiträge: 9.937	Probiers doch aus

14.11.2006, 17:44
Promaetheus Erfahrener Benutzer Registriert seit: 07.11.2003 Beiträge: 526	hab ich grad. mit einigen modifikationen und mehreren korrekturen funktioniert es. wollte nur wissen ob das auch up2date ist was ich da produzier. funktionieren tut es so: PHP-Code: <?phpfunction prod_count($prod_id, $ip, $time='1800') { $timeout = time() - $time; $sql = "SELECT prod_id FROM ".TABLE_COUNT_CONTROL." WHERE prod_id = '$prod_id' AND user_ip = '$ip'"; $query = pd_mysql_query($sql); if(mysql_num_rows($query) == 0) { $sql = "UPDATE ".TABLE_PRODUCTS." SET prod_view_count = prod_view_count + 1 WHERE prod_id = '$prod_id'"; $query = mysql_query($sql); $sql = "INSERT into ".TABLE_COUNT_CONTROL." (prod_id, user_ip, timestamp) VALUES ('$prod_id', '$ip', '".time()."')"; $query = pd_mysql_query($sql); } $sql = "DELETE FROM ".TABLE_COUNT_CONTROL." WHERE prod_id = '$prod_id' AND timestamp < '$timeout'"; $query = pd_mysql_query($sql); } ?> die konstanten und pd_mysql_query hab ich ausgelagert. __________________ mfg Alexander Haim