[Android] Button のonClickListenerの設定が色々できる件

ButtonのsetOnClickListenerですが、匿名クラスを使ったりと色々やり方があるようです。それぞれ良し悪しが個々の状況によってあると思います。すぐに使えるように確認して見ました。

Android Studio 2.3
Android 7.1.1

Button onClickListner

　
このサイトでのButton Clickのリスナーは匿名クラスを多用していますが、他にも方法があります。規模が大きなprojectになれば使い方も変えた方がメンテナンスしやすくなります。

匿名クラス
変数でまとめる
OnClickListener を implementsして onClick を定義
カスタムクラスを作成
android:onClick

匿名クラス

　
GoogleのReferenceに紹介されているやり方で、メソッド内でクラスを定義する匿名（無名）クラスを使います。ボタンが１、２個であればこれで済ませることが多いですね。
MainActivity.java

import android.app.Activity;
import android.os.Bundle;
import android.util.Log;
import android.view.View;
import android.widget.Button;

public class MainActivity extends Activity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        setContentView(R.layout.activity_main);

        final Button button1 = (Button) findViewById(R.id.button_1);
        button1.setOnClickListener(new View.OnClickListener() {
            public void onClick(View view) {
                Log.d("debug", "button1, Perform action on click");
            }
        });

        final Button button2 = (Button) findViewById(R.id.button_2);
        button2.setOnClickListener(new View.OnClickListener() {
            public void onClick(View view) {
                Log.d("debug","button2, Perform action on click");
            }
        });

    }
}

import android.app.Activity;

import android.os.Bundle;

import android.util.Log;

import android.view.View;

import android.widget.Button;

public class MainActivity extends Activity {

@Override

protected void onCreate(Bundle savedInstanceState) {

super.onCreate(savedInstanceState);

setContentView(R.layout.activity_main);

final Button button1 = (Button) findViewById(R.id.button_1);

button1.setOnClickListener(new View.OnClickListener() {

public void onClick(View view) {

Log.d("debug", "button1, Perform action on click");

}

});

final Button button2 = (Button) findViewById(R.id.button_2);

button2.setOnClickListener(new View.OnClickListener() {

public void onClick(View view) {

Log.d("debug","button2, Perform action on click");

}

});

}

　
activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical"
    android:gravity="center"
    tools:context=".MainActivity">

    <Button
        android:id="@+id/button_1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="button 1" />

    <Button
        android:id="@+id/button_2"
        android:layout_margin="10dp"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="button 2" />

</LinearLayout>

<?xml version="1.0" encoding="utf-8"?>

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"

xmlns:tools="http://schemas.android.com/tools"

android:layout_width="match_parent"

android:layout_height="match_parent"

android:orientation="vertical"

android:gravity="center"

tools:context=".MainActivity">

<Button

android:id="@+id/button_1"

android:layout_width="wrap_content"

android:layout_height="wrap_content"

android:text="button 1" />

<Button

android:id="@+id/button_2"

android:layout_margin="10dp"

android:layout_width="wrap_content"

android:layout_height="wrap_content"

android:text="button 2" />

</LinearLayout>

変数でまとめる

　
正直なところこの方法はこれが１文なので長ったらしいしごちゃごちゃする、最後の；を忘れることが多いですね。

button1.setOnClickListener(new View.OnClickListener() {
    public void onClick(View view) {
    　　　　Log.d("debug", "button1, Perform action on click");
    }
 });

button1.setOnClickListener(new View.OnClickListener() {

public void onClick(View view) {

　　　　Log.d("debug", "button1, Perform action on click");

}

});

単純に見やすくするには、これを変数に置き換えてしまうやり方があります
MainActivity.java

import android.app.Activity;
import android.os.Bundle;
import android.util.Log;
import android.view.View;
import android.widget.Button;


public class MainActivity extends Activity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        setContentView(R.layout.activity_main);

        final Button button1 = (Button) findViewById(R.id.button_1);
        button1.setOnClickListener(buttonClick);

        final Button button2 = (Button) findViewById(R.id.button_2);
        button2.setOnClickListener(buttonClick);

    }

    private View.OnClickListener buttonClick = new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            switch (view.getId()) {
                case R.id.button_1:
                    Log.d("debug","button1, Perform action on click");
                    break;
                case R.id.button_2:
                    Log.d("debug","button2, Perform action on click");
                    break;
            }
        }
    };
}

import android.app.Activity;

import android.os.Bundle;

import android.util.Log;

import android.view.View;

import android.widget.Button;

