我看手术麻醉系统

lfree · 发表于 2011-5-12 10:00

9.滥用外部连接（+）：
这个是我看到使用外部连接最多的程序，我不想说明什么，也许是编程的需要，我不好评论！
我举一个例子：

toad对于外部连接的sql语句在写成（ + ）的情况下（有空格的情况下），格式化有BUG，以上看不全sql语句。我格式化如下：
SELECT COUNT (*)
  FROM med_operation_master a, med_anesthesia_summary b
WHERE a.patient_id = b.patient_id(+)
AND a.visit_id = b.visit_id(+)
AND a.oper_id = b.oper_id(+)
AND b.patient_id IS NULL
AND oper_status > 2
AND a.anesthesia_doctor = :1
AND start_date_time > SYSDATE - 31

我第一次看的时候，感觉很奇怪，med_anesthesia_summary的主键中有patient_id,为什么要查询b.patient_id IS NULL
这样的条件出现，再细看才知道使用了外部连接。另外我前面一直没说一个问题，像oper_status，start_date_time的字段，前面没有a.,
b.的前缀，对于我开始不熟悉数据库，根本不知道这个字段是来自那个表,这个有一个适应的过程，不知道以后接手的程序员估计也会花一定的时间。

这个语句更好的写法应该这样。
SELECT COUNT (*)
  FROM med_operation_master a
WHERE a.oper_status > 2
AND a.anesthesia_doctor = :1
AND a.start_date_time > SYSDATE - 31
AND NOT EXISTS (
      SELECT NULL
         FROM med_anesthesia_summary b
         WHERE b.patient_id = a.patient_id
         AND b.visit_id = a.visit_id
AND b.oper_id = a.oper_id);
另外我查询v$sql_bind_capture视图，发现:1参数都是汉字。

而我查询表med_operation_master 字段anesthesia_doctor里面全是代码，仅仅有一个是汉字，我仅仅估计程序有BUG。

lfree · 发表于 2011-5-12 10:01

10再来看看awr报表，实际上这个系统对于dba讲，是一个小系统，看awr报表根本看不到问题，开发商应该感谢现在的硬件设备，掩盖了许多程序设计的缺陷。

2个小时snap，db time仅仅0.77分钟，我的估计即使存在上面的问题不去管，使用5－6年也不会有问题。

lfree · 发表于 2011-5-12 10:04

总结：
我优化N多项目，我真的没看到几个做医院开发做的非常好，我看到更多的是垃圾！当然从电话反馈看，系统运行应用中的问题不是很多，运行还是很稳定的，我自己更多的关注是数据库的优化。

这样的系统不优化，表的增长速度不是很快，运行5－6年，不看底层，根本看不到问题。

像MED_OPERATION_SCHEDULE表，目前不到1M，运行5－6年最多10M。
天天全表扫描，也不会很慢。

lfree · 发表于 2011-5-12 10:28

补充：

7.语句很长,我格式化：
中的条件：

  AND (    scheduled_date_time >= TRUNC (SYSDATE, 'dd') - -1
                     AND scheduled_date_time < TRUNC (SYSDATE, 'dd') - -1
                                                + 1
                  OR scheduled_date_time IS NULL
                  )

为什么要放在内层？这个字段是表MED_OPERATION_SCHEDULE中的字段。