フラグメント内のXML onClickを使用してボタンのクリックを処理する方法


440

Pre-Honeycomb(Android 3)では、各アクティビティはonClick、レイアウトのXMLのタグを介してボタンのクリックを処理するように登録されていました。

android:onClick="myClickMethod"

そのメソッド内で使用できます view.getId()、switchステートメントを使用してボタンのロジックを実行できます。

Honeycombの導入により、これらのアクティビティをフラグメントに分割し、さまざまなアクティビティ内で再利用できるようにします。ボタンの動作のほとんどはアクティビティに依存せずOnClickListenerボタンごとにを登録する古い(1.6より前の)メソッドを使用せずにコードをFragmentsファイル内に常駐させたい。

final Button button = (Button) findViewById(R.id.button_id);
button.setOnClickListener(new View.OnClickListener() {
    public void onClick(View v) {
        // Perform action on click
    }
});

問題は、レイアウトがインフレートされても、個々のフラグメントではなく、ボタンのクリックを受け取っているのはホストアクティビティであることです。どちらにも良いアプローチはありますか

  • フラグメントを登録してボタンのクリックを受け取りますか?
  • アクティビティからクリックイベントを、それらが属するフラグメントに渡しますか?

1
フラグメントのonCreate内でリスナーの登録を処理できませんか?
CL22、

24
@jodesはい、私は使用する必要はしたくないsetOnClickListenerfindViewById、なぜだと、各ボタンのためにonClick物事を簡単にするために、追加されました。
smith324

4
受け入れられた答えを見ると、XMLのonClickアプローチに固執するよりも、setOnClickListenerを使用するほうが疎結合だと思います。アクティビティがクリックごとに正しいフラグメントに「転送」する必要がある場合、これは、フラグメントが追加されるたびにコードを変更する必要があることを意味します。インターフェースを使用してフラグメントの基本クラスから分離しても、それは役に立ちません。フラグメントが正しいボタン自体に登録されている場合、アクティビティは完全に不可知のままであり、IMOより優れたスタイルです。Adorjan Princzからの回答も参照してください。
Adriaan Koster、2014

@ smith324はこれについてAdriaanに同意する必要があります。アドルジャンの答えを試してみて、その後の人生が良くなるかどうか見てみましょう。
user1567453 2017

回答:


175

あなたはこれを行うことができます:

アクティビティ:

Fragment someFragment;    

//...onCreate etc instantiating your fragments

public void myClickMethod(View v) {
    someFragment.myClickMethod(v);
}

断片:

public void myClickMethod(View v) {
    switch(v.getId()) {
        // Just like you were doing
    }
}    

カップリングを減らしてフラグメントを再利用できるようにしたい@Ameenへの対応

インターフェース:

public interface XmlClickable {
    void myClickMethod(View v);
}

アクティビティ:

XmlClickable someFragment;    

//...onCreate, etc. instantiating your fragments casting to your interface.
public void myClickMethod(View v) {
    someFragment.myClickMethod(v);
}

断片:

