แนวทางปฏิบัติที่ดีที่สุดสำหรับการสร้างอินสแตนซ์ Android ใหม่

ฉันได้เห็นวิธีปฏิบัติทั่วไปสองประการเพื่อยกตัวอย่างชิ้นส่วนใหม่ในแอปพลิเคชัน:

Fragment newFragment = new MyFragment();

และ

Fragment newFragment = MyFragment.newInstance();

ตัวเลือกที่สองใช้ประโยชน์จากวิธีการคงที่newInstance()และโดยทั่วไปมีวิธีการดังต่อไปนี้

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

ตอนแรกฉันคิดว่าประโยชน์หลักคือความจริงที่ว่าฉันสามารถโอเวอร์โหลดเมธอด newInstance () เพื่อให้ความยืดหยุ่นเมื่อสร้างอินสแตนซ์ใหม่ของ Fragment - แต่ฉันสามารถทำได้โดยการสร้าง Constructor ที่โอเวอร์โหลดสำหรับ Fragment

ฉันพลาดอะไรไปหรือเปล่า?

ประโยชน์ของวิธีการหนึ่งเหนืออีกวิธีหนึ่งคืออะไร หรือเป็นเพียงการปฏิบัติที่ดี?

หาก Android ตัดสินใจที่จะสร้าง Fragment ของคุณขึ้นใหม่ในภายหลังมันจะเรียกตัวสร้างแบบไม่มีอาร์กิวเมนต์ของแฟรกเมนต์ของคุณ ดังนั้นการสร้างมากเกินไปไม่ใช่วิธีแก้ปัญหา

เมื่อพูดถึงวิธีการส่งข้อมูลไปยัง Fragment ของคุณเพื่อให้สามารถใช้งานได้หลังจากสร้างแฟรกเมนต์ใหม่โดย Android คือการส่งบันเดิลไปยังsetArgumentsวิธีการ

ตัวอย่างเช่นถ้าเราต้องการส่งจำนวนเต็มไปยังส่วนที่เราจะใช้สิ่งที่ชอบ:

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

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

    return myFragment;
}

และในภายหลังใน Fragment onCreate()คุณสามารถเข้าถึงจำนวนเต็มนั้นได้โดยใช้:

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

Bundle นี้จะสามารถใช้งานได้แม้ว่า Android จะสร้างขึ้นมาใหม่ก็ตาม

หมายเหตุ: setArgumentsสามารถเรียกได้เฉพาะก่อนที่แฟรกเมนต์จะแนบกับกิจกรรม

วิธีการนี้ได้รับการบันทึกไว้ในเอกสารอ้างอิงสำหรับนักพัฒนาซอฟต์แวร์ Android: https://developer.android.com/reference/android/app/Fragment.html

ประโยชน์เพียงอย่างเดียวในการใช้สิ่งnewInstance()ที่ฉันเห็นมีดังต่อไปนี้:

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 ตัดสินใจที่จะสร้าง Fragment ของคุณขึ้นใหม่ในภายหลังมันจะเรียกตัวสร้างแบบไม่มีอาร์กิวเมนต์ของแฟรกเมนต์ของคุณ ดังนั้นการสร้างมากเกินไปไม่ใช่วิธีแก้ปัญหา

ก็ยังคงเป็นไปได้มากทีเดียวที่จะใช้คอนสตรัค ในการดูว่าทำไมนี่เป็นครั้งแรกที่เราต้องดูว่าทำไม Android ใช้วิธีแก้ปัญหาข้างต้น

ก่อนที่จะสามารถใช้แฟรกเมนต์จำเป็นต้องใช้อินสแตนซ์ การเรียก Android YourFragment()(ตัวสร้างอาร์กิวเมนต์ที่ไม่มีอาร์กิวเมนต์ ) เพื่อสร้างอินสแตนซ์ของแฟรกเมนต์ ที่นี่ตัวสร้างที่โอเวอร์โหลดที่คุณเขียนจะถูกละเว้นเนื่องจาก Android ไม่สามารถรู้ได้ว่าจะใช้อันไหน

