AndroidフラグメントonClickボタンメソッド


91

onClick(View v)XMLでメソッドを呼び出そうとしていますが、Fragmentでは機能しません。これはエラーです。

01-17 12:38:36.840: E/AndroidRuntime(4171): java.lang.IllegalStateException: 
Could not find a method insertIntoDb(View) in the activity class main.MainActivity 
for onClick handler on view class android.widget.Button with id 'btn_conferma'

Javaコード:

public void insertIntoDb(View v) {
...
} 

XML:

<Button
        android:id="@id/btn_conferma"
        style="?android:attr/borderlessButtonStyle"
        android:layout_width="0.0dip"
        android:layout_height="45dp"
        android:layout_marginLeft="2dp"
        android:layout_weight="1.0"
        android:background="@drawable/bottoni"
        android:gravity="center_horizontal|center_vertical"
        android:onClick="insertIntoDb"
        android:text="SALVA"
        android:textColor="#ffffff"
        android:textSize="16sp" />

1
スタックトレースと関連コードの詳細を投稿してください
Emmanuel

2
ここでdeveloper.android.com/guide/topics/ui/controls/…に記載されているように、「レイアウトをホストするアクティビティは、対応するメソッドを実装する必要があります。」あなたのケースでは、対応するメソッドをフラグメントに実装しました。このため、アクティビティで対応するメソッドが見つからなかったため、IllegalStateExceptionがスローされます。たぶん、この投稿stackoverflow.com/a/6271637/2515815に
hcelaloner 2014年

メソッドinsertIntoDb(View)は、Fragmentクラスではなく、main.MainActivityにある必要があります。
betorcs 2014年


回答:


185

あなたの活動は

public void insertIntoDb(View v) {
...
} 

フラグメントではありません。

上記の活動を望まない場合。フラグメントのボタンを初期化し、リスナーを同じに設定します。

<Button
    android:id="@+id/btn_conferma" // + missing

その後

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

   View view = inflater.inflate(R.layout.fragment_rssitem_detail,
    container, false);
   Button button = (Button) view.findViewById(R.id.btn_conferma);
   button.setOnClickListener(new OnClickListener()
   {
             @Override
             public void onClick(View v)
             {
                // do something
             } 
   }); 
   return view;
}

これにより、API 21を使用するプロジェクトでフラグメントがクラッシュします。代替の修正/提案はありますか?
ダースコーダー

@DarthCoder私はそうは思いません。コードを提示する必要があります。ここに掲載されているこのコードとは関連がない可能性があります
ラグフナンダン2015年

@Sanuロリポップ前ロリポップとは何の関係もありません。詳細を記載した別の質問を投稿してください..
ラグナンダン

上記の答えとして、クリックイベントがトリガーされることはありません。
Ted

3
上記のコメントに少し驚いています。問題はありませんが、詳細を投稿せずに「うまくいかない」と言うのは、あまり専門的ではありません...そして、私としては、「うまくいく」だけです。
zzheng

15

別のオプションは、フラグメントにView.OnClickListenerを実装させ、フラグメント内のonClick(View v)をオーバーライドすることです。フラグメントがアクティビティと通信する必要がある場合は、目的のメソッドを持つインターフェースを追加し、アクティビティにインターフェースを実装させ、そのメソッドをオーバーライドさせるだけです。

