问一个sql的优化

fals

原帖由 lfree 于 2008-6-6 10:06 发表
SELECT DISTINCT "OPERATION_BILL_ITEMS"."PATIENT_ID",
                "OPERATION_BILL_ITEMS"."OPER_ID", "PAT_MASTER_INDEX"."NAME",
                "PAT_MASTER_INDEX"."SEX", "PAT_MASTER_INDEX"."CHARGE_TYPE"
           FROM "PAT_MASTER_INDEX",
                "OPERATION_BILL_ITEMS",
                "SURGERY"."OPERATION_MASTER"
          WHERE ("SURGERY"."OPERATION_MASTER"."PATIENT_ID" =
                                           "OPERATION_BILL_ITEMS"."PATIENT_ID"
                )
            AND ("SURGERY"."OPERATION_MASTER"."VISIT_ID" =
                                             "OPERATION_BILL_ITEMS"."VISIT_ID"
                )
            AND ("SURGERY"."OPERATION_MASTER"."OPER_ID" =
                                              "OPERATION_BILL_ITEMS"."OPER_ID"
                )
            AND ("OPERATION_BILL_ITEMS"."PATIENT_ID" =
                                               "PAT_MASTER_INDEX"."PATIENT_ID"
                )
            AND (    (TO_CHAR ("OPERATION_MASTER"."START_DATE_TIME",
                               'YYYY-MM-DD'
                              ) = ：operating_date
                     )
                 AND ("OPERATION_BILL_ITEMS"."PERFORMED_BY" = ：performed_by)
                )

再贴一个,现在看军惠的sql写的真烂!!!!

这个SQL看不太明白程序员的意图，是不是想查一下在某天手术室有哪些操作员收了哪些病人的费用？为什么偏要指定执行科室？难道是其他科室要查看手术室替自己记了些什么费用么？如果是这样的话，为什么不显示明细，而只显示病人及其身份？

如果不需要限定执行科室，可以优化为如下语句：

SELECT b.PATIENT_ID,
                c.OPER_ID, b.NAME,
                b.SEX, b.CHARGE_TYPE
           FROM PAT_MASTER_INDEX b,
                SURGERY.OPERATION_MASTER c
          WHERE c.PATIENT_ID = b.PATIENT_ID
            AND ((TO_CHAR (c.START_DATE_TIME,'YYYY-MM-DD') = ：operating_date);

lfree

写错许多，改正一下。
SELECT a.retail_price, a.purchase_price
  FROM drug_export_detail a,
       drug_export_master b,
       (SELECT MAX (export_date) max_export_date
          FROM drug_export_detail m, drug_export_master x
         WHERE m.document_no = x.document_no
           AND x.STORAGE = :xx1
           AND m.drug_code = :xx2
           AND m.package_spec = :xx3
           AND m.firm_id = :xx4) c
WHERE a.document_no = b.document_no
   AND b.STORAGE = :xx1
   AND a.drug_code = :xx2
   AND a.package_spec = :xx3
   AND a.firm_id = :xx4
   AND b.export_date = c.max_export_date
   AND ROWNUM = 1

lfree

这样确实快一些，主要是可以利用export_date索引。

lfree

原帖由 fals 于 2008-6-19 17:14 发表


这个SQL看不太明白程序员的意图，是不是想查一下在某天手术室有哪些操作员收了哪些病人的费用？为什么偏要指定执行科室？难道是其他科室要查看手术室替自己记了些什么费用么？如果是这样的话，为什么不显示明细，而只显示病人及其身份？

如果不需要限定执行科室，可以优化为如下语句：

SELECT b.PATIENT_ID,
                c.OPER_ID, b.NAME,
                b.SEX, b.CHARGE_TYPE
           FROM PAT_MASTER_INDEX b,
                SURGERY.OPERATION_MASTER c
          WHERE c.PATIENT_ID = b.PATIENT_ID
            AND ((TO_CHAR (c.START_DATE_TIME,'YYYY-MM-DD') = ：operating_date);

哈哈，我也不知道为什么？

fals

原帖由 lfree 于 2008-6-19 17:21 发表
这样确实快一些，主要是可以利用export_date索引。

改得挺对，我粗心了，没有仔细去看这些表的结构。

一方面可以使用export_date索引，另一方面去掉了不必要的group by ，这个可能引起硬盘排序

fals

原帖由 lfree 于 2008-6-19 17:24 发表


哈哈，我也不知道为什么？

这个SQL是从哪个窗口里跟踪出来的？抓个图出来看看吧，或者可以修改业务。

那个distinct挺讨厌的，多数情况下都会引起硬盘排序，如果能够去掉的话，会省很多

lfree

fals 还是蛮厉害，一下就发现这个distinct。
就是这个distinct，程序员的素质太差了。

实际上不在于去掉，不知道程序员为什么这样写。

fals

是从性能视图里跟出来的么？如果是这样的话，很有必要把产生这个SQL的前台程序抓出来，多花点时间抓一下吧，估计可以去掉这个业务需求。

一条坏的SQL，可以搞死整个数据库。既然已经发现了，就不要放过去。

或者程序员自己都没明白为什么要写这个SQL呢，下面的人也就稀里糊涂地用。

husthxd

呵呵，没有任何背景知识，看这么长的sql还是有点吃力的。
补充一下，纯技术来看，可以试试用分析函数row_number代替rownum=1

wollaston

没有用过军惠,用Sqlserver的飘过