public class SomeFragment implements XmlClickable {

//...onCreateView, etc.

@Override
public void myClickMethod(View v) {
    switch(v.getId()){
        // Just like you were doing
    }
}    

52
それは私が本質的に今やっていることですが、それぞれがクリックイベントを受け取る必要がある複数のフラグメントがある場合、それは非常に面倒です。パラダイムがそれらの周りに溶け込んでいるので、私は一般的に断片で悪化しています。
smith324

128
私は同じ問題に遭遇しています。あなたの回答に感謝しますが、これはソフトウェアエンジニアリングの観点からはクリーンなコードではありません。このコードにより、フラグメントと密接に結合されたアクティビティが発生します。アクティビティーがフラグメントの実装の詳細を知らなくても、複数のアクティビティーで同じフラグメントを再利用できるはずです。
Ameen 2013

1
「switch(v.getId()){」ではなく「switch(v.getId()){」である必要があります
eb80

5
独自のインターフェイスを定義する代わりに、Euporieが述べたように既存のOnClickListenerを使用できます。
fr00tyl00p 2014

1
これを読んだとき、私は泣きそうになりました、それはとてもボイルプレートです... @AdorjanPrinczからの以下の答えは行く方法です。
Wjdavis5 2015年

603

onClickイベントの処理には、次のソリューションを使用することをお勧めします。これは、アクティビティとフラグメントでも機能します。

public class StartFragment extends Fragment implements OnClickListener{

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
            Bundle savedInstanceState) {

        View v = inflater.inflate(R.layout.fragment_start, container, false);

        Button b = (Button) v.findViewById(R.id.StartButton);
        b.setOnClickListener(this);
        return v;
    }

    @Override
    public void onClick(View v) {
        switch (v.getId()) {
        case R.id.StartButton:

            ...

            break;
        }
    }
}

9
onCreateViewで、ViewGroup vのすべての子項目をループ処理し、見つかったすべてのButtonインスタンスにonclicklistenerを設定します。すべてのボタンのリスナーを手動で設定するよりもはるかに優れています。
Aパーソン

17
投票した。これにより、フラグメントが再利用可能になります。そうでなければ、なぜフラグメントを使用するのですか?
ボレアス2013年

44
これは、1987年にプログラミングウィンドウズによって提唱されたのと同じ手法ではありませんか 心配無用。Googleは動きが速く、開発者の生産性がすべてです。イベント処理が1991-eara Visual Basicと同等になるまでは、長くはかからないと思います。
Edward Brey 2013年

7
魔女のインポートはあなたがOnClickListenerに使用しましたか?Intellijはandroid.view.View.OnClickListenerを提案し、それが機能しません:/(onClickは実行されません)
Lucas Jota

4
@NathanOsman質問はxml onClickに関連していたと思うので、承認されたansが正確なソリューションを提供します。
Naveed Ahmad 2014

29

問題は、ビューがフラグメントではなく、まだアクティビティであることです。フラグメントには独自の独立したビューはなく、親アクティビティビューにアタッチされます。そのため、イベントはフラグメントではなくアクティビティで終了します。残念ながら、これを機能させるにはコードが必要になると思います。

変換中に私がしてきたことは、古いイベントハンドラーを呼び出すクリックリスナーを追加することです。

例えば:

final Button loginButton = (Button) view.findViewById(R.id.loginButton);
loginButton.setOnClickListener(new OnClickListener() {
    @Override
    public void onClick(final View v) {
        onLoginClicked(v);
    }
});

1
ありがとう-フラグメントビュー(つまり、inflater.inflate(R.layout.my_fragment_xml_resource)の結果)をonLoginClicked()に渡してフラグメントサブビューにアクセスできるようにするため、これを1つのわずかな変更で使用しました。 EditTextなど、view.findViewById()を使用(アクティビティビューを単に通過した場合、view.findViewById(R.id.myfragmentwidget_id)を呼び出すとnullが返されます)。
Michael Nelson

これは私のプロジェクトのAPI 21では機能しません。このアプローチをどのように使用するかについての考えはありますか?
Darth Coder 2015年

かなり基本的なコードで、ほとんどすべてのアプリで使用されています。何が起こっているのか説明できますか?
Brill Pappin、2015年

1
フラグメントのレイアウトがアクティビティのビューにアタッチされているために発生した問題の説明については、この回答に賛成してください。
fllo

25

最近、この問題を解決しました。コンテキストActivityにメソッドを追加したり、OnClickListenerを実装したりする必要はありません。それが「有効な」ソリューションであるかどうかはわかりませんが、機能します。

に基づく: https //developer.android.com/tools/data-binding/guide.html#binding_events

データバインディングで実行できます。フラグメントインスタンスを変数として追加するだけで、任意のメソッドをonClickにリンクできます。

<layout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    tools:context="com.example.testapp.fragments.CustomFragment">

    <data>
        <variable android:name="fragment" android:type="com.example.testapp.fragments.CustomFragment"/>
    </data>
    <LinearLayout
        android:orientation="vertical"
        android:layout_width="match_parent"
        android:layout_height="match_parent">

        <ImageButton
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:src="@drawable/ic_place_black_24dp"
            android:onClick="@{() -> fragment.buttonClicked()}"/>
    </LinearLayout>
</layout>

そしてフラグメントリンクコードは...

public class CustomFragment extends Fragment {

    ...

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        // Inflate the layout for this fragment
        View view = inflater.inflate(R.layout.fragment_person_profile, container, false);
        FragmentCustomBinding binding = DataBindingUtil.bind(view);
        binding.setFragment(this);
        return view;
    }

    ...

}

1
私はただ感じています、この解決策を得ることができないのでいくつかの詳細が欠落しています
Tima

たぶん、あなたは、ドキュメントで「ビルド環境」から何かが欠けている:developer.android.com/tools/data-binding/...
アルドCanepa

@Aldo XMLのonClickメソッドの場合、代わりにandroid:onClick = "@ {()-> fragment。buttonClicked()}"を使用する必要があると思います。また、他の人のために、フラグメント内でbuttonClicked()関数を宣言し、ロジックを内部に置く必要があります。
Hayk Nahapetyan

xmlでは、次のようになりますandroid:name="fragment" android:type="com.example.testapp.fragments.CustomFragment"/>
COYG

1
私はこれを試したところ、タグのnameおよびtype属性に接頭辞が付いていないvariableはずです。おそらくそれを必要とする古いバージョンのAndroidレイアウトがありますか?android:
JohnnyLambada

6

バターナイフはおそらくクラッター問題の最良の解決策です。注釈プロセッサを使用して、いわゆる「古いメソッド」のボイラープレートコードを生成します。

ただし、カスタムインフレータを使用して、onClickメソッドを引き続き使用できます。

使い方

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup cnt, Bundle state) {
    inflater = FragmentInflatorFactory.inflatorFor(inflater, this);
    return inflater.inflate(R.layout.fragment_main, cnt, false);
}

