
在开发基于 arraylist 的库存管理系统时,一个常见需求是在添加或更新元素后,立即显示该元素最新的详细信息。然而,当 arraylist 中包含多个元素时,如果显示逻辑设计不当,可能会出现问题。
原始代码中,displayCurrentVehicleEntry() 方法旨在显示当前操作的车辆信息。在 addVehicle() 方法中,它能正确显示新添加的车辆,因为新添加的车辆通常位于列表的末尾。然而,在 updateVehicle() 方法中,尽管车辆数据已成功更新,但 displayCurrentVehicleEntry() 却总是显示 ArrayList 中最后一个元素的信息,而非刚刚被更新的车辆。
仔细观察 displayCurrentVehicleEntry() 的实现,可以发现问题根源:
public void displayCurrentVehicleEntry() {
try {
// 始终获取列表中的最后一个元素
AutoInv vehicle = listOfVehicles.get(listOfVehicles.size() - 1);
System.out.println("Make: " + vehicle.getMake().toUpperCase());
// ... 其他属性
} catch (Exception e) {
System.out.println("Failure");
System.out.println(e.getMessage());
e.printStackTrace();
}
}这段代码硬编码了获取 listOfVehicles.size() - 1 索引处的元素,这意味着它总是尝试获取列表中的最后一个元素。当在 updateVehicle() 方法中调用此方法时,如果被更新的车辆不是列表中的最后一个,就会导致显示错误。
解决此问题的核心在于,让 displayCurrentVehicleEntry() 方法能够接收一个参数,明确指定要显示哪个索引位置的元素。这样,无论是在添加还是更新操作中,我们都可以将正确元素的索引传递给该方法。
首先,我们需要修改 displayCurrentVehicleEntry() 方法的签名,使其接受一个 int 类型的 index 参数。
public void displayCurrentVehicleEntry(int index) { // 添加 index 参数
try {
// 使用传入的 index 获取指定元素
AutoInv vehicle = listOfVehicles.get(index);
System.out.println("Make: " + vehicle.getMake().toUpperCase());
System.out.println("Model: " + vehicle.getModel().toUpperCase());
System.out.println("Color: " + vehicle.getColor().toUpperCase());
System.out.println("Year: " + vehicle.getYear());
System.out.println("Mileage: " + vehicle.getMileage());
System.out.println("");
} catch (IndexOutOfBoundsException e) { // 捕获更具体的异常
System.out.println("错误:尝试显示一个不存在的车辆索引。");
e.printStackTrace();
} catch (Exception e) {
System.out.println("显示车辆信息失败");
System.out.println(e.getMessage());
e.printStackTrace();
}
}说明:
在 updateVehicle() 方法中,当找到匹配的车辆并完成更新后,我们现在可以将当前循环的索引 i 传递给 displayCurrentVehicleEntry() 方法。
public void updateVehicle(String makeCurrent, String modelCurrent, String colorCurrent, int yearCurrent, int mileageCurrent,
String makeUpdated, String modelUpdated, String colorUpdated, int yearUpdated, int mileageUpdated) {
try {
boolean found = false;
for (int i = 0; i < listOfVehicles.size(); i++) {
AutoInv vehicle = listOfVehicles.get(i);
if (vehicle.getMake().equalsIgnoreCase(makeCurrent)
&& vehicle.getModel().equalsIgnoreCase(modelCurrent)
&& vehicle.getColor().equalsIgnoreCase(colorCurrent)
&& vehicle.getYear() == yearCurrent
&& vehicle.getMileage() == mileageCurrent) {
// 更新车辆信息
vehicle.setMake(makeUpdated);
vehicle.setModel(modelUpdated);
vehicle.setColor(colorUpdated);
vehicle.setYear(yearUpdated);
vehicle.setMileage(mileageUpdated);
System.out.println("\nVehicle updated successfully!\n");
// 调用修改后的方法,传入当前车辆的索引 i
displayCurrentVehicleEntry(i);
found = true;
// 如果只允许更新一个匹配项,可以在这里添加 break;
break;
}
}
if (!found) {
System.out.println("\nVehicle not found in inventory!");
}
} catch (Exception e) {
System.out.println("更新车辆失败");
System.out.println(e.getMessage());
e.printStackTrace();
}
}说明:
同样,在 addVehicle() 方法中,虽然原始代码能正确工作,但为了保持一致性和最佳实践,我们也应该使用带参数的新版 displayCurrentVehicleEntry() 方法。新添加的车辆始终位于列表的末尾,因此其索引为 listOfVehicles.size() - 1。
public void addVehicle(AutoInv vehicle) throws Exception{
try {
if (listOfVehicles.add(vehicle)) {
System.out.println("\nFollowing vehicle added successfully:\n");
// 调用修改后的方法,传入新添加车辆的索引
displayCurrentVehicleEntry(listOfVehicles.size() - 1);
}
else {
throw new Exception("\nFailed to add vehicle.");
}
} catch (Exception e) {
System.out.println(e.getMessage());
}
}说明:
通过对 displayCurrentVehicleEntry() 方法进行参数化改造,使其能够接收并处理具体的元素索引,我们成功解决了在 ArrayList 中更新元素后无法正确显示其详情的问题。这种方法不仅修复了现有bug,还提升了代码的模块化和可维护性。在设计类似功能时,应始终考虑如何通过传递必要的上下文信息(如索引或对象本身)来确保方法的正确性和通用性。
以上就是如何正确显示 ArrayList 中更新后的特定元素信息的详细内容,更多请关注php中文网其它相关文章!
每个人都需要一台速度更快、更稳定的 PC。随着时间的推移,垃圾文件、旧注册表数据和不必要的后台进程会占用资源并降低性能。幸运的是,许多工具可以让 Windows 保持平稳运行。
Copyright 2014-2025 https://www.php.cn/ All Rights Reserved | php.cn | 湘ICP备2023035733号