[调查]代码习惯问题

[blazechen]

今天在项目组，同事看了我写的一些代码，说我的代码写的很别扭。
具体例子如下：
if(!"".equals(sContractNo))
{
//doSomething;
}

if("action".equals(sAction))
{
//dosomething
}
我一直都是这样写，以避免NullPointerException，但是其他人认为
!sContractNo.equals("")看起来才舒服。。。
但是如果sContractNo是null呢？？
请问你们做字符串比较的时候，喜欢怎么写呢？

[sg552]

我不喜欢你的风格。

这样避免空指针异常，其实是利用了equals函数的特性，
而并不是在运行函数之前检查。

虽然你这样的做法貌似不少人推崇，
但是我就是不喜欢。认为很别扭。

补充一句，避免空指针异常，还有很多其他方法。

[SAMUEL_NAME]

喜欢
我和你写的一样