开发更好的代码

Question

差异

  select CE1.CLASS_ID, 
         CE1.LOCATION_ID,
         count(case CE1.FORMAT_ID  when 5 then 1 end) as LIVE,
         count(case CE1.FORMAT_ID  when 14 then 1 end) as LB,
         count(case CE1.FORMAT_ID  when 15 then 1 end) as WEB,
         SUM(DECODE (CE1.ROLE_ID,5,1,0)) FACULTY,
         SUM(DECODE  (CE1.ROLE_ID ,2,1,0)) MODERATOR,
         SUM(DECODE  (CE1.ROLE_ID ,3,1,0)) PANELIST,  
         SUM(DECODE  (CE1.ROLE_ID,4,1,0)) PRESENTER,
         COUNT(CE1.USER_ID) TOT 
    from C_EDUCATION1 CE1 
GROUP BY CE1.LOCATION_ID, CE1.CLASS_ID

我们正在开发现有系统，我们发现混合使用 COUNT 和 DECODE。 如果我们理解正确的话，我们可以转换为以下代码。

  select CE1.CLASS_ID,  
         CE1.LOCATION_ID, 
         SUM(DECODE (CE1.FORMAT_ID ,5,1,0)) LIVE,
         SUM(DECODE  (CE1.FORMAT_ID ,14,1,0)) LB, 
         SUM(DECODE  (CE1.FORMAT_ID,15,1,0)) WEB,
         SUM(DECODE (CE1.ROLE_ID,5,1,0)) FACULTY,
         SUM(DECODE  (CE1.ROLE_ID ,2,1,0)) MODERATOR,
         SUM(DECODE  (CE1.ROLE_ID ,3,1,0)) PANELIST,  
         SUM(DECODE  (CE1.ROLE_ID,4,1,0)) PRESENTER,
         COUNT(CE1.USER_ID) TOT 
    from C_EDUCATION1 CE1 
GROUP BY CE1.LOCATION_ID, CE1.CLASS_ID

或者 所有 COUNT

我们想知道性能差异或其他问题。 有更好的方式存在吗？

Answer 1

两者之间不会有性能差异。

就我个人而言，我更倾向于基于 CASE 的解决方案。 CASE 是 ANSI 标准而不是 Oracle 特定的函数。来自其他数据库平台的开发人员可能必须查找 DECODE 语句的语义。无论开发人员熟悉哪种语言，CASE 对于开发人员来说也是相对明显的。几乎每种语言都有 CASE 语句，因此任何开发人员都应该能够快速了解​​您在做什么。

话虽如此，我倾向于提倡这样的方法：

SELECT ce1.class_id,
       ce1.location_id,
       SUM( CASE WHEN format_pkg.is_live( format_id ) = 'Y' THEN 1 ELSE 0 END ) live,
       SUM( CASE WHEN format_pkg.is_lb(   format_id ) = 'Y' THEN 1 ELSE 0 END ) lb,
       ...

This 使用第一个查询中的 CASE，第二个查询中的 SUM，并添加一些函数调用来确定正在讨论的格式/角色类型。这可以防止您必须在各处对 FORMAT_ID/ROLE_ID 值进行硬编码。使用 SUM 而不是 COUNT 更清晰一些，因为对于开发人员来说，COUNT 排除 NULL 值可能不会立即显而易见。