Przekazywanie parametru wartości Null do dynamicznego zapytania SQL. Jak można to z powodzeniem wykonać?

Question

Przeprowadzam dynamiczne zapytanie SQL. gdzie parametry są przekazywane dynamicznie. Najpierw napisałem podobnie do następującego kodu.

DECLARE @Id nvarchar(max); 
DECLARE @query nvarchar(max); 
SET @Id ='RMH_108' 
SET @query ='Select * 
      FROM [A06].[Syn_RMDemand] 
      WHERE RMHierarchyId =' + @Id 
PRINT @query 
EXEC(@query) 

Wtedy to pisałem.

DECLARE @Id nvarchar(max); 
DECLARE @query nvarchar(max); 
SET @Id ='RMH_108' 
SET @query ='Select * 
      FROM [A06].[Syn_RMDemand] 
      WHERE RMHierarchyId = '''+@Id+'''' 
PRINT @query 
EXEC(@query) 

Kod ten czas pomyślnie wykonana. Ponieważ parametr może być pusty, muszę przekonwertować go na wartość null. I zmodyfikowany kod i napisał ten

DECLARE @Id nvarchar(max); 
DECLARE @query nvarchar(max); 
SET @Id = '' 
SET @Id = NULLIF(@Id,'') 
-- COMMENTED SET @Id ='RMH_108' 

SET @query ='Select * FROM [A06].[Syn_RMDemand] 
      WHERE RMHierarchyId = '''+@Id+'''' 
PRINT @query 
EXEC(@query) 

bez błędów. Zapytanie się zawiesiło.
Czy ktoś ma taki problem?

Answer 1

Nie ma potrzeby, aby przekształcić go NULL po prostu skontaktować się z swoją wartość

DECLARE @Id nvarchar(max); 
DECLARE @query nvarchar(max); 
SET @Id = '' 


          -- COMMENTED SET @Id ='RMH_108' 
SET @query ='Select * FROM [A06].[Syn_RMDemand] 
      WHERE RMHierarchyId = '''+@Id+'''' 
PRINT @query 
EXEC(@query) 

Answer 2

Spróbuj wykonać następujące czynności:

SET @query ='Select * FROM [A06].[Syn_RMDemand]' 
IF @Id IS NOT NULL 
    SET @query = @query + ' WHERE RMHierarchyId = '''+@Id+'''' 

Jeśli parametr pochodzi od klienta lepiej zrobić z zapytaniem parametryzowaną (patrz @ ughai za odpowiedź) w celu wykluczenia możliwości sql injection ...

Answer 3

Spróbuj tego.

SET @query ='Select * FROM [A06].[Syn_RMDemand] 
      WHERE RMHierarchyId = ' + ISNULL(@Id , '') 

Answer 4

Powinieneś użyć sp_executeSQL i usunąć konkatenację ciągów w ten sposób. Zakładam, że jeśli przekazano NULL, wszystkie wiersze mają zostać zwrócone.

Zapytanie

DECLARE @Id nvarchar(max); 
DECLARE @query nvarchar(max); 
SET @Id ='RMH_108'SET @query ='Select * 
           FROM [A06].[Syn_RMDemand] 
           WHERE RMHierarchyId = @Id OR @ID IS NULL' 
PRINT @query 
EXEC sp_executeSQL @query,N'@Id NVARCHAR(MAX)',@Id 

Answer 5

DECLARE @Id nvarchar(max); 
DECLARE @query nvarchar(max); 

SET @query = N'SELECT * FROM [A06].[Syn_RMDemand] ' + CHAR(10) 

IF @Id IS NOT NULL BEGIN 
    SET @query = @query + N'WHERE RMHierarchyId = @param_id' 
END 
ELSE BEGIN 
    SET @query = @query + N'WHERE RMHierarchyId IS NULL' 
END 

PRINT @query 

EXEC sp_executesql 
     @statement = @query, 
     @params = N'@param_id VARCHAR(MAX)', 
     @param_id = @Id 

Należy stosować kwerendy parametryczne podczas korzystania z dynamicznego SQL, aby uniknąć SQL Injection. Przeczytaj this, aby uzyskać więcej informacji.

Jeśli chcesz zwrócić wszystkie wiersze, jeśli przekazana @Id jest NULL, po prostu usuń część ELSE.

Answer 6

Można również napisać jak:

SET @query ='Select * FROM [A06].[Syn_RMDemand] 
      WHERE 1 = case when '''+@Id+''' = '''' then 1 when RMHierarchyId =  
      '''+@Id+''' then 1 else 0 end ' 

Answer 7

Jestem ciekaw, dlaczego nikt nie sugeruje, nie ma potrzeby korzystania z dynamicznego SQL tutaj

Select * 
FROM [A06].[Syn_RMDemand] 
WHERE NULLIF(@Id, '') IS NULL OR RMHierarchyId = @Id 

Answer 8

spróbować

SET @query = 'Select * FROM [A06].[Syn_RMDemand] WHERE RMHierarchyId=''' + isnull(Convert(VARCHAR, @id), '') + ''' ' 

    EXEC (@query) 

Answer 9

Wszystko z arytmetyczną wartości NULL da NULL.

zrobiłbym ten sposób jeśli NULL jest przeznaczona do zwrotu wszystkich wierszy:

SET @query ='Select * FROM [A06].[Syn_RMDemand]' + 
      case when @Id is not null then 
      ' WHERE RMHierarchyId = ''' + @Id + '''' else '' end 

iw ten sposób, jeśli naprawdę chcesz mieć wiersze z warunku NULL:

SET @query ='Select * FROM [A06].[Syn_RMDemand] 
      WHERE RMHierarchyId ' + case when @Id is null 
      then 'is NULL' else '= ''' + @Id +'''' end 

ale potem znowu , zawsze lepiej unikać zapytania dynamicznego za wszelką cenę. Rozważmy:

Select * FROM [A06].[Syn_RMDemand] WHERE 
      (RMHierarchyId = @Id or nullif(@Id, '') is null) 

oraz:

Select * FROM [A06].[Syn_RMDemand] WHERE (RMHierarchyId = @Id or 
      (RMHierarchyId is null and @Id is null))