Discussion :

[Drackster]

Coucou,

Je suis en train de programmer mon premier portail personnel, et jusqu'ici je n'ai eu aucun problème, sauf un, qui n'est pas vraiment un réel problème mais bon, j'aimerais bien connaître la réponse.

Donc voici le code, tout simple :

INDEX.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
<?
 
include("connectSQL.php");
include("page.php");
 
$array_config = mysql_query("SELECT name FROM " . $db_prefix . "_config");
while ($row = mysql_fetch_array($array_config))
{
    echo $row['name'];
}
 
requete();
 
?>

PAGE.PHP :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<?
 
function requete()
{
	connect($db_serv, $db_name, $db_user, $db_pass);
	$array_configw = mysql_query("SELECT value FROM " . $db_prefix . "_config");
	while ($rowx = mysql_fetch_array($array_configw))
	{
	    echo $rowx['value'];
	}
}
 
?>

Le problème est que la requête de la fonction "requête()" ne s'execute pas.
Il faut que j'include à nouveau le connectSQL.php dans la fonction pour que celle-ci s'execute...

J'aimerais pouvoir appeler la fonction sans à avoir refaire la connexion à la base de donnée...

Merci d'avance !

Dans l'attente de vous lire

[s.n.a.f.u]

Tu as un problème avec la portée des variables.

Un petit tour en FAQ ?

[Drackster]

Et bien j'ai essayé beaucoup de chose, donc les variables globales, en ayant mis dans ma fonction :

Code :

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

global $sql;

Code :

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

1
2
$sql = @mysql_connect($db_serv, $db_user, $db_pass) or die("Impossible to connect the server") ;
    @mysql_select_db($db_name) or die("Impossible to connect the data-base") ;

Mais ça n'a pas fonctionné.

Faut-il mettre :

Code :

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

1
2
3
4
5
6
7
function requete()
{
	global $sql; // pour dire que la variable est globale.
	$sql; // pour l'executer.
 
	[...]
}

J'ai un peu de mal là...
J'ai regardé sur d'autres CMS, mais il n'y a aucun global du genre.
La connexion est executée une fois, et aucune variable globale n'est lancée (?).

Merci d'avance ^^.

[s.n.a.f.u]

Code :

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

1
2
$sql = @mysql_connect($db_serv, $db_user, $db_pass) or die("Impossible to connect the server") ;
    @mysql_select_db($db_name) or die("Impossible to connect the data-base") ;

Mais ça n'a pas fonctionné.

Et oui...
Il y a du mieux mais ce n'est pas encore ça.
Tu déclares $sql en global. OK, mais quid de $db_serv, $db_user, etc... Toute variable alimentée hors de la fonction doit être déclaré&e en global (beurk) ou passée en paramètre (miam, mieux).

Courage, tu es sur la voie de la lumière.

[Drackster]

Il faudrait donc que ma fonction requête devienne :

Code :

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

1
2
3
4
5
6
7
requete($db_user, $db_pass, etc)
{
global $sql;
$sql;
 
[...]
}

?

Ca ne pose pas de problème niveau sécurité ? Et je dois faire cela dans chaque fonction ?!

[s.n.a.f.u]

Il faudrait donc que ma fonction requête devienne :

Code :

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

1
2
3
4
5
6
7
requete($db_user, $db_pass, etc)
{
global $sql;
$sql;
 
[...]
}

?

Ca ne pose pas de problème niveau sécurité ? Et je dois faire cela dans chaque fonction ?!

C'est une façon de faire. Tu peux aussi déclarer des constantes qui sont accessibles partout dans le code, ça se fait pas mal. Personnellement, je préfère.
De toutes façons, c'est ton premier portail : quand il marchera, ce sera sûrement un gruyère, mais au moins il fonctionnera. Avec l'expérience et quand les bases seront acquises, tu pourras te pencher de plus près sur la sécurité.

[Deallyra]

INDEX.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
<?php
include("connectSQL.php");
include("page.php");

//tu effectues une requête... sans ouvrir de connexion à ta BdD.
$array_config = mysql_query("SELECT name FROM " . $db_prefix . "_config");
while ($row = mysql_fetch_array($array_config))
{
    echo $row['name'];
}

requete();

?>

PAGE.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
<?

function requete()
{
	//connexion (tardive) à la BdD
	connect($db_serv, $db_name, $db_user, $db_pass);
	//$array_configw est une variable de portée locale à requete.
	$array_configw = mysql_query("SELECT value FROM " . $db_prefix . "_config");
	while ($rowx = mysql_fetch_array($array_configw))
	{
	    echo $rowx['value'];
	}
}

?>

Une simple question...

Pourquoi ne pas faire plus simple (dans un premier temps), à savoir, dans une fonction, ouvrir ta connexion à ta BdD, faire tes requêtes, ensuite, fermer ta connexion à la BdD.

Ouvrir et fermer la connexion pour chaque requête autrement... Mais pour ce faire, la fonction d'ouverture et la fonction de fermeture doivent être écrites

[Drackster]

La connexion se faisait dans le connectSQL.php.
Et j'ai trouvé le problème, il n'y a aucunement besoin de faire des connexions à la base de tout partout, j'ai juste mis mysql_fetch_array, au lieu de mysql_fetch_row...

Merci pour votre aide ^^.