Discussion :

[UDSP50]

Bonsoir à tous

Voila je gère un site sur un serveur.

On m'a demandé de m'occuper d'un autre sur un autre serveur et de faire un système d'accès membre.

Du coup, j'ai repris le mien qui fonctionne très bien mais là c'est la cata j'ai un message d'erreur

voici le code qui merde

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
 
 
<?php
    // Page faisant appel aux sessions
    session_start();
 
 
    // Vérification de l'autorisation
    if (!$_SESSION["login"]) {
        // Si l'utilisateur n'est pas autorisé il est reconduit
        // sur le formulaire d'identification
        header("Location: connexion.php");
        die();
    }   
	include('db_connect.php');
	$result=$mysqli->query("SELECT * FROM users WHERE login=".$_SESSION['login']);
	while($row = $result->fetch_array()){$login=$row['login'];}
?>

il me retourne cette erreur

Code :

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

1
2
Fatal error: Call to a member function fetch_array() on a non-object in /htdocs/Nouveau site/menu.php on line 14

la ligne 14 correspond a la ligne

Code :

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

1
2
 
while($row = $result->fetch_array()){$login=$row['login'];}

je pige pas sur mon autre site cela fonctionne nickel

merci de votre aide

[UDSP50]

Bon j'ai l'impression que cela vient du WHERE car quand je l'enlève j'ai plus d'erreur mais par contre je n'ai pas le filtre que je veux

[laurentSc]

Bonsoir,
je pense que le SELECT que tu fais à la ligne 16 ne retourne rien d'où le message d'erreur.
Fais un peu de debug : afficher la valeur de $_SESSION['login'] et afficher le contenu de la table users.

[CosmoKnacki]

Une fois de plus: ne fait pas de concaténations et prépare tes requêtes avec un placeholder (un point d'interrogation) à la place de ta variable: https://www.php.net/manual/fr/mysqli.prepare.php

Aussi, avant de faire des fetch, vérifie que le résultat n'est pas false.

Code :

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

if (!$_SESSION["login"])

Ce ne serait pas plutôt: if (!isset($_SESSION['login'])) ?

[laurentSc]

Une fois de plus: ne fait pas de concaténations et prépare tes requêtes avec un placeholder (un point d'interrogation)

ou encore mieux : des marqueurs nommés (c'est plus clair)

Aussi, avant de faire des fetch, vérifie que le résultat n'est pas false.?

Oui, ça t'éviterait l'erreur fatale.

[CosmoKnacki]

ou encore mieux : des marqueurs nommés (c'est plus clair)

C'est vrai pour ce qui est de PDO, mais ici UDSP50 utilise l'extension mysqli qui ne supporte que les ?.

[laurentSc]

OK, ne connaissant pas du tout MySQLi, je savais pas...