Discussion :

[anju-chan]

Bonjour, j'essaye de récupérer la valeur du nombre de visiteurs mais j'ai cette erreur :

D’abord le script :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
function compter_visite()
{
    // On va utiliser l'objet $pdo pour se connecter, il est créé en dehors de la fonction
    // donc on doit indiquer global $pdo; au début de la fonction
    global $pdo;
 
    // On prépare les données à insérer
    $ip   = $_SERVER['REMOTE_ADDR']; // L'adresse IP du visiteur
    $date = date('Y-m-d');           // La date d'aujourd'hui, sous la forme AAAA-MM-JJ
 
    // Mise à jour de la base de données
    // 1. On initialise la requête préparée
    $query = $pdo->prepare("
        INSERT INTO agride_stats_visites (ip , date_visite , pages_vues) VALUES (:ip , :date , 1)
        ON DUPLICATE KEY UPDATE pages_vues = pages_vues + 1
    ");
    // 2. On execute la requête préparée avec nos paramètres
    $query->execute(array(
        ':ip'   => $ip,
        ':date' => $date
    ));
 
	$list_visiteur = $query->fetchAll(PDO::FETCH_ASSOC);
}

ET j'ai cette erreur :

Fatal error: Call to undefined method PDO::fetchAll() in C:\wamp64\www\function\compteur_visite.php on line 46
Call Stack

Merci de m'aider

[Geoffrey74]

Hello,

Tu fais un INSERT, pas un SELECT, donc tu n'aura pas de résultat à exploiter.

[Invité]

Bonjour,

es-tu sûr de COMPRENDRE ce que tu fais ? ce qui est écrit dans le code ?


Pourtant, il y a des explications dans ce code... : "Mise à jour de la base de données",...

On voit bien qu'il n'est pas écrit : "récupération du nombre de visiteurs"....

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

"SELECT pages_vues FROM agride_stats_visites WHERE ip = :ip"

[Celira]

Pour compléter les réponses de mes collègues : tu fais d'abord la mise à jour (avec la requête INSERT) et ensuite tu fais la récupération du nombre de visite (avec une requête SELECT)

Que compte le compteur ? le nombre de visites totales ? le nombre de visiteurs uniques ? depuis la nuit des temps ? à une date donnée ?

@jreaux : non, là tu récupères le nombre de pages visitées par un visiteur donné (ou plus exactement une IP données)

[Invité]

@Celira

compter_visite =

• 1-"compter les nombre de visiteurs"
• 2- ou "compter le nombre de visites de CE visiteur" ?

Qui a raison ?

Quel suspens !...

[anju-chan]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
// Fonction qui permet de mettre à jour le compteur de visites
function compter_visite()
{
    global $pdo;
 
    $ip   = $_SERVER['REMOTE_ADDR']; // L'adresse IP du visiteur
    $date = date('Y-m-d');           // La date d'aujourd'hui, sous la forme AAAA-MM-JJ
 
	$visiteur = $pdo->prepare("SELECT COUNT(*) as nb FROM agride_stats_visites WHERE ip='".$ip."'");
	$visiteur->execute();
	$list_visiteur = $visiteur->fetch(PDO::FETCH_ASSOC);
	$nb = $list_visiteur['nb'];
 
	if ($nb == 0 || $nb == NULL || $nb == "")
	{
		echo "echo";
		$query = $pdo->prepare("
			INSERT INTO agride_stats_visites (ip , date_visite , pages_vues) VALUES ('".$ip."' , '".$date."' , 1)
		");
		$query->execute();
	}
	elseif ($nb >= 0)
	{
		echo "Il existe deja cette IP";
	}
}

[Invité]

Bonjour,

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

if ($nb == 0 || $nb == NULL || $nb == "")

s'écrit plus simplement :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

if ( empty($nb) )

[anju-chan]

Reponse 1 donc, je pense pas avoir commis d'erreur dans ce simple bout de code !

[Invité]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

	$visiteur = $pdo->prepare("SELECT COUNT(*) as nb FROM agride_stats_visites WHERE ip='".$ip."'");

Ca, ce n'est pas une requête correctement préparée...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
		$query = $pdo->prepare("
			INSERT INTO agride_stats_visites (ip , date_visite , pages_vues) VALUES ('".$ip."' , '".$date."' , 1)
		");

Ca, non plus.

Tu n'as pas compris le principe.

[anju-chan]

A la base si, mais j'ai enlevé pour eviter toute erreur.
Je remetttrai après quand j'aurai mes ip

[Celira]

A ta place, je le remettrai tout de suite. Ce n'est pas parce que la donnée à insérer n'est pas la bonne qu'il faut faire n'importe quoi pour l'insérer.