Android StudioでNumberPickerが動きません。

Question

以前に質問させていただいたアプリの強制終了は解決したのですが、今度はNumberPickerが全く動かなくなりました。実現したい動作は３けたの数字をNumberPickerで選択して「OK」ボタンを押すと数字が表示されてその後に「次へ」を押すと次のアクティビティへ移るというものです。次のアクティビティへ移ることはできるのですがNumberPickerが全く反応せず、０が表示されるだけという状態になっています。ログを見てもエラーが見当たらず困っています。どなたか教えてください。
以下がコードです。

コードpackage com.example.soja.page1;

import androidx.appcompat.app.AppCompatActivity;

import android.content.Intent;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.NumberPicker;
import android.widget.TextView;

public class nextpage2Activity extends AppCompatActivity {
    private NumberPicker numPicker0, numPicker1, numPicker2;
    private TextView pickerTextView;

    private String[] figures = new String[2];

    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_nextpage2);

        numPicker0 = findViewById(R.id.numPicker0);
        numPicker1 = findViewById(R.id.numPicker1);
        numPicker2 = findViewById(R.id.numPicker2);

        Button pickerButton1 = findViewById(R.id.button1);

        numPicker0.setMaxValue(9);
        numPicker0.setMinValue(0);

        numPicker1.setMaxValue(9);
        numPicker1.setMinValue(0);

        numPicker2.setMaxValue(9);
        numPicker2.setMinValue(0);


        pickerButton1.setOnClickListener(new View.OnClickListener() {
            public void onClick(View v) {
                figures[0] = String.valueOf(numPicker0.getValue());
                figures[1] = String.valueOf(numPicker1.getValue());
                figures[2] = String.valueOf(numPicker2.getValue());

                String str = String.format("%s%s%s%", figures[0], figures[1], figures[2]);

                pickerTextView.setText(Integer.valueOf(str));
            }
        });
    


        Button button = findViewById(R.id.button4);

        //ボタンが押された時の処理//
        button.setOnClickListener(new View.OnClickListener() {
            public void onClick(View view) {
                //インテントの作成//
                Intent intent = new Intent(nextpage2Activity.this, nextpage3.class);
                startActivity(intent);
            }
        });
    }
}

Answer 1

OnCreate は呼ばれていないようですが？

Comments

[souja]

ご回答ありがとうございます。具体的にはどこを直した方がいいのでしょうか？

[jimbe]

まず, なぜ onCreate と OnCreate があるのでしょうか.

[souja]

すみません、OnCreateはミスでした。

[souja]

onCreateにに直したところ、onCreate(Bundle) is already defined in 'com.example.soja~'というエラーがでてしまいます。これの対処法がわからなくて困っています。

[jimbe]

でしたら, OnCreate の機能内容を onCreate に入れて, OnCreate は(紛らわしいので)削除すると良いかと思います.

