一、代码
public static final UserExcepFactor ACCOUNT_IS_EMPTY = new UserExcepFactor(
HttpStatus.BAD_REQUEST, 50, "Account is empty.","账号为空");
private static final Set<ExcepFactor> excepFactors = new HashSet<>();
private ExcepFactor( HttpStatus httpStatus,int errorCode, String errorMsg, String errorMsgCn) {
if (errorCode < 0 || errorCode > 99) {
throw new IllegalArgumentException("errorCode must between 1~99 .");
}
if (serviceId < 0 || serviceId > 99) {
throw new IllegalArgumentException("serviceId must between 1~99 .");
}
this.httpStatus = httpStatus;
this.errorCode = errorCode;
this.errorMsg = errorMsg;
this.errorMsgCn = errorMsgCn;
if(excepFactors.contains(this)){
throw new IllegalArgumentException("this error exist: "+this.getErrorCode());
}
excepFactors.add(this);
}
MatrixExceptionHelper.localMatrixException(UserExcepFactor.ACCOUNT_IS_EMPTY )
二、描述
第三段localMatrixException方法里面需要传入一个参数,该参数是一个UserExcepFactor类对象,这个对象的属性现在需要动态传入。该类中有一个HashSet类型的变量excepFactors,为了确保excepFactors变量中值唯一,该变量中属性相同的对象只能存在一个,所以就需要保证localMatrixException中传入的对象是唯一的。如果直接new一个该对象,每次请求过来后都会新创建一个对象,这样excepFactors.contains(this)就会成立,就会抛出异常,所以需要每次产生的对象实例都是同一个对象实例来避免这个问题,所以就可以用单例模式来保证生产出唯一的实例。
三、解决
public static UserExcepFactor SINGLE = null;
public static UserExcepFactor single(String errorMsg,String errorMsgCn){
if (SINGLE == null){
SINGLE = new UserExcepFactor(HttpStatus.BAD_REQUEST,98,errorMsg,errorMsgCn);
}
return SINGLE;
}
throw MatrixExceptionHelper.localMatrixException(UserExcepFactor.single("我对了","我真的对了"));
四、反思
原因说到底应该是静态属性的问题,因为创建的对象是放在静态类属性中,所以创建后就会存在虚拟机栈中,不会随着方法的结束而销毁,而继续创建的时候还会调用构造方法,而构造方法中还进行判断,所以会出现问题。所以只要保证相同属性的只调用一次构造方法就好了,所以可以用单例,保证只调用一次。个人理解,勿喷。