新しいAndroidフラグメントをインスタンス化するためのベストプラクティス

アプリケーションで新しいフラグメントをインスタンス化するための2つの一般的なプラクティスを見てきました。

Fragment newFragment = new MyFragment();

そして

Fragment newFragment = MyFragment.newInstance();

2番目のオプションは、静的メソッドを使用するnewInstance()と、一般的に次のような方法が含まれています。

public static Fragment newInstance() 
{
    MyFragment myFragment = new MyFragment();
    return myFragment;
}

最初、主な利点は、フラグメントの新しいインスタンスを作成するときに柔軟性を与えるためにnewInstance（）メソッドをオーバーロードできるということでしたが、フラグメントのオーバーロードされたコンストラクターを作成することによってもこれを行うことができました。

私は何か見落としてますか？

あるアプローチが他のアプローチより優れている点は何ですか？それともそれはただ良い習慣ですか？

Androidが後でフラグメントを再作成することを決定した場合、フラグメントの引数のないコンストラクターを呼び出します。そのため、コンストラクターのオーバーロードは解決策ではありません。

そうは言っても、AndroidによってFragmentが再作成された後に使用できるように、Fragmentにデータを渡すsetArguments方法は、メソッドにバンドルを渡すことです。

したがって、たとえば、フラグメントに整数を渡したい場合は、次のようなものを使用します。

public static MyFragment newInstance(int someInt) {
    MyFragment myFragment = new MyFragment();

    Bundle args = new Bundle();
    args.putInt("someInt", someInt);
    myFragment.setArguments(args);

    return myFragment;
}

そして、後でフラグメントonCreate()を使用して、その整数にアクセスできます。

getArguments().getInt("someInt", 0);

このバンドルは、フラグメントがAndroidによって何らかの方法で再作成された場合でも使用できます。

また、注：setArgumentsフラグメントがアクティビティにアタッチされる前にのみ呼び出すことができます。

このアプローチは、Android開発者リファレンスにも記載されています：https : //developer.android.com/reference/android/app/Fragment.html

newInstance()私が見るものを使用する唯一の利点は次のとおりです：

1. フラグメントで使用されるすべての引数を1か所にまとめることができ、フラグメントをインスタンス化するたびに以下のコードを記述する必要はありません。

   Bundle args = new Bundle();
   args.putInt("someInt", someInt);
   args.putString("someString", someString);
   // Put any other arguments
   myFragment.setArguments(args);

2. 他のクラスに忠実に動作することが期待される引数を通知するための優れた方法です（ただし、フラグメントインスタンスに引数がバンドルされていない場合は処理できるはずです）。

したがって、私は、静的newInstance()を使用してフラグメントをインスタンス化することをお勧めします。

別の方法もあります：

Fragment.instantiate(context, MyFragment.class.getName(), myBundle)

@yydlは、newInstanceメソッドが優れている理由について説得力のある理由を示していますが、

Androidが後でフラグメントを再作成することを決定した場合、フラグメントの引数のないコンストラクターを呼び出します。そのため、コンストラクターのオーバーロードは解決策ではありません。

コンストラクタを使用することは依然として可能です。これがなぜかを確認するには、まず、上記の回避策がAndroidで使用される理由を確認する必要があります。

フラグメントを使用する前に、インスタンスが必要です。AndroidはYourFragment()（引数なしのコンストラクター）を呼び出して、フラグメントのインスタンスを作成します。ここでは、Androidがどちらを使用するかを認識できないため、オーバーロードしたコンストラクターは無視されます。

アクティビティの存続期間中に、フラグメントは上記のように作成され、Androidによって複数回破棄されます。つまり、フラグメントオブジェクト自体にデータを配置した場合、フラグメントが破棄されるとデータは失われます。

回避策として、AndroidはBundle（呼び出しsetArguments()）を使用してデータを保存するように要求しますYourFragment。これは、からアクセスできます。引数bundleはAndroidによって保護されているため、永続的であることが保証されています。

このバンドルを設定する1つの方法は、静的newInstanceメソッドを使用することです。

public static YourFragment newInstance (int data) {
    YourFragment yf = new YourFragment()
    /* See this code gets executed immediately on your object construction */
    Bundle args = new Bundle();
    args.putInt("data", data);
    yf.setArguments(args);
    return yf;
}

