Discussion :

[Larger]

Bonjour,

J'ai une base de données contenant des données utilisateurs.
On y trouve notamment un champ pseudo de type VARCHAR dont une de ses propriétés est l'unicité.

Mon problème est que je dois à la fois vérifier qu'un pseudo est disponible ou non et si ce n'est pas le cas générer n pseudos aléatoirement.
Ces pseudos générés sont assez simples : c'est le pseudo initial concaténé avec un entier inférieur à 100.

Je précise que j'utilise SQL Server 2008 et que j'appelle une procédure depuis mon code C#.

La seule solution que j'ai trouvé pour le moment est que j'appelle une proc très simple qui me dit si le pseudo est utilisé ou pas. Puis je fais un appel à la proc suivante :

Code :

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

1
2
3
        SELECT us.USR_PSEUDO_CH
		FROM USR_USER us
		WHERE LOWER(us.USR_PSEUDO_CH) LIKE @PSEUDO + '[0-9][0-9]'

En gros elle sélectionne tous les pseudos déjà utilisés qui sont du type "@PSEUDO + '[0-9][0-9]" et qui me permet de déterminer en C# quels sont les pseudos qui sont disponibles.


Cette manière de faire fonctionne, mais je me demande s'il n'y a pas plus efficace notamment au niveau de mes procs.

Qu'en pensez-vous ?


Merci pour votre attention.

[SQLpro]

Lisez ceci : http://sqlpro.developpez.com/Exercic...e=part-3#LIV-F
Et dites moi si c'est ce que vous voulez...

A +

[Larger]

C'est presque ça. La principale différence est que je ne veux pas faire un insert mais un select.

[elsuket]

Bonjour,

Voici une solution, mais j'ai hâte de voir celle que proposera SQLPro

Je considère la table contenant les pseudos comme suivante :

Code :

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

1
2
3
4
5
6
CREATE TABLE pseudo
(
	pseudo_id int NOT NULL IDENTITY CONSTRAINT PKpseudo PRIMARY KEY
	, pseudo varchar(16) NOT NULL CONSTRAINT UQpseudo UNIQUE
)
GO

La procédure suivante affecte à la variable @pseudo soit le pseudo s'il n'existe pas, soit le pseudo avec un nombre déjà affecté, concaténé à celui-ci.

Elle insère 10 d'entre-eux dans la table, mais vous pouvez vous en passer.

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
CREATE PROCEDURE ps_pseudo_check
	@pseudo_to_be_checked varchar(16)
	, @pseudo varchar(16) = NULL OUTPUT
AS
BEGIN
	SET NOCOUNT ON
 
	SET	@pseudo = @pseudo_to_be_checked
 
	IF EXISTS
	(
		SELECT	*
		FROM	dbo.pseudo
		WHERE	pseudo = @pseudo_to_be_checked
	)
	BEGIN
		;WITH
			CTE AS
			(
					SELECT	0 AS n
				UNION ALL
					SELECT	n + 1
					FROM	CTE
					WHERE	n < 100
			)
		INSERT	INTO dbo.pseudo
		SELECT	TOP 10 @pseudo_to_be_checked + CAST(n AS varchar(3))
		FROM	CTE
		ORDER	BY NEWID()
		OPTION	(MAXRECURSION 100)
 
		SELECT	@pseudo = pseudo
		FROM	dbo.pseudo
		WHERE	pseudo_id = SCOPE_IDENTITY()
	END
	ELSE
	BEGIN
		INSERT	INTO dbo.pseudo (pseudo) VALUES (@pseudo)
	END
END
GO

Utilisation :

Code :

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

1
2
3
4
5
DECLARE @pseudo varchar(16) = 'elsuket'
 
EXEC dbo.ps_pseudo_check @pseudo, @pseudo OUTPUT
 
SELECT @pseudo

Si vous exécutez deux fois ce code, vous aurez successivement elsuket puis elsuketNN, où NN est un entier

@++

[Larger]

Je ne suis pas un pro de SQL Server mais je trouve votre manière de faire beaucoup plus élégante que mon LIKE.

En transformant comme ceci, j'obtiens exactement ce que je veux :

Code :

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

1
2
3
4
5
6
		SELECT TOP 10 '@pseudo_to_be_checked ' + CAST(n AS varchar(3))
		FROM	CTE
		LEFT OUTER JOIN pseudo ps ON ps.pseudo = 'pseudo_to_be_checked ' + CAST(n AS varchar(3))
		WHERE ps.pseudo IS NULL
		ORDER	BY NEWID()
		OPTION	(MAXRECURSION 100)

Merci !


J'ai également très envie de voir la proposition de SQLPro avant de clore ce sujet.