Discussion :

[olivier136978]

Bonjour,

Je n'ai plus développer depuis des années et je bute depuis une heure sur une difficulté.
Je cherche à mutualiser la fonction de connexion et de requête dans un fichier à inclure.

Voici mon fichier de fonctions : la première fonction connecte à la bdd et la seconde traite un requete. Premier fichier donc connexion.php

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 connexion(){
 
$serveur = 'localhost';
$bdd = 'mabase';
$utilisateur = "root";
$mot_de_passe = "";
 
$mysqli = new mysqli($serveur, $utilisateur, $mot_de_passe, $mabase);
 
$mysqli->set_charset("utf8");return $mysqli;
 
if ($mysqli->connect_error) {
    die('Erreur de connexion ('.$mysqli->connect_errno.')'. $mysqli->connect_error);
}
 
}
 
function requete ($requete, $mysqli){
 
$resultat = $mysqli->query($requete) or die ('Erreur '.$requete.' '.$mysqli->error);
 
 
while ($ligne = $resultat->fetch_assoc()) {
 
 $resultats[] = $ligne;
 
return $resultats; 
}
}

Et voici le second fichier avec inclusion du premier. Il lance la requête et récupère les résultats (dans cette requete il doit donc récupérer 5 résultats) :

Code :

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

1
2
3
4
5
6
7
8
9
 
include ('connexion.php');
 
$mysqli=connexion();
 
 
$requete = "SELECT DISTINCT * FROM mabase limit 5";
 
$res=requete ($requete, $mysqli);

___________________________________

Sauf que $res devrait être un tableau avec 5 résultats mais en fait ne renferme qu'un seul et unique résultat ! (mais $resultats est bien un tableau avec 5 résultats !).

Y a t'il une meilleure façon de faire ?

Merci de votre aide !

Olivier

[Invité]

Bonjour,

Avec une bonne indentation de ton code, tu verrais tout de suite l'erreur.

[olivier136978]

Vu ! Merci de ton aide.

cela te parait-il une bonne façon de gérer un site avec ces deux fonctions mutualisées de cette façon ? ou y a t'il une bonne pratique différente ?

merci !

[kevin254kl]

Bonjour,

Je pense que cela dépend de ton besoin, mais plus tu factorises plus tu gagnes du temps lors du débogage et des futurs développements.

Code :

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

1
2
3
4
5
6
while ($ligne = $resultat->fetch_assoc()) {
 
 $resultats[] = $ligne;
 
return $resultats; 
}

Par contre ici tu fais un return juste après avoir ajouté un résultat à ton tableau du coup
tu n'auras jamais plus.

Code :

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

1
2
3
while ($ligne = $resultat->fetch_assoc()) 
    $resultats[] = $ligne;
 return $resultats;

[rawsrc]

Salut,

Enfin pour faire la même chose tu pourrais te contenter juste de :

Code :

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

1
2
$resultat = $mysqli->query($requete) or die ('Erreur '.$requete.' '.$mysqli->error);
return $resultat->fetch_all(MYSQLI_ASSOC);

Généralement, on ne mélange pas dans le même fichier la connexion avec l'exécution (insertion, modification suppression, extraction, procédures stockées...)

[olivier136978]

Merci de vos conseils !

Effectivement je vais isoler la fonction de connexion dans un fichier (connexion.php par exemple) et je vais lire le tableau avec une boucle foreach pour extraire les données à afficher.
Je profite de l'occasion pour vous poser une nouvelle question sur les bonnes pratiques : comment amener le contenu des pages ?

A- mettre le contenu en tables et appeler les champs adhoc depuis la page index,
B- créer physiquement des fichiers php appelés depuis la page index ?

Je ne sais pas si je suis clair !
Le site sera une photothèque (10 000 photos aujourd'hui car c'est une refonte d'un site que j'ai développé il y a 13 ans !) avec des formulaires de recherche multicriètres

Olivier