它是优化 SQL Server 中存储过程的有效方法吗

Question

我有一个名为Optimized_GetArticlePost 的存储过程，我想对其进行优化。

当前存储过程：

CREATE PROCEDURE [dbo].[Optimized_GetArticlePost]            
    (@PostID int)
    --[Optimized_GetArticlePost] 
AS                                  
BEGIN 
    SET NOCOUNT ON;                                  

    //declare all the required variables.. 

    -- LIST ALL ATTACHMENTS                                           
    SELECT [FileName] 
    FROM CS_PostAttachments                  
    WHERE contenttype LIKE 'audio/mpeg' 
      AND FILENAME LIKE '%.mp3' 
      AND isremote = 1 
      AND postid = @Postid ------returns a seperate table

    --GET THE FB FEED IMAGES FOR THIS  POST
    SELECT ImageURL 
    FROM PostFBImageURL 
    WHERE PostID = @PostID------returns a seperate table

    --GET STORY AT GLANCE DATA                                  
    SELECT 
        StoryImage, StoryContent, ALT 
    FROM 
        PostRelatedData 
    WHERE
        PostID = @PostID------returns a seperate table

    --GET POST REFERENCES                                  
    SELECT ReferenceData 
    FROM PostReferences  
    WHERE PostID = @PostID------returns a seperate table
END 

从上面的存储过程中，每个select 语句都会返回一个表.. 我不想也想消除它

优化的存储过程：

CREATE PROCEDURE [dbo].[Optimized_GetArticlePost]            
    (@PostID int)
--[Optimized_GetArticlePost] 715049
AS                                  
BEGIN 
    ------------------------------------------------------------
    ----Declare variable table...
    DECLARE @Temp TABLE (FileName nvarchar(MAX),
                         ImageURL VARCHAR(MAX),
                         StoryImage nvarchar(MAX),
                         StoryContent nvarchar(MAX),
                         ALT nvarchar(MAX),
                         ReferenceData nvarchar(MAX) )  

    SET NOCOUNT ON;                                  

    -- LIST ALL ATTACHMENTS
    INSERT INTO @Temp (FileName)
        SELECT [FileName] 
        FROM CS_PostAttachments                  
        WHERE contenttype LIKE 'audio/mpeg' 
          AND FILENAME LIKE '%.mp3' 
          AND isremote = 1 
          AND postid = @Postid

    --GET THE FB FEED IMAGES FOR THIS POST   
    INSERT INTO @Temp (ImageURL) 
        SELECT ImageURL 
        FROM PostFBImageURL 
        WHERE PostID = @PostID

    --GET STORY AT GLANCE DATA    
    INSERT INTO @Temp (StoryImage,StoryContent,ALT) 
        SELECT StoryImage, StoryContent, ALT 
        FROM PostRelatedData 
        WHERE PostID = @PostID

    --GET POST REFERENCES  
    INSERT INTO @Temp (ReferenceData) 
        SELECT ReferenceData 
        FROM PostReferences  
        WHERE PostID = @PostID

    SELECT * FROM @postDate                             
END 

在上述优化的存储过程中，我将数据插入到一个表变量中，并在最后一次从该临时表中进行选择..

所以，我的问题是：这是否是优化存储过程的正确方法..或者想要在这种情况下如何优化的建议。

提前致谢

Answer 1

首先：为什么你的临时表有 ALL (n)varchar(max)??不要只是在任何地方都使用(N)VARCHAR(MAX)，因为您懒得去想(n)varchar 列的长度应该是多长、合理！了解您的数据 - 使用最合适的数据类型！如果您真的必须有超过 8000 个字符 - 那应该是例外，而不是规则！

另外：要么使用varchar 要么使用nvarchar - 我会尽量避免混合这两者 - 你需要 Unicode - 然后使用nvarchar - 或者你不需要它 - 然后使用varchar。

我认为这太可怕了 - 现在每个 @PostID 最多可以有 4 行 - 这真的是你想要的吗？

FileName    ImageUrl    StoryImage    StoryContent   ALT   ReferenceData
------------------------------------------------------------------------
TheFile     NULL        NULL          NULL           NULL  NULL
NULL        SomeUrl     NULL          NULL           NULL  NULL
NULL        NULL        SomeImage     Content        ALT   NULL
NULL        NULL        NULL          NULL           NULL  ReferenceData

似乎是个非常糟糕的主意。

您最有可能想要的是跨这些表的 JOIN - 这样您就可以为每个 @PostID 获得 ONE ROW： p>

CREATE PROCEDURE [dbo].[GetArticlePostWithJoins]
    (@PostID int)
AS                                  
BEGIN 
    SELECT 
        pa.[FileName],
        fb.ImageURL,
        rd.StoryImage, rd.StoryContent, rd.ALT,
        pr.ReferenceData 
    FROM 
        CS_PostAttachments pa
    LEFT OUTER JOIN
        PostFBImageURL fb ON fb.PostID = pa.PostID
    LEFT OUTER JOIN
        PostRelatedData rd ON rd.PostID = pa.PostID
    LEFT OUTER JOIN
        PostReferences pr ON pr.PostID = pa.PostID
    WHERE 
        pa.contenttype LIKE 'audio/mpeg' 
        AND pa.FILENAME LIKE '%.mp3' 
        AND pa.isremote = 1 
        AND pa.postid = @Postid ------returns a seperate table
END