Discussion :

[darkgamin]

Ma recherche ne me retourne jamais de résultat quelqu'un peut m'aider?
J'utilise sql server !

Code :

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

1
2
3
4
5
6
7
8
set ANSI_NULLS ON
set QUOTED_IDENTIFIER ON
GO
ALTER PROCEDURE [dbo].[sp_RechercheClient]  @critere varchar(50)
AS 
SELECT *
FROM dbo.CLIENTS_AVANTAGE
WHERE upper(CNOM)+upper(CRESP)+upper(CADR)+upper(CVIL) LIKE upper('%@critere%')

Merci d'avance

[Invité]

Votre façon de faire est dégueulasse et ne doit pas être très performante.
Vous feriez mieux de faire une clause différente par champs.
En plus, si un des champs est null, la concaténation retournera un null...
Sinon, c'est :

Code :

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

1
2
WHERE upper(CNOM)+upper(CRESP)+upper(CADR)+upper(CVIL) LIKE upper('%'+@critere+'%')

[darkgamin]

Votre façon de faire est dégueulasse et ne doit pas être très performante.
Vous feriez mieux de faire une clause différente par champs.
En plus, si un des champs est null, la concaténation retournera un null...
Sinon, c'est :

Code :

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

1
2
WHERE upper(CNOM)+upper(CRESP)+upper(CADR)+upper(CVIL) LIKE upper('%'+@critere+'%')

Merci pour l'encouragement Je suis débutant en sql server alors je suis ouvert au suggestion

zinzineti : Merci sa fonctionne !

[Invité]

Merci pour l'encouragement Je suis débutant en sql server alors je suis ouvert au suggestion

Désolé de vous ouvrir les yeux, mais les pistes sont dans la suite du message :

Vous feriez mieux de faire une clause différente par champs.
En plus, si un des champs est null, la concaténation retournera un null...

Si vous voulez plus de détails, je reste à votre disposition.

[darkgamin]

Désolé de vous ouvrir les yeux, mais les pistes sont dans la suite du message :
Si vous voulez plus de détails, je reste à votre disposition.

En effet j'aimerais bien améliorer ma procédure. Cependant je ne comprend pas très bien ce que vous voulez dire par "faire une clause différente par champ" Merci!

[Invité]

En effet j'aimerais bien améliorer ma procédure. Cependant je ne comprend pas très bien ce que vous voulez dire par "faire une clause différente par champ" Merci!

Faites comme ceci, comme ça si vous avez un champ à null, ça pourra tout de même vous ramener la ligne :

Code :

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

1
2
3
4
WHERE CNOM LIKE '%'+@critere+'%'
	or CRESP LIKE '%'+@critere+'%'
	or CADR LIKE '%'+@critere+'%'
	or CVIL LIKE '%'+@critere+'%'

De plus, MS Sql server n'est pas case sensitive par défaut - (sauf si la collation a été modifié pour que la bd le soit), alors les upper ne sont pas vraisemblablement pas nécessaire dans votre requête.

[zinzineti]

Essaye ceci pour voir

Code :

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

1
2
3
4
5
6
7
8
9
10
SET ANSI_NULLS ON
SET QUOTED_IDENTIFIER ON
GO
ALTER PROCEDURE [dbo].[sp_RechercheClient]  @critere varchar(50)
AS 
DECLARE @varUpper VARCHAR (1024)
SET @varUpper = upper (@critere)
SELECT *
FROM dbo.CLIENTS_AVANTAGE
WHERE upper(CNOM)+upper(CRESP)+upper(CADR)+upper(CVIL) LIKE '%' + @varUpper + '%'