Discussion :

[BenoitM]

Bonjour,
Nous avons un petit soucis quand nous créons des SP, si le nom de la table n'est pas valide il accepte de créer la SP.

Il y a moyen d'éviter ça?

[elsuket]

Bonjour,

On peut par défaut créer une procédure stockée en référençant une table qui n'existe pas, étant donné que la compilation est déférée au moment de l'exécution.
Quel est exactement le message d'erreur que vous obtenez ?

@++

[BenoitM]

Aucun message la SP se crée correctement hélas

Mais bon le jour où j’exécute réellement la celle-ci me dira que la table X n’existe pas.

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
 
CREATE PROCEDURE Test
(
    @NO_ADR INT
)
AS
BEGIN
 
    SET NOCOUNT ON;
 
    SELECT *
    FROM	Adrese
    WHERE	NO_ADR = @NO_ADR;
 
END

Code :

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

1
2
3
 
EXEC	@return_value = [dbo].[Test] @NO_ADR = 1
Invalid object name 'Adrese'.

C'est un peu bête de se rendre compte seulement lors de l'utilisation du programme que le développeur à taper "Adrese" au lieu de "Adresse"
Il n'y aurait pas une solution simple pour éviter ce genre de problème?
Je me vois mal demander à chaque développeurs de tester l’exécute manuellement de chaque SP qu'ils créent ou modifient

[BenoitM]

J'ai peut-être trouvé une solution

Code :

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

1
2
3
4
5
 
select o.type, o.name, ed.referenced_entity_name, ed.is_caller_dependent
from sys.sql_expression_dependencies ed
join sys.objects o on ed.referencing_id = o.object_id
where ed.referenced_id is null

[elsuket]

Dans quel schéma se trouve la table Adresse ?

SELECT OBJECT_SCHEMA_NAME(OBJECT_ID('T_EMP'), DB_ID())

@++

[mikedavem]

Une solution un peu radicale mais qui doit marcher après avoir créer la procédure stockée:

Code :

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

EXEC sp_recompile <procedure>

++

[elsuket]

Ou alors sp_refreshsqlmodule ?

@++

[BenoitM]

Code :

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

EXEC sp_recompile <procedure>

indique simplement

Object 'test' was successfully marked for recompilation.

Code :

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

sys.sp_refreshsqlmodule 'dbo.Test'

Command(s) completed successfully.

Avec la requête par contre j'obtiens

type	Schema	Name	referenced_entity_name
P	dbo	Test	Adrese

Donc j'ai pu identifier que ma SP dbo.Test fait appel à une table non référencé Adrese

Merci de votre aide

[elsuket]

Votre procédure stockée référence la table Adrese, donc cela devrait fonctionner.
Que vous retourne la petite requête que je vous ai donné ?

Je me vois mal demander à chaque développeurs de tester l’exécute manuellement de chaque SP qu'ils créent ou modifient

Comment réalisez-vous les tests unitaires alors ?
Mon avis c'est que toute modification ou création de code doit être testée.

@++

[BenoitM]

Votre procédure stockée référence la table Adrese, donc cela devrait fonctionner.
Que vous retourne la petite requête que je vous ai donné ?


Comment réalisez-vous les tests unitaires alors ?
Mon avis c'est que toute modification ou création de code doit être testée.

@++

la table "Adrese" n'existe pas le but est de le détecter
j'ai testé ys.sp_refreshsqlmodule 'dbo.Test' qui m'a indiqué "Command(s) completed successfully."

Des testes unitaires?
Hélas c'est inconnu ici

[elsuket]

Si vous voulez tester l'existence d'une table :

Code :

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

1
2
3
4
IF OBJECTPROPERTYEX(OBJECT_ID('dbo.Adrese'), 'IsTable') = 1
BEGIN
	...
END

Cela fait que je ne comprends pas ce que vous souhaitez faire avec la procédure stockée.
Pourriez-vous clarifier ?

@++