ただし、コンストラクタ：

public YourFragment(int data) {
    Bundle args = new Bundle();
    args.putInt("data", data);
    setArguments(args);
}

newInstanceメソッドとまったく同じことができます。

当然、これは失敗し、AndroidがこのnewInstanceメソッドを使用してほしい理由の1つです。

public YourFragment(int data) {
    this.data = data; // Don't do this
}

さらに説明すると、Androidのフラグメントクラスは次のとおりです。

/**
 * Supply the construction arguments for this fragment.  This can only
 * be called before the fragment has been attached to its activity; that
 * is, you should call it immediately after constructing the fragment.  The
 * arguments supplied here will be retained across fragment destroy and
 * creation.
 */
public void setArguments(Bundle args) {
    if (mIndex >= 0) {
        throw new IllegalStateException("Fragment already active");
    }
    mArguments = args;
}

Androidは、引数が構築時にのみ設定されることを要求し、これらが保持されることを保証することに注意してください。

編集：@JHHのコメントで指摘されているように、引数を必要とするカスタムコンストラクターを提供している場合、Javaは引数のないデフォルトコンストラクターをフラグメントに提供しません。したがって、これには引数なしのコンストラクタを定義する必要があります。これは、newInstanceファクトリメソッドで回避できるコードです。

編集：Androidはフラグメントにオーバーロードされたコンストラクターを使用することを許可しなくなりました。newInstanceメソッドを使用する必要があります。

私は yydiの答えに同意しません：

Androidが後でフラグメントを再作成することを決定した場合、フラグメントの引数のないコンストラクターを呼び出します。そのため、コンストラクターのオーバーロードは解決策ではありません。

私はそれが解決策であり良いものだと思います。これがまさにJavaコア言語によって開発された理由です。

Androidシステムがを破壊して再作成できることは事実ですFragment。だからあなたはこれを行うことができます：

public MyFragment() {
//  An empty constructor for Android System to use, otherwise exception may occur.
}

public MyFragment(int someInt) {
    Bundle args = new Bundle();
    args.putInt("someInt", someInt);
    setArguments(args);
}

システムによって再作成された場合でもsomeInt、getArguments()後者からプルすることができますFragment。これはstaticコンストラクタよりもエレガントなソリューションです。

私の意見では、staticコンストラクタは役に立たないので使用しないでください。また、将来的にこれを拡張Fragmentしてコンストラクタに機能を追加したい場合にも制限されます。ではstaticコンストラクタあなたはこれを行うことはできません。

更新：

Androidは、デフォルト以外のすべてのコンストラクターにエラーのフラグを立てる検査を追加しました。
上記の理由により、無効にすることをお勧めします。

いくつかのkotlinコード：

companion object {
    fun newInstance(first: String, second: String) : SampleFragment {
        return SampleFragment().apply {
            arguments = Bundle().apply {
                putString("firstString", first)
                putString("secondString", second)
            }
        }
    }
}

そして、これで引数を得ることができます：

val first: String by lazy { arguments?.getString("firstString") ?: "default"}
val second: String by lazy { arguments?.getString("secondString") ?: "default"}

Androidで引数を使用してフラグメントをインスタンス化するベストプラクティスは、フラグメントに静的なファクトリメソッドを含めることです。

public static MyFragment newInstance(String name, int age) {
    Bundle bundle = new Bundle();
    bundle.putString("name", name);
    bundle.putInt("age", age);

    MyFragment fragment = new MyFragment();
    fragment.setArguments(bundle);

    return fragment;
}

フラグメントのインスタンスでフィールドを設定することは避けてください。Androidシステムがフラグメントを再作成するたびに、システムがより多くのメモリを必要とする場合は、引数なしのコンストラクターを使用してフラグメントを再作成するためです。

引数を使用してフラグメントをインスタンス化するためのベストプラクティスについて詳しくは、こちらをご覧ください。

ベストプラクティスについての質問から、いくつかのREST Webサービスを操作するときにフラグメントを作成するためにハイブリッドアプローチを使用することが非常によくあることを付け加えておきます

ユーザーフラグメントを表示する場合、複雑なオブジェクト、たとえば一部のユーザーモデルを渡すことはできません