public class MainActivity extends Activity {

@Override

protected void onCreate(Bundle savedInstanceState) {

super.onCreate(savedInstanceState);

setContentView(R.layout.activity_main);

final Button button1 = (Button) findViewById(R.id.button_1);

button1.setOnClickListener(buttonClick);

final Button button2 = (Button) findViewById(R.id.button_2);

button2.setOnClickListener(buttonClick);

}

private View.OnClickListener buttonClick = new View.OnClickListener() {

@Override

public void onClick(View view) {

switch (view.getId()) {

case R.id.button_1:

Log.d("debug","button1, Perform action on click");

break;

case R.id.button_2:

Log.d("debug","button2, Perform action on click");

break;

}

};

}

　
レイアウトは同じです。
　

OnClickListener を implementsして onClick を定義

　
匿名クラスとしない場合はどうするかというと、OnClickListenerをimplementsします。するとpublic void onClick(View view){}というメソッドを作るようにように要求されます。

MainActivity.java

import android.app.Activity;
import android.os.Bundle;
import android.util.Log;
import android.view.View;
import android.widget.Button;

public class MainActivity extends Activity implements View.OnClickListener {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        setContentView(R.layout.activity_main);

        final Button button1 = (Button)findViewById(R.id.button_1);
        button1.setOnClickListener(this);

        final Button button2 = (Button)findViewById(R.id.button_2);
        button2.setOnClickListener(this);
    }

    public void onClick(View view){
        switch (view.getId()) {
            case R.id.button_1:
                Log.d("debug","button1, Perform action on click");
                break;
            case R.id.button_2:
                Log.d("debug","button2, Perform action on click");
                break;
        }
    }
}

import android.app.Activity;

import android.os.Bundle;

import android.util.Log;

import android.view.View;

import android.widget.Button;

public class MainActivity extends Activity implements View.OnClickListener {

@Override

protected void onCreate(Bundle savedInstanceState) {

super.onCreate(savedInstanceState);

setContentView(R.layout.activity_main);

final Button button1 = (Button)findViewById(R.id.button_1);

button1.setOnClickListener(this);

final Button button2 = (Button)findViewById(R.id.button_2);

button2.setOnClickListener(this);

}

public void onClick(View view){

switch (view.getId()) {

case R.id.button_1:

Log.d("debug","button1, Perform action on click");

break;

case R.id.button_2:

Log.d("debug","button2, Perform action on click");

break;

}

activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical"
    android:gravity="center"
    tools:context=".MainActivity">

    <Button
        android:id="@+id/button_1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="button 1" />

    <Button
        android:id="@+id/button_2"
        android:layout_margin="10dp"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="button 2" />

</LinearLayout>

<?xml version="1.0" encoding="utf-8"?>

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"

xmlns:tools="http://schemas.android.com/tools"

android:layout_width="match_parent"

android:layout_height="match_parent"

android:orientation="vertical"

android:gravity="center"

tools:context=".MainActivity">

<Button

android:id="@+id/button_1"

android:layout_width="wrap_content"

android:layout_height="wrap_content"

android:text="button 1" />

<Button

android:id="@+id/button_2"

android:layout_margin="10dp"

android:layout_width="wrap_content"

android:layout_height="wrap_content"

android:text="button 2" />

</LinearLayout>

　
ボタンクリックでの処理を１か所にまとめられるので可読性が上がるかもしれません。
　

カスタムクラスを作成

　
メソッドでクリックを受けていますが、カスタムクラスにしてしまうこともできます。その場合は、OnClickListenerを継承したクラスとします。

MainActivity.java

import android.app.Activity;
import android.os.Bundle;
import android.util.Log;
import android.view.View;
import android.widget.Button;

public class MainActivity extends Activity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        setContentView(R.layout.activity_main);

        final Button button1 = (Button) findViewById(R.id.button_1);
        button1.setOnClickListener(new buttonClick());

        final Button button2 = (Button) findViewById(R.id.button_2);
        button2.setOnClickListener(new buttonClick());

    }
}

class buttonClick implements View.OnClickListener {
    @Override
    public void onClick(View view) {
        switch (view.getId()) {
            case R.id.button_1:
                Log.d("debug","button1, Perform action on click");
                break;
            case R.id.button_2:
                Log.d("debug","button2, Perform action on click");
                break;
        }    }
}

import android.app.Activity;

import android.os.Bundle;

import android.util.Log;

import android.view.View;

import android.widget.Button;

