在按钮网格上调用 removeViewAt 时应用程序崩溃

Question

我编写了以下应用程序，其中显示了 3 x 4 的按钮网格，用户可以通过单击菜单项来更改网格尺寸。问题是“deleteSomething”函数发生了一些奇怪的事情。下面是完整代码：

package com.example.myapplication;

import android.os.Bundle;
import android.support.v7.app.AppCompatActivity;
import android.view.Gravity;
import android.view.Menu;
import android.view.MenuItem;
import android.widget.Button;
import android.widget.LinearLayout;

public class MainActivity extends AppCompatActivity {

    LinearLayout.LayoutParams params;
    LinearLayout linearLayout;
    int _row;
    int column;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        linearLayout = new LinearLayout(this);
        linearLayout.setOrientation(LinearLayout.VERTICAL);  //Can also be done in xml by android:orientation="vertical"
        params = new LinearLayout.LayoutParams(LinearLayout.LayoutParams.FILL_PARENT, LinearLayout.LayoutParams.FILL_PARENT);
        params.weight = 1.0f;
        params.gravity = Gravity.TOP;
        //layout.setBackgroundColor(0xFFFFFFFF);
        _row=3;
        column=4;
        update();

    }


    public void update(){
        for (int i = 0; i < _row; i++) {
            LinearLayout row = new LinearLayout(this);
            row.setLayoutParams(params);

            for (int j = 0; j < column; j++) {
                Button btnTag = new Button(this);
                btnTag.setLayoutParams(params);
                btnTag.setText("Button " + (j + 1 + (i * column)));
                btnTag.setId(j + 1 + (i * column));
                if ((i+j) % 2 == 0) {
                    btnTag.setBackgroundColor(0xFFFF0000);
                } else {
                    btnTag.setBackgroundColor(0x00000000);
                }

                row.addView(btnTag);
            }
            linearLayout.addView(row);
        }

        setContentView(linearLayout);
    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        menu.add(Menu.NONE, 1, Menu.NONE, "Item name");
        menu.add(Menu.NONE, 2, Menu.NONE, "Item name");
        return true;
    }

    public void deleteSomething(){

        ((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(0);
        ((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(1);
        ((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(2);
    }

    @Override
    public boolean onOptionsItemSelected(MenuItem item) {

        switch (item.getItemId()) {
            case MENU_ITEM_ITEM1:
                deleteSomething();
                //linearLayout.removeAllViews();
                //_row=4;
                //column=5;
                //update();
                return true;
            case 2:
                deleteSomething();
                //linearLayout.removeAllViews();
                _row=6;
                column=3;
                update();
            default:
                return false;
        }


    }
}

这个特定部分有些奇怪：

public void deleteSomething(){

            ((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(0);
            ((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(1);
            ((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(2);
        }

注释掉前两行，第二行第三个按钮被删除没有问题；注释掉第二个，第二个中的前两个按钮也可以毫无问题地删除。但是，当您包含所有这三行时，应用程序会在您单击下拉菜单中的第一项后立即崩溃。

有谁知道为什么吗？

Answer 1

问题

您的问题是您正在按索引引用视图。每次删除视图时，子视图的数量都会减少。让我们来看看你的例子：

LinearLayout 以 3 个孩子开始：
[甲乙丙]

第一次通话

((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(0);

现在你的 LinearLayout 有 2 个孩子：
[乙、丙]

执行下一个调用

((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(1);

现在你的 LinearLayout 有 1 个孩子：
[乙]

执行下一个调用

(LinearLayout) linearLayout.getChildAt(1)).removeViewAt(2);

这会引发异常，因为索引 2 超出了您的孩子的范围（现在大小为 1）。

解决方案

首先获取对要删除的任何视图的引用。然后使用

removeView(View view)

将它们全部从 LayoutGroup 中删除。下面是一个简单的示例，说明您如何出于演示目的执行此操作：

LinearLayout layout = (LinearLayout) linearLayout.getChildAt(1);
View a = layout.getChildAt(0);
View b = layout.getChildAt(1);
View c = layout.getChildAt(2);
layout.removeView(a);
layout.removeView(b);
layout.removeView(c);

Answer 2

首先感谢dharms的解释。没有他/她的回应，我将无法想出这个解决方案。虽然他/她的解释很有帮助，但他/她提出的解决方案是不正确的。我将在下面发布正确的解决方案： 改变这个

public void deleteSomething(){

    ((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(0);
    ((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(1);
    ((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(2);
}

至：

public void deleteSomething(){
    for (int i = _row-1; i >= 0; i--) {
        for (int j = column - 1; j >= 0; j--) {
            ((LinearLayout) linearLayout.getChildAt(i)).removeViewAt(j);
        }
    }
}

基本思路是删除的时候不想改变id，所以要倒着引用他们的id。