[HaBo]

$oul · 09.05.07, 18:27

Hallo Leute,
folgendes Problem: Ich will jedem Besucher meiner Seite einen Counter zuweisen, der pro Klicks auf einen Link den der User bekommt, der eine ID enthält, um eins verringert wird. Das ganze lässt sich jedoch meinerseits nur schwer realisieren, ich komm einfach nicht drauf, wie ich das Problem angehen soll. Momentan habe ich diesen Code:

PHP-Code:

  <?
    include 'config.php';

    @mysql_connect(MYSQL_HOST, MYSQL_USER, MYSQL_PASS) OR die(mysql_error());
    mysql_select_db(MYSQL_DATABASE) OR die(mysql_error());
    
    $ip=$_GET['id'];
    $date=date("d.m.Y - H:i:s");
    
    $sql = "SELECT IP,ClickCount FROM ".IDTable." WHERE IP='".$ip."';";
    $resulta = mysql_query($sql) OR die(mysql_error());
    while($row = mysql_fetch_assoc($resulta))
        {
            $ccnt = $row['ClickCount'];
            echo "Dir fehlen noch ".$ccnt." Klicks!<p>";
            echo $ccnt." Klicks<p>";
            echo $ip."<p>";
        }
    $ccnt=$ccnt-1;
    
    $sql = "SELECT IP,ClickCount FROM ".IDTable." WHERE IP='".$ip."';";
    $resulta = mysql_query($sql) OR die(mysql_error());
    
    if (resulta=="Resource id #6")
    {
        $realip=getenv("REMOTE_ADDR");
        $sql="INSERT INTO ".IDTable." (IP,Datum,ClickCount) VALUES ('$realip','$date','$ccnt')";
        $resultb = @mysql_query($sql);
    }
    else
    {
        $sql="UPDATE ".IDTable." SET ClickCount=".$ccnt." WHERE IP='".$ip."';";
        $resultc = @mysql_query($sql);
    }
?>

Die if-Anweisung klappt komischerweise auch nicht.
Meine grundsätzliche Frage ist: Wie kann ich das beschriebene realisieren?

mfg soul

Indi · 09.05.07, 18:40

mal abgesehen davon, dass dir das $-Zeichen in der if-abfrage fehlt, prüft man eine sql-ergebnis-menge im normalfall nicht mit "resource id #6" ab. Besser wäre da zb. mysql_num_rows() zu verwenden.

Nachtrag:
Desweiteren macht es wenig Sinn bei unregistrierten Usern die IP-Adresse zu verwenden, weil dadurch Ergebnisse verfälscht werden können, wenn sich mehrere User in einem Netzwerk auf deiner Seite bewegen, aber alle dann die gleiche IP-Adresse haben.

Anzeige

- Anzeige -

.smash · 09.05.07, 23:39

Code:

    $ip=$_GET['id'];
    $date=date("d.m.Y - H:i:s");
    
    $sql = "SELECT IP,ClickCount FROM ".IDTable." WHERE IP='".$ip."';";

Code:

        $sql="UPDATE ".IDTable." SET ClickCount=".$ccnt." WHERE IP='".$ip."';";
        $resultc = @mysql_query($sql);

Ich würde mal sagen, dass sind 2 sehr einfach ausnutzbare sql-injections
So wie es scheint, kann man gezielt bei einem selbst oder bei anderen countern die klickanzahl beliebig editieren...

Ist zwar jetzt keine antwort auf diene Frage, aber wenn du einen richtigen service bieten willst, solltest genauso auf die sicherheit achten.

mfg

Henni · 21.05.07, 09:06

Wie ich das aus dem Code sehen kann, hast du in der Variable "$ip" die ID gespeichert...

Mal davon abgesehen der HTML Code ist daraus so wie ich es sehen kann auch net ganz richtig^^

[quote]

PHP-Code:

       while($row = mysql_fetch_assoc($resulta))         {
            $ccnt = $row['ClickCount'];
            echo "Dir fehlen noch ".$ccnt." Klicks!<p>";
            echo $ccnt." Klicks<p>";
            echo $ip."<p>";
        }

Hast du da etwa 3 mal einen P Tag geöffnet?Funktioniert der Code jetzt eigentlich? sonst würde ich eigentlich auch nur noch das Finden was schon gesagt wurde^^mfgHenni

Eydeet · 21.05.07, 14:54

Musst du eigentlich unbedingt MySQL verwenden? Mit Sessions würde es nämlich viel einfacher funktionieren:

PHP-Code:

  <?php
session_start();
define("CLICKS_REQUIRED", 10);

if (!isset($_SESSION["clicksLeft"]))
{
    $_SESSION["clicksLeft"] = CLICKS_REQUIRED;
}
else
{
    --$_SESSION["clicksLeft"];
}
?>
<p>Dir fehlen noch <?=$_SESSION["clicksLeft"]?> Klicks.</p>

Anzeige

- Anzeige -