public class MainActivity extends Activity {

@Override

protected void onCreate(Bundle savedInstanceState) {

super.onCreate(savedInstanceState);

setContentView(R.layout.activity_main);

final Button button1 = (Button) findViewById(R.id.button_1);

button1.setOnClickListener(new buttonClick());

final Button button2 = (Button) findViewById(R.id.button_2);

button2.setOnClickListener(new buttonClick());

}

class buttonClick implements View.OnClickListener {

@Override

public void onClick(View view) {

switch (view.getId()) {

case R.id.button_1:

Log.d("debug","button1, Perform action on click");

break;

case R.id.button_2:

Log.d("debug","button2, Perform action on click");

break;

} }

}

activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical"
    android:gravity="center"
    tools:context=".MainActivity">

    <Button
        android:id="@+id/button_1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="button 1" />

    <Button
        android:id="@+id/button_2"
        android:layout_margin="10dp"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="button 2" />

</LinearLayout>

<?xml version="1.0" encoding="utf-8"?>

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"

xmlns:tools="http://schemas.android.com/tools"

android:layout_width="match_parent"

android:layout_height="match_parent"

android:orientation="vertical"

android:gravity="center"

tools:context=".MainActivity">

<Button

android:id="@+id/button_1"

android:layout_width="wrap_content"

android:layout_height="wrap_content"

android:text="button 1" />

<Button

android:id="@+id/button_2"

android:layout_margin="10dp"

android:layout_width="wrap_content"

android:layout_height="wrap_content"

android:text="button 2" />

</LinearLayout>

android:onClick

　
XMLレイアウトの中でandroid:onClickを定義してメソッドを実行させるケースです。これもGoogleのReferenceで紹介されています。
　

import android.app.Activity;
import android.os.Bundle;
import android.util.Log;
import android.view.View;

public class MainActivity extends Activity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        setContentView(R.layout.activity_main);
    }

    public void selfDestruct(View view) {
        switch (view.getId()) {
            case R.id.button_1:
                Log.d("debug","button1, Perform action on click");
                break;
            case R.id.button_2:
                Log.d("debug","button2, Perform action on click");
                break;
        }
    }
}

import android.app.Activity;

import android.os.Bundle;

import android.util.Log;

import android.view.View;

public class MainActivity extends Activity {

@Override

protected void onCreate(Bundle savedInstanceState) {

super.onCreate(savedInstanceState);

setContentView(R.layout.activity_main);

}

public void selfDestruct(View view) {

switch (view.getId()) {

case R.id.button_1:

Log.d("debug","button1, Perform action on click");

break;

case R.id.button_2:

Log.d("debug","button2, Perform action on click");

break;

}

activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical"
    android:gravity="center"
    tools:context=".MainActivity">

    <Button
        android:id="@+id/button_1"
        android:onClick="selfDestruct"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="button 1" />

    <Button
        android:id="@+id/button_2"
        android:onClick="selfDestruct"
        android:layout_margin="10dp"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="button 2" />

</LinearLayout>

<?xml version="1.0" encoding="utf-8"?>

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"

xmlns:tools="http://schemas.android.com/tools"

android:layout_width="match_parent"

android:layout_height="match_parent"

android:orientation="vertical"

android:gravity="center"

tools:context=".MainActivity">

<Button

android:id="@+id/button_1"

android:onClick="selfDestruct"

android:layout_width="wrap_content"

android:layout_height="wrap_content"

android:text="button 1" />

<Button

android:id="@+id/button_2"

android:onClick="selfDestruct"

android:layout_margin="10dp"

android:layout_width="wrap_content"

android:layout_height="wrap_content"

android:text="button 2" />

</LinearLayout>

　
ActivityのコードにはButtonのインスタンスもありません、これが気持ち悪いと感じるかどうかです。Fragmentではうまくつかえるのでしょうか…
　

5種類のButton Click

　
この５種類をテストするコードを書いてみました。
MainActivity.java

import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.TextView;

public class MainActivity extends AppCompatActivity implements View.OnClickListener {

    private TextView textView;
    private boolean flag = true;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        textView = (TextView)findViewById(R.id.text_view);

        // 匿名クラス
        final Button button0 = (Button) findViewById(R.id.button_0);
        button0.setOnClickListener(new View.OnClickListener() {
            public void onClick(View view) {
                if (flag) {
                    textView.setText("Type0: Hello");
                    flag = false;
                }
                else {
                    textView.setText("Type0: World");
                    flag = true;
                }
            }
        });
        // 変数でまとめる
        final Button button1 = (Button) findViewById(R.id.button_1);
        button1.setOnClickListener(buttonClick);

        // OnClickListener を implementsして onClick を定義
        final Button button2 = (Button)findViewById(R.id.button_2);
        button2.setOnClickListener(this);

        // カスタムクラス
        final Button button3 = (Button) findViewById(R.id.button_3);
        button3.setOnClickListener(new buttonClick());

    }


    // 変数でまとめる
    private View.OnClickListener buttonClick = new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            switch (view.getId()) {
                case R.id.button_1:
                    if (flag) {
                        textView.setText("Type1: Hello");
                        flag = false;
                    }
                    else {
                        textView.setText("Type1: World");
                        flag = true;
                    }
                    break;
            }
        }
    };

    // OnClickListener を implementsして onClick を定義
    public void onClick(View view){
        switch (view.getId()) {
            case R.id.button_2:
                if (flag) {
                    textView.setText("Type2: Hello");
                    flag = false;
                }
                else {
                    textView.setText("Type2: World");
                    flag = true;
                }
                break;
        }
    }

    // カスタムクラス
    class buttonClick implements View.OnClickListener {
        @Override
        public void onClick(View view) {
            switch (view.getId()) {
                case R.id.button_3:
                    if (flag) {
                        textView.setText("Type3: Hello");
                        flag = false;
                    } else {
                        textView.setText("Type3: World");
                        flag = true;
                    }
                    break;
            }
        }
    }

    // android:onClick
    public void selfDestruct(View view) {
        switch (view.getId()) {
            case R.id.button_4:
                if (flag) {
                    textView.setText("Type4: Hello");
                    flag = false;
                } else {
                    textView.setText("Type4: World");
                    flag = true;
                }
                break;
        }
    }
}