実装

public class FragmentInflatorFactory implements LayoutInflater.Factory {

    private static final int[] sWantedAttrs = { android.R.attr.onClick };

    private static final Method sOnCreateViewMethod;
    static {
        // We could duplicate its functionallity.. or just ignore its a protected method.
        try {
            Method method = LayoutInflater.class.getDeclaredMethod(
                    "onCreateView", String.class, AttributeSet.class);
            method.setAccessible(true);
            sOnCreateViewMethod = method;
        } catch (NoSuchMethodException e) {
            // Public API: Should not happen.
            throw new RuntimeException(e);
        }
    }

    private final LayoutInflater mInflator;
    private final Object mFragment;

    public FragmentInflatorFactory(LayoutInflater delegate, Object fragment) {
        if (delegate == null || fragment == null) {
            throw new NullPointerException();
        }
        mInflator = delegate;
        mFragment = fragment;
    }

    public static LayoutInflater inflatorFor(LayoutInflater original, Object fragment) {
        LayoutInflater inflator = original.cloneInContext(original.getContext());
        FragmentInflatorFactory factory = new FragmentInflatorFactory(inflator, fragment);
        inflator.setFactory(factory);
        return inflator;
    }

    @Override
    public View onCreateView(String name, Context context, AttributeSet attrs) {
        if ("fragment".equals(name)) {
            // Let the Activity ("private factory") handle it
            return null;
        }

        View view = null;

        if (name.indexOf('.') == -1) {
            try {
                view = (View) sOnCreateViewMethod.invoke(mInflator, name, attrs);
            } catch (IllegalAccessException e) {
                throw new AssertionError(e);
            } catch (InvocationTargetException e) {
                if (e.getCause() instanceof ClassNotFoundException) {
                    return null;
                }
                throw new RuntimeException(e);
            }
        } else {
            try {
                view = mInflator.createView(name, null, attrs);
            } catch (ClassNotFoundException e) {
                return null;
            }
        }

        TypedArray a = context.obtainStyledAttributes(attrs, sWantedAttrs);
        String methodName = a.getString(0);
        a.recycle();

        if (methodName != null) {
            view.setOnClickListener(new FragmentClickListener(mFragment, methodName));
        }
        return view;
    }

