当我将 ArrayList 的 get 方法传递给一个变量时，它使用的是整个 ArrayList

Question

由于某种原因，当我传递我的 ArrayList（它是来自 Java 的 Lang.* 的基本 ArrayList）一个 int 变量时，而不是在变量 int 的索引处获取项目（例如： i = 0; arrayList.get(i); 应该是第一个ArrayList 中的项目） - 它调用 WHOLE 列表。

下面是有问题的方法：

public ZipCode findZip (int zip) {
    ZipCode aZip = new ZipCode(00000);
     //System.out.println(zips.get(0)); here gives 99501,ANCHORAGE,AK
     //System.out.println(zips.get(0).getZipCode()); here gives 99501
        for(int i = 0; i < zips.size(); i++) {
            if(zips.get(i).getZipCode() == zip)
                aZip = zips.get(i);
              //System.out.print(aZip); here gives the zip codes from the whole array
            else
                aZip = null;
        }
    return aZip;
    //Therefore, aZip is ALWAYS null. Even if it exists.
}

我尝试了一系列故障排除来解决这个问题。到目前为止，我了解到当我输入一个直整数时不会出现问题（例如： arrayList.get(1); 正常获取第二项）。不幸的是，没有要显示的编译器错误。

它运行时总是给我 null 。 我需要使用一个 int 所以我可以增加它来遍历列表，这也很不幸。 更大的程序片段：（与我之前的问题相同）

import java.util.*;
import java.io.*;
import java.lang.Math;
public class ZipCodeDatabase {
   //Field
    private ArrayList<ZipCode> zips;

   //Constructor
   public ZipCodeDatabase () {
      zips = new ArrayList<ZipCode> (); 
   }

   //Mutator Method
   public void readZipCodeData(String filename) {
      Scanner inFS = null; 
      FileInputStream fileByteStream = null;
      try{
       // open the File and set delimiters
         fileByteStream = new FileInputStream(filename);
         inFS = new Scanner(fileByteStream);
         inFS.useDelimiter("[,\r\n]+");
       // continue while there is more data to read
         while(inFS.hasNext()) {
            //read in all input
            int aZip = inFS.nextInt();
            String aCity = inFS.next();
            String aState = inFS.next();
            double aLat = inFS.nextDouble();
            double aLon = inFS.nextDouble();
            //Create and add new zipcode
            ZipCode newZip = new ZipCode(aZip, aCity, aState, aLat, aLon);
            zips.add(newZip);
         }
         fileByteStream.close();
         // Could not find file
         }catch(FileNotFoundException error1) {
            System.out.println("Failed to read the data file: " + filename);
          // error while reading the file                      
         }catch(IOException error2) {
             System.out.println("Oops! Error related to: " + filename);
      }        
   }

   //Accessor Methods
   public ZipCode findZip (int zip) {
      ZipCode aZip = new ZipCode(00000);
      for(int i = 0; i < zips.size(); i++) {
          if(zips.get(i).getZipCode() == zip)
            aZip = zips.get(i);
         else
            aZip = null;
      }
      return aZip;
   }

这是 getZipCode() 方法（它返回邮政编码 int，是包含纬度和经度的一小部分 int 的一部分；我有一个数组）：

public int getZipCode () {
       return zipCode;
}

Answer 1

当您找到邮政编码时，您不会停止循环，因此您在循环中的下一次迭代通常会将您的返回值设置为 null。将您的声明更改为：

为您的教授编辑：

 ZipCode aZip = null;
 for(int i = 0; i < zips.size() && aZip == null; i++) {

    if(zips.get(i).getZipCode() == zip) {
            aZip = zips.get(i);
    }
 }
 if (aZip == null) {
    aZip = new ZipCode(00000);  
    // return new ZipCode(00000); would be better
 }

 return aZip;

Answer 2

由于您在找到匹配项后没有停止循环，因此下一次迭代的结果将覆盖您的匹配项，这很可能是一个不匹配项，因此用null 覆盖它。有几种选择：

break 出：

public ZipCode findZip (int zip) {
  ZipCode aZip = null;
  for(int i = 0; i < zips.size(); i++) {
      if(zips.get(i).getZipCode() == zip) {
        aZip = zips.get(i);
        break;
     }
     else
        aZip = null;
  }
  return aZip;
}

改变循环条件：

public ZipCode findZip (int zip) {
  ZipCode aZip = null;
  for(int i = 0; i < zips.size() && aZip==null; i++) {
      if(zips.get(i).getZipCode() == zip)
        aZip = zips.get(i);
     else
        aZip = null;
  }
  return aZip;
}

不要覆盖：

public ZipCode findZip (int zip) {
  ZipCode aZip = null;
  for(int i = 0; i < zips.size(); i++) {
      if(zips.get(i).getZipCode() == zip)
        aZip = zips.get(i);
     else
        continue; // obsolete, but you said, your prof insist on it…
  }
  return aZip;
}

---

最后一点，最干净的解决方案是在 ZipCode 类中提供正确的 equals 方法。如果ZipCode 类有正确的equals 实现，您可以简单地实现该方法：

public ZipCode findZip (int zip) {
  ZipCode aZip = new ZipCode(zip);
  if(zips.contains(aZip))
    return aZip;
  else
    return null;
}

或者，甚至更短：

public ZipCode findZip (int zip) {
  ZipCode aZip = new ZipCode(zip);
  return zips.contains(aZip)? aZip: null;
}

Answer 3

谢谢大家！

虽然其他答案确实有效，但这是一个不同的解决方案，它给出了似乎正确的答案：

public ZipCode findZip (int zip) {
      ZipCode aZip = null;
      for(int i = 0; i < zips.size(); i++) {
          if(zips.get(i).getZipCode() == zip) {
            aZip = zips.get(i);
          }
      }
      return aZip;
   }

Answer 4

问题在于，当您找到目标 ZipCode 时，您并没有停止迭代，而是在后续迭代中将 null 分配给您的 aZip 变量。

要解决它，您可以使用条件来停止迭代：

ZipCode aZip = null;

for (int i = 0; (i < zips.size()) && (aZip == null); i++) {
    if (zips.get(i).getZipCode() == zip) {
        aZip = zips.get(i); // When you set this, aZip will no longer be == null
    }
}