Declaration de classe PHP5 [Résolu]

[sim0m0]

Bonjour,

Avant de poser ma question je vous montre ces deux bouts de code ayant le meme resultat (fonctionnent tous les deux bien, pas d'erreur) en local.

Premiere code:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
<?php
 class Voiture{
 	var $couleur;
 	function Voiture(){
 		$this->couleur = 'rouge';
 		}
 	function changer_couleur($nv_couleur){
 		$this->couleur = $nv_couleur;
 		echo '<br />la couleur de votre voiture est maintenant: ',$this->couleur; 		
 		}
 }
 
 $mercedes = new Voiture();
 echo 'le couleur de votre voiture est: ',$mercedes->couleur;
 $mercedes->changer_couleur('blanche');
?>

deuxieme code

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
<?php
 class Voiture{
 	var $couleur='rouge';
 	function changer_couleur($nv_couleur){
 		$this->couleur = $nv_couleur;
 		echo '<br />la couleur de votre voiture est maintenant: ',$this->couleur; 		
 		}
 }
 
 $mercedes = new Voiture;
 echo 'le couleur de votre voiture est: ',$mercedes->couleur;
 $mercedes->changer_couleur('blanche');
?>

Personnailement je me suis penché vers le deuxieme, pensant qu'il est plus simple surtout que toute les instance de ma classe auront au depart la meme couleur (je ne vois pas l'interet du constructeur). Mais je suis debutant et je peux bien me trompé, alors je me tourne vers vous pour confirmer ce que je pense ou bien le corriger, aussi pour savoir si mon code est optimal et respecte les standard sinon une proposition ou plutot une correction serait la bienvenue surtou avec des explications.


Merci d'avance pour les gens qui essayeront de m'aider.

[stealth35]

c'est des classes PHP 4 ca
en 5 c'est :

Code :

1
2
3
4
5
6
7
8
9
10
class Voiture
{
    private $_couleur = 'rouge';
 
    public function changer_couleur($nv_couleur)
    {
        $this->couleur = $nv_couleur;
    	echo '<br />la couleur de votre voiture est maintenant: ', $this->_couleur;	
    }
}

[sim0m0]

Merci pour la réponse, je viens d'essayer votre suggestion mais le code ne fonctionne pas bien voici le code:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
<?php
class Voiture
{
    private $_couleur = 'rouge';
 
    public function changer_couleur($nv_couleur)
    {
        $this->couleur = $nv_couleur;
    	echo '<br />la couleur de votre voiture est maintenant: ', $this->_couleur;	
    }
}
 
 $mercedes = new Voiture;
 echo 'le couleur de votre voiture est: ',$mercedes->couleur;
 $mercedes->changer_couleur('verte');
?>

et voici le résultat:

Code :

1
2
le couleur de votre voiture est:
la couleur de votre voiture est maintenant: rouge

[andry.aime]

Bonjour,

$_couleur est déclaré private donc tu ne dois pas avoir accès avec $mercedes->couleur.

$mercedes = new Voiture;

Code :

$mercedes = new Voiture();

A+.

[syl2095]

Code :

1
2
3
 $mercedes = new Voiture();
 echo 'le couleur de votre voiture est: ',$mercedes->_couleur;
 $mercedes->changer_couleur('verte');

[andry.aime]

Envoyé par andry.aime

$_couleur est déclaré private donc tu ne dois pas avoir accès avec $mercedes->couleur.

Envoyé par syl2095

Code :

echo 'le couleur de votre voiture est: ',$mercedes->_couleur;

[grunk]

Code :

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
<?php
class Voiture
{
    private $_couleur;
 
    public function __construct() 
    {
	$this->_couleur = 'rouge';
    }
 
    public function changer_couleur($nv_couleur)
    {
        $this->couleur = $nv_couleur;
    	echo '<br />la couleur de votre voiture est maintenant: ', $this->_couleur;	
    }
 
