代码中的魔鬼数字

魔鬼数字的定义:在代码中没有具体含义的数字、字符串。

魔鬼数字主要影响了代码可读性,读者看到的数字无法理解其含义,从而难以理解程序的意图。当程序中出现的魔鬼数字过多时,代码的可维护性将会急剧下降,代码变得难以修改,并容易引入错误。

例如:

修改后:

解决方法:用常量定义魔鬼数字。


   将数字定义为常量的最终目的是为了使代码更容易理解,所以并不是只要将数字定义为常量就不是魔鬼数字了。如果常量的名称没有意义,无法帮助理解代码,同样是一种魔鬼数字。

   在个别情况下,将数字定义为常量反而会导致代码更难以理解,此时就不应该强求将数字定义为常量。

案例

// 魔鬼数字,无法理解3具体代表产品的什么状态

if (product.getProduct().getProductStatus() != 3)

{

    throw new PMSException(PMSErrorCode.Product.ADD_ERROR);

}

// 仍然是魔鬼数字,无法理解NUM_THREE具体代表产品的什么状态

if (product.getProduct().getProductStatus() != NUM_THREE)

{

    throw new PMSException(PMSErrorCode.Product.ADD_ERROR);

}



魔鬼代码

01-23

procedure UnpackPlate(sourceFile: string; image:p);rnvarrn k,j,i:integer;rn height,width :integer;rn PTmp:p;rn readbyte:byte;rn writebyte:byte;rn shiftbyte:byte;rn sourceStream :TFileStream;rnbeginrn width := 112; //解压后的宽度rn height := 20; //解压后的高度rn PTmp := image; //临时指针rn if not FileExists(sourceFile) then exit;rn tryrn sourceStream := TFileStream.Create(sourceFile,fmOpenRead or fmShareDenyNone);rn tryrn for i := 0 to height-1 dorn beginrn sourceStream.Position := (height-1-i)*14;rn for j := 0 to (width div 8)-1 dorn beginrn sourceStream.Read(readbyte,1);rn shiftbyte := 1;rn for k := 0 to 7 do // k 值从8开始??????rn beginrn if (shiftbyte and readbyte) > 0 thenrn beginrn writebyte := 0;rn endrn elsern beginrn writebyte := 215;rn end;rn PTmp^:= writebyte;rn inc(PTmp);rn shiftbyte := shiftbyte shl 1; //一位8次就会报“边界溢出”错rn end; //end of for krn end; //end of for jrn end; // end of for irn finallyrn sourceStream.Free;rn end;rn exceptrn exit;rn end;rnend;rnrn这是一个过程,其中image是一块内存的指针,在过程外面已经分配好了。这段代码rn在以前运行都是好的,昨天发现问题了。rn首先,K值竟然是从8开始,共循环8次,到1时就退出K循环了。rn其二,shiftbyte := shiftbyte shl 1; 这个左移操作在第8次的时候会抛出一个异常:rn---------------------------rnDebugger Exception Notificationrn---------------------------rnProject demo.exe raised exception class ERangeError with message 'Range check error'. rn其三,还有跟绝的,如果我把左移操作该成:rnif k < 7 thenrnbeginrnshiftbyte := shiftbyte shl 1;rnend;rn这个时候再跟踪K值,竟然发现他又是从0开始的了。rn其四,i,k值都可以在单步调试的时候可以查看它们的值,而查看j值确报告:由于优化,不能查看rn其值????,怎么回事?如果我一定要知到j值呢?rnrnrn以上四个问题,请大家讨论!rn我用的2000 server,d7rn昨天还重装过d7,在别的机器上测试这段代码也有同样的问题。rnrnrn

没有更多推荐了,返回首页

私密
私密原因:
请选择设置私密原因
  • 广告
  • 抄袭
  • 版权
  • 政治
  • 色情
  • 无意义
  • 其他
其他原因:
120
出错啦
系统繁忙,请稍后再试