100

101

102

103

104

105

106

107

108

109

110

111

112

113

114

115

import android.support.v7.app.AppCompatActivity;

import android.os.Bundle;

import android.view.View;

import android.widget.Button;

import android.widget.TextView;

public class MainActivity extends AppCompatActivity implements View.OnClickListener {

private TextView textView;

private boolean flag = true;

@Override

protected void onCreate(Bundle savedInstanceState) {

super.onCreate(savedInstanceState);

setContentView(R.layout.activity_main);

textView = (TextView)findViewById(R.id.text_view);

// 匿名クラス

final Button button0 = (Button) findViewById(R.id.button_0);

button0.setOnClickListener(new View.OnClickListener() {

public void onClick(View view) {

if (flag) {

textView.setText("Type0: Hello");

flag = false;

}

else {

textView.setText("Type0: World");

flag = true;

}

});

// 変数でまとめる

final Button button1 = (Button) findViewById(R.id.button_1);

button1.setOnClickListener(buttonClick);

// OnClickListener を implementsして onClick を定義

final Button button2 = (Button)findViewById(R.id.button_2);

button2.setOnClickListener(this);

// カスタムクラス

final Button button3 = (Button) findViewById(R.id.button_3);

button3.setOnClickListener(new buttonClick());

}

// 変数でまとめる

private View.OnClickListener buttonClick = new View.OnClickListener() {

@Override

public void onClick(View view) {

switch (view.getId()) {

case R.id.button_1:

if (flag) {

textView.setText("Type1: Hello");

flag = false;

}

else {

textView.setText("Type1: World");

flag = true;

}

break;

}

};

// OnClickListener を implementsして onClick を定義

public void onClick(View view){

switch (view.getId()) {

case R.id.button_2:

if (flag) {

textView.setText("Type2: Hello");

flag = false;

}

else {

textView.setText("Type2: World");

flag = true;

}

break;

}

// カスタムクラス

class buttonClick implements View.OnClickListener {

@Override

public void onClick(View view) {

switch (view.getId()) {

case R.id.button_3:

if (flag) {

textView.setText("Type3: Hello");

flag = false;

} else {

textView.setText("Type3: World");

flag = true;

}

break;

}

// android:onClick

public void selfDestruct(View view) {

switch (view.getId()) {

case R.id.button_4:

if (flag) {

textView.setText("Type4: Hello");

flag = false;

} else {

textView.setText("Type4: World");

flag = true;

}

break;

}

　
activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical"
    android:gravity="center"
    android:padding="50dp"
    tools:context=".MainActivity">

    <TextView
        android:id="@+id/text_view"
        android:text="Text Button Click"
        android:textColor="#000"
        android:textSize="30sp"
        android:layout_margin="10dp"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content" />

    <Button
        android:id="@+id/button_0"
        android:layout_margin="10dp"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:text="button 1" />

    <Button
        android:id="@+id/button_1"
        android:layout_margin="10dp"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:text="button 1" />

    <Button
        android:id="@+id/button_2"
        android:layout_margin="10dp"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:text="button 2" />

    <Button
        android:id="@+id/button_3"
        android:layout_margin="10dp"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:text="button_3" />

    <Button
        android:id="@+id/button_4"
        android:onClick="selfDestruct"
        android:layout_margin="10dp"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:text="button_4" />
</LinearLayout>