เหตุใดฉันจึงต้องการหลีกเลี่ยงสิ่งก่อสร้างที่ไม่ใช่ค่าเริ่มต้นในเศษ

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

Avoid non-default constructors in fragments: use a default constructor plus Fragment#setArguments(Bundle) instead

มีใครบอกฉันได้ไหมว่าทำไมนี่จึงไม่ใช่ความคิดที่ดี

คุณยังสามารถแนะนำว่าฉันจะทำสิ่งนี้ให้สำเร็จได้อย่างไร:

public static class MenuFragment extends ListFragment {
    public ListView listView1;
    Categories category;

    //this is my "non-default" constructor
    public MenuFragment(Categories category){
        this.category = category;
    }....

หากไม่ใช้ตัวสร้างที่ไม่ใช่ค่าเริ่มต้น

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

Bundle args = new Bundle();
args.putLong("key", value);
yourFragment.setArguments(args);

หลังจากนั้นในข้อมูลการเข้าถึงชิ้นส่วนของคุณ:

Type value = getArguments().getType("key");

นั่นคือทั้งหมดที่

ดูเหมือนว่าไม่มีคำตอบที่จริงตอบ "ทำไมต้องใช้ชุดรวมสำหรับการส่งพารามิเตอร์มากกว่าตัวสร้างเริ่มต้นที่ไม่ใช่"

เหตุผลที่คุณควรจะได้รับการส่งผ่านพารามิเตอร์ผ่านมัดเป็นเพราะเมื่อระบบคืนค่าfragment(เช่นเกี่ยวกับการเปลี่ยนแปลงการตั้งค่า) bundleก็จะเรียกคืนโดยอัตโนมัติ

การเรียกกลับที่ชอบonCreateหรือonCreateViewควรอ่านพารามิเตอร์จากbundle- ด้วยวิธีนี้คุณรับประกันได้ว่าจะเรียกคืนสถานะของfragmentอย่างถูกต้องให้อยู่ในสถานะเดียวกันกับที่fragmentถูกกำหนดค่าเริ่มต้นด้วย (โปรดทราบว่าสถานะนี้อาจแตกต่างจากonSaveInstanceState bundleที่ผ่านไปonCreate/onCreateView)

คำแนะนำในการใช้newInstance()วิธีการสแตติกเป็นเพียงคำแนะนำ คุณสามารถใช้คอนสตรัคเตอร์ที่ไม่ใช่ค่าเริ่มต้น แต่ต้องแน่ใจว่าคุณใส่พารามิเตอร์การเริ่มต้นในส่วนbundleของเนื้อหาของตัวสร้าง และอ่านพารามิเตอร์เหล่านั้นในonCreate()หรือonCreateView()วิธีการ

คุณFragmentไม่ควรมีคอนสตรัคเตอร์เพราะFragmentManagerมันสร้างอินสแตนซ์ได้อย่างไร คุณควรมีnewInstance()วิธีการสแตติกที่กำหนดไว้กับพารามิเตอร์ที่คุณต้องการจากนั้นรวมเข้าด้วยกันและตั้งเป็นอาร์กิวเมนต์ของแฟรกเมนต์ซึ่งคุณสามารถเข้าถึงได้ในภายหลังด้วยBundleพารามิเตอร์

ตัวอย่างเช่น:

public static MyFragment newInstance(int title, String message) {
    MyFragment fragment = new MyFragment();
    Bundle bundle = new Bundle(2);
    bundle.putInt(EXTRA_TITLE, title);
    bundle.putString(EXTRA_MESSAGE, message);
    fragment.setArguments(bundle);
    return fragment ;
}

และอ่านข้อโต้แย้งเหล่านี้ได้ที่onCreate:

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    title = getArguments().getInt(EXTRA_TITLE);
    message = getArguments().getString(EXTRA_MESSAGE);

    //...
}

วิธีนี้หากแยกออกและเชื่อมต่ออีกครั้งสถานะของวัตถุสามารถจัดเก็บผ่านอาร์กิวเมนต์ได้เช่นเดียวbundlesกับที่แนบมากับIntents

หากคุณใช้พารามิเตอร์สำหรับบางคลาส ลองนี้

SomeClass mSomeInstance;
public static final MyFragment newInstance(SomeClass someInstance){
    MyFragment f = new MyFragment();
    f.mSomeInstance = someInstance;
    return f;
}

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

ในโครงการของฉันฉันใช้ Kotlin และใช้แฟรกเมนต์กับตัวสร้าง no-arg หลักและตัวสร้างรองสำหรับอาร์กิวเมนต์ที่เพิ่งเก็บไว้ในบันเดิลและตั้งค่าเป็นอาร์กิวเมนต์ Fragment ทุกอย่างทำงานได้ดี

ถ้าแฟรกเมนต์ใช้คอนสตรัคเตอร์ที่ไม่ใช่ค่าเริ่มต้นหลังจากเปลี่ยนการกำหนดค่าแฟรกเมนต์จะสูญเสียข้อมูลทั้งหมด