Discussion :

[randriano]

Bonjour,

Je me bloque sur une requête SQL, c'est du Linq Entity plutôt

Code :

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

1
2
3
4
5
6
7
8
9
10
Dim ListOfCampagnes As List(Of campagnes) = (From campagne In context.campagnes.Include("annonceurs").Include("secteurs") _
                                                    Where _
                                                        (campagne.Nom.Contains(argKeyWord) Or _
                                                         campagne.annonceurs.Nom.Contains(argKeyWord) Or _
                                                         campagne.secteurs.Nom.Contains(argKeyWord)) _
                                                        And _
                                                        (campagne.annonceurs.IdAnnonceur = argIdAnnonceur Or argIdAnnonceur = 0) _
                                                    Order By _
                                                        campagne.annonceurs.Nom Ascending _
                                                    Select campagne Distinct).ToList

Comment gérer cette requête en cas de NULL dans certains champs des tables présentes? Ca crashe (NullReferenceException). Je cherche un équivalent à du LEFT JOIN en fait mais bon c'est du Linq

[GuruuMeditation]

Pour faire un LEFT join en LINQ, il faut utiliser la fonction DefaultIfEmpty()

Exemple :

Code :

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

1
2
3
 
var i = from a in table1 join b in table2 on a.ID = b.ID into temp
from c in temp.DefaultIfEmpty() select c;

avec DefaultIfEmpty(), LINQ va retourner une sequence vide au lieu de rien du tout => le le join va se faire

[randriano]

Merci pour le truc de DefaultEmpty mais ce qu'il me faut c'est en fait ne pas considérer les enregistrements nuls (INNER JOIN)
Par exemple la requête suivante crashe car une ligne de ListOfBooks ayant un IdLibrary vide (FK vers la table libraries)

Code :

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

ListOfBooks = (From c In ListOfBooks.DefaultIfEmpty() Order By c.libraries.Name Select c).ToList

[anthyme]

Tu es sûr que tu n'es pas en linq to object sur cette requete ?

Car en effet en linq to object on ne peut pas faire d'orderby avec des reference null de façon standard

[randriano]

J'utilise du Linq to Entities mais pas du Linq to Object mais peut que linq2entities ne permet pas aussi de faire d'orderby avec des NULL

[The_badger_man]

Merci pour le truc de DefaultEmpty mais ce qu'il me faut c'est en fait ne pas considérer les enregistrements nuls (INNER JOIN)
Par exemple la requête suivante crashe car une ligne de ListOfBooks ayant un IdLibrary vide (FK vers la table libraries)

Code :

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

ListOfBooks = (From c In ListOfBooks.DefaultIfEmpty() Order By c.libraries.Name Select c).ToList

bin il suffit de rajouter un filtre avec libraries qui n'est pas null.

Code :

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

ListOfBooks = (From c In ListOfBooks Where c.libraries <> Null Order By c.libraries.Name Select c).ToList

ps: je ne suis pas sur de la syntaxe en VB.NET.

[randriano]

En fait, ce n'est pas c.libraries qui contient des lignes vides causant l'erreur mais plutôt ListOfBooks qui contient des "Nom" vides, en fait "<> NULL" ou "IsNot Nothing"?

Ma remarque: en cherchant dans ce forum, je n'ai pas trouvé beaucoup de gens qui utilisent des Include() comme dans mes requêtes:

Code :

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

1
2
Return (From a In Context.libraries.Include("sections").Include("sections.rangees").Include("sections.rangees.messages").Include("sections.rangees.messages.tarifs").Include("sections.rangees.messages.tarifs.produitsvendus").Include("sections.rangees.messages.tarifs.produitsvendus.distributions").Include("sections.rangees.messages.tarifs.produitsvendus.distributions.produits").Include("sections.rangees.messages.tarifs.produitsvendus.distributions.fournisseurs").Include("sections.rangees.messages.tarifs.produitsvendus.supports").Include("sections.rangees.messages.tarifs.produitsvendus.supports.medias").Include("sections.rangees.messages.parutions") _
                Select a).ToList()

[GuruuMeditation]

Ma remarque: en cherchant dans ce forum, je n'ai pas trouvé beaucoup de gens qui utilisent des Include() comme dans mes requêtes:

Code :

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

1
2
Return (From a In Context.libraries.Include("sections").Include("sections.rangees").Include("sections.rangees.messages").Include("sections.rangees.messages.tarifs").Include("sections.rangees.messages.tarifs.produitsvendus").Include("sections.rangees.messages.tarifs.produitsvendus.distributions").Include("sections.rangees.messages.tarifs.produitsvendus.distributions.produits").Include("sections.rangees.messages.tarifs.produitsvendus.distributions.fournisseurs").Include("sections.rangees.messages.tarifs.produitsvendus.supports").Include("sections.rangees.messages.tarifs.produitsvendus.supports.medias").Include("sections.rangees.messages.parutions") _
                Select a).ToList()

Les Include() c'est très facile mais ça peut vite partir en vrille si il y en a beaucoup (comme dans ton exemple, ça doit donner un graphe enorme).
En général, si beaucoup d'include, je préfère faire une StoreProc ou une View qui me renvoie uniquement les champs dont j'ai besoin, et faire un LINQ dessus, plutot que tout ramener.

[randriano]

StoreProc? Une procédure stockée au niveau du serveur

J'ai montré ce code car en effet ça me fait une exception EntityCommandExecutionException sur certaines machines! J'ai pensé à un enregistrement vide mais la BD est la même donc je ne sais pas

J'en profite pour demander: comment déboguer du LINQ? Voir le graphe comme tu le dis là, EquinoxeDotnet?

[GuruuMeditation]

