不管你想不相信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安装目录下\Source\Samples\Spin.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;
|
本类阅读TOP10