Discussion :

[kritopal]

Bonjour à tous.

Je désire faire une requête qui recherche des personnes suivant 3 critères : nom, prénom, date de naissance.
L'utilisateur peut mettre 1,2 ou 3 critères.

Lorsque nom et prénom ne sont pas remplis je sais construire ma requête pour qu'il ne tienne pas compte de ces critères (valeur par défaut %).

Par contre pour la date j'ai un problème :
Sur Management studio la requête ci-dessous m'affiche toutes les personnes dont le champs date de naissance n'est pas NULL :

SELECT * FROM [tbl_personnes] WHERE (([per_nom] LIKE '%%%') AND ([per_prenom] LIKE '%%%') AND ([per_dtdn] LIKE '%%%' ))

Sur ASP.net 2 j'ai une erreur car il ne peut pas convertir la chaine en datetime. N'étant pas contrariant je modifie le type de donnée de la date de naissance qui est renvoyée par mon SQLDataSource sur asp.net (datetime en string).

Sur ce il accepte que la date de naissance soit vide mais si je mets une date de naissance il ne retourne aucune personne !

Si quelqu'un a déjà été confronté au problème je suis preneur d'information !

Bonne journée

[bossun]

vu que tu bosses sur SQL Server, je te propose de faire une procédure stockée pour faire ça..

ça va te faciliter grandement la vie

Code :

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

1
2
3
4
5
6
7
8
create procedure Ta_Procedure
(
@nom varchar(100) = NULL,@prenom varchar(100) = NULL,@dtdn datetime =NULL
)
as
 
--tu fais ta série de tests, ici pour voir quels elements sont renseignés et construire ta requete...
RETURN

Evidement pour des raisons de simplicité, je te conseille de faire plusieurs requetes en fonction des parametres renseignés...

[Kelpan]

Bonjour,

tu peux aussi bien générer ta requete par code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
 
MaRequete = _
    "SELECT * FROM [tbl_personnes] WHERE 1 = 1 " 
 
If RechercheParNom Then
    MaRequete &= "AND [per_nom] LIKE @Nom "
End If
 
If RechercheParPreNom Then
...

[robri]

pas très propre le "1=1".

ça serait mieux comme ça :

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
MaRequete = _
    "SELECT * FROM [tbl_personnes] WHERE " 
 
dim lClauseWhere as System.Text.Stringbuilder
If RechercheParNom Then
    lClauseWhere.Append("[per_nom] LIKE @Nom AND ")
End If
 
If RechercheParPreNom Then
...
 
'pour enlever le " AND " de la fin
If Not String.IsNullOrEmpty(lClauseWhere.ToString) Then
MaRequete &= lClauseWhere.ToString.substring(0, lClauseWhere.length - 5)
'execution de la requete
End If

[Kelpan]

Code :

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

1
2
3
4
5
6
 
'pour enlever le " AND " de la fin
If Not String.IsNullOrEmpty(lClauseWhere.ToString) Then
MaRequete &= lClauseWhere.ToString.substring(0, lClauseWhere.length - 5)
'execution de la requete
End If

Franchement....

Je préfère ma solution pas très propre ...

J'ai déjà détecté un risque de bug avec la tienne...

[Delphi-ne]

Le problème du 1=1 est que SQL serveur risque de na pas retrouver l'index à utiliser
J'ai le cas avec l'application sur laquelle je travaille en ce moment et là, en fonction du nombre d'enregistrements de la table, les temps de réponse explosent
On peut donc forcer l'index

Code :

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

1
2
3
Select xx from LaTable With (Index(NomIndex))
Where 1=1
and ...

[robri]

J'ai déjà détecté un risque de bug avec la tienne...

quel risque tu as trouvé stp?

[Kelpan]

Le problème du 1=1 est que SQL serveur risque de na pas retrouver l'index à utiliser
J'ai le cas avec l'application sur laquelle je travaille en ce moment et là, en fonction du nombre d'enregistrements de la table, les temps de réponse explosent
On peut donc forcer l'index

Code :

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

1
2
3
Select xx from LaTable With (Index(NomIndex))
Where 1=1
and ...

Je ne vois pas en quoi le test de 2 constantes à un rapport avec l'index à utiliser.
Quoiqu'il en soit, je ne m'avance pas et ce que tu affirmes mérite une étude un peu plus poussé.

quel risque tu as trouvé stp?

Je te mets sur la voie, si l'utilisateur décide de ne faire aucunes sélections, quelle sera ta requete ?

[kritopal]

Bonjour et merci pour vos réponses.

La solution est effectivement d'utiliser une procédure stockée.

Cependant ma méthode n'est pas très élégante pour la date : j'ai mis une valeur par défaut sur le serveur asp sous forme de chaîne.
La procédure stockée teste la valeur de la chaîne et applique une requête différente s'il y a une valeur date envoyé ou non.

[robri]

Je te mets sur la voie, si l'utilisateur décide de ne faire aucunes sélections, quelle sera ta requete ?

il n'y aura tout simplement pas de requète.

Code :

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

1
2
3
4
If Not String.IsNullOrEmpty(lClauseWhere.ToString) Then
MaRequete &= lClauseWhere.ToString.substring(0, lClauseWhere.length - 5)
'execution de la requete
End If

vu que je lance la requète seulement si la clause where n'est pas vide.