Discussion :

[flozza]

Bonjour

Jusque là en php je programmais en procédural, mais j'aimerais exploiter à fond ce language, alors je me lance dans l'objet.Le passage est un peu déconcertant.

J'apprends en créant des scripts anodins, et je bloque sur le premier..

Code php :

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
 
<?php
class commande {
	public $client_nom;
	public $client_prenom;
	public $contenu;
 
 public function create($nom, $prenom) {
	 $this->client_nom = $nom;
	 $this->client_prenom = $prenom;
 }
 
 public function addPizza($pizza, $quantite) {
	 switch($pizza) {
	 	case marguerite:
			$this->contenu[0] = $quantite; 
		break;
 
		case royale:
			$this->contenu[1] = $quantite;
		break;
	 }
 }
}
 
$commande = new commande();
$commande->create($_GET['nom'], $_GET['prenom']);
if($_GET['commander']) {
	$com = explode("-", $_GET['commander']);
	$commande->addPizza($com[0], $com[1]);
}
echo <<<EOT
$commande->client_prenom $commande->client_nom a commandé $commande->contenu[0] marguerites et $commande->contenu[1] royales
<br>Miam !
EOT;
?>

Mon problème est sur l'array $contenu. Le résultat m'affiche Array[0] et Array[1]. Alors ma question est pourquoi?

[stealth35]

je te conseil de mettre ton niveau d'erreur au maxium (php.ini error_reporting = E_ALL | _STRICT), t'as des erreurs de syntax

fait plutôt un sprintf que du heredoc

[Atomya Rise]

Tu utilises un EOT, il faut donc que tu entoure tes champs par des accolades parce que ce sont des tableaux :

Code :

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

1
2
3
4
echo <<<EOT
$commande->client_prenom $commande->client_nom a commandé {$commande->contenu[0]} marguerites et {$commande->contenu[1]} royales
<br>Miam !
EOT;

[stealth35]

Tu utilises un EOT

Heredoc

[Atomya Rise]

Heredoc

En effet

[flozza]

Merci à vous deux!

[dourouc05]

Puisque ton objectif est d'apprendre la POO, deux remarques sur ton code :
- les constructeurs, c'est mieux que ce que tu as fait, ça a l'air beaucoup plus naturel et tu es sûr que la méthode de construction est appelée :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<?php
class commande {
	public $client_nom;
	public $client_prenom;
	public $contenu;
 
 public function __construct($nom, $prenom) {
	 $this->client_nom = $nom;
	 $this->client_prenom = $prenom;
 }
}
 
$commande = new commande($_GET['nom'], $_GET['prenom']);

Avec ton code, qu'on appelle ou pas create(), ton objet est quand même créé, tu n'as aucune certitude que ces champs soient remplis ;
- ensuite, des champs d'une classe en public... ! Cela viole l'encapsulation. Donc tu passes ça en private et tu mets des getter/setter :

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
<?php
class commande {
	private $client_nom;
	private $client_prenom;
	private $contenu;
 
 public function __construct($nom, $prenom) {
	 $this->client_nom = $nom;
	 $this->client_prenom = $prenom;
 }
 
 public function getName() { return $this->client_nom; }
 public function getFirstName() { return $this->client_prenom; }
}
 
$commande = new commande($_GET['nom'], $_GET['prenom']);

Ainsi, tu seras plus proche des principes de l'OO.