首页 > 代码库 > 事务和异常易出现的错误

事务和异常易出现的错误

1、Exit方法
原以为Exit方法执行后,会马上退出过程,但是真正做了一个例子来测试后,才让我改变了想法。请看下面这
个例子,flag最后被赋值为‘C‘。

============================================================
var
    flag: string;
begin
    try
      flag := ‘A‘;

      Exit;

      flag := ‘B‘;
    finally
      flag := ‘C‘;
    end;

    flag := ‘D‘;
end;

===========================================================
分析:不论try子句如何结束,finally 子句总是被执行。(多谢ylmg网友)

2、一个能让整个系统停止运作的小问题

在数据库系统设计中,经常使用事务操作来保证数据的完整性,但是设计不当,却容易产生比较大的影响,下面举个例子说明虽然数据完整性保证了,但是可能令到系统完全停止运作:
============================================================
AdoConnection1.BeginTrans;
try   
    ...

    if Application.MessageBox(‘是否确定删除?‘, ‘询问‘, MB_YESNO+MB_ICONQUESTION)<>IDYes then //(1)
    begin

      ...

    end;

    Application.MessageBox(‘操作失败‘, ‘警告‘, MB_OK); //(2)

    AdoConnection1.CommitTrans;
except
    Application.MessageBox(‘操作失败‘, ‘警告‘, MB_OK); //(3)

    AdoConnection1.RollbackTrans;
end;
============================================================

分析:上面代码中的问题都是由于(1)、(2)、(3)的Application.MessageBox引起,但是引起问题的并不是Application.MessageBox本身,而是它将程序挂起,需要用户干预后,才继续执行后面的操作;如果用户这时候离开了计算机,或者没有对这些对话框进行确定操作的话,可想而知,整个系统因为这个事务没有结束而通通处于等待状态了。

为了避免这个问题,原则有两个:
(1)、事务启动后,无需用户干预,程序可以自动结束事务;
(2)、在事务里面做时间最短的操作。


3、try...except...end结构

下面举个例子来说明try结构,还是使用事务操作的例子:

有问题的代码:
============================================================
try
    ...

    AdoConnection1.BeginTrans;

    ...

    AdoConnection1.CommitTrans;
except
    AdoConnection1.RollbackTrans;
end;
===========================================================

分析:如果try之后到AdoConnection1.BeginTrans这段代码中出现异常,将跳转到AdoConnection1.RollbackTrans执行,但是AdoConnection1因为出错并没有启动事务,所以AdoConnection1.RollbackTrans执行时出错了。

正确的代码: ============================================================
AdoConnection1.BeginTrans;
try
    ...

    ...

    AdoConnection1.CommitTrans;
except
    AdoConnection1.RollbackTrans;
end;
============================================================

总之,try的架构是用来保护异常的操作的,try...except之间的产生异常都会执行except...end之间的操作,设计try命令时一定要注意架构的合理性。

4、欺骗了自己的事务保护

在做数据库应用软件时,我们经常需要碰到下面的问题:对原有数据进行判断,然后做出相应的修改。这个问题看似比较简单,但是如果考虑到网络上还有别的人在使用同一个系统,那么,你就不的不考虑可能被意外改变的问题了。我的同事比较粗心,虽然在我的提示下考虑了多用户问题,但是他还是写下了有问题的代码:
============================================================
var
    adsTemp: TAdoDataSet;
    isOk: boolean;
begin
    adsTemp := TAdoDataSet.Create(self);
    try
      adsTemp.Connection := AdoConnection1;
      adsTemp.CommandText := ‘select fid, fnumber from tb1 where fid=120‘;
      adsTemp.Open;

      isOk := adsTemp.FieldByName(‘fnumber‘).AsInteger>100;
    finally
      adsTemp.Free;
    end;

    if not isOk then
      Exit;

    AdoConnection1.BeginTrans;
    try
      AdoConnection1.Execute(‘update tb1 set ffull=ffull + 1 from tb1 where fid=120‘;
      ...
      ...

      AdoConnection1.CommitTrans;
    except
      AdoConnection1.RollbackTrans;
    end;
end;
============================================================
分析:不知大家看出问题来了没有,在AdoConnection1.BeginTrans之前判断数据,然后使用AdoConnection1.Execute改变数据,如果这个数据是共享的,那么在判断之后到AdoConnection1.BeginTrans之前的这段时间里头,tb1的数据可能已经发生了改变,这个事务保护是没有用处的。

正确的方法是判断和修改必须是同一份数据,下面示例了有两个方法(区别在于启动事务的位置不相同):

代码1(使用事务保护以后,判断的和修改的是同一数据):
============================================================
var
    adsTemp: TAdoDataSet;
    isOk: boolean;
begin
    AdoConnection1.BeginTrans;
    try
      adsTemp := TAdoDataSet.Create(self);
      try
        adsTemp.Connection := AdoConnection1;
        adsTemp.CommandText := ‘select fid, fnumber, ffull from tb1 where fid=120‘;
        adsTemp.Open;

        if adsTemp.FieldByName(‘fnumber‘).AsInteger>100 then
        begin
          adsTemp.Edit;
          adsTemp.FieldByName(‘ffull‘).AsInteger := adsTemp.FieldByName(‘ffull‘).AsInteger + 1;
          adsTemp.Post;
        end;

      finally
        adsTemp.Free;
      end;

      AdoConnection1.CommitTrans;
    except
      AdoConnection1.RollbackTrans;
    end;
end;
============================================================

代码2(使用异常捕捉,假如判断和修改的不是同一份数据,adsTemp.Post时会出现异常,这个是ADODataSet对象具有的特性):
============================================================
var
    adsTemp: TAdoDataSet;
    isOk: boolean;
begin
    adsTemp := TAdoDataSet.Create(self);
    try
      adsTemp.Connection := AdoConnection1;
      adsTemp.CommandText := ‘select fid, fnumber, ffull from tb1 where fid=120‘;
      adsTemp.Open;

      if adsTemp.FieldByName(‘fnumber‘).AsInteger>100 then
      begin
        AdoConnection1.BeginTrans;
        try
          adsTemp.Edit;
          adsTemp.FieldByName(‘ffull‘).AsInteger := adsTemp.FieldByName(‘ffull‘).AsInteger + 1;
          adsTemp.Post;

          AdoConnection1.CommitTrans;
        except
          AdoConnection1.RollbackTrans;
        end;
      end;
    finally
      adsTemp.Free;
    end;
end;

事务和异常易出现的错误