最近写代码写晕了。 - 迷失网络

搜索

仅本站注册用户可用，请注册用户或者登录

公告

lostinet@lostinet.com这个油箱不能用了。因为空间没了，lostinet.com指向为127.0.0.1 。。。

Feedbacks

# re: 最近写代码写晕了。

典型的代码风格造成的问题，问题的原因你知我知天知地知，其他人知不知我就不知了。还是不要点破，留点想象空间给大家吧。

9/3/2004 7:11 AM | sumtec

# re: 最近写代码写晕了。

Keep in mind to always use code block delimiters (like curly braces or BEGIN..END) for code blocks, unless you were working with Python (where you use indentation to describe code blocks, purely :)

9/3/2004 7:22 AM | JGTM'2004 [MVP]

# re: 最近写代码写晕了。

现在流行这种错误吧！～～heihei
我前两天还在跟朋友讲我犯了这种错误，今天又碰到一个！～

9/3/2004 7:38 AM | xinyu

# re: 最近写代码写晕了。

这种写法else是跟随最近的if走~

9/3/2004 8:33 AM | minbear

# re: 最近写代码写晕了。

看来括号是很重要的。

9/3/2004 8:57 AM | webmote

# re: 最近写代码写晕了。

pascal也是这样，即使if后有一行代码也要加括号，保证以后再加代码不会有错。

9/3/2004 9:22 AM | 学习中

# re: 最近写代码写晕了。

我习惯即使只有一行代码也加括号，所以我从未有这种错误。

9/3/2004 9:26 AM | 雷俊强

# re: 最近写代码写晕了。

可能，IDE也有一定的关系。如果环境能够，把自动缩进体现的更明显点，这种错误就可以被发现了吧。那个else太靠前了！

9/3/2004 9:30 AM | irrational

# re: 最近写代码写晕了。

还是python好，通过缩进解决这个问题：
if(x.IsAdmin):
　　　　if(!y.IsAdmin)return -1;
else:
　　　　if(y.IsAdmin)return 1;

9/3/2004 9:31 AM | rIPPER

# re: 最近写代码写晕了。

修改一下：）

if x.IsAdmin:
　　　　if not y.IsAdmin:
　　　　　　　　return -1;
else:
　　　　if y.IsAdmin:
　　　　　　　　return 1;

9/3/2004 9:33 AM | rIPPER

# re: 最近写代码写晕了。

看了这篇我想去code review了，好像team里面有个人从来不喜欢多用括号

9/3/2004 9:37 AM | Alexander

# re: 最近写代码写晕了。

我写代码都是大量的使用{}，即使只有一行语句的if。因为这样的错误有时实在是难以发现（单步调试倒是很容易发现），我不想在后期为这个纠缠……

9/3/2004 10:05 AM | Magicloud

# re: 最近写代码写晕了。

我以后都不敢这样写了。555。

9/3/2004 11:37 AM | Lostinet

# re: 最近写代码写晕了。

典型的代码风格造成的。

9/3/2004 11:51 AM | elssann

# re: 最近写代码写晕了。

To XinYu:
哈哈，你找到同路人了！！

看来类似的错误，还是满普遍的哦~~

9/3/2004 1:19 PM | Young

# re: 最近写代码写晕了。

You editor is not smart/good enough in auto-indent

CB400

9/3/2004 2:14 PM | CB400

# re: 最近写代码写晕了。

"反过来，如果心里没有认为它没有问题，那么就很难发现错误了。"
文字比代码更有意思.

9/3/2004 3:00 PM | zzm

# re: 最近写代码写晕了。

有时候的确就是看不出来错误在哪，大部分首先看的是程序的思路，并没在意代码规范。

9/3/2004 4:22 PM | 漆黑

# re: 最近写代码写晕了。

if (x.IsAdmin)
{
if (!y.IsAdmin)
{
return -1;
}
}
else
{
if (y.IsAdmin)
{
return 1;
}
}

何必节省那么几行?

9/3/2004 4:22 PM | feilng

# re: 最近写代码写晕了。

偶一行代码也加括号^&^

9/3/2004 4:42 PM | ayya__@hotmail.com

# re: 最近写代码写晕了。

养成好的习惯好啊~好象我不会有这样的错误^_^

9/3/2004 6:27 PM | ttyp

# re: 最近写代码写晕了。

在开发产品的时候应该尽量用建议的标准格式去写，不要嫌太繁琐（排错的时间不比写个括号长？）。如果只是写个例子或试验一下考考自己或者别人倒也无妨:-)

9/3/2004 11:00 PM | niuke

# re: 最近写代码写晕了。

严格来讲x.IsAdmin这样的方法调用是不能放在if后面的。if后面只能放变量和简单表达式。

9/4/2004 11:54 AM | Ninputer

# re: 最近写代码写晕了。

to Ninputer:

这里的 IsAdmin 是个属性，不是方法（C#中方法是要加括号的）。属性一般可以看作是简单表达式了，比如常见的：

if (Page.IsPostBack)
{
//... some codes
}

所以这里应该不会有什么效率问题，除非 IsAdmin 属性实现时在 get 访问器的代码中进行了许多操作。

9/4/2004 5:45 PM | 破宝

# re: 最近写代码写晕了。

从来就应该加括号，应该有这种习惯！

9/7/2004 11:11 AM | 仪表

# re: 最近写代码写晕了。

缩成一句，不过返回值是 0或1，返回值用0,1比-1,1好用。
if(x.IsAdmin!=y.IsAdmin)
return y.IsAdmin;
如果要返回-1,1也可以一句实现。

9/7/2004 4:03 PM | fan-fan

# re: 最近写代码写晕了。

只需一个if 根相不用两个

if(x.IsAdmin!=y.IsAdmin)
return y.IsAdmin*1-x.IsAdmin*1;

9/7/2004 4:07 PM | fan-fan

# re: 最近写代码写晕了。

实际上楼主要实现的罗辑是：如果两人权限不一致，则返回两人的权限差。所以只需一个if 用两个if是多余的，而且第一个else是同第二个if配对。你的写法错了。
用下面的语句就可实现你的逻辑
if(x.IsAdmin!=y.IsAdmin)
return y.IsAdmin*1-x.IsAdmin*1;

9/7/2004 4:15 PM | fan-fan