Question

编辑：参见附带的错误日志。

我正在尝试创建一个简单的应用来在Android Studio中滚动不同的侧面骰子。到目前为止这是我的代码： MainActivity.java

package com.example.thomb.tutorialspoint;

import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.RadioButton;
import android.widget.TextView;

import java.util.Random;

public class MainActivity extends AppCompatActivity {

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    Button button = (Button) findViewById(R.id.buttonRoll);
    button.setOnClickListener(new View.OnClickListener() {
        public void onClick(View v) {
            int roll = RollDice(sidesChosen);
            TextView tv1 = (TextView)findViewById(R.id.textView);
            tv1.setText(roll);
            setContentView(tv1);
        }
    });
}

public int sidesChosen;
public int RollDice(int sides) {
    Random r = new Random();
    return r.nextInt(sides)+1;
} //method
public void onRadioButtonClicked(View view) {
    // Is the button now checked?
    boolean checked = ((RadioButton) view).isChecked();

    // Check which radio button was clicked
    switch(view.getId()) {
        case R.id.radioButtonD4:
            if (checked)
                sidesChosen = 4;
                break;
        case R.id.radioButtonD6:
            if (checked)
                sidesChosen = 6;
                break;
        case R.id.radioButtonD8:
            if (checked)
                sidesChosen = 8;
                break;
        case R.id.radioButtonD10:
            if (checked)
                sidesChosen = 10;
                break;
        case R.id.radioButtonD12:
            if (checked)
                sidesChosen = 12;
                break;
        case R.id.radioButtonD20:
            if (checked)
                sidesChosen = 20;
                break;
    } //switch
} //method
} //class

这就是布局的样子： http://i.imgur.com/IeIbMlz.png

当我点击滚动按钮时，应用程序崩溃，但我不知道为什么。 ID都是正确的，单选按钮按预期工作。我使用的是API级别25.我对Java很新，但我对C＃非常熟悉，所以问题可能在于代码语法，尽管Android Studio报告没有错误。如果您需要查看布局的XML，请告诉我。

E/AndroidRuntime: FATAL EXCEPTION: main
              Process: com.example.thomb.tutorialspoint, PID: 5029
              java.lang.IllegalStateException: The specified child already has a parent. You must call removeView() on the child's parent first.
                  at android.view.ViewGroup.addViewInner(ViewGroup.java:4310)
                  at android.view.ViewGroup.addView(ViewGroup.java:4146)
                  at android.view.ViewGroup.addView(ViewGroup.java:4087)
                  at android.view.ViewGroup.addView(ViewGroup.java:4060)
                  at android.support.v7.app.AppCompatDelegateImplV9.setContentView(AppCompatDelegateImplV9.java:279)
                  at android.support.v7.app.AppCompatActivity.setContentView(AppCompatActivity.java:145)
                  at com.example.thomb.tutorialspoint.MainActivity$1.onClick(MainActivity.java:25)
                  at android.view.View.performClick(View.java:5280)
                  at android.view.View$PerformClick.run(View.java:21239)
                  at android.os.Handler.handleCallback(Handler.java:739)
                  at android.os.Handler.dispatchMessage(Handler.java:95)
                  at android.os.Looper.loop(Looper.java:234)
                  at android.app.ActivityThread.main(ActivityThread.java:5526)
                  at java.lang.reflect.Method.invoke(Native Method)
                  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
                  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)

Answer 1

setText()是一种重载方法，有两种类型：一种是String，另一种是int。这里的int应该是字符串资源ID。这是您的错误所在。在下面的代码中，您使用的是setText(int)，但未传递有效的字符串资源ID。

int roll = RollDice(sidesChosen);
...
tv1.setText(roll);

执行setText(String.valueOf(roll)将其转换为字符串

日志后编辑：

您的错误原因是您将0传递给nextInt()方法。这可能是因为您从未选择过RadioButton（默认情况下sidesChosen为0），或者即使选择了RadioButton，也没有输入case的{{1}}。

第二次登录后编辑:( ...）

删除switch这用于将布局附加到活动。你为什么在这里使用它？

Answer 2

似乎您将0传递给了Random.nextInt()方法，这是不允许的。

java.lang.IllegalArgumentException: n <= 0: 0
              at java.util.Random.nextInt(Random.java:182)

如果没有达到case分支，则会发生这种情况。所以看来，处理单选按钮点击的方式有问题。

根据边数滚动骰子

2 个答案: