if文がよく分かりません。ボタンが３つありひとつずつ押すとメッセージを表示させたいです。

前提・実現したいこと

if文がよく分かりません。ボタンが３つありひとつずつ押すとメッセージを表示させたいです
ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

実行されない

該当のソースコード

class MainActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)
    }

    fun showMessage(view: View) {
        if (val button1 = findViewById<Button>(view.id)){
       Toast.makeText(this, "1が押されました", Toast.LENGTH_LONG).show()}
        else (val button2 = findViewById<Button>(view.id)){
            Toast.makeText(this, "2が押されました", Toast.LENGTH_LONG).show()}
        else (val button3 = findViewById<Button>(view.id)){
            Toast.makeText(this, "3が押されました", Toast.LENGTH_LONG).show()}
    }
    
}


<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity">

    <TextView
        android:id="@+id/textView"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="ボタンを押してね！"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintHorizontal_bias="0.498"
        app:layout_constraintLeft_toLeftOf="parent"
        app:layout_constraintRight_toRightOf="parent"
        app:layout_constraintTop_toTopOf="parent" />

    <Button
        android:id="@+id/button2"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:onClick="showMessage"
        android:text="Button2"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/textView" />

    <Button
        android:id="@+id/button1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:onClick="showMessage"
        android:text="Button1"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toStartOf="@+id/button2"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/textView"
        app:layout_constraintVertical_bias="0.503" />

    <Button
        android:id="@+id/button3"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:onClick="showMessage"
        android:text="Button3"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toEndOf="@+id/button2"
        app:layout_constraintTop_toBottomOf="@+id/textView" />

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

K_3578

2021/04/28 08:47

ソースコードはMarkDownの<code>ブロック内に貼り付けてください。 ```言語名ソースコード ``` 上記のようにしてみてください。 JavaなのかKotlinなのかはっきりしましょう。

hoshi-takanori

2021/04/28 08:54

layout xml ファイルがないのでよく分かりませんが、if 文の条件は view.id == R.id.button1 とかでは。

hoshi-takanori

2021/04/28 09:07

android:onClick="showMessage2" や android:onClick="showMessage3" ってことは、button 2 や 3 が押された場合はそれぞれ showMessage ではなく showMessage2 や showMessage3 というメソッドが呼ばれるはずなので、showMessage の中で if 文を使って場合わけする必要はありません。

tanakariiii

2021/04/28 09:10

fun showMessage(view: View) { if (view.id == R.id.button1){ Toast.makeText(this, "1が押されました", Toast.LENGTH_LONG).show()} else (view.id == R.id.button2){ Toast.makeText(this, "2が押されました", Toast.LENGTH_LONG).show()} else (view.id == R.id.button3){ Toast.makeText(this, "3が押されました", Toast.LENGTH_LONG).show()} } このelseからの条件はどうしたらいいですか？

行動規範の内容に同意します

回答1件

ベストアンサー

showMessage メソッドの中で場合分けするのであれば、次のように書くと良いでしょう。
else の後ろに別の条件を書くには else if を使います。
また、} は show() の後ろではなく、次の行に書くのが一般的です。

Kotlin
1    fun showMessage(view: View) {
2        if (view.id == R.id.button1) {
3            Toast.makeText(this, "1が押されました", Toast.LENGTH_LONG).show()
4        } else if (view.id == R.id.button2) {
5            Toast.makeText(this, "2が押されました", Toast.LENGTH_LONG).show()
6        } else if (view.id == R.id.button3) {
7            Toast.makeText(this, "3が押されました", Toast.LENGTH_LONG).show()
8        }
9    }

また、Kotlin では when (Java の switch に相当) を使って次のように書くこともできます。

Kotlin
1    fun showMessage(view: View) {
2        when (view.id) {
3            R.id.button1 -> {
4                Toast.makeText(this, "1が押されました", Toast.LENGTH_LONG).show()
5            }
6            R.id.button2 -> {
7                Toast.makeText(this, "2が押されました", Toast.LENGTH_LONG).show()
8            }
9            R.id.button3 -> {
10                Toast.makeText(this, "3が押されました", Toast.LENGTH_LONG).show()
11            }
12        }
13    }