android - 如何在 onCreate 中使用 findViewById() 而不返回 null？

Question

我不知道如何使用 findViewById(R.id.nav_timetable); 而不返回 null 并创建 NPE。我想要做的是让 TimetableFragment 默认加载，然后人们可以使用抽屉转到其他 fragment 。

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
    setSupportActionBar(toolbar);

    setupDrawer(toolbar);

    drawer.openDrawer(GravityCompat.START); // Opened this on first run so they know what's in it.

    // TODO Change this later to a home page and remove goToTimetablePage();
    // goToTimetablePage();
    goToFragmentFromNavMenuItem((MenuItem) findViewById(R.id.nav_timetable));
}

@Override
public boolean onNavigationItemSelected(MenuItem menuItem) {
    // Handle navigation view item clicks here.

    goToFragmentFromNavMenuItem(menuItem);
    return true;
}

private void goToFragmentFromNavMenuItem(MenuItem menuItem) {
    // Create a new fragment and specify the fragment to show based on nav item clicked
    Fragment fragment = null;
    Class fragmentClass;

    switch (menuItem.getItemId()) {
        case R.id.nav_slideshow: // TODO fragmentClass = SlideshowFragment.class; break;
        case R.id.nav_share: // TODO fragmentClass = ShareFragment.class; break;
        case R.id.nav_timetable:
            fragmentClass = TimetableFragment.class;
            // goToTimetablePage();
            break;
        default:
            fragmentClass = TimetableFragment.class;
            // TODO Change this later to a home page:
            // TODO fragmentClass = HomeFragment.class; Remove "goToTimetablePage();"
    }


    //noinspection TryWithIdenticalCatches
    try {
        fragment = (Fragment) fragmentClass.newInstance();
    } catch (InstantiationException e) {
        e.printStackTrace();
    } catch (IllegalAccessException e) {
        e.printStackTrace();
    }

    FragmentManager fragmentManager = getSupportFragmentManager();

    fragmentManager.beginTransaction().replace(R.id.flContent, fragment).commit();

    menuItem.setChecked(true);
    setTitle(menuItem.getTitle());
    drawer.closeDrawer(GravityCompat.START);
}

错误信息:

java.lang.RuntimeException: Unable to start activity ComponentInfo{com.gmail.gogobebe2.thedayahead/com.gmail.gogobebe2.thedayahead.MainActivity}: java.lang.NullPointerException: Attempt to invoke interface method 'int android.view.MenuItem.getItemId()' on a null object reference
  at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2416)
  at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2476)
  at android.app.ActivityThread.-wrap11(ActivityThread.java)
  at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1344)
  at android.os.Handler.dispatchMessage(Handler.java:102)
  at android.os.Looper.loop(Looper.java:148)
  at android.app.ActivityThread.main(ActivityThread.java:5417)
  at java.lang.reflect.Method.invoke(Native Method)
  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)
  Caused by: java.lang.NullPointerException: Attempt to invoke interface method 'int android.view.MenuItem.getItemId()' on a null object reference
  at com.gmail.gogobebe2.thedayahead.MainActivity.goToNavFragment(MainActivity.java:123)
  at com.gmail.gogobebe2.thedayahead.MainActivity.onCreate(MainActivity.java:59)
  at android.app.Activity.performCreate(Activity.java:6237)
  at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1107)
  at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2369)
  at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2476) 
  at android.app.ActivityThread.-wrap11(ActivityThread.java) 
  at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1344) 
  at android.os.Handler.dispatchMessage(Handler.java:102) 
  at android.os.Looper.loop(Looper.java:148) 
  at android.app.ActivityThread.main(ActivityThread.java:5417) 
  at java.lang.reflect.Method.invoke(Native Method) 
  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726) 
  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616) 

我尝试使用 onCreateView 方法，但出于某种原因它只是生成了 stackoverflow 消息。

编辑:以下是消除任何混淆的所有代码:https://github.com/gogobebe2/TheDayAhead

EDIT2:确保您不用担心 goToTimetablePage() 方法。目前未使用，我计划将其移至 TimetableFragment fragment 。

EDIT3:我将方法名称更改为 goToFragmentFromNavMenuItem() 以便它更好地描述它的清理操作。我还想让你们知道 onNavigationItemSelected(MenuItem menuItem) 方法与 goToFragmentFromNavMenuItem() 方法完美配合，只是在内部使用时不起作用onCreate(Bundle savedInstanceState) 因为 findViewById(R.id.nav_timetable) 一直返回 null。

此外，这是我制作的第一个 Android 应用程序，我在制作它的同时也在学习。昨晚我什至不知道什么是 fragment ，所以我对 Android 真的很陌生，请不要假设任何知识。

Answer 1

为避免由于 NPE 导致的崩溃，您可以在传递返回值之前测试返回值是否与 null 不同:

MenuItem menuItem = (MenuItem) findViewById(R.id.nav_timetable);
if (menuItem != null) {
    goToNavFragment(menuItem);
}

无论如何，这不会解决您的问题，事实上 findViewById(int) 正在返回 null 因为您要搜索的 View ID 不存在于您的 Activity 布置。

findViewById() 方法，它来自 View 类，您在 this 上调用它，这是一个 Activity(一个扩展 View 的类)，在当前展开的布局中搜索 View ID(来自 R)。

现在，如果您查看 onCreate 实现，您会发现您正在调用 setContentViewMethod(R.layout.activity_main)；这意味着您正在设置 this Activity 的布局，从 id 为 activity_main 的 xml 布局中获取它。

这里最大的问题是您传递的布局 (activity_main) 不包含您要查找的 ID。看看 activity_main.xml 布局文件，您会发现没有 ID 为 R.id.nav_timetable 的对象。

一个 Activity 一次只能加载一个布局 Activity 文件。您仍然可以为菜单或特定 View 对象(Spinner、List 等)加载其他布局文件。

这里最重要的是您正在尝试检索 MenuItem；为此，您必须覆盖两个方法 onCreateOptionsMenu() 和 onOptionsItemSelected()。

---

onCreateOptionsMenu() 的功能如其名。它会创建您将要使用的选项菜单。在这里你可以 Hook 你的 Action ，但你必须做的最重要的事情是膨胀菜单布局。否则，您将无法应用自定义菜单布局和使用 findViewById。

因此，在调用 findViewById() 之前，您应该这样做:

@Override
public boolean onCreateOptionsMenu(Menu menu) {
    MenuInflater inflater = getMenuInflater();
    inflater.inflate(R.menu.your_menu_id, menu);

    // findViewById()

    return true;
}

onOptionsItemSelected() 将在您从之前创建的菜单中选择一个项目时调用，它会将所选的 MenuItem 作为参数提供给您，您可以在此处实现菜单项单击操作。标准实现可以是以下内容:

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    switch (item.getItemId()) {
        case R.id.action_one:
            doSomething()
            return true;
        case R.id.action_two:
            doSomethingElse();
            return true;
        default:
            return super.onOptionsItemSelected(item);
    }
}