Discussion :

[ClaudeBg]

Bonjour,

Je rencontre un problème avec linq to entities.

Pour cerner au mieux le problème, j'ai une table client avec des clients contenant un nom (not null) et un prénom (qui peut être null)

Si j'écris la condition suivante :

Code :

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

1
2
3
4
5
 if( (from c in Bdd.Client
               where c.Nom.Equals(clientsource.Nom) 
               &&  c.Prenom.Equals(clientsource.Prenom)
               select c).FirstOrDefault() == null)
           {...

ou évidemment :

Code :

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

1
2
3
4
5
if((from c in Bdd.Client
             where c.Nom == clientsource.Nom 
             && c.Prenom == clientsource.Prenom
             select c).FirstOrDefault() == null)
             {...

Je m'attend à valider la condition SI dans ma base de données il n'y a pas déjà un client avec le même nom et le même prénom que mon client "clientsource".

Or, voici ce que j'obtiens avec 3 exemples simples (nom, prénom) :

Code :

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

1
2
3
4
Base contient          clientsource                    résultat
Albert,Chose           Albert,Chose                    not null (OK)
Albert,Chose           Albert,Machin                   null (OK)
Albert,null            Albert,null                     null (PAS BON)

Bref, si le nom existe déjà dans la base, mais que les prénoms sont null, alors linq to entities considère que les prénoms sont différents.

Quelqu'un peut m'expliquer ce problème qui consiste à constater que null != null?

Merci d'avance

Claude

[Arnard]

Code :

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

1
2
3
4
5
 if( (from c in Bdd.Client
               where c.Nom.Equals(clientsource.Nom) 
               &&  c.Prenom.Equals(clientsource.Prenom)
               select c).FirstOrDefault() == null)
           {...

Si ton prénom est Null, l'appel en gras lève une exception. Donc tester juste avant si c.Prenom n'est pas null, ou dire à clientsource.Prenom de traiter le Equals si cette valeur est toujours existante.

Code :

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

1
2
3
4
5
if( (from c in Bdd.Client
               where c.Nom.Equals(clientsource.Nom) 
               &&  (( c.Prenom == null && clientsource.Prenom == null) || 
                     ( c.Prenom != null  && c.Prenom.Equals(clientsource.Prenom))
               select c).FirstOrDefault() == null)

L'exception entraine que FirstOrDefault retourne null, et donc que ta condition se vérifie alors que ca ne devrait pas être le cas.

[ClaudeBg]

Salut
-----

Si ton prénom est Null, l'appel en gras lève une exception. Donc tester juste avant si c.Prenom n'est pas null, ou dire à clientsource.Prenom de traiter le Equals si cette valeur est toujours existante.

Ok, merci, je comprends.
Dommage que cette exception ne soit pas reportée pour qu'on puisse s'en apercevoir. En fait, il faut la deviner, ce n'est pas très efficace d'avoir une erreur de ce type non signalée.

Avec ton code, effectivement ça fonctionne, même si c'est un peu plus lourd.

Encore merci

Claude

[Arnard]

En effet, il faut faire attention aux NULL avec Linq, sinon on se retrouve avec es données incohérentes.

En effet c'est un peu plus lourd, après si le client fournit toujours un prénom pour contrôle, base toi sur ce prénom pour tester l'égalité dans ta requête.

Et n'oublies pas le tag résolu

[ClaudeBg]

Salut

Oui, merci.

Pour éviter ce genre de problème, j'ai modifié tous les champs de toutes les tables pour qu'ils soient "NOT NULL", et au lieu d'engistrer une valeur null pour un champs non utilisé, j'y met la valeur string.empty (""). C'est moins élégant, mais vu les circonstances c'est plus sûr.

Encore merci

Claude