09.05.07, 18:27	#1 (permalink)
$oul Registriert seit: 19.12.04 Likes: 2	PHP: Jedem User ohne Anmeldung eine ID zuweisen und eigenen Counter Anzeige Hallo Leute, folgendes Problem: Ich will jedem Besucher meiner Seite einen Counter zuweisen, der pro Klicks auf einen Link den der User bekommt, der eine ID enthält, um eins verringert wird. Das ganze lässt sich jedoch meinerseits nur schwer realisieren, ich komm einfach nicht drauf, wie ich das Problem angehen soll. Momentan habe ich diesen Code: PHP-Code: <? include 'config.php'; @mysql_connect(MYSQL_HOST, MYSQL_USER, MYSQL_PASS) OR die(mysql_error()); mysql_select_db(MYSQL_DATABASE) OR die(mysql_error()); $ip=$_GET['id']; $date=date("d.m.Y - H:i:s"); $sql = "SELECT IP,ClickCount FROM ".IDTable." WHERE IP='".$ip."';"; $resulta = mysql_query($sql) OR die(mysql_error()); while($row = mysql_fetch_assoc($resulta)) { $ccnt = $row['ClickCount']; echo "Dir fehlen noch ".$ccnt." Klicks!<p>"; echo $ccnt." Klicks<p>"; echo $ip."<p>"; } $ccnt=$ccnt-1; $sql = "SELECT IP,ClickCount FROM ".IDTable." WHERE IP='".$ip."';"; $resulta = mysql_query($sql) OR die(mysql_error()); if (resulta=="Resource id #6") { $realip=getenv("REMOTE_ADDR"); $sql="INSERT INTO ".IDTable." (IP,Datum,ClickCount) VALUES ('$realip','$date','$ccnt')"; $resultb = @mysql_query($sql); } else { $sql="UPDATE ".IDTable." SET ClickCount=".$ccnt." WHERE IP='".$ip."';"; $resultc = @mysql_query($sql); } ?> Die if-Anweisung klappt komischerweise auch nicht. Meine grundsätzliche Frage ist: Wie kann ich das beschriebene realisieren? mfg soul __________________ Der eigene Wille müsste stets ein wenig mächtiger sein, als es das eigene Selbst ist. Aber Wunschdenken stellt ein Risiko dar und so etwas können wir uns in einem Weltkapitalismus nicht leisten.

09.05.07, 23:39	#3 (permalink)
.smash Registriert seit: 14.03.07 Likes: 0	Code: $ip=$_GET['id']; $date=date("d.m.Y - H:i:s"); $sql = "SELECT IP,ClickCount FROM ".IDTable." WHERE IP='".$ip."';"; Code: $sql="UPDATE ".IDTable." SET ClickCount=".$ccnt." WHERE IP='".$ip."';"; $resultc = @mysql_query($sql); Ich würde mal sagen, dass sind 2 sehr einfach ausnutzbare sql-injections So wie es scheint, kann man gezielt bei einem selbst oder bei anderen countern die klickanzahl beliebig editieren... Ist zwar jetzt keine antwort auf diene Frage, aber wenn du einen richtigen service bieten willst, solltest genauso auf die sicherheit achten. mfg

21.05.07, 09:06	#4 (permalink)
Henni Registriert seit: 03.01.07 Likes: 0	Wie ich das aus dem Code sehen kann, hast du in der Variable "$ip" die ID gespeichert... Mal davon abgesehen der HTML Code ist daraus so wie ich es sehen kann auch net ganz richtig^^ [quote] PHP-Code: `while($row = mysql_fetch_assoc($resulta)) { $ccnt = $row['ClickCount']; echo "Dir fehlen noch ".$ccnt." Klicks!<p>"; echo $ccnt." Klicks<p>"; echo $ip."<p>"; }` Hast du da etwa 3 mal einen P Tag geöffnet?Funktioniert der Code jetzt eigentlich? sonst würde ich eigentlich auch nur noch das Finden was schon gesagt wurde^^mfgHenni

21.05.07, 14:54	#5 (permalink)
Eydeet Registriert seit: 14.04.06 Likes: 4	Musst du eigentlich unbedingt MySQL verwenden? Mit Sessions würde es nämlich viel einfacher funktionieren: PHP-Code: `<?php session_start(); define("CLICKS_REQUIRED", 10); if (!isset($_SESSION["clicksLeft"])) { $_SESSION["clicksLeft"] = CLICKS_REQUIRED; } else { --$_SESSION["clicksLeft"]; } ?> <p>Dir fehlen noch <?=$_SESSION["clicksLeft"]?> Klicks.</p>`

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Ansicht
Linear-Darstellung Zur Hybrid-Darstellung wechseln Zur Baum-Darstellung wechseln

09.05.07, 18:40	#2 (permalink)
Indi Member of Honour Registriert seit: 02.10.01 Likes: 0	mal abgesehen davon, dass dir das $-Zeichen in der if-abfrage fehlt, prüft man eine sql-ergebnis-menge im normalfall nicht mit "resource id #6" ab. Besser wäre da zb. mysql_num_rows() zu verwenden. Nachtrag: Desweiteren macht es wenig Sinn bei unregistrierten Usern die IP-Adresse zu verwenden, weil dadurch Ergebnisse verfälscht werden können, wenn sich mehrere User in einem Netzwerk auf deiner Seite bewegen, aber alle dann die gleiche IP-Adresse haben.

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Autostart ohne Registry vor Anmeldung	Nick H.	Windows	3	07.08.06 22:09
Suche : Useronline Rekord counter ohne Mysql	XeonX	(Web-) Design und webbasierte Sprachen	0	21.08.04 22:04
Counter ohne JS-Code	Flou	(Web-) Design und webbasierte Sprachen	2	05.12.02 16:07
Counter ohne JS-Code	Flou	Internet Allgemein	1	27.08.02 03:51

[HaBo]

PHP: Jedem User ohne Anmeldung eine ID zuweisen und eigenen Counter