ご質問に, 修正したコードとエラー全文を追記して頂けますか.
その際は, 入力欄上部の <code> を押すと出てくる, ``` で挟まれた 「コード」を置き換える形にすると, 専用の枠内にコードが表示されるようになりますので, ご利用頂くと見易くなって助かります.

[souja]

謝って自己解決の方に書いてしまいましたが上の質問も編集しました。

[jimbe]

同じメソッドが2つ以上あればエラーになるのは java (に限りませんが) では当然です.
落ち着いて, 一つに纏めてください.

[souja]

なるほど！エラーが消えました！ありがとうございます！
ですが、NumberPickerがまだ動かないです、、、

[jimbe]

修正されたコードをご提示願えますか.

[souja]

すいません。一番上の質問の部分を更新して修正したコードをのせています。

[souja]

たびたびすみません。NumberPickerは動くようになったのですが、その後の表示ボタンを押してもNumberPickerの数字が表示欄に表示されず、アプリが停止してしまいます。

[souja]

以下がエラーです。
E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.example.soja.page1, PID: 26476
java.util.UnknownFormatConversionException: Conversion = 'End of String'
at java.util.Formatter$FormatSpecifierParser.peek(Formatter.java:2641)
at java.util.Formatter$FormatSpecifierParser.<init>(Formatter.java:2602)
at java.util.Formatter.parse(Formatter.java:2557)
at java.util.Formatter.format(Formatter.java:2504)
at java.util.Formatter.format(Formatter.java:2458)
at java.lang.String.format(String.java:2770)
at com.example.soja.page1.nextpage2Activity$1.onClick(nextpageActivity.java:47)
at android.view.View.performClick(View.java:6294)
at android.view.View$PerformClick.run(View.java:24770)
at android.os.Handler.handleCallback(Handler.java:790)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:164)
at android.app.ActivityThread.main(ActivityThread.java:6494)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)
Application terminated.E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.example.soja.page1, PID: 26476
java.util.UnknownFormatConversionException: Conversion = 'End of String'
at java.util.Formatter$FormatSpecifierParser.peek(Formatter.java:2641)
at java.util.Formatter$FormatSpecifierParser.<init>(Formatter.java:2602)
at java.util.Formatter.parse(Formatter.java:2557)
at java.util.Formatter.format(Formatter.java:2504)
at java.util.Formatter.format(Formatter.java:2458)
at java.lang.String.format(String.java:2770)
at com.example.soja.page1.nextpageActivity$1.onClick(nextpageActivity.java:47)
at android.view.View.performClick(View.java:6294)
at android.view.View$PerformClick.run(View.java:24770)
at android.os.Handler.handleCallback(Handler.java:790)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:164)
at android.app.ActivityThread.main(ActivityThread.java:6494)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)
Application terminated.

[souja]

switch文などでidごとに分岐などをしたほうがいいのでしょうか？

[jimbe]

> 一番上の質問の部分を更新して修正したコードをのせています。
ご質問にあるコードは, @Override onCreate() が変な位置にありますし, {} の対応が取れていません.
不要になったコードは残さず消してください. 編集履歴は残りますので, 古いコードが必要になったら取り出せますので.

エラー表示されている中で発生個所と思われる部分は
> com.example.srkmmtky1260.page1.nextpage2Activity$1.onClick(nextpageActivity.java:47
ですが, パッケージ名がご提示のコードと違います. ということは, ご提示のコードは最新ではないということで, 問題を探すことができません.

こちらは, コードの枠の右上にある [+] を押して android studio にコピペして確認していますので, nextpage2Activity.java の全文をコード部分に上書きして頂けますか.

[souja]

・onCreateを消去しました。素人でそんな機能があるとは知らなかったので助かります。
・一番上の質問を編集しました。そこにnextpage2Activityのコードをのせました。また、エラーコードもそれに対応するものに編集しました。何度もすみません。よろしくおねがいします。

[jimbe]

ご提示のコード上で無名クラスの onClock 内で String.format を使っている所ですと, 44行目の
> String str = String.format("%s%s%s%", figures[0], figures[1], figures[2]);
になります.
format 文字列 "%s%s%s%" は最後の '%' は要らないのではないでしょうか.

[souja]

ご回答ありがとうございます。
ここの%は確かに不必要でした。凡ミスです。これを修正して実行してみたところまだエラーが消えません。よろしくお願いします。
以下がエラーコードです。
E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.example.soja.page1, PID: 29689
java.lang.ArrayIndexOutOfBoundsException: length=2; index=2
at com.example.soja.page1.nextpage2Activity$1.onClick(nextpage2Activity.java:44)
at android.view.View.performClick(View.java:6294)
at android.view.View$PerformClick.run(View.java:24770)
at android.os.Handler.handleCallback(Handler.java:790)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:164)
at android.app.ActivityThread.main(ActivityThread.java:6494)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)

[jimbe]

エラー(この場合は正確には"例外"ですが)をお読みになっていらっしゃいますか.
> java.lang.ArrayIndexOutOfBoundsException: length=2; index=2
> at com.example.soja.page1.nextpage2Activity$1.onClick(nextpage2Activity.java:44)
44 行目で配列のアクセスに index=2 が使用されていますが, 配列の長さが 2 なので領域外にアクセスしようとしている... と言われています.
> private String[] figures = new String[2];
figures は2要素しか用意していません.

[souja]

あまり読んでいませんでした。すみません。配列の長さは修正をしました。

[jimbe]

いきなりアルファベットの羅列が大量に表示されるとビックリして混乱してしまったりするのは分かりますが, 何が起きたのかを調べるには最も大切な情報です.
見覚えのあるパッケージ名やクラス・メソッド名があれば, そこで問題が発生したことが分かりますし行番号まで出ています.
全ての行を理解する必要は全くありません. 最初のほうにある ～Exception と, 発生個所さえ分かればいいはずですので, ぜひ挑戦して頂ければと思います.

[souja]

教えていただいてありがとうございます。ご指摘いただいた通り人任せにせずまずは自分で考えてからわからないところを投稿しようと思います。またご質問させていただくことがあると思いますのでよろしければまたご回答していただきたいです。

[souja]

エラーを見直して修正したところNumberPickerが動きました！ありがとうございました！

[jimbe]

おつかれさまでした.