StoreProc? Une procédure stockée au niveau du serveur?

Oui. Ou alors une vue.

J'en profite pour demander: comment déboguer du LINQ? Voir le graphe comme tu le dis là, EquinoxeDotnet?

Perso j'utilise LINQPad. C'est vrailent pratique. La version de base est gratuite, et la version payante (avec intellisense) est de 39US$, et ça les vaut largement! (Je ne sais pas si c'est toujours le cas mais quand je l'ai acheté c'etait mises à jour à vie compris! Si ça c'est pas une bonne affaire!).
http://www.linqpad.net/
(NOTE : Il semble que la l'intellisense de la version payante ne marche qu'avec C#)

[randriano]

Génial ce LinqPad, merci pour le tuyau, je le teste avec la version standalone et ça me suffit pour le moment

[randriano]

Je reviens sur ce sujet car j'ai encore quelques questions: est-ce que ma requête avec beaucoup d'Include() consomme de la mémoire ou nécessite beaucoup de ressource processeur? En fait, il paraît que l'erreur EntityCommandExecutionException est due au fait que le Command Timeout ne suffit plus, où peut-on élever sa valeur en dehors du code?

[GuruuMeditation]

La connectionstring se trouve dans le fichier .config de ton appli. Tu peux rajouter un "Connection Timeout=XX".
Mais il vaut mieux regarder la cause, voir si tu peux optimiser tes query.
Avec beaucoup d'include, ca va consommer du temps DB, de la mémoire principalement. Si ça doit passer par des WCF ou autres, si va utiliser pas mal de resources en plus.

[randriano]

Avec beaucoup d'include, ca va consommer du temps DB, de la mémoire principalement..

Du temps DB? Tu m'avais dit avant de préférer une vue ou une procédure stockée donc cela consommerait tous du temps DB, non?

Dans le cadre d'optimisation de ma query, je rencontre une situation bizarre:

Code :

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

From par In parutions.Include("messages").Include("messages.tarifs").Include("messages.tarifs.formats").Include("messages.tarifs.produitsvendus.distributions.fournisseurs").Include("messages.parutions").Include("messages.tarifs.produitsvendus.supports").Include("messages.tarifs.produitsvendus.supports.medias") Where par.messages.plansmedias.IdPlanMedia = 12 And par.messages.tarifs.produitsvendus.distributions.fournisseurs.IdFournisseur = 5 And par.messages.tarifs.produitsvendus.supports.medias.IdMedia = 2 Select par

s'exécute rapidement mais la seconde query suivante provoque un EntityCommandExecutionException:

Code :

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

From par In parutions.Include("messages").Include("messages.tarifs").Include("messages.tarifs.formats").Include("messages.tarifs.produitsvendus.distributions.fournisseurs").Include("messages.parutions").Include("messages.tarifs.produitsvendus.supports").Include("messages.tarifs.produitsvendus.supports.medias") Where par.messages.plansmedias.IdPlanMedia = 12 And par.messages.tarifs.produitsvendus.distributions.fournisseurs.IdFournisseur = 5 And par.messages.tarifs.produitsvendus.supports.medias.IdMedia = 2 And par.messages.HeureParution = "7h30-7h45" Select par

dont la différence est seulement un AND par.messages.HeureParution = "7h30-7h45"

[Nathanael Marchand]

Du temps DB? Tu m'avais dit avant de préférer une vue ou une procédure stockée donc cela consommerait tous du temps DB, non?

Dans le cadre d'optimisation de ma query, je rencontre une situation bizarre:

Code :

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

From par In parutions.Include("messages").Include("messages.tarifs").Include("messages.tarifs.formats").Include("messages.tarifs.produitsvendus.distributions.fournisseurs").Include("messages.parutions").Include("messages.tarifs.produitsvendus.supports").Include("messages.tarifs.produitsvendus.supports.medias") Where par.messages.plansmedias.IdPlanMedia = 12 And par.messages.tarifs.produitsvendus.distributions.fournisseurs.IdFournisseur = 5 And par.messages.tarifs.produitsvendus.supports.medias.IdMedia = 2 Select par

s'exécute rapidement mais la seconde query suivante provoque un EntityCommandExecutionException:

Code :

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

From par In parutions.Include("messages").Include("messages.tarifs").Include("messages.tarifs.formats").Include("messages.tarifs.produitsvendus.distributions.fournisseurs").Include("messages.parutions").Include("messages.tarifs.produitsvendus.supports").Include("messages.tarifs.produitsvendus.supports.medias") Where par.messages.plansmedias.IdPlanMedia = 12 And par.messages.tarifs.produitsvendus.distributions.fournisseurs.IdFournisseur = 5 And par.messages.tarifs.produitsvendus.supports.medias.IdMedia = 2 And par.messages.HeureParution = "7h30-7h45" Select par

dont la différence est seulement un AND par.messages.HeureParution = "7h30-7h45"

C'est peut être un problème d'indexation sur la table.
Le mieux serait de prendre le profiler SQL, de regarder la requête et son plan d'execution. Tu sauras si tu fais un index lookup ou un full scan de la table. Vu la tête des données je dirais que c'est pas indexé

[randriano]

En effet, il n'y a pas d'index sur ce champ de type varchar

Le bizarre selon moi c'est que sous LinqPad la durée d'exécution de ces 2 requêtes n'est que de 1 seconde!

[GuruuMeditation]

Du temps DB? Tu m'avais dit avant de préférer une vue ou une procédure stockée donc cela consommerait tous du temps DB, non?

Oui mais (pour simplifier) ça prends moins de temps de faire une vue ou stored procedure optimisée que de "bêtement" tout ramener, c'est ce que je voulais dire.