应用程序在removeView上崩溃

时间:2017-11-03 21:08:32

标签: android android-layout android-linearlayout programmatically

我编写了以下应用程序,其中显示了一个3乘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);
        }

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

有谁知道为什么?

2 个答案:

答案 0 :(得分:0)

问题

您的问题是您是按索引引用视图。每次删除视图时,子项数都会减少。我们来看看你的例子:

LinearLayout以3个孩子开头:
[A,B,C]

第一次打电话 ((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(0);

现在你的LinearLayout有两个孩子:
[B,C]

执行下一个呼叫
((LinearLayout) linearLayout.getChildAt(1)).removeViewAt(1);

现在你的LinearLayout有一个孩子:
[B]

执行下一个电话
(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);

答案 1 :(得分:0)

首先,我要感谢达姆斯的解释。如果没有他/她的回应,我将无法提出这个解决方案。虽然他/她的解释是有帮助的,但他/她提出的解决方案是不正确的。我将在下面发布一个正确的解决方案: 改变这个

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。