gpt4 book ai didi

java - 验证由另一个参数确定的构造参数

转载 作者:行者123 更新时间:2023-11-30 10:28:55 28 4
gpt4 key购买 nike

我有以下 enum

public enum StudentType {
Domestic, International;
}

和一个具有以下构造函数的 Student 类:

//Left out additonal parameters and validation
public Student(StudentType type, List<String> documents){

if(type == null){
throw new IllegalArgumentException("You must provide Student type");
}
this.type = type;
this.documents = this.validateList(documents);
}

和一个验证列表的私有(private)方法:

private List<String> validateList(List<String> validate){
if(this.type == StudentType.Domestic && validate.isEmpty()){
return validate;
}
else
if(this.type == StudentType.Domestic && !validate.isEmpty()){
return Collections.emptyList();
}
return new ArrayList<String>(validate);
}

我的计划是让 Student 类不可变。

解释:

只有国际学生才需要文件(护照等)。国内学生不需要任何文件。

在我的 constructor我检查以确保类型不是 null , 客户必须提供国内或国际。

在我的私有(private)验证列表中 method我检查学生类型,如果 List是否为空。如果它是国内的且为空,则返回列表,如果它是国内的且不为空,则返回一个空集合,否则返回 ArrayList<String>文件。

我的问题是检查私有(private)方法中的类型是否有代码异味?我担心的是,一个参数(文件列表)是由学生类型决定/验证的。如果检查私有(private)方法中的类型是代码味道,我应该怎么做呢?

最佳答案

我看到您发布的代码有几个问题,但不是您认为的问题。

首先,我们不知道文档是从哪里来的。我假设这只是代码中的一个错误,它实际上是第二个参数。

现在,代码:

private List<String> validateList(List<String> validate){
if(this.type == StudentType.Domestic && validate.isEmpty()){
return validate;
}
else
if(this.type == StudentType.Domestic && !validate.isEmpty()){
return Collections.emptyList();
}
return new ArrayList<String>(validate);
}

首先,不是在参数无效时拒绝该参数(就像您对空学生类型所做的那样),而是忽略该参数并改用一个空列表。这通常不是一个好主意。如果调用者传递一个非空列表,它当然不会期望这个列表被静默忽略。如果列表应该为空,则通过抛出异常来拒绝非空列表。

第二:代码的最后一行建议您制作作为参数传递的列表的防御性副本。但是您不会在 if block 的第一个分支中执行此操作。由于这两个分支无论如何都存储一个空集合,因此可以将其替换为

private List<String> validateList(List<String> validate) {
return this.type == StudentType.Domestic ? Collections.emptyList() : new ArrayList<>(validate);
}

最后,(回到我的第一点),如果学生类型是国内的,调用者永远不应该传递非空列表,你可以通过使用两个工厂方法而不是构造函数来使这不可能:

private Student(StudentType type, List<String> documents) {
this.type = type;
this.documents = documents;
}

public static Student createDomestic() {
return new Student(StudentType.DOMESTIC), Collections.emptyList());
}

public static Student createInternational(List<Document> documents) {
return new Student(StudentType.INTERNATIONAL, new ArrayList<>(documents);
}

关于java - 验证由另一个参数确定的构造参数,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/44345658/

28 4 0
Copyright 2021 - 2024 cfsdn All Rights Reserved 蜀ICP备2022000587号
广告合作:1813099741@qq.com 6ren.com