Discussion :

[éric1]

Bonjour,

J'ai du mal à comprendre mon tutoriel et je n'ai que les explications ci-dessous. Ce que je ne comprends pas est en rouge.

Merci d'avance.

Mon tutoriel me dit :
"Quand vous passez des paramètres à une méthode, vous pouvez exiger de ces paramètres un type à respecter. Par exemple, on peut exiger de la méthode frapper un paramètre de type Personnage (car on veut frapper un personnage ). Ainsi, le code pourrait ressembler à ceci :

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
<?php
    class Personnage
    {
        private $force;
        private $localisation;
        private $experience;
        private degats;
        
        public function deplacer()
        {
        
        }
        
        public function frapper(Personnage $persoAFrapper) // On veut un paramètre de type Personnage.
        {
            $persoAFrapper->recevoirDegats ($this->force);
        }
        
        public function gagnerExperience()
        {
            $this->experience++;
        }
        
        public function recevoirDegats ($force)
        {
            $this->degats += $force;
        }
    }
?>

[RideKick]

Ce que le tutoriel veut dire c'est que :

Code :

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

1
2
3
4
public function frapper(Personnage $persoAFrapper) // On veut un paramètre de type Personnage.
        {
            $persoAFrapper->recevoirDegats ($this->force);
        }

est different de

Code :

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

1
2
3
4
public function frapper($persoAFrapper) // On veut un paramètre de type Personnage.
        {
            $persoAFrapper->recevoirDegats ($this->force);
        }

Dans le premier cas $persoAFrapper est obligatoirement une personne, dans le deuxieme tu passe ce que tu veux mais par contre si ce n'est pas une personne la methode appliquéee recevoirDegats(); va renvoyer une erreur .

[éric1]

Ok pour ton explication.

Mais c'est surtout ce petit bout de code qui est assez obscur pour moi :

$persoAFrapper->recevoirDegats ($this->force);

Je comprends bien que '$this->force' va cherche dans la classe la valeur de 'force'.
Mais je ne comprends pas ce qu'il y a devant (et donc toute la ligne !) :
$persoAFrapper->recevoirDegats.

Autre petite question.
Suis-je obligé de mettre public devant function? Ce n'est pas public par défaut?

Merci.

[Alshten]

"$persoAFrapper->recevoirDegats" permet d'appeler la méthode recevoirDegats de l'objet $persoAFrapper en lui passant donc en paramètre $this->force. Comme ça le personnage frappé se prend les dégats correspondants à la force du personnage qui frappe.

Pour public, il me semble que c'est obligatoire, en tout cas je le mets toujours et c'est conseillé de toujours le mettre.

[bricecol]

cela veut dire que tu appelles la méthode "recevoirDegats" sur le personnage que ton personnage frappe. en gros, quand tu frappes quelqu'un, tu lui dis "reçois x de dégat", x correspondant ici à la force de ton personnage. c'est pour cela qu'on doit vérifier que l'objet passé à la fonction est bien un objet de la classe Personnage qui est capable de recevoir des dégats.

[RideKick]

Pour public, il me semble que c'est obligatoire, en tout cas je le mets toujours et c'est conseillé de toujours le mettre.

Pas tellement non ....mais bon la on en vient carrement a revoir les bases de la POO, je vous conseille d'aller voir les tutos qui traitent du sujet..

[Alshten]

Pas tellement non ....mais bon la on en vient carrement a revoir les bases de la POO, je vous conseille d'aller voir les tutos qui traitent du sujet..

Je crois que je me suis mal exprimé. Je voulais pas dire qu'il fallait mettre toutes les méthodes en public mais qu'il fallait mettre le mot clé public pour les méthode que l'on veut mettre publique parce qu'il demandait si le mot clé était obligatoire.

[éric1]

Bon ... Que dois-je retenir?

Ceci ?
"Une méthode est public par défaut même si on ne le précise pas dans le code."
"Par convention, il est préférable d'écrire public".

A vous de me dire.

[Petibidon]

Je vote pour : "par convention il est préférable d'écrire public".

Disons que si quelqu'un relit ton code (ou toi quelques temps plus tard) déclarer la visibilité "public" revient à appuyer ton intention de la rendre publique. Ne pas déclarer la visibilité pourrait revenir à dire "c'est public mais ça pourrait ne pas l'être, j'm'en fous". D'ailleurs en Java par exemple, ne pas déclarer une méthode publique lui donne une visibilité limitée. Tiens d'ailleurs ça me fait penser qu'il est possible que ce soit aussi le cas en PHP au niveau des namespaces. A confirmer, je me suis pas encore penché sur la question

[Celira]

Pour des raisons de lisibilité et de maintenabilité du code, il vaut mieux indiquer systématiquement la visibilité des éléments. Comme ça, on sait qui est quoi et on ne s'emmêle pas les pinceaux avec les visibilités par défaut (des fois que les concepteurs de php se mettent en tête de faire un truc comme Java où par défaut c'est un état batard entre public et protected)

[éric1]

Merci à tous!