SQL 代码简化或者优化

[shuohuameijiang]

select '零售自营收入',SUM(HasTaxMoney) totalHasTaxMoney from Mall_FinaceData 
where VersioId in(
  select ID from Mall_FinaceDictVersion 
  where DictIdSnap like '%'+ cast((select ID from Mall_FinaceDict where DictName='自营') as nvarchar(100)) +'%'
  and DictIdSnap like '%'+ cast((select ID from Mall_FinaceDict where DictName='零售') as nvarchar(100)) +'%'
  ) 
union all
select '零售联营收入',SUM(HasTaxMoney) totalHasTaxMoney from Mall_FinaceData 
where VersioId in(
  select ID from Mall_FinaceDictVersion 
  where DictIdSnap like '%'+ cast((select ID from Mall_FinaceDict where DictName='联营') as nvarchar(100)) +'%'
  and DictIdSnap like '%'+ cast((select ID from Mall_FinaceDict where DictName='零售') as nvarchar(100)) +'%'
 )

表中的数据如下图，请教如何简化和优化SQL代码？

[中国风]

你的表结构决定了语句不会高效，最少你要满足3范式设计
，数据量不大时可尝试用以下方法，

SELECT CASE WHEN MFDV2.DictName=N'自营' THEN N'零售自营收入'
            ELSE N'零售联营收入'
       END AS Type
     , SUM(HasTaxMoney) AS totalHasTaxMoney
FROM Mall_FinaceData AS MFD
     CROSS APPLY
		(SELECT TOP 1
			 DictName
		 FROM Mall_FinaceDictVersion AS MFDV
		 WHERE MFD.VersioId=MFDV.ID
			   AND EXISTS (SELECT 1
						   FROM Mall_FinaceDict
						   WHERE DictName IN ( N'自营', N'联营' )
								 AND MFDV.DictIdSnap LIKE '%'+RTRIM(ID) +'%')
			   AND EXISTS (SELECT 1
						   FROM Mall_FinaceDict
						   WHERE DictName=N'零售'
								 AND MFDV.DictIdSnap LIKE '%'+RTRIM(ID) +'%')) AS MFDV2
GROUP BY MFDV2.DictName;

[shuohuameijiang]

你写的我不是很理解。DictName 不属于 Mall_FinaceDictVersion表。上面我只截取了一部分脚本，完整的SQL见图，多了发不上来。

引用 1 楼 roy_88 的回复:

你的表结构决定了语句不会高效，最少你要满足3范式设计
，数据量不大时可尝试用以下方法，

SELECT CASE WHEN MFDV2.DictName=N'自营' THEN N'零售自营收入'
            ELSE N'零售联营收入'
       END AS Type
     , SUM(HasTaxMoney) AS totalHasTaxMoney
FROM Mall_FinaceData AS MFD
     CROSS APPLY
		(SELECT TOP 1
			 DictName
		 FROM Mall_FinaceDictVersion AS MFDV
		 WHERE MFD.VersioId=MFDV.ID
			   AND EXISTS (SELECT 1
						   FROM Mall_FinaceDict
						   WHERE DictName IN ( N'自营', N'联营' )
								 AND MFDV.DictIdSnap LIKE '%'+RTRIM(ID) +'%')
			   AND EXISTS (SELECT 1
						   FROM Mall_FinaceDict
						   WHERE DictName=N'零售'
								 AND MFDV.DictIdSnap LIKE '%'+RTRIM(ID) +'%')) AS MFDV2
GROUP BY MFDV2.DictName;

[文盲老顾]

select * from (
	select sum(HasTaxMoney) as _sum,VersionID 
	from Mall_FinaceData
	where PeriodDate between '2017-1-1' and '2017-12-31'
	and isDelete=0
	group by VersionID
) a
cross apply (
	select (
		select y.DictName from Mall_FinaceDictVersion x
		inner join Mall_FinaceDict y on charindex(convert(nvarchar(100),y.ID),x.DictIdSnap)>0
		for xml path('')
	) as tp
) b

[吉普赛的歌]

建表的时候就应该有一个专门的分类表， 清楚明确地对应， 而不是用 LIKE。
用了模糊查询， 要优化就很麻烦的了， 效率受影响是绝对的。

可以分步骤来做， 部分数据查出来放临时表， 后面再连接操作

[文盲老顾]

引用 4 楼 yenange 的回复:

建表的时候就应该有一个专门的分类表， 清楚明确地对应， 而不是用 LIKE。
用了模糊查询， 要优化就很麻烦的了， 效率受影响是绝对的。

可以分步骤来做， 部分数据查出来放临时表， 后面再连接操作

历史遗留问题是没办法弄的，真要改数据库结构，等于整站重做一遍，我现在的数据库结构调整完了两年，网站代码还没全部重写完成。。。。只能用触发器同步数据库数据

[shuohuameijiang]

你的执行后结果如图：

但我想要的效果是这样的：

请教你，应该怎样改呢？

引用 3 楼 superwfei 的回复:

select * from (
	select sum(HasTaxMoney) as _sum,VersionID 
	from Mall_FinaceData
	where PeriodDate between '2017-1-1' and '2017-12-31'
	and isDelete=0
	group by VersionID
) a
cross apply (
	select (
		select y.DictName from Mall_FinaceDictVersion x
		inner join Mall_FinaceDict y on charindex(convert(nvarchar(100),y.ID),x.DictIdSnap)>0
		for xml path('')
	) as tp
) b

[文盲老顾]

那你最好提供几个数据的示例，然后才能想办法

Mall_FinaceDictVersion的DiciIdSnap的值是什么样的，Mall_FinaceDict表的ID和DictName值是什么样的

[文盲老顾]

select * from (
    select sum(HasTaxMoney) as _sum,VersionID 
    from Mall_FinaceData
    where PeriodDate between '2017-1-1' and '2017-12-31'
    and isDelete=0
    group by VersionID
) a
cross apply (
    select (
        select distinct ''+y.DictName from Mall_FinaceDictVersion x
        inner join Mall_FinaceDict y on charindex(convert(nvarchar(100),y.ID),x.DictIdSnap)>0
		where y.DictName in ('自营','零售','游乐','收入','联营','水吧')
		order by charindex(y.DictName,'游乐零售水吧自营联营收入')
        for xml path('')
    ) as tp
) b