Início / dba / Perguntas / 343226
Accepted
Jefferson
Asked: 2024-10-26 01:12:22 +0800 CST

Tabelas temporárias simplificam consultas

  • 772

Estou tentando acessar a tabela Customer a partir dos dados na tabela SiteConfiguration.

Estou pegando os dados e configurando-os para @AdditionalVoteViewers, que seria uma sequência de endereços de e-mail. Então, divido a sequência e insiro os resultados em uma tabela temporária.

Nesse ponto, unirei a tabela temporária com a tabela Cliente para retornar o CustomerId .

Só quero ver se há uma maneira melhor de fazer isso.

Dados na tabela SiteConfiguration

[email protected];[email protected]

Dados na tabela Cliente

[email protected] 1
[email protected] 2

Código:

declare @AdditionalVoteViewers VARCHAR(100) 
Select @AdditionalVoteViewers = 
(
    SELECT [Value] FROM [SiteConfiguration] 
    where Name = 'AdditionalVoteViewers'
)
print @AdditionalVoteViewers
--CREATE TABLE #TempAdditionalVoteViewers(email  varchar(MAX) )

select * into #TempAdditionalVoteViewers 
from [dbo].[Split] (@AdditionalVoteViewers,';')
--select * from #TempAdditionalVoteViewers

select [CustomerId] into #TempAdditionalVoteViewersId
from [dbo].[Customer] c 
join #TempAdditionalVoteViewers t on c.Email = t.val


select * from #TempAdditionalVoteViewersId
sql-server

2 respostas

  1. Best Answer

    está bem

    Você está fazendo as coisas corretamente, no sentido de que não está dividindo a string na sua consulta e unindo aos resultados. Isso geralmente é uma receita para desastre de desempenho .

    Se eu pudesse fazer uma sugestão, seria garantir que a instrução de criação da tabela temporária ou a seleção em converta a coluna de e-mail em um tipo de dados e comprimento que corresponda à coluna de e-mail na tabela Cliente.

    No seu código de exemplo, você está fazendo isso:

    --CREATE TABLE #TempAdditionalVoteViewers(email  varchar(MAX) )

select * into #TempAdditionalVoteViewers from [dbo].[Split] (@AdditionalVoteViewers,';')

    Usar o máximo de tipos de dados para colunas ou parâmetros/variáveis ​​fará com que você não consiga buscar em nenhum índice útil que possa ter. Não tenho ideia do que o código da sua Splitfunção faz, mas a maioria dos que vi copiados e colados de blogs do SQL Server com mais de 20 anos são genéricos e geralmente retornam tipos de saída ridículos.

    • 3 
  2. -- as a general rule, I never do an INSERT INTO
-- I want the table to exist before the insert with whatever
-- DDL provisions that I see necessary for performance and integrity

CREATE TABLE #TempAdditionalVoteViewers (
    [email] varchar(max) PRIMARY KEY 
    -- I never have a temp table without a primary key unless there is
    -- a good reason (which almost never happens).
)

-- I never ever ever use implicit column order in insert statements
-- even if there is only one column like here
INSERT #TempAdditionalVoteViewers ([email])
SELECT DISTINCT splt.[email]  
FROM [dbo].[Split] (@AdditionalVoteViewers,';') as splt
WHERE LEN(splt.[email]) > 0


-- You could see the entire [SiteConfiguration] table shredded in one 
-- command, this way

SELECT sc.[Name], x.[email]
FROM [SiteConfiguration] as sc
CROSS APPLY (
    SELECT DISTINCT splt.[email] 
    from [dbo].[Split] (sc.[Value],';') as splt
    WHERE LEN(splt.[email]) > 0
) as x


-- This might be the answer that you are hoping for. No temp tables

SELECT sc.[Name], x.[email]
FROM [SiteConfiguration] as sc
CROSS APPLY (
    SELECT DISTINCT splt.[email] 
    FROM [dbo].[Split] (sc.[Value],';') as splt
    WHERE LEN(splt.[email]) > 0
) as x
WHERE sc.[Name] = 'AdditionalVoteViewers'
    • -2

relate perguntas