public void process(File file) {
    if (file != null) {
        if (file.exists()) {
            if (!file.isDirectory()) {
                if (file.canRead()) {
                    // do something;
                }
            }
        }
    }
}

import com.google.common.base.Preconditions.checkNotNull;
public void process(File file){
    checkNotNull(file);
    if(file.isFile() && file.canRead(){
        ...
    }
}

------解决方案--------------------
探讨

确实有些隐患，一些特殊情况下不执行任何代码就跳过了，在后继的代码中不好界定处于什么状态


------解决方案--------------------
checkNotNull很多开源项目的做法是Assert.notNull(file,"参数不能为空");

很多项目中都有自己的Assert类


探讨


Java code
import com.google.common.base.Preconditions.checkNotNull;
public void process(File file){
    checkNotNull(file);
    if(file.isFile() && file.canRead(){
        ...
    }
}


------解决方案--------------------
是啊！！是初学者

------解决方案--------------------
重构太多会导致混乱的，楼主不要使用这么多嵌套

------解决方案--------------------
使用太多的嵌套是不好 但是有时候需要多层嵌套解决问题更加容易

------解决方案--------------------
嵌套太多了，肯定要重构。

------解决方案--------------------
支持楼主~~

------解决方案--------------------
代码中的 for、if 的嵌套三四层应该是可以接受的吧
只是可以接受,尽量不要这样写就好了

------解决方案--------------------
Java code

//一般我喜欢这么做
public void process(File file) {
        if (file == null)return;
        if(!file.exists())return;
        if(file.isDirectory())return;
        if(!file.canRead())return;
        .....
        doSomething
        .....
    }

------解决方案--------------------
嵌套太多不好

------解决方案--------------------
怎么重构啊？给个例子，莫非就是把几个if弄到一块，弄个&&吗？
探讨

引用:
确实有些隐患，一些特殊情况下不执行任何代码就跳过了，在后继的代码中不好界定处于什么状态

超过的话得想办法重构，怎么重构法？谢谢


------解决方案--------------------
com.google.common.base.Preconditions.checkNotNull
这个类原来是androidSDK里的啊，终于找到了，谢谢，又开眼界了

------解决方案--------------------
大家不要把新手当老手看，他们考虑问题的时候跟我们想的不一样；大家刚出来的时候未必就没写过恶心代码，随着经验的增长，再看以前写的代码，总觉得以前自己好白痴。

------解决方案--------------------