Discussion :

[azhek]

Bonjour à tous je me demande comment accéder à la propriété (get ; set) d'une classe enfant en polymorphisme. Voilà j'explique ... Je suis entrain de faire un petit jeu sur monogame en c#. J'ai la classe GameObject qui à ses propriétés et une classe bouclier qui possède toute les propriétés de la classe GameObject sauf une qui est l'énergie du bouclier. Donc dans mon main j'ai un GameObject bouclier qui est également un bouclier puisque
bouclier = new Bouclier();. Mais au niveau de mon affichage j'aimerais pouvoir afficher la quantité d'énergie restante du dit bouclier. Mais je ne peux pas utiliser bouclier.Energie puisqu'il n'est pas accessible. Dans ma classe le champs energie est private et l'accesseur public. Je me demande donc comment y accéder à partir du main ??

[Pol63]

Code :

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

1
2
3
4
5
if (lavariable is Bouclier) // test du type
{
  var energie = ((Bouclier)lavariable).Energie; // cast explicite
  //
}

il y a des patterns plus efficaces pour faire du spécifique sans le savoir, mais bon ce code est simple et reste propre

[StringBuilder]

Question performance, "as" n'est-il pas plus performant que le cast ?

Code csharp :

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

1
2
3
4
5
6
 
if (lavariable is Bouclier) // test du type
{
  var energie = (lavariable as Bouclier).Energie; // pas de création d'un nouveau handle
  //
}

[popo]

Question performance, "as" n'est-il pas plus performant que le cast ?

Code csharp :

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

1
2
3
4
5
6
 
if (lavariable is Bouclier) // test du type
{
  var energie = (lavariable as Bouclier).Energie; // pas de création d'un nouveau handle
  //
}

Le "as" est moins performant puisqu'il refait implicitement un "is".

Dans les docs Microsoft, ils conseillent de faire comme ceci puisque le "as" ne va affecter la variable que si le "is" renvoie true :

Code :

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

1
2
3
4
5
var bouclier = (lavariariable as bouclier);
if (bouclier != null)
{
     var energie = bouclier.Energie;
}

[StringBuilder]

Dans ce cas... Ceci ?

Code csharp :

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

var energie = (lavariable as Bouclier)?.Energie;

[popo]

Sur le principe, oui.

Même si personnellement, je ne suis pas fan de cette syntaxe car l’œil peut facilement manquer le point d'interrogation.

[StringBuilder]

Ça c'est ce qu'il se passe quand on lit trop vite sans se poser de question

[wallace1]

Ça c'est ce qu'il se passe quand on lit trop vite sans se poser de question

Ça arrive a tout le monde ne t'en fais pas

[popo]

Il y aussi :
- des gens pour lesquels cette syntaxe ne sera pas parlante,
- des gens qui vont l'utiliser là où ce n'est pas utile, juste pour éviter de se poser la question
- des gens qui vont abuser de cette syntaxe au point que ça devienne illisible

Voici le genre de chose que j'ai déjà vu dans du code (et encore je l'ai simplifié)
Qu'est-ce qui te parait le plus compréhensible :

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
string tutu;
if (A > 0)
{
    if (B > 0) { tutu = "A +, B +";} 
    else if (B < 0) { tutu = "A +, B -"; }
    else { tutu = "A +, B Z"; }
}
else if (A < 0)
{
    if (B > 0) { tutu = "A -, B +"; }
    else if (B < 0) { tutu = "A -, B -"; }
    else { tutu = "A -, B Z"; }
}
else
{
    if (B > 0) { tutu = "A Z, B +"; }
    else if (B < 0) { tutu = "A Z, B -"; }
    else { tutu = "A Z, B Z"; }
}

Code :

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

1
2
3
4
var titi = (A > 0) ? 
    (B > 0) ? "A +, B +" : (B < 0) ? "A +, B -" : "A +, B Z" : 
    (A < 0) ? (B > 0) ? "A -, B +" : (B < 0) ? "A -, B -" : "A -, B Z" : 
    (B > 0) ? "A Z, B +" : (B < 0) ? "A Z, B -" : "A Z, B Z";

OK, on ne parle plus du Nullable<T> ici, mais c'est pour dire que raccourcir la syntaxe n'est pas systématiquement une bonne idée.

[StringBuilder]

Voici le genre de chose que j'ai déjà vu dans du code (et encore je l'ai simplifié)
Qu'est-ce qui te parait le plus compréhensible :

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
string tutu;
if (A > 0)
{
    if (B > 0) { tutu = "A +, B +";} 
    else if (B < 0) { tutu = "A +, B -"; }
    else { tutu = "A +, B Z"; }
}
else if (A < 0)
{
    if (B > 0) { tutu = "A -, B +"; }
    else if (B < 0) { tutu = "A -, B -"; }
    else { tutu = "A -, B Z"; }
}
else
{
    if (B > 0) { tutu = "A Z, B +"; }
    else if (B < 0) { tutu = "A Z, B -"; }
    else { tutu = "A Z, B Z"; }
}

Code :

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

