Question

我知道从一个方法实例化一个类变量不是一个好主意，但是不幸的是，我遇到了没有其他解决方案的情况。我创建了一些自定义验证器我有一个对象（rawField），其中填充了一些来自GUI的信息。为了填充该实例变量，我使用了javax.faces.Validator中的validate方法。

因此，通过此事件，我从GUI获取每个字段的信息。

我的问题是：这是一种好的设计模式吗？关于我该如何实例化此变量，有人有更好的主意吗？

父类：

 public abstract class FormFieldValidator extends BaseValidator implements IFormFieldValidator, Validator {

            protected RawField rawField;

            @Override
            public abstract RawField doInitialize(Object inputObject);

            @Override
            public void validate(FacesContext context, UIComponent component, Object value) throws ValidatorException {

            rawField = doInitialize(value);

            if (rawField == null) {

                throw new IllegalArgumentException("The field cannot be empty");

            }

            doBasicValidation();
            }
    }

儿童班

@FacesValidator(Constants.Validators.USERNAME_VALIDATOR)

public class UsernameValidator extends FormFieldValidator {

    @Override
    public RawField doInitialize(Object guiField) {

    ValidationConditions validationConditions = new ValidationConditions

        .Builder(Entities.ParamsSize.USERNAME_MIN_LENGTH, Entities.ParamsSize.USERNAME_MAX_LENGTH)

            .setRegex(Constants.Regex.USERNAME_REGEX).setNullable(false).setUnique(true).build();


    FieldDetails fieldDetails = new FieldDetails(guiField, "Username");

    RawField rawField = new RawField(fieldDetails, validationConditions);

    return rawField;
    }

}

Answer 1

用方法调用数据填充对象不一定是反模式。您的代码中不正确的是方法validate所做的验证工作更多。首先，它初始化变量，然后验证对象。最好将其命名为populateAndValidate或从正文中删除doInitialize并确保由客户端代码调用。对rawField字段进行空检查可能是验证过程的一部分。

从方法实例化类变量-Java

1 个答案: