- html - 出于某种原因,IE8 对我的 Sass 文件中继承的 html5 CSS 不友好?
- JMeter 在响应断言中使用 span 标签的问题
- html - 在 :hover and :active? 上具有不同效果的 CSS 动画
- html - 相对于居中的 html 内容固定的 CSS 重复背景?
我有以下 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/
简而言之:我想从可变参数模板参数中提取各种选项,但不仅通过标签而且通过那些参数的索引,这些参数是未知的 标签。我喜欢 boost 中的方法(例如 heap 或 lockfree 策略),但想让它与 S
我可以对单元格中的 excel IF 语句提供一些帮助吗? 它在做什么? 对“BaselineAmount”进行了哪些评估? =IF(BaselineAmount, (Variance/Baselin
我正在使用以下方法: public async Task Save(Foo foo,out int param) { ....... MySqlParameter prmparamID
我正在使用 CodeGear RAD Studio IDE。 为了使用命令行参数测试我的应用程序,我多次使用了“运行 -> 参数”菜单中的“参数”字段。 但是每次我给它提供一个新值时,它都无法从“下拉
我已经为信用卡类编写了一些代码,粘贴在下面。我有一个接受上述变量的构造函数,并且正在研究一些方法将这些变量格式化为字符串,以便最终输出将类似于 号码:1234 5678 9012 3456 截止日期:
MySql IN 参数 - 在存储过程中使用时,VarChar IN 参数 val 是否需要单引号? 我已经像平常一样创建了经典 ASP 代码,但我没有更新该列。 我需要引用 VarChar 参数吗?
给出了下面的开始,但似乎不知道如何完成它。本质上,如果我调用 myTest([one, Two, Three], 2); 它应该返回元素 third。必须使用for循环来找到我的解决方案。 funct
将 1113355579999 作为参数传递时,该值在函数内部变为 959050335。 调用(main.c): printf("%d\n", FindCommonDigit(111335557999
这个问题在这里已经有了答案: Is Java "pass-by-reference" or "pass-by-value"? (92 个回答) 关闭9年前。 public class StackOve
我真的很困惑,当像 1 == scanf("%lg", &entry) 交换为 scanf("%lg", &entry) == 1 没有区别。我的实验书上说的是前者,而我觉得后者是可以理解的。 1 =
我正在尝试使用调用 SetupDiGetDeviceRegistryProperty 的函数使用德尔福 7。该调用来自示例函数 SetupEnumAvailableComPorts .它看起来像这样:
我需要在现有项目上实现一些事件的显示。我无法更改数据库结构。 在我的 Controller 中,我(从 ajax 请求)传递了一个时间戳,并且我需要显示之前的 8 个事件。因此,如果时间戳是(转换后)
rails 新手。按照多态关联的教程,我遇到了这个以在create 和destroy 中设置@client。 @client = Client.find(params[:client_id] || p
通过将 VM 参数设置为 -Xmx1024m,我能够通过 Eclipse 运行 Java 程序-Xms256M。现在我想通过 Windows 中的 .bat 文件运行相同的 Java 程序 (jar)
我有一个 Delphi DLL,它在被 Delphi 应用程序调用时工作并导出声明为的方法: Procedure ProduceOutput(request,inputs:widestring; va
浏览完文档和示例后,我还没有弄清楚 schema.yaml 文件中的参数到底用在哪里。 在此处使用 AWS 代码示例:https://github.com/aws-samples/aws-proton
程序参数: procedure get_user_profile ( i_attuid in ras_user.attuid%type, i_data_group in data_g
我有一个字符串作为参数传递给我的存储过程。 dim AgentString as String = " 'test1', 'test2', 'test3' " 我想在 IN 中使用该参数声明。 AND
这个问题已经有答案了: When should I use "this" in a class? (17 个回答) 已关闭 6 年前。 我运行了一些java代码,我看到了一些我不太明白的东西。为什么下
我输入 scroll(0,10,200,10);但是当它运行时,它会传递字符串“xxpos”或“yypos”,我确实在没有撇号的情况下尝试过,但它就是行不通。 scroll = function(xp
我是一名优秀的程序员,十分优秀!