public static void add(ArrayList<stu> arr){ stu s=new stu(); Scanner sc=new Scanner(System.in); System.out.println("名字"); String name=sc.nextLine(); System.out.println("年龄"); String age=sc.nextLine(); System.out.println("学号"); String id=sc.nextLine(); System.out.println("地址"); String adddrese=sc.nextLine(); s.setAdresss(adddrese); s.setAge(age); s.setId(id); s.setName(name); arr.add(s); boolean b=judge(arr); if(b==false){ arr.remove(arr.size()-1); } if(b==true){ System.out.println("添加成功"); } } public static boolean judge(ArrayList<stu> arr){ stu s2=arr.get(arr.size()-1); for (int i=0;i<arr.size();i++){ stu s1=arr.get(i); if(s1.getId().equals(s2.getId())){ return false; } } return true; } }这段代码中 judge方法哪里出错了?怎样修改?
时间: 2024-04-17 22:22:52 浏览: 116
Lambda表达式学习
在judge方法中,出错的地方是在判断是否有重复学号的逻辑中。当前的逻辑是,遍历arr中的每个stu对象,与最后一个添加的stu对象s2进行比较。如果发现有重复的学号,则返回false。
问题出在,比较的时候仅仅使用了学号(id)进行比较,而没有考虑其他属性(如姓名、年龄、地址)是否相同。因此,如果有两个学生对象的学号相同,但其他属性不同,也会判断为重复。
要修改这个问题,可以在判断学号是否相同的基础上,再判断其他属性是否相同。可以使用equals()方法来比较字符串属性是否相同。
下面是修改后的judge方法:
```java
public static boolean judge(ArrayList<stu> arr) {
stu s2 = arr.get(arr.size() - 1);
for (int i = 0; i < arr.size() - 1; i++) { // 注意遍历范围是 arr.size()-1
stu s1 = arr.get(i);
if (s1.getId().equals(s2.getId()) && s1.equals(s2)) { // 判断学号和其他属性都相同
return false;
}
}
return true;
}
```
这样修改后,judge方法将会比较两个stu对象的学号和其他属性是否都相同,如果有任何一个不同,则认为不是重复的学号。
阅读全文