Discussion :

[Wazzouille]

Bonjour,

Je débute en c# et j'aimerais contourner un problème qui ma foi est peu handicapant mais ennuyant à la longue.

Imaginons que nous avons un objet appelé 'Maison' et que cet objet a une propriété qui est elle-même un objet appelé Porte. Cet objet Porte a elle-même différente caractéristique comme par exemple 'hauteur'.

Pourquoi doit-on obligatoirement regarder dans une condition que :

Code :

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

Maison.Porte != null && Maison.Porte.Hauteur != null

Et pas tout simplement

Code :

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

Maison.Porte.Hauteur != null

Je comprend le procédé réalisé par la machine derrière lors de l’exécution et le besoin de procédé de la première manière, mais n'y a-t-il pas un moyen de le contourner car lorsque l'on a un embriquement d'objet assez conséquent, cela pour devenir assez lourd à écrire ?

D'avance merci pour vos réponses,
Wazzouille

[popo]

Tu peux toujours t'assurer que la porte soit créée dans le constructeur de la maison ou bien lorsque tu la déclare.

Code C# :

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

1
2
3
4
5
6
7
8
9
public class Maison
{
    public Porte Porte { get; private set; }
 
    public Maison()
    {
        Porte = new Porte();
    }
}

ou

Code C# :

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

1
2
3
4
5
6
7
8
public class Maison
{
    Porte porte = new Porte();
    public Porte Porte
    {
         get { return porte; }
     }
 }

Mais si laisse libre accès à cette propriété, tu ne peux pas empêcher le consommateur de ta classe de libérer cet Objet.

[Pol63]

parce que si porte est null ca revient à écrire null.hauteur ce qui plante

par contre il y a une syntaxe qui va te convenir c'est

Code :

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

if (Maison.Porte?.Hauteur != null)

un ? derrière quelque chose permet de renvoyer null sans tester la suite
on peut en enchainer plusieurs genre Maison?.Porte?.Hauteur

[Noxen]

La propriété Hauteur de ta Porte devrait être un float, double ou même int, qui sont des types valeur, donc non nullable par défaut. Sinon pour ton problème C#6 introduit justement l'opérateur ?. qui vérifie que l'objet sur lequel on appelle une méthode est bien instancié avant de faire l'appel, et renvoie null sinon ; à combiner avec l'opérateur ?? qui fait un null check sur l'objet passé avant et permet de proposer une valeur de remplacement.

Si mon logger n'est pas null, je logge mon message, sinon c'est ignoré :

Code :

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

logger?.Log(message);

Si mon item n'a pas d'enfants, je renvoie un tableau vide :

Code :

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

public IEnumerable<Item> Children { get { return _children ?? new Item[0]; } }

On peut chaîner les ?. et renvoyer une valeur par défaut au besoin :

Code :

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

var hauteur = maison?.Porte?.Hauteur ?? 0.0;

On peut faire de l'initialisation paresseuse avec ces opérateurs ; si mon objet est null, je l'initialise avant de le renvoyer :

Code :

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

public List<Item> Items { get { return _items ?? (_items = new List<Item>())); } }

Pour terminer, je dirais que tu peux aussi avoir une approche un peu "conservative" dans certains cas. Si un objet a une propriété sensée être initialisée à tout moment, ce n'est pas la peine de faire un null check : si la propriété est nulle à un moment ou un autre, c'est qu'il y a un bug quelque part ; tu as alors vraiment envie que ton application casse pendant ton debug, afin d'être informé de l'existence de ce bug.

[Wazzouille]

le Maison?.Porte?.Hauteur est exactement ce qu'il me fallait

Merci pour vos réponses !