public class FragName extends Fragment implements View.OnClickListener { 
    public FragmentCommunicator fComm;
    public ImageButton res1, res2;
    int c;

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        return inflater.inflate(R.layout.fragment_test, container, false);
    }

    @Override
    public void onAttach(Activity activity) {
        super.onAttach(activity);
        try {
            fComm = (FragmentCommunicator) activity;
        } catch (ClassCastException e) {
            throw new ClassCastException(activity.toString()
                    + " must implement FragmentCommunicator");
        }
    }

    @Override
    public void onActivityCreated(Bundle savedInstanceState) {
        res1 = (ImageButton) getActivity().findViewById(R.id.responseButton1);
        res1.setOnClickListener(this);
        res2 = (ImageButton) getActivity().findViewById(R.id.responseButton2);
        res2.setOnClickListener(this);
    }
    public void onClick(final View v) { //check for what button is pressed
            switch (v.getId()) {
                case R.id.responseButton1:
                  c *= fComm.fragmentContactActivity(2);
                  break;
                case R.id.responseButton2:
                  c *= fComm.fragmentContactActivity(4);
                  break;
                default:
                  c *= fComm.fragmentContactActivity(100);
                  break;
    }
    public interface FragmentCommunicator{
        public int fragmentContactActivity(int b);
    }



public class MainActivity extends FragmentActivity implements FragName.FragmentCommunicator{
    int a = 10;
    //variable a is update by fragment. ex. use to change textview or whatever else you'd like.

    public int fragmentContactActivity(int b) {
        //update info on activity here
        a += b;
        return a;
    }
}

http://developer.android.com/training/basics/firstapp/starting-activity.html http://developer.android.com/training/basics/fragments/communicating.html


3
私はいつもこれをやっていますが、それでも醜いと思います
Alexander Farber 2015年

私も最初はやりましたが、今ではほとんど中古になりました。さらに、それは実際にはより複雑なアプリで役立ちましたが、基本的なものにとってはやり過ぎのようです。
cjayem13 2015

@ cjayem13、「なぜやり過ぎなのか」について詳しく説明していただけますか?
eRaisedToX 2017年

9

これは問題ではありません。これはAndroidのデザインです。こちらをご覧ください

各フラグメントは、モジュール化された再利用可能なアクティビティコンポーネントとして設計する必要があります。つまり、各フラグメントは独自のレイアウトと独自のライフサイクルコールバックを使用した独自の動作を定義するため、1つのフラグメントを複数のアクティビティに含めることができるため、再利用のために設計し、あるフラグメントを別のフラグメントから直接操作しないようにする必要があります。

可能な回避策は、MainActivityで次のようなことを行うことです。

Fragment someFragment;    

...onCreate etc instantiating your fragments

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

そしてあなたのフラグメントクラスで:

public void myClickMethod(View v){
    switch(v.getid()){
       // Your code here
    }
 } 

4

他の人は、onClickのメソッドはフラグメントではなくアクティビティで検索されるとすでに述べています。それでも、本当に必要な場合可能です。

基本的に、各ビューにはタグ(おそらくnull)があります。ルートビューのタグを、そのビューを拡張したフラグメントに設定します。次に、ビューの親を検索し、クリックしたボタンを含むフラグメントを取得するのは簡単です。次に、メソッド名を見つけ、リフレクションを使用して、取得したフラグメントから同じメソッドを呼び出します。かんたん!

拡張するクラスでFragment

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
    View rootView = inflater.inflate(R.layout.fragment_id, container, false);

    OnClickFragments.registerTagFragment(rootView, this); // <========== !!!!!

    return rootView;
}

public void onButtonSomething(View v) {
    Log.d("~~~","~~~ MyFragment.onButtonSomething");
    // whatever
}

すべてのアクティビティは、同じButtonHandlingActivityから派生しています。

public class PageListActivity extends ButtonHandlingActivity

ButtonHandlingActivity.java:

public class ButtonHandlingActivity extends Activity {

    public void onButtonSomething(View v) {
        OnClickFragments.invokeFragmentButtonHandlerNoExc(v);
//or, if you want to handle exceptions:
//      try {
//          OnClickFragments.invokeFragmentButtonHandler(v);
//      } catch ...
    }

}

すべてのxml onClickハンドラーのメソッドを定義する必要があります。

com / example / customandroid / OnClickFragments.java:

package com.example.customandroid;

import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;
import android.app.Fragment;
import android.view.View;

