发信人: tellmewhy (tellmewhy), 信区: Java
标 题: 发一段code review的代码
发信站: 水木社区 (Thu Sep 19 15:42:06 2013), 站内

大致就是在for循环中检查一个json的数据中是否有message字段,如果没有或值长度为0的话就continue跳过,review前代码:
if(json.has(\"message\"))
{
if(json.getString(\"message\").trim().length() == 0)
continue;
}
else
{
continue;
}

review后代码:
if(!json.has(\"message\") || json.getString(\"message\").trim().length() == 0 )
continue;

在此想问大家觉得有必要必须改成review后代码吗,或者大家看到review前这段代码是什么心情?

--

※ 修改:·tellmewhy 于 Sep 19 15:46:16 2013 修改本文·[FROM: 123.120.8.*]
※ 来源:·水木社区 newsmth.net·[FROM: 123.120.8.*]