PHP [POO] Problem de class

Version imprimable

Bonjour a tous

A l'heure actuelle j'ai refait mes class, mais il y a un problème, j'essaye d'afficher le résultat de ma requête dans l'index.php mais j'ai une page blanche qui s'affiche.
Quelqu'un pourrait m'aider svp?

Connexion.class.php

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
<?php
class Connexion {
    private $tabConnexion;
    private $link;
    public function __construct ($serveur, $login, $mdp, $database){
        $this->tabConnexion=array("serveur"=>$serveur,"login"=>$login,"mdp"=>$mdp,"bdd"=>$database);
        $this->link=mysql_connect($this->tabConnexion['serveur'],$this->tabConnexion['login'],$this->tabConnexion['mdp']);
        mysql_select_db($this->tabConnexion['bdd']);
        if (!$this->link){
            throw new exception ('Impossible de se connecter à la base de donnée');
        }
    }
    public function Deconnecter(){
        $test=mysql_close($this->link);
        if (!$test){
            echo "Deco impossible";
        }
    }
}
?>

Index1.php

Code:

1
2
3
4
5
6
7
8
9
10
 
<?php
include ("./connexion.class.php" );
include ("./config.php" );
include ("./verification.class.php" );
$connexion=new Connexion($serveur,$login,$mdp,$database);
$dee=new verification();
$dee->execRequete("SELECT* FROM tbl_user" );
$connexion->Deconnecter();
?>

Verification.class.php

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
<?php
class verification {
    private $loginDb;
    private $passDb;
    public function execRequete($query){
        $result=mysql_query($query);
        while ($row=mysql_fetch_object($result));
        $this->passDb=$row->login;
        echo $this->passDb;
    }
}
?>

Config.php

Code:

1
2
3
4
5
6
7
 
<?php
$login='root';
$mdp='';
$serveur='localhost';
$database='telesurveillance';
?>

PS: J'ai vérifier avec un var_dump a l'intérieur de la class verification tout marche, mais dans l'index j'ai rien.
Merci

je comprends pas bien ce que tu veux faire la :

Code:

1
2
3
while ($row=mysql_fetch_object($result));
        $this->passDb=$row->login;
        echo $this->passDb;

soit tu fais une boucle soit tu prends le resultat :

Code:

1
2
3
4
while ($row=mysql_fetch_object($result)){
        $this->passDb=$row->login;
        echo $this->passDb;
}

Code:

1
2
3
$row=mysql_fetch_object($result);
        $this->passDb=$row->login;
        echo $this->passDb;

Citation:
Envoyé par deep21
Config.php
Code:

1 2 3 4 5 6 7 <?php $login='root'; $mdp=''; $serveur='localhost'; $database='telesurveillance'; ?>
PS: J'ai vérifier avec un var_dump a l'intérieur de la class verification tout marche, mais dans l'index j'ai rien.
Pourquoi ne pas mettre ces infos directement dans la classe ?
Exemple :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 class Connexion { protected $host = 'localhost'; // nom du serveur protected $user = 'root'; // protected $mdp = ''; // Mot de passe protected $bdd = 'la_bdd'; public function __construct() { $this->link = mysql_connect($this->host, $this->user, $this->mdp); mysql_select_db($this->bdd, $this->link); } } // Dans index $connexion=new Connexion();
Si tu veux quand même avoir un fichier config, alors utilise des constantes, car par définition il sera impossible de les redéfinir, de plus les constante ont une portées globales, elles seront donc accessible dans la classe.

Config .php :
Code:

1 2 3 4 5 6 7 <?php define('DB_HOST', 'localhost'); define('DB_USER', 'root'); define('DB_MDP', ''); define('DB_NAME', 'la_db'); ?>
Classe :
Code:

1 2 3 4 5 6 7 class Connexion { public function __construct() { $this->link = mysql_connect(DB_HOST, DB_USER, DB_MDP); mysql_select_db(DB_NAME, $this->link); } }
/!\ Attention à bien mettre le $link dans toutes les fonctions de mysql_* réclamant cet ID de connexion dans ta classe, sinon il y a un risque d'avoir des comportement étranges (perte de l'ID de connexion).
Ici je l'ai rajouté dans le mysql_select_db()

Ensuite, tu as une autre classe "Vérification", mais elle ne connait pas $connexion, il y a donc un problème de portée (visibilité, accessibilité) sur cette variable.
Le plus simple c'est de la déclarer en global dans cette classe.
Il est inutile alors de re-déclarer
Exemple :
Code:

1 2 3 4 5 6 class Verification { global $connexion; ... code etc ... }
Mais c'est pas tout car, si tu lance la fonction mysql_query(requête, link) en dehors de la classe Connexion, l'ID de connexion risque fort d'être perdu.
L'idéal serait de créer une méthode équivalante à mysql_query dans la classe connexion, une méthode genre query($query).
Idem avec toutes les fonctions de mysql_* réclamant cet ID et dont tu aura l'usage.

Exemple :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 class Connexion { protected $link; public function __construct() { ... bla bla bla ... } public function query($query) { return mysql_query($query, $this->link); // On conserve le même ID de connexion } }
Cette classe ne serait donc pas une simple classe de connexion, mais une classe MySQL.

Si tu as une page blanche alors qu'ici tu devrais avoir au moins 1 erreur, regarde dans ta config du php.ini, il doit être défini que les messages ne devront pas être affichés (dispaly_errors à Off).
Si tu développe, il vaut mieux les afficher (On), ça aide à comprendre.
Rajoute aussi des : or die(mysql_error())

Une question omme ça au pif. Pourquoi ne pas utiliser PDO ?
qui est, à lire de ce qui se dit, le remplaçant des mysql_* qui sous entend que cette dernière sera obsolète un de ces 4 matins.