Discussion :

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

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

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

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 :

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

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 :

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

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

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 :

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

$mercedes = new Voiture();

A+.

[syl2095]

Code :

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

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

[andry.aime]

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

Code :

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

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

[sim0m0]

après cette modification:

Code :

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

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 :

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

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

[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