我目前正在研究一些非常旧的代码以扩展它,并且我不断遇到可能很容易返回 void 的方法。我心目中最臭名昭著的一条是:
public int dbInsert(int userID)
{
Map<String, Integer> foo = new HashMap<String, Integer>();
foo.put("bar", 0);
foo.put("user", userID);
sqlInsert(foo);
return foo.get("bar") != null ? (Integer) foo.get("bar") : 0;
}
我认为无论如何都会有很大的重做,但我只是想知道将这样的方法剥离为 void 返回类型(因为上面的代码片段总是返回 0)会影响性能或编译器是否会承认冷漠并忽略它?