- 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/
我正在使用 Selenium Web 驱动程序 3.0,并且想要从打开的两个对话框(一个在后台,第二个在前台)的 Activity 对话框中单击“确定”按钮。如何从 html 下面的父 div 单击前
actions: [ FlatButton( onPressed: () {
我有一个问题有点超出我的范围(我真的很高兴我是 Beta)涉及重复项(所以 GROUP BY, HAVING, COUNT),通过将解决方案保留在 SQLite 附带的标准函数中而变得更加复杂。我正在
使用DBI是否可以确定SELECT语句的已执行语句句柄是否返回任何行而不从中获取行? IE。就像是: use DBI; ... my $sth = $dbh->prepare("SELECT ..."
是否可以为“确定”和“关闭”按钮指定回调函数? 如果是JQuery Modal,则可以在初始化时使用按钮字典指定回调函数。 Semantic-ui模态是否提供类似的功能?按下确定后,我该如何寻求其他逻
我想阅读警报中的消息。 示例:如果警报显示“错误的电子邮件地址”。怎么读呢?意味着我想将该消息存储在字符串中。 如何在“警报”中单击“确定”...?? 如何使用 Selenium 来做到这一点? 最佳
我有一个删除按钮: 我试图首先查明是否已选择一个网站,如果已选择一个网站,我需要确定是否已选择一个或多个列表项,如果是,则继续删除这些项目。 我的 if 语句不断返回“您必须首先选择您的列表”,即使它
部分出于好奇——我们想知道在我们的应用程序中发生了什么——部分是因为我们需要在我们的代码中找到一些潜在的问题,我喜欢在我们的网络应用程序运行时跟踪一些一般值。这尤其包括某些对象图的分配内存。 我们的应
我将 SweetAlert 与 Symfony 结合使用,我希望用户在完成删除操作之前进行确认。 发生的情况是,当用户单击删除按钮时,SweetAlert 会弹出,然后立即消失,并且该项目被删除。 在
我们有一个应用程序可以生成不包括字母 O 的随机基数 35 [0-9A-Z]。我正在寻找一种解决方案来查找包含任何淫秽英语单词的代码,而无需搜索包含 10,000 个条目的列表每个生成的代码。每秒生成
这是我做的: #include #include int betweenArray(int a, int b){ int *arr,i,range; range = b - a +
我知道如何创建 警报和确认框,但我不知道如何做的是实际单击“确定”。我有一个弹出确认框的页面。 我想使用 Java Script 插件单击“确定”。基本上,我希望我的代码单击页面上的链接,然后在出现提
代码: swal('Your ORDER has been placed Successfully!!!'); window.location="index.php"; 甜蜜警报工
>>> import re >>> s = "These are the words in a sentence" >>> regex = re.compile('are|words') >>> [m
使用确定的理想散列函数给出随机期望线性时间算法两个数组 A[1..n] 和 B[1..n] 是否不相交,即 A 的元素是否也是 B 的元素。 谁能告诉我如何做到这一点,甚至如何开始考虑它? 最佳答案
我在计算机科学课上有这段代码: int input=15; while (input < n ) { input = input *3;} 这段代码有 log3(n/15) 次循环的上限。我们怎样才能
我有一个允许 2 位玩家玩 TicTacToe 的程序。在每个玩家移动之后,它应该在那个点显示棋盘并返回一个名为 Status 的枚举,显示玩家是否应该继续,如果玩家赢了,还是平局。但是,该算法要么返
给定一个 y 值数组,例如 [-3400, -1000, 500, 1200, 3790],我如何确定“好的”Y 轴标签并将它们放置在网格上? ^ ---(6,000)-|---
假设我有一个检查用户登录的 SQL 语句: SELECT * FROM users WHERE username='test@example.com', password='abc123', expi
teradata中有返回表中哪一列被定义为主索引的命令吗?我没有制作一些我正在处理的表,也没有尝试优化我对这些表的连接。谢谢! 最佳答案 有dbc.IndicesV,其中IndexNumber=1表示
我是一名优秀的程序员,十分优秀!