关于java方法里的非空判断的疑问，谢谢

Question

   public static void main(String[] args) {

        String result = replaceStr("jaja");

        if (result != null) {            //在这里又要判断下非空

              int count = 50 + Integer.parseInt(result);

        }

    }

    

    public static String replaceStr(String tem){

        String result;

        if (tem == null) {

            result = null;

        } else {

            String substring = tem.substring(0, 1);

            result = substring;

        }

        return result;

    }

如上代码，我的疑惑时，当参数tem为空，方法返回null，那岂不是又要在调用replaceStr这个方法的地方又要判断次返回值是否为null?那这个代码看上去不是很难看，好多if(**!=null)这样的语句。

我想问的是：有没有一种设计，可以只要在一处判断非空，其他地方只要写业务逻辑就行了？

谢谢

Answer 1

个人做法：除非需要null返回，否则别返回null值；别传递null值【《Clean Code》也这么写】如果不返回null，处理方式一：直接抛NPE；处理方式二：用Optional（可以避免我们忘记！=null的校验，同时比到处!=看着优雅）

Answer 2

而且你的提问有些费解，看不懂你想问什么问题。

Answer 3

首先，你代码都写错了        if (tem != null) {            result = null;        } else {            String substring = tem.substring(0, 1);            result = substring;        }  应该是if (tem == null) 其次，即使这样改过来还是不够严谨,tem=""时else里面不就有问题了。 再次，你这才判断两次为null就嫌多啊。 最后，判断的次数是根据你的业务逻辑来的，即使在某个地方减少了判断的次数，在别的地方还是要还的。

Answer 4

你别把null 作为方法的返回值就不用检测null了，比如你这个例子里面返回"0"就可以了。

Answer 5

如果确定你的term入参在正常业务场景下必须不能为null，那么没必要去做null值得判断，尽早抛出空指针异常是合理的。

Answer 6

可以通过业务逻辑严密设计来减少 非空的判断，但是依然不能保证维护代码产生的畸变，所以为了代码的健壮性，还是使用之前首先判断， 如果散落代码 str == null || "".equals(str) 很不好看，可以使用自己封装工具或使用开源工具，比如apache的StringUtils 等.