恢复状态时,android onCreateOptionsMenu调用了两次

时间:2012-06-12 19:59:44

标签: android android-layout android-fragments

这是我创建的一个简单的Android应用程序来演示我的问题:

public class OptionMenuTest extends Activity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        Log.d("test", "create activity");
        setContentView(R.layout.options_layout);
        if(getFragmentManager().findFragmentByTag("frag") == null) {
            getFragmentManager().beginTransaction().add(R.id.option_fragment_container, new OptionMenuFragment(), "frag").commit(); 
        }

    }

    @Override
    protected void onSaveInstanceState(Bundle outState) {
        Log.d("test", "saving Activity state");
        super.onSaveInstanceState(outState);
    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        Log.d("test", "create Activity options menu");
        menu.add("activity");
        return true;
    }
}

片段:

public class OptionMenuFragment extends Fragment {

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        Log.d("test", "create fragment");
        setHasOptionsMenu(true);
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        TextView tv = new TextView(getActivity());
        tv.setText("Hello world");
        Log.d("test", "create fragment view");
        return tv;
    }

    @Override
    public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) {
        menu.add("fragment");
        Log.d("test", "create fragment options menu");
    }
}

Layout只是一个LinearLayout,用于将片段转储到:

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:id="@+id/option_fragment_container"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    >
</LinearLayout>

很简单吧?当我运行它时,我得到了预期的以下输出:

06-12 15:42:51.415: D/test(957): create activity
06-12 15:42:51.446: D/test(957): create fragment
06-12 15:42:51.446: D/test(957): create fragment view
06-12 15:42:51.446: D/test(957): create Activity options menu
06-12 15:42:51.446: D/test(957): create fragment options menu

现在,当我旋转手机时,我会遇到一些奇怪的行为:

06-12 15:43:11.251: D/test(957): saving Activity state
06-12 15:43:11.290: D/test(957): create fragment
06-12 15:43:11.290: D/test(957): create activity
06-12 15:43:11.306: D/test(957): create fragment view
06-12 15:43:11.306: D/test(957): create Activity options menu
06-12 15:43:11.306: D/test(957): create fragment options menu
06-12 15:43:11.306: D/test(957): create Activity options menu
06-12 15:43:11.306: D/test(957): create fragment options menu

为什么onCreateOptionMenu和片段onCreateOptionsMenu上的活动被调用了两次?如果我从片段中删除选项菜单,我会按预期调用onCreateOptionsMenu上的活动:

06-12 15:50:03.610: D/test(1076): create fragment
06-12 15:50:03.610: D/test(1076): create fragment view
06-12 15:50:03.813: D/test(1076): create Activity options menu
06-12 15:50:08.392: D/test(1076): saving Activity state // <-- rotate happens here
06-12 15:50:08.446: D/test(1076): create fragment
06-12 15:50:08.446: D/test(1076): create activity
06-12 15:50:08.462: D/test(1076): create fragment view
06-12 15:50:08.470: D/test(1076): create Activity options menu

我不明白这一点,似乎没有其他人遇到过这个问题。真正的问题是我的SearchView无法恢复配置更改(手机旋转)的状态,因为onCreateOptionMenu被调用两次。它第一次似乎有它的状态,但第二次被清除并重置。我无法弄清楚我做错了什么。

提前致谢。

3 个答案:

答案 0 :(得分:5)

我想我找到了这个问题的答案。

看看这个:

https://stackoverflow.com/a/7225296/48468

这个问题似乎与Android在活动被销毁时(旋转设备时)不破坏碎片的事实有关。

基本上我补充道:

setRetainInstance(true);

到我的片段构造函数,问题就解决了。

希望它有所帮助!

答案 1 :(得分:2)

@Gerardo Contijoch的回答是误导性的,除了一个事实:

在您的示例中,ActivityFragment在轮换时被销毁并再次创建。 这就是onCreateOptionsMenu()被调用两次的原因。这是正确和预期的行为。

通过setRetainInstance(true)告诉Android不要销毁片段。如果没有UI的片段没有Activity的上下文(对AsyncTasks协调和其他类似服务的东西有用),这可能会很有用。

在其他情况下,碎片可能会导致内存泄漏,您应该避免这种情况。

答案 2 :(得分:0)

我在充气和工作之前刚刚清除了菜单

 @Override
 public void   onCreateOptionsMenu(Menu menu, MenuInflater inflater) {
      menu.clear();
      inflater.inflate(R.menu.call_menu, menu);
      super.onCreateOptionsMenu(menu, inflater);

 }