City[] array = new City[6];
String [] arr = {"Paris","London","Rome","Los Angeles","New York","San Francisco"};
int [] arr2 = {200000, 100000, 80000, 60000, 50000, 45000};
String [] arr3 = {"Breitzel", "Statute of Liberty", "Tramways"};
for (int i = 0; i < 6; i++){
if (i<3){
City V = new City (arr[i], "EU", tab2[i]);
array[i] = V;
}
else {
for (int j = 0; j < 3; j++){
Capitale C = new Capitale (arr[i], "USA", arr2[i], arr3[j]);
array[i] = C;
}
}
}
第一个递增循环效果很好,即创建城市的循环(System.out.printline 显示 City[] 获得 6 个元素,即 6 个城市。
但是:这 3 个美国城市应该分别分配一个 arr3 元素。这是行不通的。 j 不会增加,所有 3 个美国城市都有“有轨电车”。
我不明白为什么......
最佳答案
相反,j
正在递增。
但是内部循环的每次迭代都会覆盖前一次迭代的结果。
最后一次迭代的 j = 2,并且 arr3[2]
是“Tramways”。
我想你会得到你正在寻找的结果。
for (int i = 0; i < 6; i++){
if (i<3){
City V = new City (arr[i], "EU", tab2[i]);
array[i] = V;
}
else {
Capitale C = new Capitale (arr[i], "USA", arr2[i], arr3[j - 3]);
array[i] = C;
}
}
也就是说,代码并不是那么清晰。 3 的魔法值尚不清楚。
您可以使用命名常量来解决这个问题:
final int INDEX_OF_FIRST_CAPITALE = 3;
或者让 arr3
与其他两个数组平行
String [] arr3 = { null, null, null, "Breitzel", "Statute of Liberty", "Tramways"};
或者使用从首都名称到景点的 map 。
Map<String,String> = new HashMap<String,String>();
关于java 用 for 递增只能运行一次,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/31926539/