メソッドはかけたけどそれをどのように実装すればいいかわからない

Question

Androidの計算問題アプリを作っていて自分で出題される問題の数字の範囲を変えることができるように設定画面を作っています。
Edit Textでどこからどこまでという範囲を入力できるようにしたのですが入力されていないときエラーメッセージを表示させたいためそのメソッドを書きました。しかしメソッドを書いただけで何も起こりません。
入力が完了したときにチェックをしたいのですがどうすれば良いでしょうか。

設定画面にはEdit Textが4つと設定が終わった後に問題画面に移行するためのボタンがありますonClickはそのボタンが押された時の処理です。

public class Setting extends AppCompatActivity {

public void　check(View view) {
        EditText editText1 = (EditText)findViewById(R.id.number1_1);
        String num1_1 = editText1.getText().toString();
        if (num1_1.isEmpty()) {
            editText1.setError("文字を入力してください");
        }
    }
public void onClick(View target) {
        //入力されたテキストを取得
        EditText editText1 = (EditText)findViewById(R.id.number1_1);
        String num1_1 = editText1.getText().toString();

        EditText editText2 = (EditText)findViewById(R.id.answer);
        String num1_2 = editText2.getText().toString();

        EditText editText3 = (EditText)findViewById(R.id.number2_1);
        String num2_1 = editText3.getText().toString();
        
        EditText editText4 = (EditText)findViewById(R.id.number2_2);
        String num2_2 = editText4.getText().toString();

        //int型に変換
        number[0]=Integer.parseInt(num1_1);
        number[1]=Integer.parseInt(num1_2);
        number[2]=Integer.parseInt(num2_1);
        number[3]=Integer.parseInt(num2_2);

        //次の画面に設定データを渡して画面移動
        Intent intent = new Intent(this,Question.class);
        for (int i=0;i<4;i++) {
            intent.putExtra("data_"+i,number[i]);
        }
        intent.putExtra("mark",mark_number);//mark_numberは足し算、引き算などの符号の設定を番号で表したものです
        startActivity(intent);
    }

}

Comments

[Zuishin]

呼び出していないのになぜ何かが起こると思ったのか謎です。
これをどこで呼び出すか答えよという宿題ですか？

[yuruyuru]

はい。
onClick内で使ってもデータが入っていないのは変わらないので画面が落ちてしまいます。
レベルが低いかもしれませんが宿題ではないです。（学校の課題なら自分でやれと思われる可能性があるので一応否定しておきました。不快に思われたのならすいません。）

[m.ts10806]

そもそもどのように使われるかというのがあっての「実装」なので、メソッドだけ書いて後はどうしたらいいかというのは手順として間違ってます。

[yuruyuru]

入力が完了したときにこのメソッドを使いたいです。
画面が移動する前にエラーメッセージを表示させてアプリを使っている人に修正させるようにしたです。
これがどのように使われるかということではないのでしょうか。。
このような機能を実装したいのですが難しいでしょうか

[hoshi-takanori]

ボタンが押された時に onClick を実行したいということなら、onCreate メソッドで setOnClickListener すれば良いでしょう。
あと、設定画面ということなら入力内容は次の画面に渡すだけでなく、SharedPreference などに保存したいところですし、入力欄が空だったらエラーではなくデフォルト値 (たぶん全範囲) にする方が自然な気がします。

[m.ts10806]

メソッドだけ書いて後はどうしたらいいかというのは手順として間違ってます。

>難しいでしょうか
基準が質問者さんなら「難しい」
そうでないなら「大抵のことはできる」です。
もっと基本的なところから見直したほうが良いのではと。ここで答えだけもらったとて身につきません。（それに設計どうなってるか分からないと他者には答えようがないところもあります）

[yuruyuru]

hoshi-takanoriさん
ありがとうございます。
試してみます。

[yuruyuru]

t.ts10806さん
すいません。自分の実力的に難しいかどうかを聞いたわけではありませんでした。
どのようにすればできるかという質問なのでみになるかどうかは関係ないです。
いろいろ考えた上でおしゃってくださっているのかもしれませんが本当にすいません。

[m.ts10806]

細かいこと言うとあくまで赤の他人としては「書けた」と「どうやって実装すれば」に矛盾を感じるという点です。
動作確認できてないものを「書けた」と言ってるのはおかしいと思います。実装できたなら「書けた」と言ってもいいでしょうけど。
こういう細かいことはけっこう大事です。回答者はあくまで赤の他人ですから。
メソッド単体でモックデータでも投入して動作確認されたのでしたらその旨も書かれないと他人には伝わりませんので。

Answer 1

質問依頼の欄でアドバイスがあったので自己解決ということにさせていただきます。

Comments

[m.ts10806]

どのようにして解決に至ったのか書かれないと「回答」としても「自己解決」としても不十分です。
下記参考に。
> https://teratail.com/help#resolve-myself
また後から読む人のためにも、具体的な解決手順を記載してください。

ちなみに、メソッドの使い方すら分からない状態では何もできないので、やはり基礎からやってください。
質問内容から感じたことそのままです。
howばかり追い求めても何もできません。
他人に作ってもらいたいわけじゃないですよね？

[m.ts10806]

メソッドだけ出されたところでコンポーネントや要素がどう配置されているか、イベントがどう設計、設定されているか分からないことには他人には答えようがないという意図のコメントもしています。設計が〜の部分。
メソッドだけコード出されても「使うように実装しては」くらいしか答えようがないのです。