メニューからのメソッド呼び出しでAndroidアプリケーションがクラッシュする

Question

Gridlayout（下のコード）にボタンのグリッドを作成し、そのリスナーを設定し、それぞれにno_padding_buttonというカスタムレイアウトを追加するcreateGridという関数があります。

は、ユーザーが私をクリックしたどのボタン（複数可）に応じて、グリッドを作成した後gridlayoutを反復処理とは、そのタグに応じて、クリックされたボタンの数を返しますsaveDesignと呼ばれる別の機能を持っています。私はsaveDesignをメニュー項目から呼び出します。

私がsaveDesignメソッドを呼び出すとすぐに、アプリがクラッシュするという問題があります。 グリッドがうまくいき、ボタンをクリックすると色が変わりタグが変わりますが、gridlayoutを読み取ろうとする何らかの理由でアプリがクラッシュします。

私のコードのエラーは何ですか？

おかげ

//method to create a new grid, the number of rows and columns come from the dialogue inside the activity and then passed to this function 
void createGrid(final Context context, GridLayout gridLayout) { 

    gridLayout.setColumnCount(totalColumns); //set the number of rows of the gridlayout 
    gridLayout.setRowCount(totalRows);   //set the number of columns of the grid layout 

    //add the buttons and implement their listeners 
    for (int i = 0; i < totalRows; i++) { 
     for (int j = 0; j < totalColumns; j++) { 
      Button button = new Button(context); 
      //no padding 
      button.setBackground(context.getResources().getDrawable(R.drawable.no_padding_button)); 
      //set the name of the button according to its position 
      button.setText(Integer.toString(i) + "," + Integer.toString(j)+","); 
      //hide the name, the user does not need to see this information at this moment 
      button.setTextColor(View.INVISIBLE); 

      //setting up the layout parameters for each button 
      GridLayout.LayoutParams param = new GridLayout.LayoutParams(); 
      param.setMargins(0, 0, 0, 0); 
      button.setLayoutParams(param); 

      //button listener 
      button.setOnClickListener(new View.OnClickListener() { 
       boolean already_clicked = false; 
       @Override 
       public void onClick(View v) { 

        //on click, hide the button 
        Button button = (Button) v; 
        if(!already_clicked){ 
        //change the color of the selected buttons as an indication 
        button.setBackgroundColor(context.getResources().getColor(R.color.selected_button)); 
         button.setTag(1); 
         already_clicked =true; 
        } 
        else{ 
         button.setBackgroundColor(context.getResources().getColor(R.color.unselected_button)); 
         button.setTag(0); 
         already_clicked =false; 
        } 
       } 
      }); 

      gridLayout.addView(button); 
     } 
    } 

    Toast.makeText(context, "Grid Created", Toast.LENGTH_SHORT).show(); 
} 

saveDesing方法：

public void saveDesign(){ 
    int temp=0; 
    for (int i=0;i<gridLayout.getChildCount(); i++){ 
     Button childButton = (Button)gridLayout.getChildAt(i); 
     if(childButton.getTag().toString() == "1"){ 
      temp++; 
     } 
    } 
    Toast.makeText(this, Integer.toString(temp), Toast.LENGTH_SHORT).show(); 
} 

グリッドレイアウトを保持アクティビティのXMLコード：

<?xml version="1.0" encoding="utf-8"?> 
<ScrollView 
    xmlns:android="http://schemas.android.com/apk/res/android" 
    android:layout_width="wrap_content" 
    android:layout_height="wrap_content" 
    android:layout_gravity="center"> 
    <HorizontalScrollView android:id="@+id/HorizontalScrollView" 
     android:layout_width="wrap_content" 
     android:layout_height="wrap_content" 
     android:layout_gravity="center"> 
     <GridLayout xmlns:android="http://schemas.android.com/apk/res/android" 
      android:id="@+id/gridLayout" 
      android:layout_width="wrap_content" 
      android:layout_height="wrap_content" 
      android:layout_gravity="center"> 

     </GridLayout> 
    </HorizontalScrollView> 
</ScrollView> 

UPDATE 1エラーログには、私も試みstring.equals（ "1"）、同じエラーが発生しました：

E/AndroidRuntime: FATAL EXCEPTION: main 
        Process: com.example.test.gridtest, PID: 3247 
        java.lang.NullPointerException: Attempt to invoke virtual method 'boolean java.lang.Object.equals(java.lang.Object)' on a null object reference 
         at com.example.abtin.gridtest.MainActivity.saveDesign(MainActivity.java:85) 
         at com.example.abtin.gridtest.MainActivity.onOptionsItemSelected(MainActivity.java:109) 
         at android.app.Activity.onMenuItemSelected(Activity.java:3204) 
         at android.support.v4.app.FragmentActivity.onMenuItemSelected(FragmentActivity.java:406) 
         at android.support.v7.app.AppCompatActivity.onMenuItemSelected(AppCompatActivity.java:195) 
         at android.support.v7.view.WindowCallbackWrapper.onMenuItemSelected(WindowCallbackWrapper.java:103) 
         at android.support.v7.app.AppCompatDelegateImplV9.onMenuItemSelected(AppCompatDelegateImplV9.java:667) 
         at android.support.v7.view.menu.MenuBuilder.dispatchMenuItemSelected(MenuBuilder.java:810) 
         at android.support.v7.view.menu.MenuItemImpl.invoke(MenuItemImpl.java:152) 
         at android.support.v7.view.menu.MenuBuilder.performItemAction(MenuBuilder.java:957) 
         at android.support.v7.view.menu.MenuBuilder.performItemAction(MenuBuilder.java:947) 
         at android.support.v7.widget.ActionMenuView.invokeItem(ActionMenuView.java:616) 
         at android.support.v7.view.menu.ActionMenuItemView.onClick(ActionMenuItemView.java:153) 
         at android.view.View.performClick(View.java:5610) 
         at android.view.View$PerformClick.run(View.java:22260) 
         at android.os.Handler.handleCallback(Handler.java:751) 
         at android.os.Handler.dispatchMessage(Handler.java:95) 
         at android.os.Looper.loop(Looper.java:154) 
         at android.app.ActivityThread.main(ActivityThread.java:6077) 
         at java.lang.reflect.Method.invoke(Native Method) 
         at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:865) 
         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:755) 

Answer 1

あなたがボタンをクリックしない場合は、それがタグ付けされていません。したがって、childButton.getTag（）はnullを返し、toStringはアプリケーションをクラッシュさせます。ボタンを作成するとき だから）（saveDesignに

if(childButton.getTag() != null) { 
    if(childButton.getTag().toString() == "1"){ 
     temp++; 
    } 
} 

または設定ボタンのタグを確認してください。