Discussion :

[amnay]

Bonjour à vous
voilà avant je vais essayer d'exposer ce que je chercher à bidouiller.
pour mieux organiser mes fichiers et pr un code bien fait, j ai pensé à mettre dans un fichier
function.php les qlq fonction dont j'aurais besoin, exemple : se connecter à une base mysql avec choix de la base;

j'ai alors le fichier fonction.php suivant :

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
 
<?php
 
function openDB(){
 
echo 'je suis fonction.php '; // message témoin, 
 
  global $db_host;
  global $db_login;
  global $db_pass;
  global $db_basename;
  global $dbConnect;
  global $dbSelect;
 
  $db_host="127.0.0.1";
  $db_login="root";
  $db_pass="";
  $db_basename="gestdoc";
 
  //connextion à une base de donnée 
  $dbConnect = mysql_connect($db_host, $db_login, $db_pass); 
  $dbSelect = mysql_select_db($db_basename);
 
}
 
clodeDB(){
  mysql_close();
}
?>

et dans un autre fichier, recherche.php (par exemple ) j'ai mis :

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
 
<?php
funcrion recherche(){
 
    include ('fonction.php') 
     openDB();  // ici appel à la fonction qui ouvre la connexion à MySQL avecsélection de la base
 
     $sql ="select .... "; 
 
    $result = mysql_query($dbConnect,$sql);
 
.... 
 
closeDB(); 
 
} 
?>

Mais ça marche pas

qlq un à une idée, comment mieux organiser ses scriptes avec des fonction qu'on peut appeler par tout.

Merci

[ThomasR]

Bonjour,

1 - funcrion à remplacer par function
2 - Ou appelles-tu la fonction recherche() ?

Car a mon avis c'est ca qui coince, tu mets ton traitement dans une fonction que tu n'appelles pas.

[Bisûnûrs]

Il faut que tu remettes tes déclarations "global" au début de la fonction recherche.
La liste des arguments de mysql_query est inversée.

[amnay]

la fonction rechercher est appelé par

un

Code :

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

1
2
 
if (isset($_GET['fonction']) && $_GET['fonction']=="find") rechercher();

c'est juste pr avoir les url : rechercher.php?fonction=find et ça présente pas d eproblème si j'ajoute un

Code :

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

1
2
 
echo 'messahe temoin rechercher' ;

ça s'affiche




j'ai essayé de déclarer mes variable global au début de la fonction, mais ça marche pas,
même le message témoin que j'ai ajouté au début de fonction.php ne s'affiche pas,

Merci

[Sh4dow49]

dans ta fonction qui se connecte a ta base de données ajoute a la fin un

Code :

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

return $dbConnect;

pour récupérer la connexion que tu viens d'ouvrir.

Ensuite appelle la fonction comme ceci :

Code :

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

$dbConnect = openDB();

car sinon ta bariable $dbConnect ne vaut rien dans la fonction recherche (et donc sa ne marchera pas)

Normalement sa devrait mieux marcher comme ca

[Sh4dow49]

et ta fonction closeDb(), passe lui la variable $dbConnect en paramètre, et modifie donc la fonction comme ceci :

Code :

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

1
2
3
clodeDB($dbConnect){
  mysql_close($dbConnect);
}

comme ca tu fermes la connexion que tu viens d'ouvrir (si tu veux faire un code propre et ménager ta base c'est pas mal)

[amnay]

je reçois tjrs

Fatal error: Call to undefined function openDB()

comme message d erreur

le prob est ici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
<?php
...
 
 global $dbConnect;
 
  include('config.php');
// cette ligne est la cause de l erreur 
$dbConnect = openDB(); 
...
 
?>

[Sh4dow49]

tu fais un include du fichier "config.php" mais tu as dis que tes fonctions étaient dans le fichier "fonction.php"

c'est une erreur ou tu as renomé le fichier qui contient les fonctions ?

