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

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

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

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

输出的结果是：

第一次加薪，工资总数：65535
第二次加薪，工资总数：0

工资被清零了。

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

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

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

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

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

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