	public function get_couleur()
	{
		return $this->_couleur;
	}
 
}
 
 $mercedes = new Voiture;
 echo 'le couleur de votre voiture est: ',$mercedes->get_couleur();
 $mercedes->changer_couleur('verte');

Pour récupérer la couleur (qui est déclarée privée) il faut passer par un getter (méthode retournant la valeur).
C'est histoire de gout , mais je suis plus partisan d'utiliser le constructeur pour initialiser les variable plutôt que de le faire directement dans la déclaration.

[stealth35]

Envoyé par grunk

C'est histoire de gout , mais je suis plus partisan d'utiliser le constructeur pour initialiser les variable plutôt que de le faire directement dans la déclaration.

Arrive un moment ou tu vas avoir des problème se sérialisation, y'a des moments ou ca passe pas par __construct, et niveau Doc c'est pas top non plus

[syl2095]

Envoyé par andry.aime

oui mais comme ça, on a une erreur Fatal error: Cannot access private property Voiture::$_couleur ...

c'est plus parlant

[sim0m0]

après cette modification:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
class Voiture
{
    private $_couleur = 'rouge';
    public function changer_couleur($nv_couleur)
    {
        $this->couleur = $nv_couleur;
    	echo '<br />la couleur de votre voiture est maintenant: ', $this->_couleur;	
    }
}
 $mercedes = new Voiture();
 echo 'le couleur de votre voiture est: ',$mercedes->_couleur;
 $mercedes->changer_couleur('verte');
?>>

J'obtient ceci:

le couleur de votre voiture est:
Fatal error: Cannot access private property Voiture::$_couleur in C:\xampp\htdocs\azerty\index.php on line 12

PS: la ligne 12 correspond a celle ci:

Code :

 echo 'le couleur de votre voiture est: ',$mercedes->_couleur;

[andry.aime]

Envoyé par andry.aime

$_couleur est déclaré private donc tu ne dois pas avoir accès avec $mercedes->couleur.

Et grunk a aussi proposé une correction.
comme si tu n'as pas lu tous les post .

A+.

[sim0m0]

En essayant la correction de grunk (un copier coller du code, j'évite de le réécrire pour éviter le flood) j'ai eu ce résultat:

le couleur de votre voiture est: rouge
la couleur de votre voiture est maintenant: rouge

[andry.aime]

Code :

1
2
3
4
5
 public function changer_couleur($nv_couleur)
    {
        $this->_couleur = $nv_couleur;
    	echo '<br />la couleur de votre voiture est maintenant: ', $this->_couleur;	
    }

A+.

[sim0m0]

Merci la classe fonctionne très bien avec ce code:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<?php
class Voiture
{
    private $_couleur;
    public function __construct() 
    {
	$this->_couleur = 'rouge';
    }
    public function changer_couleur($nv_couleur)
    {
        $this->_couleur = $nv_couleur;
    	echo '<br />la couleur de votre voiture est maintenant: ', $this->_couleur;	
    }
	public function get_couleur()
	{
		return $this->_couleur;
	}
}
 $mercedes = new Voiture;
 echo 'le couleur de votre voiture est: ',$mercedes->get_couleur();
 $mercedes->changer_couleur('verte');
 ?>

Une dernière question, est ce que le nom d'une propriété doit toujours commencer par un $_ pour respecter les normes, parce que j'ai essayer $couleur à la place de $_couleur, et ça a fonctionné sans problème.

Merci d'avance.

[grunk]

Une dernière question, est ce que le nom d'une propriété doit toujours commencer par un $_ pour respecter les normes, parce que j'ai essayer $couleur à la place de $_couleur, et ça a fonctionné sans problème.

L'underscore (_) est utilisé pour dire d'une variable ou d'une méthode qu'elle est privée. Mais c'est juste cosmétique , tu appelle tes variables comme bon te semble.
C'est comme pour ceux qui préfixe leur variable avec un i quand c'est un int , o pour objet , etc ... Ca marche sans , ca aide juste à voir rapidement à quoi on à a faire dans le code sans devoir remonté au déclaration des variables.

[sim0m0]

Merci beaucoup pour les explications et la correction