Discussion :

[kanabzh29]

Bonjour,

J'essaye de créer un classe, celle-ci doit se connecter à une base de données puis afficher le contenu d'une table, seulement ma page je s'affiche pas, et pire, j'ai le droit à l'erreur HTTP 500 de IE.
Je suis pas très fort en POO et j'aimerais avoir si possible vos remarques sur mon 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
26
27
28
29
30
31
 
class Mycv{
 
	private $dbhote = '',
	private $dbbase = '',
	private $dbuser = '',
	private $dbpass = '',
	private $dsn = '';	
	private $dbh = '';
 
	public function __construct($dbhote = 'localhost', $dbbase = 'test', $dbuser = 'root', $dbpass = '')
	{
		$this->dbhote = $Serveur;
		$this->dbbase = $Bdd;
		$this->dbuser = $Identifiant;
		$this->dbpass = $Mdp;
		$this->dsn = 'mysql:dbname=$this->dbbase;host=$this->dbhote';
		$this->dbh = new PDO($this->dsn, $this->dbuser, $this->dbpass);
	}
 
	public function viewData(){
		$sql = 'SELECT * FROM fic_opex_cla';
		$res = $this->dbh->query($sql);
		while($row = $res->fetch(PDO::FETCH_NUM)){
			echo $row[1].'<br/>';
		}
	}
}
 
$test = new Mycv();
$test->viewData();

Merci d'avance.

[sabotage]

Les variables ne sont pas interpretées à l'interieur des guillemets simples.

Code :

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

$this->dsn = "mysql:dbname=$this->dbbase;host=$this->dbhote";

[kanabzh29]

Cela me fait le mème problème en mettant :

Code :

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

1
2
 
$this->dsn = 'mysql:dbname='.$this->dbbase.';host='.$this->dbhote;

ou en mettant :

Code :

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

1
2
 
$this->dsn = "mysql:dbname=$this->dbbase;host=$this->dbhote";

Cela ne viendrait pas du fait que j'utilise les PDO à l'intérieur d'une classe ??? même si je ne pense pas.

[zyongh]

Tu dois remplacer les virgules par des points-virgules ou enlever les private à partir de ta seconde variable de déclarée.

De plus ton constructeur possède dans son corps des variables qui ne sont définies nulle part: $Serveur, $Bdd, $Identifiant, $Mdp.

Et il n'y a aucune utilité à mettre des variables private si c'est pour qu'elle soit affectée par des variables qui peuvent changer de l'extérieur.

[kanabzh29]

C'est bon, j'ai fais quelques petites corrections avec votre aide, et cela fonctionne, j'ai rajouté une variable et une fonction (en gras) qui doit calculer le nombre de résultat, mais cela n'affiche rien, mème pas d'erreur :

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
32
33
34
35
36
37
38
class Mycv{
	public $dbhote = '';
	public $dbbase = '';
	public $dbuser = '';
	public $dbpass = '';
	public $dsn = '';	
	public $dbh = '';
	
	public $nbres = 0;
	
	public function __construct($dbhote = 'localhost', $dbbase = 'test', $dbuser = 'root', $dbpass = '')
	{
		$this->dbhote = $dbhote;
		$this->dbbase = $dbbase;
		$this->dbuser = $dbuser;
		$this->dbpass = $dbpass;
		$this->dsn = "mysql:dbname=$this->dbbase;host=$this->dbhote";
		$this->dbh = new PDO($this->dsn, $this->dbuser, $this->dbpass);
	}
	
	public function viewData(){
		$this->sql = 'SELECT * FROM fic_opex_cla';
		$this->res = $this->dbh->query($this->sql);
		while($this->row = $this->res->fetch(PDO::FETCH_NUM)){
			echo $this->row[1].'<br/>';
			++$this->nbres;
		}
	}
	
	public function nombreRes(){
		return $this->nbres;
	} 
}

$test = new Mycv();
$test->viewData();
$test->nombreRes();

[sabotage]

Ta fonction en gras n'a pas d'echo.