ในช่วงชีวิตของกิจกรรมชิ้นส่วนจะถูกสร้างขึ้นดังกล่าวข้างต้นและทำลายโดย Android หลายครั้ง ซึ่งหมายความว่าหากคุณใส่ข้อมูลในชิ้นส่วนวัตถุเองมันจะหายไปเมื่อชิ้นส่วนถูกทำลาย

ในการแก้ปัญหา, Android ขอให้คุณเก็บข้อมูลโดยใช้Bundle(เรียกsetArguments()) YourFragmentซึ่งจากนั้นจะสามารถเข้าถึงได้จาก อาร์กิวเมนต์bundles ได้รับการคุ้มครองโดย Android และด้วยเหตุนี้มีการรับประกันว่าถาวร

วิธีหนึ่งในการตั้งค่าบันเดิลนี้คือการใช้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วิธีนี้:

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

เพื่อเป็นการอธิบายเพิ่มเติมต่อไปนี้เป็นคลาส Fragment ของ 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 จะไม่ให้ชิ้นส่วนของคุณด้วยตัวสร้างเริ่มต้นที่ไม่มี arg ดังนั้นสิ่งนี้จะทำให้คุณต้องกำหนดคอนสตรัคเตอร์หาเรื่องไม่ซึ่งเป็นรหัสที่คุณสามารถหลีกเลี่ยงได้ด้วยnewInstanceวิธีการจากโรงงาน

แก้ไข : Android ไม่อนุญาตให้ใช้ตัวสร้างที่โอเวอร์โหลดสำหรับชิ้นส่วนอีกต่อไป คุณต้องใช้newInstanceวิธีการ

ฉันไม่เห็นด้วยกับ yydi ตอบว่า:

หาก Android ตัดสินใจที่จะสร้าง Fragment ของคุณขึ้นใหม่ในภายหลังมันจะเรียกตัวสร้างแบบไม่มีอาร์กิวเมนต์ของแฟรกเมนต์ของคุณ ดังนั้นการสร้างมากเกินไปไม่ใช่วิธีแก้ปัญหา

ฉันคิดว่ามันเป็นทางออกและเป็นสิ่งที่ดีนี่คือเหตุผลที่มันถูกพัฒนาโดยภาษาหลักของ 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และเพิ่มฟังก์ชันการทำงานให้กับตัวสร้าง ด้วยstaticConstructor คุณไม่สามารถทำได้

ปรับปรุง:

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 บางแห่ง

เราไม่สามารถส่งผ่านวัตถุที่ซับซ้อนได้ตัวอย่างเช่นบางรุ่นของผู้ใช้สำหรับกรณีของการแสดงส่วนของผู้ใช้

แต่สิ่งที่เราสามารถทำได้คือการตรวจสอบonCreateผู้ใช้นั้น! = null และถ้าไม่ - จากนั้นก็นำเขาจากชั้นข้อมูลมิฉะนั้น - ใช้ที่มีอยู่

วิธีนี้เราจะเพิ่มความสามารถในการสร้างโดย userId ทั้งสองกรณีในส่วนของ Android และ snappiness สำหรับการกระทำของผู้ใช้เช่นเดียวกับความสามารถในการสร้างชิ้นส่วนโดยถือเพื่อคัดค้านตัวเองหรือเพียง 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% แก้ปัญหาของคุณ

ใส่รหัสนี้ในส่วนแรกของการจัดเรียง

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 ที่เป็นค่าเริ่มต้นหรือสร้างเมธอดจากโรงงานเพื่ออินสแตนซ์ของแฟรกเมนต์
ข้อควรระวัง: สร้างตัวสร้างที่ว่างเปล่าหนึ่งตัวในแฟรกเมนต์อื่น ๆ

เมื่อเร็ว ๆ นี้ฉันอยู่ที่นี่ แต่สักวันฉันเพิ่งรู้ว่าอาจช่วยคุณได้เล็กน้อย

หากคุณใช้งานจาวาก็ไม่มีอะไรเปลี่ยนแปลงอีกมากมาย แต่สำหรับนักพัฒนา 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")

• คุณสามารถขยายการดำเนินการของ parent parent ในคลาส fragment child โดย:

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;
   }