Início / dba / Perguntas / 269332
Accepted
Jefferson
Asked: 2020-06-18 05:22:03 +0800 CST

instrução de caso de procedimento armazenado

  • 772

Estou construindo essa condição where concatenando a variável @whereClause no meu procedimento armazenado. Neste ponto, estou usando o caso para adicionar valores adicionais. se existe uma maneira melhor de fazer isso?

SET @whereClause  = 
    ( CASE  
        -- APPType Clinical
         WHEN (@ApplicationType = 'Clinical') THEN @whereClause + ' AND vpgl.AppType = Clinical'
         -- APPType Patients
         WHEN (@ApplicationType = 'Patients') THEN @whereClause + ' AND vpgl.AppType = Patients'
          -- APPType Web
         WHEN (@ApplicationType = 'Web') THEN @whereClause + ' AND vpgl.AppType = null'
         ELSE @whereClause
       END  
    )
sql-server sql-server-2016

2 respostas

  1. Best Answer

    Acho que esta é uma abordagem melhor, mais segura, mais autodocumentada e mais escalável:

    IF @ApplicationType IN ('Clinical','Patients')
BEGIN
  SET @whereClause += N' AND vpgl.AppType = @ApplicationType';
END

IF @ApplicationType = 'Web'
BEGIN
  SET @whereClause += N' AND vpgl.AppType IS NULL';
END

...
SET @sql = @sql + @whereClause;

EXEC sys.sp_executesql @sql,
     N'@ApplicationType varchar(32)',
     @ApplicationType;

    Se você realmente quiser usar uma CASEexpressão (que não se destina ao controle de fluxo ), você pode dizer:

    SET @whereClause += CASE 

   WHEN @ApplicationType IN ('Clinical','Patients')
   THEN N' AND vpgl.AppType = @ApplicationType'

   WHEN @ApplicationType = 'Web'
   THEN N' AND vpgl.AppType IS NULL';

   ELSE N'' END;

...
SET @sql = @sql + @whereClause;

EXEC sys.sp_executesql @sql,
     N'@ApplicationType varchar(32)',
     @ApplicationType;
    • 2

  2. Você pode torná-lo mais curto:

    SET @whereClause += (CASE @ApplicationType
                          -- APPType Clinical
                          WHEN 'Clinical' THEN ' AND vpgl.AppType = Clinical'
                          -- APPType Patients
                          WHEN 'Patients' THEN ' AND vpgl.AppType = Patients'
                          -- APPType Web
                          WHEN 'Web' THEN ' AND vpgl.AppType = null'
                          ELSE ''
                    END);
    • 0

relate perguntas