为什么我要避免片段中的非默认构造函数？

Question

我正在使用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;
    }....

不使用非默认构造函数？

Answer 1

似乎没有一个答案实际回答"为什么使用bundle传递参数而不是非默认构造函数"

您应该通过bundle传递参数的原因是因为当系统恢复fragment(例如,在配置更改时),它将自动恢复您的bundle.

回调类似onCreate或onCreateView应该从中读取参数bundle- 这样您就可以保证将fragment正确的状态恢复到fragment初始化的相同状态(注意此状态可以与onSaveInstanceState bundle传递给的状态不同onCreate/onCreateView)

使用静态newInstance()方法的建议只是一个建议.您可以使用非默认构造函数,但请确保在该bundle构造函数的主体内部填充初始化参数.并在onCreate()或onCreateView()方法中读取这些参数.

Answer 2

创建一个bundle对象并插入你的数据(在这个例子中是你的Category对象).请注意,除非可序列化,否则不能将此对象直接传递到包中.我认为在片段中构建对象更好,并且只将id或其他内容放入bundle中.这是创建和附加包的代码:

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

之后,在您的片段访问数据中:

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

就这样.

Answer 3

你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一样.

Answer 4

如果您为某些类使用参数.试试这个

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