Discussion :

[new_wave]

Bonjour,

soit la liste d'animaux suivante

Code :

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

1
2
3
4
5
6
7
8
9
 
List<Animal> animaux = new List<Animal>();
                    Animal chien_1 = new Chien();
 
                    Animal chat_1 = new Chat();
 
                    Animal animal_1 = new Animal();
 
                    Chien chien_2 = new Chien();

Soit la classe Chien avec sa méthode Aboyer

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
 
class Chien : Animal
    {
        public Chien(): base() { }
 
        public Chien(string prenomDuChien):base(prenomDuChien)
    {
 
    }//fin de constructeur*/
 
     public void Aboyer()
     {
         Console.WriteLine("Wouaf !");
     }//fin de méthode Aboyer*/
    }//fin de classe

Si j'écris ce code pour faire aboyer les chiens

Code :

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

1
2
3
4
5
6
7
8
9
 
 
foreach (Animal a in animaux)
          {   
               if (a is Chien)
          {
              ((Chien)a).Aboyer();
           }//fin de if 
}//fin de foreach

Est que le cast en Chien s'executera aussi pour l'instance chien_2 alors qu'il n'est pas nécessaire, ou est ce que ce dernier cast ne sera pas réalisé car chien_2 est déjà un Chien qui peut Aboyer?


Merci beaucoup de votre réponse

Bien cordialement.

new_wave

[popo]

C# fait ce que tu lui demandes de faire.
Si tu forces un CAST, il sera effectué.

D'ailleurs ce code génère un avertissement S3247 à cause du double CAST.
https://rules.sonarsource.com/csharp/RSPEC-3247/

Il est préférable d'utiliser la syntaxe suivante :

Code :

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

1
2
3
4
5
6
7
foreach (Animal a in animaux)
{
    if (a is Chien chien)
    {
        chien.Aboyer();
    }
}

[umfred]

ça s'appelle un modèle (ou pattern): la doc : https://learn.microsoft.com/fr-fr/do...-type-patterns

[François DORIN]

A noter qu'il est possible d'écrire autrement le code encore, en le simplifiant d'autant plus, si tu ne cherches à exécuter un traitement spécifique que sur une seule classe.

Code :

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

1
2
3
4
foreach (Chien chien in animaux.OfType<Chien>())
{   
   chien.Aboyer();           
}

Par contre, s'il y en a plusieurs, il vaut mieux utiliser l'approche de popo (et avec un switch, c'est encore mieux).

[new_wave]

C# fait ce que tu lui demandes de faire.
Si tu forces un CAST, il sera effectué.

D'ailleurs ce code génère un avertissement S3247 à cause du double CAST.
https://rules.sonarsource.com/csharp/RSPEC-3247/

Il est préférable d'utiliser la syntaxe suivante :

Code :

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

1
2
3
4
5
6
7
foreach (Animal a in animaux)
{
    if (a is Chien chien)
    {
        chien.Aboyer();
    }
}

Bonjour,

Dans ce cas , si je comprends bien, quand a est déjà un Chien avec toutes les méthodes d'un Chien, il n'y a pas de conversion et quand a est un Chien avec les méthodes d'un Animal, il est converti en "vrai" Chien .
is peut donc aussi permettre un CAST, si j'ai bien compris.

Merci de ton retour.

Bien cordialement.

new_wave

[new_wave]

C'est la solution que j'ai finalement retenue.Je pense que c'est la plus efficace!

Bien cordialement.

new_wave

A noter qu'il est possible d'écrire autrement le code encore, en le simplifiant d'autant plus, si tu ne cherches à exécuter un traitement spécifique que sur une seule classe.

Code :

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

1
2
3
4
foreach (Chien chien in animaux.OfType<Chien>())
{   
   chien.Aboyer();           
}

Par contre, s'il y en a plusieurs, il vaut mieux utiliser l'approche de popo (et avec un switch, c'est encore mieux).

[popo]

Dans ce cas , si je comprends bien, quand a est déjà un Chien avec toutes les méthodes d'un Chien, il n'y a pas de conversion

Non, tu n'a pas compris.
Si tu forces un CAST, il sera effectué.

quand a est un Chien avec les méthodes d'un Animal, il est converti en "vrai" Chien .

Quand tu déclare une variable avec un type Animal, il faut obligatoirement le CASTER pour qu'il soit considérer comme un Chien.
Si tu as un Chat et que tu forces le CAST en Chien, tu vas planter.

is peut donc aussi permettre un CAST, si j'ai bien compris.

is fait effectivement un CAST, tout comme [C]as|/C].

C'est la solution que j'ai finalement retenue.Je pense que c'est la plus efficace!

On ne prend pas une décision sur une impression.
On la prend sur des faits et sur la réflexion qu'on réalise à partir de ces fait.
Dans les faits, François DORIN t'as donné un avertissement sur cette syntaxe : il ne fonctionne que si tu gère uniquement des Chiens
Dans les faits, tu gère aussi des chats d'après le code que tu as donné.
Donc, à toi t'en tirer des conclusions.