POO Début d'apprentissage [Résolu]

[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 :

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 :

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]

Envoyé par Atomya Rise

Tu utilises un EOT

Heredoc

[Atomya Rise]

Envoyé par stealth35

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 :

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 :

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.

[flozza]

Hmm je vois je n'étais pas encore au chapitre du constructeur. Je n'ai pas encore bien compris le système de private, protected, var et public. J'ai tout mis en public pour ne pas avoir un problème d'accès à la variable.Merci pour ces astuces thibault!

[dourouc05]

Envoyé par flozza

Merci pour ces astuces thibault!

Sans l pour moi .

var, c'est le mal, à ne pas utiliser (résumé).
private, accès uniquement depuis la classe, rien d'autre. Énormément de choses devraient être en private, tout ce qui n'a pas d'utilité directe aux autres quand on parle de méthodes, tout pour les attributs.
protected, accès pour la classe et les classes filles.
public, accès pour tout le monde, y compris les martiens. C'est très bien pour des méthodes, rien de plus.