编写高质量代码改善C#程序的157个建议——建议113:声明变量前考虑最大值

建议113:声明变量前考虑最大值

假设正在开发一个工资系统,其中一个模块负责处理加薪。代码如下:

        static void Main(string[] args)
{
ushort salary = ;
salary = (ushort)(salary + );
Console.WriteLine(string.Format("第一次加薪,工资总数:{0}", salary));
salary = (ushort)(salary + );
Console.WriteLine(string.Format("第二次加薪,工资总数:{0}", salary));
}

输出的结果是:

第一次加薪,工资总数:65535
第二次加薪,工资总数:0

工资被清零了。

如果让一个刚入行的程序员来写工资系统,他可能会给我们设计一个ushort字段来存储月薪。65535元?够多了,谁的月薪会超过这个数?当然,这里举的是一个极端的例子。在这个例子中我们存在很多机会去修正这个Bug。但是,即便是最有经验的程序员,也会犯这种错误。在一个千万级的数据表中统计历史数据,你认为某个字段的求和极值会是多少?

上面的错误貌似愚蠢低级,可是一不留神,我们就可能掉入这样的陷阱。所以应该始终在声明变量时考虑最大值。

在C#中,如果要避免犯类似的错误,有一个补救措施,那就是为运算加上checked关键字。在运行溢出的时候它会抛出一个异常:

        static void Main(string[] args)
{
ushort salary = ;
checked
{
salary = (ushort)(salary + );
Console.WriteLine(string.Format("第一次加薪,工资总数:{0}", salary));
salary = (ushort)(salary + );
Console.WriteLine(string.Format("第二次加薪,工资总数:{0}", salary));
}
}

经过修正的代码在运行到第二次加薪的时候,会抛出System.OverflowException:算术运算导致溢出。

转自:《编写高质量代码改善C#程序的157个建议》陆敏技

上一篇:Filter、Listener学习笔记


下一篇:Java Web基础