2つあるEditTextの一方でも空の場合にメッセージを表示

Question

前提・実現したいこと

2つあるEditTextの一方でも空の場合にメッセージを表示させたいです。

発生している問題・エラーメッセージ

A.toString() == "" || B.toString() == ""
                -> Toast.makeText(this, "数値が入力されていません!!", Toast.LENGTH_SHORT).show()

上記部分に
1.Expecting'->'
2.Expecting an expression, is-condition or in-condition
のエラーが表示されており、解決できません。ご教授いただけますでしょうか。

該当のソースコード

class MainActivity : AppCompatActivity(), View.OnClickListener {

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)
        setContentView(R.layout.editor_layout)

        button1.setOnClickListener(this)
        button2.setOnClickListener(this)
        button3.setOnClickListener(this)
        button4.setOnClickListener(this)
    }

    override fun onClick(v: View?) {
 
        val intent = Intent(this, SecondActivity::class.java)

          //EditTextの文字列を取得し、数値に変換
          val A = editText1.text.toString().toDouble()
          val B = editText2.text.toString().toDouble()

         when(v?.id){
             R.id.button1 -> intent.putExtra("VALUE", (A + B))
             R.id.button2 -> intent.putExtra("VALUE", (A - B))
             R.id.button3 -> intent.putExtra("VALUE", (A * B))
             R.id.button4 -> intent.putExtra("VALUE", (A / B))

            else
                this.A.toString() == "" || this.B.toString() == ""
                 -> Toast.makeText(this, "数値が入力されていません!!", Toast.LENGTH_SHORT).show()

        }

        startActivity(intent)
    }
}

Comments

[AtsushiYonenaga]

ソースコードはMarkdownの機能を使って、インデントが反映されるようにしてください。

Answer 1

エラーの原因？

else節はどの条件にも当てはまらなかった場合に通る部分だと思うので、elseの後に条件を書くことはできないのではないですか？
該当部分のelseが不要なのではないでしょうか。

else
    A.toString() == "" || B.toString() == ""
        -> Toast.makeText(this, "数値が入力されていません!!", Toast.LENGTH_SHORT).show()

その前に、ロジックは正しい？

when文を見ると、クリックされたボタンが１〜４だった場合と、空白チェックが同じwhen文の中に入ってしまっています。

whenは複数の処理から１つを選んで実行するものなので、この書き方だとクリックされたものがボタン4つのうちのどれかであれば空白チェック部分は通らない、と言う意図に見えます。

おそらく、「クリックされたボタンがどれであっても空白チェックは行う ＝ when文の外側に空白チェックがある」か「特定のボタンがクリックされた場合のみ空白チェックを行う = when内のいずれかの節の中に空白チェックがある」が、意図した仕様なのではないでしょうか？

Comments

[nep8810]

ご指摘いただきありがとうございます。
先程のエラーは解消されましたが、今度は
incompatible type:Boolean and Int?というエラーが発生してしまいました。
記述の仕方が間違っているのでしょうか？

[AtsushiYonenaga]

文法エラーですね。

おそらく条件式を入れるべきところに、評価したい変数を直接入れてしまうなどのミスをしてしまっていると思います。

正確な回答が必要な場合は、新たに質問するなどして修正後のソースコードを再掲するのが良いでしょう。

[AtsushiYonenaga]

ベストアンサーありがとうございました m(_ _)m

[nep8810]

解決しました。ありがとうございます。