当我尝试添加人时，我仍然遇到问题。该代码被认为是检测无效的名称和生日，并且不会将此人添加到personList中。但是，因为在代码读取之前我无法获得生日，所以只能在循环结束时添加person对象。当我尝试测试添加一个invlid人名和/或生日时，会出现问题。它显示：
无法添加人员：名称无效!!
新记录：
名称：null“
生日：未知的DOB
已添加！
Exception in thread "main" java.lang.NullPointerException
at Instruction.readInstruction(Instruction.java:272)
at EPB.main(EPB.java:12)
else if(words[0].equalsIgnoreCase("add"))
{
Person p = new Person();
String s1 = line.substring(words[0].length()).trim();
String[] s2 = s1.split(";");
if(s2.length>=2)// s2[0] = " name Testing Three" s2[1] = " birthday 13-05-1981" and s2[2]=" address.."
{
for(int i=0; i<s2.length;i++)
{
String s3 = s2[i].trim(); // "delete the leading space" s2[0] = "name Testing Three" s2[1] = "birthday 13-05-1981"
String[] s4 = s3.split("\\s+"); //s4[0] = name; s4[1] = Testing; s4[2]=Three
if(s4.length>=2) // s2[1]=birthday 13-05-1986 only has length of 2
{
if(s4[0].equalsIgnoreCase("name"))
{
//System.out.println(s4[1]);
String name="";
if(Functions.nameValidation(s4[1]))
{
for(int j=2;j<s4.length;j++)
{
name = s4[1] + " " + s4[j];
}
if(Functions.nameValidation(name))
{
p.setName(name);
}
else
{
System.out.println("Failed to add person: Invalid name!");
break;
}
}
else
{
System.out.println("Failed to add person: Invalid name!!");
break;
}
}//end of word equals to name
//-----------------------------------------------------------------
else if(s4[0].equalsIgnoreCase("birthday") && s4.length ==2)
{
if(Functions.dateValidation(s4[1]))
{
try
{
p.setBirthdayString(s4[1]);
}
catch (ParseException e)
{
//e.printStackTrace();
}
}
else
{
System.out.println("Failed to add person: Invalid Birthday Format");
break;
}
}
//end of word equals to birthday
//-----------------------------------------------------------------
boolean notFound = false;
for(Person p1: personList)
{
if(p1.getName().equals(p.getName()))
{
if(p1.getBirthdayString().equals(p.getBirthdayString()))
{
System.out.println("Information in record" +"\n" + "name: "+ p.getName() + "\n" + "Birthday: " + p.getBirthdayString() + "\n" +"has been updated");
p1.setEmail(p.getEmail());
p1.setPhone(p.getPhone());
p1.setAddress(p.getAddress());
notFound = true;
}
}
}
if (!notFound)
{
if(Functions.nameValidation(p.getName()) && Functions.dateValidation(p.getBirthdayString()))
{
System.out.println("New record: " +"\n"+"Name: " + p.getName() + "\""+ "\n"+ "Birthday: " + p.getBirthdayString() + "\nhas been added!");
personList.add(p);
}
FileIO.outData(personList, outputFileName);
}
System.out.println();
}
答案 0 :(得分：8)
这是问题所在：
for(Person p1: personList)
{
...
personList.add(p);
}
在迭代时，您无法修改集合。最简单的修复可能是有一个单独的列表，您在迭代personList时构建，然后在以后添加所有内容：
List<Person> newPeople = new ArrayList<Person>();
for(Person p1: personList)
{
...
newPeople.add(p);
}
personList.addAll(newPeople);
（正如Peter指出的那样，你可能不想再添加 - 但这就是你获得异常的原因。）
答案 1 :(得分：3)
我认为在最后一个循环中要做什么而不是迭代personList并在每个步骤检查它是否包含p的名称和生日，是为每个检查这个条件迭代中的元素有这样的：
p1.getName().equals(p.getName())
而不是personList.contains(p.getName())
和
p1.getBirthdayString().equals(p.getBirthdayString())
而不是personList.contains(p.getBirthdayString())
这会修复逻辑错误，也会解决异常，因为在迭代集合时你不会调用contains。
编辑：
然后你仍然会遇到一个错误，你试图在同一个循环中将p添加到personList（在“else”块中：personList.add(p)）。你现在这样做的方式不仅仅是让你成为例外，而是试图为列表中不等于它的每个p添加人p1。要解决此问题，您需要在循环内部保留一个布尔值，以检查是否找到任何匹配项（如果检查了上述任何条件，则为true），并将p添加到personList外部循环，如果说布尔值不是真的。
总之，这是迭代循环的修改代码：
boolean found = false;
for(Person p1: personList)
{
if(p1.getName.equals(p.getName()))
{
if(p1.getBirthdayString.equals(p.getBirthdayString()))
{
System.out.println("Information in record" +"\n" + "name: "+ p.getName() + "\n" + "Birthday: " + p.getBirthdayString() + "\n" +"has been updated");
p1.setEmail(p.getEmail());
p1.setPhone(p.getPhone());
p1.setAddress(p.getAddress());
FileIO.outData(personList, outputFileName);
found = true;
}
}
}
if (!found)
{
System.out.println("New record has been added!");
personList.add(p);
FileIO.outData(personList, outputFileName);
}
答案 2 :(得分：0)
您正在迭代personList并检查此列表是否包含Name和BirthdayString值。这没有意义，因为此列表仅包含Person对象。
您的问题是，如果包含检查失败，则在迭代时将新对象添加到此列表中，这会导致异常：
System.out.println("New record has been added!");
personList.add(p);
基本上，您需要重新考虑包含的检查，而不是修改您正在迭代的集合。
答案 3 :(得分：0)
我认为你的问题是你不知道在哪里看。你有问题
for(Person p1: personList)
{
if(personList.contains(p.getName()))
{
if(!personList.contains(p.getBirthdayString()))
{
System.out.println("New record has been added!");
personList.add(p);
您在迭代时修改列表。
这很可能是一个错误，而不是你应该采取的另一种方式。