[amnay]

c'était juste une erreur commise bêtement quand j'ai voulu expliquer ici sur le forum, mais sur mes scripts les noms des fichiers pour include sont bon.


Merci

[Sh4dow49]

Oki

C'est résolu alors ? (ou tu as toujours l'erreur ?)

[Eusebe]

Bonjour,

as-tu activé l'affichage des avertissements ? (mettre error_reporting à E_ALL dans le php.ini)

Je soupçonne moi aussi une erreur lors de l'include. Tu peux aussi essayer en utilisant require à la place d'include (entraine une erreur fatale si le fichier à inclure n'existe pas)

[Sh4dow49]

En effet je pensais aussi a remplacer include par require, vérifie le chemin aussi

[amnay]

j'ai déjà essayé require, c'est pareil, le fichier existe, pour avancer, j'ai copié les fonctions dans le même fichier ou seront appeler question de ne pas tarder sur ça, après je dois les ressortir pour organiser mes scripts
mais j'ai toujours le problème d'appel de la fonction

[Eusebe]

Tu peux nous donner les codes complets de tes fichiers ? (ou mieux encore, essayer de les réduire à un code le plus court possible reproduisant l'erreur, avec 5 lignes de code max par fichier)

[amnay]

peut être je ferais mieux d'expliquer ce que je cherche à faire comme ça si quelqu'un à une autre idée plus pratique, je suis toujours prôneur.

je cherche à mettre des fonctions qui gèrent la base de données dans un seul fichier, et de pouvoir appeler telle ou telle fonction selon le besoin.
(je crois que le but principale des fonctions est d'éviter la répétition bloc de code )

premier fichier ( on l'appelé config.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
 
<?php
 
function openDB(){
  global $db_host;
  global $db_login;
  global $db_pass;
  global $db_basename;
 
 
  $db_host="127.0.0.1";
  $db_login="root";
  $db_pass="";
  $db_basename="gestdocs";
 
//connextion à MySQL 
mysql_connect($db_hodt, $db_login, $_db_pass);
// selection de la base de donnée  
mysql_select_db($db_basename);
 
}
 
?>

le deuxième fichier ( rechercher.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
 
<?php
 
recherche();
 
function recherche(){
 
    include ('config.php') ;
     openDB();  // ici appel à la fonction qui ouvre la connexion à MySQL avecsélection de la base
 
     $sql ="select * from doc "; 
    $result = mysql_query($sql);
 
  echo '<h1>'.mysql_num_rows($result).'</h1>' ;
 
} 
?>

[Eusebe]

J'ai copié/collé le contenu de tes deux fichiers pour tester en local chez moi, et tout fonctionne (la fonction openDB est bien exécutée)...

[amnay]

je viens de changer le concept de ce que je voulais faire,
j'ai ajouté une nouvelle fonction qui ne fait que exécuter la raquette sql passée en paramètre et renvoi le résultat de cette raquette et ça marche super bien :p ici le code

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
 
config.php
<?php
 
function openDB(){
 
  $db_host="127.0.0.1";
  $db_login="root";
  $db_pass="";
  $db_basename="gestdoc";
 
  //connextion à MySQL 
  $dbConnect = mysql_connect($db_host, $db_login, $db_pass) or die (mysql_error());
  // selection de la base de donnée  
  $dbSelect = mysql_select_db($db_basename, $dbConnect ) or die (mysql_error());
}
 
function requette($sql){
  openDB();
  $result =  mysql_query($sql);
      if (!$result) die('Requête invalide : ' . mysql_error());
      return $result;
}
 
?>

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
 
rechercher.php
<?php
 
recherche();
 
function recherche(){
 
    include ('config.php') ;
 
    $sql ="select * from doc "; 
 
    $result = requette($sql);
 
  echo '<h1>'.mysql_num_rows($result).'</h1>' ;
 
} 
?>

Merci à vous tous