改良控件-Delphi自带控件Bug的消除

页面导航:首页 > 软件编程 > Delphi > 改良控件-Delphi自带控件Bug的消除

改良控件-Delphi自带控件Bug的消除

来源: 作者: 时间:2016-01-23 10:15 【

不管你想不相信Delphi自带的组件里竟然存在着一个不容忽视的Bug。开始我一点都没有想到是Delphi自带的控件有Bug。害的我调试了很多遍,后来经过跟踪才发现的。看到Samples页上的TSpinE

不管你想不相信Delphi自带的里竟然存在着一个不容忽视的Bug。
开始我一点都没有想到是Delphi自带的控件有Bug。害的我调试了很多遍,后来经过跟踪才发现的。
看到Samples页上的TSpinEdit控件了吗?他有MaxValue(最大值)、MinValue(最小值)的属性。
Bug1:先把Value设为7,再把MaxValue设为5,MinValue设为0,Value竟然不会自动改变!!!
Bug2:你设置一下MaxValue为-7,MinValue为7。看到了吗?最大值竟然可以比最小值还小。
Bug3:当最大值和最小值相等时Value竟然可以随便设置...


我不明白这个作者当时是如何设计的这么多的Bug,我不明白Borland为何采用这个控件。也许Borland的把关人员是位GG,而这位开发这是位MM,于是......

言归正转让我们打开Delphi安装目录下SourceSamplesSpin.Pas


找到property MaxValue: LongInt read FMaxValue write FMaxValue;
    property MinValue: LongInt read FMinValue write FMinValue;
Bug1、Bug2同时被找到!竟然连判断都没有,直接设置FMaxValue、FMinValue的值,也就是最大最小值竟然不受限制可以随便设置。设置完最大最小值也不刷新Value,导致了Bug1的产生。
改为:
    property MaxValue: LongInt read FMaxValue write SetMaxValue;
    property MinValue: LongInt read FMinValue write SetMinValue;
在Private中添加两个过程:
    procedure SetMaxValue(Value: LongInt);
    procedure SetMinValue(Value: LongInt);
内容如下:

procedure TSpinEdit.SetMaxValue(Value: LongInt);
begin
  if Value >= FMinValue then
    FMaxValue := Value;
  SetValue(Self.Value);
end;

procedure TSpinEdit.SetMinValue(Value: LongInt);
begin
  if Value <= FMaxValue then
    FMinValue := Value;
  SetValue(Self.Value); 
end;

 


它的Private中明明有CheckValue函数嘛,让我来看看。

function TSpinEdit.CheckValue (NewValue: LongInt): LongInt;
begin
  Result := NewValue;
  if (FMaxValue <> FMinValue) then
  begin
    if NewValue < FMinValue then
      Result := FMinValue
    else if NewValue > FMaxValue then
      Result := FMaxValue;
  end;
找到了Bug3的原因此控件作者竟然没有判断FMaxValue、FMinValue相等的情况
更改为:
function TSpinEdit.CheckValue (NewValue: LongInt): LongInt;
begin
  Result := NewValue;
  if (FMaxValue <> FMinValue) then
  begin
    if NewValue < FMinValue then
      Result := FMinValue
    else if NewValue > FMaxValue then
      Result := FMaxValue;
  end
  else
  begin
    Result:=FMaxValue;
  end;
end;

Tags:

相关文章

    文章评论

    最 近 更 新
    热 点 排 行
    Js与CSS工具
    代码转换工具
    
    <