Passa il parametro del valore Null alla query SQL dinamica. Come può essere eseguito con successo?

Question

Sto eseguendo una query SQL dinamica. dove i parametri vengono passati dinamicamente. Ho scritto per la prima volta in modo simile al seguente codice.

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) 

Poi ho scritto questo.

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) 

Questo codice volta eseguito con successo. Poiché il parametro può essere vuoto, è necessario convertirlo in null. Ho modificato il codice e scritto questo

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) 

Nessun errore. La query si è interrotta.
Qualcuno ha questo tipo di problema?

Answer 1

Non c'è bisogno di convertirlo in NULL basta controllare con il vostro valore

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

provare quanto segue:

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

Se il parametro viene da cliente è meglio fare con la domanda parametrizzata (vedi @ risposta di ughai) al fine di escludere la possibilità sql injection ...

Answer 3

Prova questa.

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

Answer 4

È necessario utilizzare sp_executeSQL e rimuovere la concatenazione di stringhe come questa. Suppongo che se si passa a NULL, si desidera che vengano restituite tutte le righe.

Query

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 

Si dovrebbe utilizzare query con parametri quando si utilizza SQL dinamico per evitare SQL Injection. Leggi this per ulteriori informazioni.

Se si desidera restituire tutte le righe se il numero @Id superato è NULL, è sufficiente rimuovere la parte ELSE.

Answer 6

Si può anche scrivere come:

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

Answer 7

Sono curioso perché nessuno suggerendo non v'è alcuna necessità di utilizzare SQL dinamico qui

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

Answer 8

provare questo

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

    EXEC (@query) 

Answer 9

Qualsiasi cosa con l'aritmetica di NULL restituirà NULL.

farei questo modo se NULL è pensato per la restituzione di tutte le righe:

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

e in questo modo, se davvero vuole avere righe con condizione NULL:

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

Ma poi di nuovo , sarebbe sempre meglio evitare query dinamiche a tutti i costi. Considerate:

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

E:

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