为什么每个人都告诉我编写这样的代码是一种不好的做法?
if (foo) Bar(); //or for(int i = 0 i < count; i++) Bar(i);
我省略花括号的最大论据是,有时花括号可能是花括号的两倍。例如,以下代码为C#中的标签绘制发光效果。
using (Brush br = new SolidBrush(Color.FromArgb(15, GlowColor))) { for (int x = 0; x <= GlowAmount; x++) { for (int y = 0; y <= GlowAmount; y++) { g.DrawString(Text, this.Font, br, new Point(IconOffset + x, y)); } } } //versus using (Brush br = new SolidBrush(Color.FromArgb(15, GlowColor))) for (int x = 0; x <= GlowAmount; x++) for (int y = 0; y <= GlowAmount; y++) g.DrawString(Text, this.Font, br, new Point(IconOffset + x, y));
您还可以获得链接usings在一起的额外好处,而不必缩进一百万次。
usings
using (Graphics g = Graphics.FromImage(bmp)) { using (Brush brush = new SolidBrush(backgroundColor)) { using (Pen pen = new Pen(Color.FromArgb(penColor))) { //do lots of work } } } //versus using (Graphics g = Graphics.FromImage(bmp)) using (Brush brush = new SolidBrush(backgroundColor)) using (Pen pen = new Pen(Color.FromArgb(penColor))) { //do lots of work }
花括号的最常见参数涉及维护编程,以及在原始if语句及其预期结果之间插入代码会引起的问题:
if (foo) Bar(); Biz();
实际上,唯一一次真正困扰我的时间是在调试时,并注释掉了bar():
if(foo) // bar(); doSomethingElse();
除此之外,我倾向于使用:
if(foo) bar();
可以解决上述情况。
编辑 感谢您澄清这个问题,我同意,我们不应该将代码写到最低公分母。