My AppがListラジオボタンでクラッシュする

Question

ラジオボタンをクリックすると、アプリケーションがクラッシュします。 ラジオボタンが動的に生成されます。しかし、私がそのボタンを参照するとクラッシュします。

クラッシュが発生するここでは、スナックバーが表示されます。私は を選択したボタンのメソッドgetTextを呼び出すと信じています。

どうすれば修正できますか？

public class ListActivity extends AppCompatActivity { 
@Override 
protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_list); 
    Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar); 
    setSupportActionBar(toolbar); 
    String[] filename = getApplicationContext().fileList(); 

    addRadioButtons(filename.length, filename); 

    FloatingActionButton fab = (FloatingActionButton) findViewById(R.id.fab); 
    fab.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View view) { 
      RadioGroup rd = (RadioGroup) findViewById(R.id.file_radio_group); 
      int id_Btn = rd.getCheckedRadioButtonId() ; 
      RadioButton selectedRB = (RadioButton) findViewById(id_Btn) ; 

      Snackbar.make(view, "Replace with your own action "+selectedRB.getText().toString(), Snackbar.LENGTH_LONG) 
        .setAction("Action", null).show(); 
     } 
    }); 



} 
public void addRadioButtons(int number,String[] filenames) { 

    for (int row = 0; row < 1; row++) { 
     RadioGroup ll = new RadioGroup(this); 
     ll.setOrientation(LinearLayout.VERTICAL); 

     for (int i = 1; i <= number; i++) { 
      RadioButton rdbtn = new RadioButton(this); 
      rdbtn.setId((row * 2) + i); 
      rdbtn.setText(filenames[i-1]); 
      //rdbtn.setText("Radio " + rdbtn.getId()); 
      rdbtn.setTextSize(25); 
      ll.addView(rdbtn); 
     } 
     ((ViewGroup) findViewById(R.id.file_radio_group)).addView(ll); 
    } 
} 
} 

はここで、私のLogcatとにかく

11-01 19:04:26.460 6743-6743/alahdal.amjad.newloginproject E/AndroidRuntime: FATAL EXCEPTION: main 
Process: alahdal.amjad.newloginproject, PID: 6743 
java.lang.NullPointerExceptionat 
alahdal.amjad.newloginproject.ListActivity$1.onClick(ListActivity.java:39) 
at android.view.View.performClick(View.java:4438) 
at android.view.View$PerformClick.run(View.java:18422) 
at android.os.Handler.handleCallback(Handler.java:733) 
at android.os.Handler.dispatchMessage(Handler.java:95) 
at android.os.Looper.loop(Looper.java:136) 
at android.app.ActivityThread.main(ActivityThread.java:5017) 
at java.lang.reflect.Method.invokeNative(Native Method) 
at java.lang.reflect.Method.invoke(Method.java:515)  atcom.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:779) 
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:595) 
at dalvik.system.NativeStart.main(Native Method) 

で、それがクラッシュした理由を私は知っている」いけない：

は、ここに私のコードです。助けてください

Answer 1

public class ListActivity extends AppCompatActivity { 
private RadioGroup ll; 
@Override 
protected void onCreate(Bundle savedInstanceState) { 
super.onCreate(savedInstanceState); 
setContentView(R.layout.activity_list); 
Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar); 
setSupportActionBar(toolbar); 
String[] filename = getApplicationContext().fileList(); 

addRadioButtons(filename.length, filename); 

FloatingActionButton fab = (FloatingActionButton) findViewById(R.id.fab); 
fab.setOnClickListener(new View.OnClickListener() { 
    @Override 
    public void onClick(View view) { 
     RadioGroup rd = (RadioGroup) findViewById(R.id.file_radio_group); 
     int id_Btn = rd.getCheckedRadioButtonId() ; 
     RadioButton selectedRB = (RadioButton) findViewById(id_Btn) ; 

     Snackbar.make(view, "Replace with your own action "+selectedRB.getText().toString(), Snackbar.LENGTH_LONG) 
       .setAction("Action", null).show(); 
    } 
}); 



} 


    public void addRadioButtons(int number,String[] filenames) { 

for (int row = 0; row < 1; row++) { 
    ll = new RadioGroup(this); 
    ll.setOrientation(LinearLayout.VERTICAL); 

    for (int i = 1; i <= number; i++) { 
     RadioButton rdbtn = new RadioButton(this); 
     rdbtn.setId((row * 2) + i); 
     rdbtn.setText(filenames[i-1]); 
     //rdbtn.setText("Radio " + rdbtn.getId()); 
     rdbtn.setTextSize(25); 
     ll.addView(rdbtn); 
    } 
    ((ViewGroup) findViewById(R.id.file_radio_group)).addView(ll); 
} 
} 
} 

Answer 2

ここに行く、私はあなたの要件とはっきりしていないが、もう1つのラジオグループを既にxmlに追加する考えを得ることはできません。ここでうまく動作する修正バージョンです。何のラジオボタンが選択されていない場合

@Override 
protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_main); 
    final RadioGroup radioGroup = (RadioGroup) findViewById(R.id.file_radio_group); 
    String[] filename = getApplicationContext().fileList(); 
    addRadioButtons(radioGroup, filename.length, filename); 
    FloatingActionButton fab = (FloatingActionButton) findViewById(R.id.fab); 
    fab.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View view) { 
      int id_Btn = radioGroup.getCheckedRadioButtonId(); 
      for (int count = 0; count < radioGroup.getChildCount(); count++) { 
       if (radioGroup.getChildAt(count).getId() == id_Btn) { 
        RadioButton radioButton = (RadioButton) radioGroup.getChildAt(count); 
        Snackbar.make(view, "Replace with your own action " + radioButton.getText().toString(), Snackbar.LENGTH_LONG) 
          .setAction("Action", null).show(); 
       } 
      } 
     } 
    }); 
} 

public void addRadioButtons(RadioGroup radioGroup, int number, String[] filenames) { 
    radioGroup.setOrientation(LinearLayout.VERTICAL); 
    for (int i = 1; i <= number; i++) { 
     RadioButton rdbtn = new RadioButton(this); 
     rdbtn.setId((i * 2) + i); 
     rdbtn.setText(filenames[i - 1]); 
     rdbtn.setTextSize(25); 
     radioGroup.addView(rdbtn); 
    } 
} 

Answer 3

rd.getCheckedRadioButtonId() は-1を返しますし、それはとてもfindViewByIdビューを見つけることができません任意のビューのidではないでしょう。

id_Btnを使用して再生回数を確認する前に-1を確認してください。

public void onClick(View view) { 
    RadioGroup rd = (RadioGroup) findViewById(R.id.file_radio_group); 
    int id_Btn = rd.getCheckedRadioButtonId() ; 
    if (id_Btn != -1) { 
     RadioButton selectedRB = (RadioButton) findViewById(id_Btn) ; 
    } else { 
     //Handle case if no button is selected 
    } 

    Snackbar.make(view, "Replace with your own action "+selectedRB.getText().toString(), Snackbar.LENGTH_LONG) 
       .setAction("Action", null).show(); 
    }