java.lang.IndexOutOfBoundsException：无效的索引2，Arraylist中的大小为2？

Question

我有一个侦听器，当我单击日期时，它将搜索arraylist并获取单击的日期，然后当我更改月份时，将删除日期，然后在当前月份中添加日期。

public void onSelectDate(Date date, View view) {
    JSONObject jsonObj = new JSONObject(jsonStr);
    JsonArray datesIn = jsonObj.getJSONArray(TAG_SCHEDULE);
    for (int i = 0; i < datesIn.length(); i++) {
        JSONObject c = datesIn.getJSONObject(i);
        String dateD = c.getString(TAG_DATED);
        String statusD = c.getString(TAG_STATUS);
        HashMap<String, String> contact = new HashMap<String, String>();
        contact.put("Tag_dated", dateD);
        contact.put("Tag_status", statusD);
        contactList.add(contact);
        Date s = formatter.parse(dateD);

        ArrayList<HashMap<String, String>> contactList= new ArrayList<HashMap<String,String>>();;
        if(formatter.format(date).equals(contactList.get(0).get(TAG_DATED))){
            if(contactList.get(0).get(TAG_STATUS).equals("Occupied")){
                Toast.makeText(getApplicationContext(),"S",Toast.LENGTH_SHORT).show();
            }else if(contactList.get(0).get(TAG_STATUS).equals("Reserved"))
                Toast.makeText(getApplicationContext(),"R",Toast.LENGTH_SHORT).show();
        }else if(formatter.format(date).equals(contactList.get(1).get(TAG_DATED))){
            if(contactList.get(1).get(TAG_STATUS).equals("Occupied")){
                Toast.makeText(getApplicationContext(),"S",Toast.LENGTH_SHORT).show();
            }else if(contactList.get(1).get(TAG_STATUS).equals("Reserved")){
                Toast.makeText(getApplicationContext(),"R",Toast.LENGTH_SHORT).show();
            }
        }else if(formatter.format(date).equals(contactList.get(2).get(TAG_DATED))){
            if(contactList.get(2).get(TAG_STATUS).equals("Occupied")){
                Toast.makeText(getApplicationContext(),"S",Toast.LENGTH_SHORT).show();
            }else if(contactList.get(2).get(TAG_STATUS).equals("Reserved")){
                Toast.makeText(getApplicationContext(),"R",Toast.LENGTH_SHORT).show();
            }
        }else if(formatter.format(date).equals(contactList.get(3).get(TAG_DATED))){
            if(contactList.get(3).get(TAG_STATUS).equals("Occupied")){
                Toast.makeText(getApplicationContext(),"S",Toast.LENGTH_SHORT).show();
            }else if(contactList.get(3).get(TAG_STATUS).equals("Reserved")){
                Toast.makeText(getApplicationContext(),"R",Toast.LENGTH_SHORT).show();
            }
        }else if(formatter.format(date).equals(contactList.get(4).get(TAG_DATED))){
            if(contactList.get(4).get(TAG_STATUS).equals("Occupied")){
                Toast.makeText(getApplicationContext(),"S",Toast.LENGTH_SHORT).show();
            }else if(contactList.get(4).get(TAG_STATUS).equals("Reserved")){
                Toast.makeText(getApplicationContext(),"R",Toast.LENGTH_SHORT).show();
            }
        }else if(formatter.format(date).equals(contactList.get(5).get(TAG_DATED))){
            if(contactList.get(5).get(TAG_STATUS).equals("Occupied")){
                Toast.makeText(getApplicationContext(),"S",Toast.LENGTH_SHORT).show();
            }else if(contactList.get(5).get(TAG_STATUS).equals("Reserved")){
                Toast.makeText(getApplicationContext(),"R",Toast.LENGTH_SHORT).show();
            }
        }else{
            Toast.makeText(getApplicationContext(),"NP",Toast.LENGTH_SHORT).show();
        }
    }

我的问题是当我更改月份然后单击以获取错误日期。

public void onChangeMonth(int month, int year) {
    contactList.clear();
    HashMap<String, String> contact = new HashMap<String, String>();
    contact.put("Tag_dated", dateD);
    contact.put("Tag_status", statusD);
    contactList.add(contact);
}

Logcat错误：

http://i.stack.imgur.com/Is35y.png

Answer 1

您正在创建一个名为contactList的新ArrayList ，并向其中添加一个元素。 那么您正在尝试从中检索几个元素！

Answer 2

错误是海峡向前，大小是两个。 索引从0开始，因此可用索引为0和1而不是1和2。您的循环可能需要大小-1。

而不是硬编码您的语句，例如：if（formatter.format（date）.equals（contactList.get（0）.get（TAG_DATED）））{

尝试以循环方式进行

public void updateToast(Date date, ArrayList<HashMap<String, String>> contactList) {
    String formattedDate = formatter.format(date);
    for (int i = 0; i < contactList.size(); i++)
    {
        if (formattedDate.equals(contactList.get(i).get(TAG_DATED))) 
        {
           if(contactList.get(i).get(TAG_STATUS).equals("Occupied"))
           {
              Toast.makeText(getApplicationContext(),"S",Toast.LENGTH_SHORT).show();
              return;
           } else if(contactList.get(i).get(TAG_STATUS).equals("Reserved")) 
           {
              Toast.makeText(getApplicationContext(),"R",Toast.LENGTH_SHORT).show();
              return;
           }
        }
    }
    // If here then its not found in any lists throw up your other text.
    Toast.makeText(getApplicationContext(),"NP",Toast.LENGTH_SHORT).show();
}

其中一些可能是由于它在这里的闲置完成而有些不足。

Answer 3

在您的代码onSelectDate中，您说的是：

contactList.add(contact);

它将仅将Hashmap添加到列表一次，因此您可以在Arraylist上使用contactList.get（0）。 但是向前说

contactList.get(1);//even two

因此它没有超越第一的元素，因此会引发异常。

Answer 4

让我们重构一下代码，摆脱这种丑陋的，不安全的if / else恐怖，并将其更改为美观的循环

    boolean found = false;
    String formattedDate = formatter.format(date);
    for (int i = 0; i < contactList.size(); i++) {
        if (formattedDate.equals(contactList.get(i).get(TAG_DATED))) {
            if (contactList.get(i).get(TAG_STATUS).equals("Occupied")) {
                Toast.makeText(getApplicationContext(), "S",
                        Toast.LENGTH_SHORT).show();
            } else if (contactList.get(1).get(TAG_STATUS)
                    .equals("Reserved")) {
                Toast.makeText(getApplicationContext(), "R",
                        Toast.LENGTH_SHORT).show();
            }
            found = true;
            break;
        }
    }
    if (!found) {
        Toast.makeText(getApplicationContext(), "NP",
                Toast.LENGTH_SHORT).show();
    }

此代码应该做与您完全相同的事情，但是不应该给您ArrayIndexOutOfBoundsException异常