    private static class FragmentClickListener implements OnClickListener {

        private final Object mFragment;
        private final String mMethodName;
        private Method mMethod;

        public FragmentClickListener(Object fragment, String methodName) {
            mFragment = fragment;
            mMethodName = methodName;
        }

        @Override
        public void onClick(View v) {
            if (mMethod == null) {
                Class<?> clazz = mFragment.getClass();
                try {
                    mMethod = clazz.getMethod(mMethodName, View.class);
                } catch (NoSuchMethodException e) {
                    throw new IllegalStateException(
                            "Cannot find public method " + mMethodName + "(View) on "
                                    + clazz + " for onClick");
                }
            }

            try {
                mMethod.invoke(mFragment, v);
            } catch (InvocationTargetException e) {
                throw new RuntimeException(e);
            } catch (IllegalAccessException e) {
                throw new AssertionError(e);
            }
        }
    }
}

6

私はコードを使用するよりもコードでクリック処理をしたい onClickフラグメントを操作するときは、XMLで属性。

これは、アクティビティをフラグメントに移行するときにさらに簡単になります。android:onClickcaseブロックから直接クリックハンドラー(以前はXMLで設定されていました)を直接呼び出すことができます。

findViewById(R.id.button_login).setOnClickListener(clickListener);
...

OnClickListener clickListener = new OnClickListener() {
    @Override
    public void onClick(final View v) {
        switch(v.getId()) {
           case R.id.button_login:
              // Which is supposed to be called automatically in your
              // activity, which has now changed to a fragment.
              onLoginClick(v);
              break;

           case R.id.button_logout:
              ...
        }
    }
}

フラグメントでのクリックの処理に関しては、これは私よりも簡単に見えandroid:onClickます。


5

これは別の方法です:

1.次のようにBaseFragmentを作成します。

public abstract class BaseFragment extends Fragment implements OnClickListener

2.使用

public class FragmentA extends BaseFragment 

の代わりに

public class FragmentA extends Fragment

3.あなたの活動:

public class MainActivity extends ActionBarActivity implements OnClickListener

そして

BaseFragment fragment = new FragmentA;

public void onClick(View v){
    fragment.onClick(v);
}

それが役に立てば幸い。


回答から1年、1か月、1日:すべてのFragmentクラスでOnClickListenerの実装を繰り返さずに抽象BaseFragmentを作成する以外に理由はありますか?
Dimitrios K.

5

私の使用例では、1つのonClickメソッドにフックするために必要な50の奇妙なImageViewがあります。私の解決策は、フラグメント内のビューをループして、それぞれに同じonclickリスナーを設定することです。

    final View.OnClickListener imageOnClickListener = new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            chosenImage = ((ImageButton)v).getDrawable();
        }
    };

    ViewGroup root = (ViewGroup) getView().findViewById(R.id.imagesParentView);
    int childViewCount = root.getChildCount();
    for (int i=0; i < childViewCount; i++){
        View image = root.getChildAt(i);
        if (image instanceof ImageButton) {
            ((ImageButton)image).setOnClickListener(imageOnClickListener);
        }
    }

2

私が答えを見ると、彼らは何とか古いです。最近Googleは、onClickの処理やxmlでの割り当てがはるかに簡単なDataBindingを導入しています。

これを処理する方法を確認できる良い例を次に示します。

