Q&A
1. そもそもどういう状況でダイアログを表示するshowAlertが呼ばれるのでしょうか？
2. コードにおいてandroid.util.Logをimportして，onClickの最初に
Log.d("Click", "Clicked);
(引数は適当に指定していい)を入れて，問題の状況でボタンをクリックしてLogcatにログが出力されるかを確認してください．
アプリ開発初心者です。
現在私は、様々なサイトを参考にしながら計算問題アプリを
作成しているのですが、上手くいきません。
以下のコードだと、ダイアログで表示されるOK,Cancelをタップすると次から値を入力することができない仕様になってしまっています。
OK,Cancelをタップして呼び出すメソッドに
onCreate(Bundle savedInstanceState)を
指定すればできるかと思いましたが、ダメでした。
何か解決策や、もしくはヒントを教えてください。
回答よろしくお願いします。
public class Keisan extends Activity implements View.OnClickListener { /** * Buttonの配列 */ Button mButton[]; /** * Idの配列 */ int mId[] = {R.id.btn0, R.id.btn1, R.id.btn2, R.id.btn3, R.id.btn4, R.id.btn5, R.id.btn6, R.id.btn7, R.id.btn8, R.id.btn9, R.id.clearbtn}; /** * キー */ private final int KEY_EQUAL = 12; private final int KEY_CLEAR = 14; /** * TextView */ private TextView mTextView; private TextView mAnswer; private TextView qa1; private TextView qa2; private int n; private int m; /** * 答えのリスト */ private String[] answers = new String[4]; /** * 前の処理 */ private int beforeStatus = 0; /** * 計算中の値 */ private ArrayList<String> calcArray; /** * 計算する時の配列 */ private ArrayList<String> signArray; @Override public void onCreate(Bundle savedInstanceState) { requestWindowFeature(Window.FEATURE_NO_TITLE); super.onCreate(savedInstanceState); setContentView(R.layout.prac); Question(); // 1. AlertDialog.Builder クラスのインスタンスを生成 AlertDialog.Builder builder = new AlertDialog.Builder(this);//// // 表示用TextView mTextView=(TextView) findViewById(R.id.text1); mAnswer=(TextView) findViewById(R.id.text2); // Button mButton=new Button[mId.length]; // Buttonの取り込みとイベントのはりつけ for( int i = 0; i<mId.length;i++) { // buttonを取り込む mButton[i] = (Button) findViewById(mId[i]); // buttonのイベント処理 mButton[i].setOnClickListener(this); } calcArray=new ArrayList<String>(); signArray=new ArrayList<String>(); } public void Question(){ qa1 = null; qa2 = null; Random r = new Random(); n = r.nextInt(9) + 1; Random t = new Random(); m = r.nextInt(9) + 1; qa1 = (TextView) findViewById(R.id.text3); qa2 = (TextView) findViewById(R.id.text4); qa1.setText(String.valueOf(n)); qa2.setText(String.valueOf(m)); // 表示用TextView mTextView=(TextView) findViewById(R.id.text1); mAnswer=(TextView) findViewById(R.id.text2); // Button mButton=new Button[mId.length]; // Buttonの取り込みとイベントのはりつけ for( int i = 0; i<mId.length;i++) { // buttonを取り込む mButton[i] = (Button) findViewById(mId[i]); // buttonのイベント処理 mButton[i].setOnClickListener(this); } calcArray=new ArrayList<String>(); signArray=new ArrayList<String>(); } @Override public void onClick(View view) { // 押されたボタンがどのボタンかを判定 for (int i = 0; i < mId.length; i++) { if (view.equals(mButton[i])) { String nowValue = mTextView.getText().toString(); // CLEAR if (i == KEY_CLEAR) { mTextView.setText(""); calcArray.clear(); signArray.clear(); beforeStatus = KEY_CLEAR; } else if (i < 10) { nowValue = checkDisplay(nowValue); // 0しか入力されていない場合は0が２個以上並ばないようにする if (nowValue.equals("0") && i == 0) { return; } else if (nowValue.equals("0") && i != 0) { nowValue = ""; } nowValue = nowValue + i; mTextView.setText(nowValue); beforeStatus = i; } break; } } } private String checkDisplay(String now) { if (beforeStatus == KEY_EQUAL) { mTextView.setText(""); return "0"; } return now; } private double calc() { if (calcArray.size() == 0) { return 0.0; } if (calcArray.size() == 1) { return Double.parseDouble(calcArray.get(0)); } double passive = Double.parseDouble(calcArray.get(0)); double active; int j = 0; for (int i = 1; i < calcArray.size(); i++) { active = Double.parseDouble(calcArray.get(i)); if (signArray.get(j).equals("+")) { passive += active; } else { passive -= active; } j++; } return passive; } private void showAlert(double ans) { // 表示用TextView int total = 0; total += n; total += m; String title = getString(R.string.title); //インスタンス作成 AlertDialog.Builder builder = new AlertDialog.Builder(this); builder.setTitle(title); builder.setMessage(String.format(getString(R.string.msg), n, m, total)); builder.setPositiveButton("OK", new DialogInterface.OnClickListener() { public void onClick(DialogInterface dialog, int which) { Question(); } }); builder.setNegativeButton("Cancel", new DialogInterface.OnClickListener() { public void onClick(DialogInterface dialog, int which) { Question(); } }); builder.create().show(); } }
参考にしたコード
https://github.com/gcluelabs/DentakuQuiz001/blob/master/src/com/example/dentaku/Dentaku.java
KEY_EQUALボタンがタップされた時点でshowAlertが呼ばれてるのだと思います。
double ans等は引用してきたコードで使われていたものが残っていて消し忘れていました。
android.util.Logをインポートして確認したところ、D/Click﹕ Clickedというログが出力されました。
回答3件
0
ベストアンサー
見ただけではよくわかりませんが、onClick()の中でボタンのオブジェクトと
比較しているのが臭いと思います。
どのボタンがクリックされたのかは、view.getId()でIdを取得できるので、
if (view.equals(mButton[i])) {
を
if (mId[i] == view.getId()) {
としてみてはどうでしょうか？
投稿2015/10/20 08:18
総合スコア3579
回答ありがとうございます。
試してみましたが変化ありませんでした。
showAlert()がどこで呼び出されているのか分からないので
なんとも言えませんが、おそらく
mButton[i].setOnClickListener(this);
を、Question()の中でやってるのが原因だと思います。
Question()はAlertDialogでも呼び出してますよね。
つまり、
mButton[i].setOnClickListener(this);
が、何回も呼び出されているのが原因ではないかと。推測ですが、リスナー処理の
中からshowAlert()が呼び出されるような処理になっていないですか？
つまり、リスナー処理の中でリスナーの設定をしてしまってるんじゃないかと
思われます。（すべてのコードがないのでなんとも言えません）
もしそうなら、リスナー処理中にリスナーの処理関数の設定を変えてしまってるのでおかしくなってるのではないか？ということです。
基本的には、リスナーのセットはonCreate()の処理の中で1回にすべきです。
Question()の中での再設定は不要です。
できれば、
public View.OnClickListener mOnClickListener = new View.OnClickListener() {
@Override
public void onClick(View v) {
// ここにボタンのクリック処理を書く
・・・・
}
};
と、リスナーを定義しておいて
mButton[i].setOnClickListener(mOnClickListener);
と、リスナーを設定するほうがわかりやすいです。
コード全文追記致しました。
すいません。
PineMatsu様の方法で書き直してみましたが、
public class PurasuAct extends Activity implements View.Listenerで
赤い波線が表示されてしまいました。
>public class PurasuAct extends Activity implements View.Listenerで
>赤い波線が表示されてしまいました。
ActivityにView.OnClickListenerがimplementsされていたんですね。
すいませんでした。
で、ほぼ同じようなアプリを試しに作成し、OKボタンを押したらshowAlert()を
呼び出す様にして、ダイアログを出すようにしてみましたが、投稿にあるような現象は
発生しません。OK,Cancelをクリックした後も、ボタンを押して値を入力することができます。（確認はエミュレータで行いました。）
もしかするとAndroid SDKのバージョンかもしれません。作成するときのSDKバージョンは何を指定しましたか？ちなみに、私は4.1(API 16)を指定して作成しました。
appのbuild.gradleでdefaultConfig.minSdkVersionをみればわかります。
Androidの1～3で作成したことはないですが、もし古いバージョンで作成していたら
こういうこともあるのかもしれません。
ターゲットのデバイスが何かわかりませんがよほど古いデバイスでない限り、今はAndroid4.0以上で動いていると思います。
すいません。
OKボタンを押したらshowAlert()が呼び出されるようにしたというのは、
入力が終わって、ダイアログを呼び出すためのボタンのことですか？
それとも、ダイアログに表示されるOKボタンのことでしょうか？
public void onClick(DialogInterface dialog, int which) {
内にshowAlert()を記入したらダイアログ内のOK,Cancelボタンタップ時に同じ画面が出力される状態になりました。
いやダイアログのOKボタンではないです。それだと自分自身を呼び出してしまうことになります。（OKやCancelを押す度にメモリを使いきってメモリ不足で落ちてしまうでしょう）
そもそも、投稿されたソースにはダイアログを表示するshowAlert()を呼び出しているところがなかったので、それを呼び出すためのボタンを追加して、そのボタンをクリックしたらshowAlert()を呼び出すようにしただけです。
投稿にはレイアウトのXMLファイルがなかったので、適当にソースを見ながら貼り付けました。(KEY_CLEAR=14となってますが、mId[]のR.Id.clearbtnはインデックスが11とか、ちょっと細かいところで？？というのがありましたが、質問の本質とは関係ないので適当にソースを作りました）
すいません。
申し訳ありませんがPineMatsu様が書いたソースを載せていただきたいのですが、
よろしいでしょうか？
分かりました。どうせなので、Android Studioのプロジェクトでお渡しします。著作権は放棄しますのでご自由に。（笑）
https://free.filesend.to/filedn_infoindex?rp=82be19e904e71a7d9349ebdf3d737f4o
からダウンロードして下さい。
2015年10月24日 18時51分以降自動で消去されます。
なお、環境はWindows7でAndroid Studioは1.4です。もしかしたら、Android SDKのアップデートをしないとコンパイルできないかもしれません。(ActivityではなくAppComppatActivityに変わっています)
ほとんどは、mokumokuさんのソースそのままですが、不要と思われるものは削除しています。
お手数おかけして申し訳ございません。
ありがとうございました。
0
ちょっとコード読んでるだけだとわかりにくいので，onClickの条件分岐のブロックに適宜Logメソッドを置いてログを取ってみてはいかがでしょうか？
onClickは確実に通っているはずなので，その後どういう経路を通っているのかを確認したほうがいいです．
それでも分からなければデバッグ機能で1行ずつステップ実行していって，変数の中身も確認しつつ見ていったほうがいいと思います．
投稿2015/10/20 07:36
総合スコア20651
あなたの回答
tips
プレビュー
