Discussion :

[LampeRouge]

Bonjour. J'espère être dans la bonne cat? Mon problème se situe au niveau des nouvelles function msqli
Voila ce que j'ai fait: J'ai 2 functions:

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
27
28
29
30
31
function sqliconnect() {
    debug("sqliconnect() ");
    $mysqli = mysqli_connect(SERVER,LOGIN,PASSWORD,DB);
    if (mysqli_connect_errno($mysqli)) {
        echo "Echec lors de la connexion a MySQL : " . mysqli_connect_error();
        return false;
    }
 
    global $mysqli;
    return $mysqli;
}
 
//-----------------------------------------------------------------------------------------------------------------------------
function sqli_select($sql) {  
    debug("sqli_select()");
    if(!substr_count(strtolower($sql), 'select') ) {
       print"Expression mauvaise. sqli_select() est annulé.   ".$sql.'</br>';
       return FALSE;
    }
    if(isset($GLOBALS[mysqli])) {$mysqli= $GLOBALS[mysqli]; }
    else    { $mysqli= sqliconnect();}
 
    if(isset($GLOBALS[mysqli])) {$mysqli= $GLOBALS[mysqli]; }
    else    { $mysqli= sqliconnect();}
 
    $query = mysqli_query($mysqli, $sql );  
    $row_count= mysqli_num_rows($query);
    if(!$row_count){ 
        print"Aucun enregistrement trouvé".mysqli_error($query).'</BR> SQL= '.$sql.'</BR>';
        return FALSE;        
    }

sqli_select($sql) est chargé de renvoyer la réponse à la requète $sql....
Pour se faire la connexion doit être établie avec la table mysql.
C'est le rôle de la 1ere function sqliconnect()
Je n'ai aucune erreur dans la 1ere function, mais la 2e ne fonctionne pas. On dirait que la variable n'est pas transmise alors que l'on est dans la même page. $row_count=null

Si quelqu'un veut bien se pencher sur mon cas, Ça me dépannerait bien. Ça fait 3 heures que je galère.
(J'ai répété 2 fois l'appel à sqliconnect() pour être sur à travers les logs, que c'est bien $mysqli qui ne passe pas.)

Je vous remercie.

[sabotage]

Tu fais un contresens : "global" se met dans la fonction qui utilise la variable, pas dans la fonction qui produit la variable.

[LampeRouge]

Merci de ton aide. J'ai bien supprimé

Code :

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

global $mysqli;

Mais le problème reste inchangé.
Le nom de ma variable est peut être un nom réservé .....

[ABCIWEB]

Salut,

Normalement on passe l'identifiant de connexion comme argument dans la fonction.

Si non on fait une classe de connexion statique, exemple ici. Adapté à ton cas cela pourrait donner ça :

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
27
class MySqliConnect {
 
    private static $instance;
 
	private function __construct() {
		self::$instance = @mysqli_connect(SERVER,LOGIN,PASSWORD,DB) or exit("Impossible de se connecter à la base de donnée");
 
                // Si tu travaille en utf-8
                mysqli_set_charset(self::$instance,'utf8');
	}
 
        public static function getC() {
		if(!isset(self::$instance) || self::$instance == null) {
			new MySqliConnect();
		}
		return self::$instance; 
	}
}
 
 
// Maintenant tu peux appeler une instance de ta connexion depuis n'importe qu'elle fonction ou classe avec "MySqliConnect::getC()".
function sqli_select($sql) {  
 
    $mysqli = MySqliConnect::getC();
 
   //...
}

L'avantage c'est que ta connexion ne sera faite qu'une fois quelque soit le nombre d'appels à la fonction de connexion et c'est bien plus propre et sécurisé qu'une variable globale.

[LampeRouge]

Merci de vous être penché sur ce cas. C'est sympa. J'ai résolu mon problème en déclarant ma variable de connexion global avant de la créer. J'y accède maintenant aisément. Je pense que pour la sécurité comme en parle ABCIWEB il y a des questions à se poser, mais pour l'instant je reconstruis un projet procédurale et non orienté objet.
Je reste ouvert au sujet. Encore merci.

[ABCIWEB]

Tu fais du procédural mais rien ne t'empêche d'utiliser une classe de connexion dans ton code procédural, il n'y a aucun inconvénient, au contraire.

L'avantage de ce type de classe statique est qu'elle ne sera instanciée qu'une fois et donc tu peux l'appeler autant de fois que tu veux sans refaire une nouvelle connexion en utilisant celle déjà ouverte.

Ce n'est pas le cas avec ta fonction de connexion qui tente d'ouvrir une nouvelle connexion chaque fois qu'on l'appelle. Donc prends soin de ne l'utiliser qu'une seule fois car refaire de nouvelles connexions est pénalisant pour le serveur et pour la rapidité d'exécution du code.

Concrètement tu ne dois pas faire cela dans tes fonctions :

Code :

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

1
2
3
function sqli_select($sql) {  
    ...
    else    { $mysqli= sqliconnect();}

car si ce code est répété dans d'autres fonctions, ton code risque de tenter d'ouvrir de nombreuses connexions alors qu'il devrait se servir de celle déjà ouverte.

Un autre avantage d'avoir une classe de connexion est que tu ne risque plus de conflits de variables comme avec des variables globales.

Et puis de nombreuses fonctions php sont maintenant des classes donc même en procédural tu utilises des classes, ce n'est pas pour autant que ton code est orienté objet. "Orienté objet" c'est une philosophie de développement, ce n'est pas juste utiliser une ou plusieurs classes dans le code.
Utiliser une classe de connexion ne t'oblige en rien à modifier ton code ou sa logique, c'est simplement un truc pratique pour appeler ta connexion depuis n'importe quel endroit de ton script sans te poser de question

[LampeRouge]

Bonjour. En globalisant la variable, 1 seule connexion se fait par page PHP. Si j'ai de multiples demande MYSQL ma variable reste active. Par contre contre comme elle n'est pas transmise en POST ou en GET, elle se perd, et un nouvel appel est lancé. Je ne connais pas de solution pour garder cette variable en mode procédural
Avec le langage Orienté Objet, je ne sais pas, non plus, comment cette variable devrait être conservée. Il manque quelque chose à ma culture....
Merci de ton intérêt.

[ABCIWEB]

Quand tu change de page c'est assez normal de tout rafraichir, par contre quand tu fais une requête post pour traiter un formulaire par exemple rien ne t'empêche de faire exécuter le code dans la même page, soit directement, soit en appelant un fichier par include.

[LampeRouge]

Merci de cette réponse pertinente. Je vais faire quelques essais....