<?xml version="1.0" encoding="utf-8"?>
<layout xmlns:android="http://schemas.android.com/apk/res/android">
   <data>
       <variable name="handlers" type="com.example.Handlers"/>
       <variable name="user" type="com.example.User"/>
   </data>
   <LinearLayout
       android:orientation="vertical"
       android:layout_width="match_parent"
       android:layout_height="match_parent">
       <TextView android:layout_width="wrap_content"
           android:layout_height="wrap_content"
           android:text="@{user.firstName}"
           android:onClick="@{user.isFriend ? handlers.onClickFriend : handlers.onClickEnemy}"/>
       <TextView android:layout_width="wrap_content"
           android:layout_height="wrap_content"
           android:text="@{user.lastName}"
           android:onClick="@{user.isFriend ? handlers.onClickFriend : handlers.onClickEnemy}"/>
   </LinearLayout>
</layout>

DataBindingに関する非常に優れたチュートリアルもあり、ここで見つけることができます。


2

コールバックをXMLレイアウトの属性として定義できます。記事「カスタムAndroidウィジェットのカスタムXML属性」では、カスタムウィジェットに対してそれを行う方法について説明します。クレジットはKevin Dionに送られます:)

基本のFragmentクラスにスタイル可能な属性を追加できるかどうかを調査しています。

基本的な考え方は、onClickコールバックを処理するときにViewが実装するのと同じ機能を持つことです。


1

Blundellの答えに加えて
、多くのonClicksでさらにフラグメントがある場合:

アクティビティ:

Fragment someFragment1 = (Fragment)getFragmentManager().findFragmentByTag("someFragment1 "); 
Fragment someFragment2 = (Fragment)getFragmentManager().findFragmentByTag("someFragment2 "); 
Fragment someFragment3 = (Fragment)getFragmentManager().findFragmentByTag("someFragment3 "); 

...onCreate etc instantiating your fragments

public void myClickMethod(View v){
  if (someFragment1.isVisible()) {
       someFragment1.myClickMethod(v);
  }else if(someFragment2.isVisible()){
       someFragment2.myClickMethod(v);
  }else if(someFragment3.isVisible()){
       someFragment3.myClickMethod(v); 
  }

} 

あなたのフラグメントで:

  public void myClickMethod(View v){
     switch(v.getid()){
       // Just like you were doing
     }
  } 

1

android:Onclick = ""を使用してxmlに登録すると、フラグメントが属するコンテキスト(getActivity())の尊重されるアクティビティにコールバックが与えられます。そのようなメソッドがアクティビティで見つからない場合、システムは例外をスローします。


おかげで、クラッシュが発生した理由を誰も説明しませんでした

1

分離されたイベントにEventBusを使用することを検討することをお勧めします。非常に簡単にイベントをリッスンできます。イベントがuiスレッドで受信されていることを確認することもできます(すべてのイベントサブスクリプションに対して自分でrunOnUiThread ..を呼び出すのではありません)。

https://github.com/greenrobot/EventBus

Githubから:

アクティビティ、フラグメント、スレッド、サービスなどの間の通信を簡素化する、Androidに最適化されたイベントバス。コードの削減、品質の向上


1

Adjorn Linkzの回答に追加したい

複数のハンドラが必要な場合は、ラムダ参照を使用できます

void onViewCreated(View view, Bundle savedInstanceState)
{
    view.setOnClickListener(this::handler);
}
void handler(View v)
{
    ...
}

ここでの秘訣は、そのhandlerメソッドの署名が署名と一致することView.OnClickListener.onClickです。この方法では、View.OnClickListenerインターフェースは必要ありません。

また、switchステートメントは必要ありません。

悲しいことに、このメソッドは、単一のメソッドまたはラムダを必要とするインターフェースにのみ制限されています。


1

データバインディングに依存する優れた回答をいくつか見つけましたが、そのアプローチでは、XMLでフラグメントのないレイアウト定義を可能にしながら、フラグメント解決を有効にするという意味で、十分な解決策が見られませんでした。

したがって、データバインディング有効になっていると、ここに私が提案できる一般的なソリューションがあります。少し長いですが、確実に機能します(いくつかの注意事項があります):