しかし、私たちができることは、onCreateそのユーザーをチェックインすることです！= null、そうでない場合-データレイヤーから彼を連れてきます。それ以外の場合は、既存のものを使用します。

このようにして、Androidによるフラグメントの再作成の場合にuserIdによって再作成する機能と、ユーザーアクションのスナップ性、およびオブジェクト自体またはそのIDのみを保持してフラグメントを作成する機能の両方を取得します

これが好きなもの：

public class UserFragment extends Fragment {
    public final static String USER_ID="user_id";
    private User user;
    private long userId;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        userId = getArguments().getLong(USER_ID);
        if(user==null){
            //
            // Recreating here user from user id(i.e requesting from your data model,
            // which could be services, direct request to rest, or data layer sitting
            // on application model
            //
             user = bringUser();
        }
    }

    public static UserFragment newInstance(User user, long user_id){
        UserFragment userFragment = new UserFragment();
        Bundle args = new Bundle();
        args.putLong(USER_ID,user_id);
        if(user!=null){
            userFragment.user=user;
        }
        userFragment.setArguments(args);
        return userFragment;

    }

    public static UserFragment newInstance(long user_id){
        return newInstance(null,user_id);
    }

    public static UserFragment newInstance(User user){
        return newInstance(user,user.id);
    }
}

このコードを使用して問題を100％修正します

このコードをfirstFragmentに入力します

public static yourNameParentFragment newInstance() {

    Bundle args = new Bundle();
    args.putBoolean("yourKey",yourValue);
    YourFragment fragment = new YourFragment();
    fragment.setArguments(args);
    return fragment;
}

このサンプルはブールデータを送信します

そして、でSecendFragment

yourNameParentFragment name =yourNameParentFragment.newInstance();
   Bundle bundle;
   bundle=sellDiamondFragments2.getArguments();
  boolean a= bundle.getBoolean("yourKey");

最初のフラグメントの値は静的である必要があります

幸せなコード

フラグメントをインスタンス化する最良の方法は、デフォルトのFragment.instantiateメソッドを使用するか、フラグメントをインスタンス化するファクトリメソッドを作成する ことです
注意：フラグメントメモリを復元するとランタイム例外がスローされますが、フラグメント内に常に1つの空のコンストラクターを作成してください。

私は最近ここにいます。しかし、私が今知っていることは少し役立つかもしれません。

Javaを使用している場合、変更することはほとんどありません。しかし、kotlin開発者のために、実行するための地下室を作ることができると私が思う以下のスニペットがあります：

• 親フラグメント：

inline fun <reified T : SampleFragment> newInstance(text: String): T {
    return T::class.java.newInstance().apply {
        arguments = Bundle().also { it.putString("key_text_arg", text) }
    }
}

• 通常の通話

val f: SampleFragment = SampleFragment.newInstance("ABC")
// or val f = SampleFragment.newInstance<SampleFragment>("ABC")

• 次の方法で、子フラグメントクラスの親init操作を拡張できます。

fun newInstance(): ChildSampleFragment {
    val child = UserProfileFragment.newInstance<ChildSampleFragment>("XYZ")
    // Do anything with the current initialized args bundle here
    // with child.arguments = ....
    return child
}

ハッピーコーディング。

setArguments()役に立たない。それは混乱をもたらすだけです。

public class MyFragment extends Fragment {

    public String mTitle;
    public String mInitialTitle;

    public static MyFragment newInstance(String param1) {
        MyFragment f = new MyFragment();
        f.mInitialTitle = param1;
        f.mTitle = param1;
        return f;
    }

    @Override
    public void onSaveInstanceState(Bundle state) {
        state.putString("mInitialTitle", mInitialTitle);
        state.putString("mTitle", mTitle);
        super.onSaveInstanceState(state);
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle state) {
        if (state != null) {
            mInitialTitle = state.getString("mInitialTitle");
            mTitle = state.getString("mTitle");
        } 
        ...
    }
}

私はこれに対してもっと簡単な解決策があると信じています。

public class MyFragment extends Fragment{

   private String mTitle;
   private List<MyObject> mObjects;

   public static MyFragment newInstance(String title, List<MyObject> objects)
   MyFragment myFrag = new MyFragment();
   myFrag.mTitle = title;
   myFrag.mObjects = objects;
   return myFrag;
   }