Discussion :

[Erakis]

Bonjour à tous,

Je dois récupérer le dernier numéro à la suite d'un segment.
Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
Exemple #1 :
    Pour le segment '1.1.2', je veux récupérer le '2'

Exemple #2
    Pour le segment '10.12,, je veux récupérer le '12'

Exemple #3
    Pour le segment '5', je veux récupérer le '5'

Voilà le code de test que j'ai écris jusqu'ici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
SELECT 
       CASE
	WHEN CHARINDEX('.', '1') = 0 THEN
	    '1'
	ELSE
	    RIGHT('1', CHARINDEX('.', REVERSE('1')) - 1)
            /* Ou cette méthode alternative
            SUBSTRING('1', LEN('1') - CHARINDEX('.', REVERSE('1')) + 2, LEN('1') - (LEN('1') - CHARINDEX('.', REVERSE('1')) + 1))
            */
 	END

Le problème, c'est que MS SQL me retourne l'erreur suivante :
Paramètre de longueur non valide passé à la fonction substring.

C'est n'importe quoi ! Comment la première condition du case peut-elle échouer ?

CHARINDEX('.', '1') donne bien 0 comme résultat, alors pourquoi va-t-il dans le ELSE ???

Merci

[SQLpro]

Voici tous les éléments pour comprendre votre erreur !

Code :

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

1
2
3
4
5
6
7
CREATE TABLE T_SEG
(SEG_ID INT IDENTITY,
 SEG_DATA   VARCHAR(32))
 
INSERT INTO T_SEG VALUES ('1.1.2')
INSERT INTO T_SEG VALUES ('10.12')
INSERT INTO T_SEG VALUES ('5')

-- la solution détaillée :

Code :

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

1
2
3
4
5
6
7
8
9
SELECT SEG_ID, SEG_DATA, CHARINDEX('.', SEG_DATA) AS FIRST_POSITION, 
       LEN(SEG_DATA) - CHARINDEX('.', REVERSE(SEG_DATA)) + 1 AS LAST_POSITION,
       CHARINDEX('.', REVERSE(SEG_DATA)) - 1 AS LONGEUR,
       CASE
          WHEN CHARINDEX('.', SEG_DATA) = 0 
             THEN SEG_DATA
          ELSE SUBSTRING(SEG_DATA, LEN(SEG_DATA) - CHARINDEX('.', REVERSE(SEG_DATA)) + 2, CHARINDEX('.', REVERSE(SEG_DATA)) - 1)   
       END AS EXTRAIT
FROM   T_SEG

-- plus simple avec le REVERSE :

Code :

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

1
2
3
4
5
6
7
SELECT SEG_ID, SEG_DATA,
       CASE
          WHEN CHARINDEX('.', SEG_DATA) = 0 
             THEN SEG_DATA
          ELSE REVERSE(SUBSTRING(REVERSE(SEG_DATA), 1, CHARINDEX('.', REVERSE(SEG_DATA)) - 1))
       END AS EXTRAIT
FROM   T_SEG

A +

[Waldar]

Ca n'empêche pas le moteur de valider la syntaxe employée.
Vous demandez de faire une extraction de chaîne de longueur négative...

[Erakis]

Je comprend mieux pourquoi à présent. La raison, c'est qu'il valide les arguments car ils sont de nature constants.

Si je reprend mon exemple précédant :

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
 
DECLARE @TT TABLE
(
    WBS   VARCHAR(40)
)
 
INSERT INTO @TT VALUES ('1.1.2')
INSERT INTO @TT VALUES ('10.12')
INSERT INTO @TT VALUES ('5')
 
SELECT 
     CASE
          WHEN CHARINDEX('.', WBS) = 0 THEN
              WBS
           ELSE
              RIGHT(WBS, CHARINDEX('.', REVERSE(WBS)) - 1)
     END
FROM @TT

À ce moment là, tout fonctionne très bien !

Eh bien ! À l'avenir je testerai mes fonctions directement sur des tables temporaires, au lieu de valeurs contantes sans table.

Merci à tous