public abstract class OnClickFragments {
    public static class FragmentHolder {
        Fragment fragment;
        public FragmentHolder(Fragment fragment) {
            this.fragment = fragment;
        }
    }
    public static Fragment getTagFragment(View view) {
        for (View v = view; v != null; v = (v.getParent() instanceof View) ? (View)v.getParent() : null) {
            Object tag = v.getTag();
            if (tag != null && tag instanceof FragmentHolder) {
                return ((FragmentHolder)tag).fragment;
            }
        }
        return null;
    }
    public static String getCallingMethodName(int callsAbove) {
        Exception e = new Exception();
        e.fillInStackTrace();
        String methodName = e.getStackTrace()[callsAbove+1].getMethodName();
        return methodName;
    }
    public static void invokeFragmentButtonHandler(View v, int callsAbove) throws IllegalAccessException, IllegalArgumentException, InvocationTargetException, NoSuchMethodException {
        String methodName = getCallingMethodName(callsAbove+1);
        Fragment f = OnClickFragments.getTagFragment(v);
        Method m = f.getClass().getMethod(methodName, new Class[] { View.class });
        m.invoke(f, v);
    }
    public static void invokeFragmentButtonHandler(View v) throws IllegalAccessException, IllegalArgumentException, InvocationTargetException, NoSuchMethodException {
        invokeFragmentButtonHandler(v,1);
    }
    public static void invokeFragmentButtonHandlerNoExc(View v) {
        try {
            invokeFragmentButtonHandler(v,1);
        } catch (NoSuchMethodException e) {
            e.printStackTrace();
        } catch (IllegalAccessException e) {
            e.printStackTrace();
        } catch (IllegalArgumentException e) {
            e.printStackTrace();
        } catch (InvocationTargetException e) {
            e.printStackTrace();
        }
    }
    public static void registerTagFragment(View rootView, Fragment fragment) {
        rootView.setTag(new FragmentHolder(fragment));
    }
}

そして次の冒険は難読化を防ぐことです...

PS

これは、データが住むようにアプリケーションを設計するのはあなた次第もちろんであるモデル(あるという活動またはフラグメントに比べてコントローラからのMVCモデル-ビュー-コントローラの視点)。ビューは、あなたがXMLを介し定義するものである、プラスカスタムビュークラス(あなたがそれらを定義した場合、ほとんどの人は単に、すでにあるものを再利用します)。経験則:一部のデータが画面の回転を確実に切り抜ける必要がある場合、それらはModelに属します。


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

    View view = inflater.inflate(R.layout.writeqrcode_main, container, false);
    // Inflate the layout for this fragment

    txt_name = (TextView) view.findViewById(R.id.name);
    txt_usranme = (TextView) view.findViewById(R.id.surname);
    txt_number = (TextView) view.findViewById(R.id.number);
    txt_province = (TextView) view.findViewById(R.id.province);
    txt_write = (EditText) view.findViewById(R.id.editText_write);
    txt_show1 = (Button) view.findViewById(R.id.buttonShow1);

    txt_show1.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            Log.e("Onclick","Onclick");

            txt_show1.setVisibility(View.INVISIBLE);
            txt_name.setVisibility(View.VISIBLE);
            txt_usranme.setVisibility(View.VISIBLE);
            txt_number.setVisibility(View.VISIBLE);
            txt_province.setVisibility(View.VISIBLE);
        }
    });
    return view;
}

大丈夫 !!!!


3
これが質問にどのように答えるかを説明できますか?最後の文章はどういう意味ですか?
Teepeemm 2015年

2

Kotlinユーザーの場合:

override fun onCreateView(
    inflater: LayoutInflater,
    container: ViewGroup?, 
    savedInstanceState: Bundle?) : View?
{
    // Inflate the layout for this fragment
    var myView = inflater.inflate(R.layout.fragment_home, container, false)
    var btn_test = myView.btn_test as Button
    btn_test.setOnClickListener {
        textView.text = "hunny home fragment"
    }

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