ステップ1:カスタムOnClick実装

これは、タップされたビュー(ボタンなど)に関連付けられたコンテキストを介して、フラグメント対応の検索を実行します。


// CustomOnClick.kt

@file:JvmName("CustomOnClick")

package com.example

import android.app.Activity
import android.content.Context
import android.content.ContextWrapper
import android.view.View
import androidx.fragment.app.Fragment
import androidx.fragment.app.FragmentActivity
import java.lang.reflect.Method

fun onClick(view: View, methodName: String) {
    resolveOnClickInvocation(view, methodName)?.invoke(view)
}

private data class OnClickInvocation(val obj: Any, val method: Method) {
    fun invoke(view: View) {
        method.invoke(obj, view)
    }
}

private fun resolveOnClickInvocation(view: View, methodName: String): OnClickInvocation? =
    searchContexts(view) { context ->
        var invocation: OnClickInvocation? = null
        if (context is Activity) {
            val activity = context as? FragmentActivity
                    ?: throw IllegalStateException("A non-FragmentActivity is not supported (looking up an onClick handler of $view)")

            invocation = getTopFragment(activity)?.let { fragment ->
                resolveInvocation(fragment, methodName)
            }?: resolveInvocation(context, methodName)
        }
        invocation
    }

private fun getTopFragment(activity: FragmentActivity): Fragment? {
    val fragments = activity.supportFragmentManager.fragments
    return if (fragments.isEmpty()) null else fragments.last()
}

private fun resolveInvocation(target: Any, methodName: String): OnClickInvocation? =
    try {
        val method = target.javaClass.getMethod(methodName, View::class.java)
        OnClickInvocation(target, method)
    } catch (e: NoSuchMethodException) {
        null
    }

private fun <T: Any> searchContexts(view: View, matcher: (context: Context) -> T?): T? {
    var context = view.context
    while (context != null && context is ContextWrapper) {
        val result = matcher(context)
        if (result == null) {
            context = context.baseContext
        } else {
            return result
        }
    }
    return null
}

注:元のAndroid実装に大まかに基づいています(https://android.googlesource.com/platform/frameworks/base/+/a175a5b/core/java/android/view/View.java#3025を参照)

ステップ2:レイアウトファイルでの宣言型アプリケーション

次に、データバインディング対応のXMLで:

<layout>
  <data>
     <import type="com.example.CustomOnClick"/>
  </data>

  <Button
    android:onClick='@{(v) -> CustomOnClick.onClick(v, "myClickMethod")}'
  </Button>
</layout>

注意事項

  • 「モダン」FragmentActivityベースの実装を想定
  • スタック内の「最上位」(つまり、最後)のフラグメントのメソッドのみを検索できます(必要に応じて修正できます)

0

これは私のために働いています:(Android studio)

 @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {

        View rootView = inflater.inflate(R.layout.update_credential, container, false);
        Button bt_login = (Button) rootView.findViewById(R.id.btnSend);

        bt_login.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {

                System.out.println("Hi its me");


            }// end onClick
        });

        return rootView;

    }// end onCreateView

1
これは、@ Brill Pappinの回答を複製したものです
naXa 2014

0

最高のソリューション私見:

断片的に:

protected void addClick(int id) {
    try {
        getView().findViewById(id).setOnClickListener(this);
    } catch (Exception e) {
        e.printStackTrace();
    }
}

public void onClick(View v) {
    if (v.getId()==R.id.myButton) {
        onMyButtonClick(v);
    }
}

次に、フラグメントのonViewStateRestoredで:

addClick(R.id.myButton);

0

使用したはずのように、アクティビティはコールバックを受信して​​います:

mViewPagerCloth.setOnClickListener((YourActivityName)getActivity());

フラグメントでコールバックを受信する場合は、次のようにします。

mViewPagerCloth.setOnClickListener(this);

onClickListenerフラグメントにインターフェースを実装する

弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.