1
2
3
4
var titi = (A > 0) ? 
    (B > 0) ? "A +, B +" : (B < 0) ? "A +, B -" : "A +, B Z" : 
    (A < 0) ? (B > 0) ? "A -, B +" : (B < 0) ? "A -, B -" : "A -, B Z" : 
    (B > 0) ? "A Z, B +" : (B < 0) ? "A Z, B -" : "A Z, B Z";

OK, on ne parle plus du Nullable<T> ici, mais c'est pour dire que raccourcir la syntaxe n'est pas systématiquement une bonne idée.

Euh... Ceci ? (en tout cas, si c'est pas plus lisible ça a au moins le mérite très bien plus performant !)

Code csharp :

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

1
2
3
 
char[] note = new char[]() { '-', 'Z', '+'};
string tutu = $"A {note[Math.Sign(A) + 1]}, B {note[Math.Sign(B) + 1]}";

Évidement, dans la vraie vie, je remplacerais "note[Math.Sign(A) + 1]" par un appel à une méthode statique.

Code csharp :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
        private static readonly char[] Signs = new char[] { '-', 'Z', '+' };
        private static char GetSignChar(int note)
        {
            return Signs[Math.Sign(note) + 1];
        }
 
        static void Main(string[] args)
        {
            int A = 10, B = -1;
            string tutu = $"A {GetSignChar(A)}, B {GetSignChar(B)}";
            Console.WriteLine(tutu);
 
            Console.ReadKey(true);
        }

Accessoirement, si y'a bien un truc que je trouve illisible, c'est bien cette manie de foutre des "var" à tout bout de champ.
Même JavaScript s'est mis aux types forts, j'arrive pas à comprendre cette mode à absolument vouloir faire abstraction des types qu'on utilise.

[popo]

Accessoirement, si y'a bien un truc que je trouve illisible, c'est bien cette manie de foutre des "var" à tout bout de champ.
Même JavaScript s'est mis aux types forts, j'arrive pas à comprendre cette mode à absolument vouloir faire abstraction des types qu'on utilise.

Je suis bien d'accord avec toi concernant le var.
Mais ça c'est une question de gout.

Dans mon équipe par exemple j'ai un type qui trouve que ceci est redondant et je me fait traiter de tout les noms si je ne mets pas var

Code :

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

using (WebClient client = new WebClient()){ ... }

Alors que ceci acceptable :

Code :

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

string x = GetSomething();

Par contre, je n'ai encore jamais vu cette syntaxe avec le dollar.
Par curiosité, je l'ai essayé mais ça ne compile pas, j'ai le message : Unexpected character '$' (j'essayé en 4.6.2)

[StringBuilder]

C'est une nouveauté de la 4.7

Je trouve cette syntaxe bien pratique, ça économise énormément de code.
Par contre, ça peut vite devenir le bordel si on est crasseux...

Il est préférable de se limiter aux variables ou simples appels de méthodes simples... Comme pour les arguments d'un String.Format()...

[azhek]

Merci beaucoup ça fonctionne nickel. J'avais déjà tenté un cast (Bouclier)bouclier.Energie; mais avec une parenthèse en moins ce n'est pas la même chose. Mais par contre je ne ferai plus cette erreur

[Pol63]

Question performance, "as" n'est-il pas plus performant que le cast ?

le mieux doit être

Code :

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

if (variable is bouclier b) console.writeline(b.propriete)

[StringBuilder]

le mieux doit être

Code :

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

if (variable is bouclier b) console.writeline(b.propriete)

Ah oui c'est vrai que j'avais oublié cette évolution du is qui fait un as

[popo]

le mieux doit être

Code :

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

if (variable is bouclier b) console.writeline(b.propriete)

Compile pas en 4.6.2...
C'est aussi une nouveau du 4.7 ?

[StringBuilder]

Compile pas en 4.6.2...
C'est aussi une nouveau du 4.7 ?

Oui c'est un raccourci de 4.7

[François DORIN]

Question performance, "as" n'est-il pas plus performant que le cast ?

Code csharp :

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

1
2
3
4
5
6
 
if (lavariable is Bouclier) // test du type
{
  var energie = (lavariable as Bouclier).Energie; // pas de création d'un nouveau handle
  //
}

Cela dépend complètement de la situation. Il serait intéressant de refaire des tests aujourd'hui (comme les compilateurs ont beaucoup évolué), mais caster un objet pour faire du downcasting est différent que caster un objet en int? par exemple.

Dans le premier cas, l'usage de as était plus performant qu'un cast. Dans le second cas, l'usage du cast (associé à is) était plus performant car le compilateur pouvait éviter les opérations de box/unboxing.

Quoi qu'il en soit, d'un point de vue performance, la cast avec as ou avec le nouveau pattern is de C# 7 sont équivalent en terme de performance.

[Pol63]

Quoi qu'il en soit, d'un point de vue performance, la cast avec as ou avec le nouveau pattern is de C# 7 sont équivalent en terme de performance.

persuadé que ce n'était qu'un syntaxic sugar j'avais regardé l'IL généré de if (instance is type newvar) comparé à as et il y avait quelques lignes en moins avec le is qui créé une variable (en c# 7.3 il me semble)
après je sais pas lire l'IL donc je pourrais pas dire la différence ^^
et c'était peut etre l'utilisation que j'en faisais ensuite qui a pu amener une optimisation