isSuccess作为变量名好不好

看到过很多代码如下：

private Boolean success;
private Boolean isSuccess;
private Boolean isSuccess(){}

其实都不推荐，因为is类似于get等在，自动生成和序列化的时候容易混淆。
可以做个试验，idea在有isSuccess方法的时候不会生成getSuccess():

private boolean success = true;
public boolean isSuccess() { 
    return this.success;
}

判断的时候有的人偏向于用

if(isSuccess()){ }

其实是多余的，因为直接用如下代码就可以判断了

if(success){ // 如果成功...
}

如果要易于观看的话可以命名为successFlag,表示这是标志字段。
读代码的时候不读flag即可。如:

if(successFlag){  //  如果成功...(这里的flag不用读，只是让人知道他是个boolean标记)
}