0

Сначала я брал данные (допустим, введённые с клавиатуры заметки) из БД в onCreateView фрагмента, где это заметки отображаются, но когда проработал функцию добавления новых заметок в другой активности, то обнаружил, что при возврате в предыдущую активность новых заметок не появляется, пока не перезапустить приложение. Я сделал вывод, что это при возврате в предыдующую активность метод onCreateView не выполняется и надо прописать код получения данных в onResume:

@Override
public void onResume() {
    super.onResume();
    getNoteData();
}

Как известно, onResume() выполняется всегда: как следом после onCreate, так и без него, потому получать данные во избежание дублирования кода можно только в onResume. Теперь код имеет вид:

@Nullable
@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
    view = inflater.inflate(LAYOUT, container, false);
    notesDBHelper = new notesDBHelper(context);

    return view;
}

@Override
public void onResume() {
    super.onResume();
    getNoteData();
}

Приведённая ниже ошибка возникает при перезапуске главной активности из Android Studio; других условий возникновения данной ошибки пока не обнаружил. Вероятно, после окончания разработки эта ошибка и не будет возникать, но сейчас надо понять, почему она возникает.

E/AndroidRuntime: FATAL EXCEPTION: main
                  Process: jp.co.yd.infowajg.ydjks, PID: 11599
                  java.lang.RuntimeException: Unable to resume activity {ru.example/ru.example.MainActivity}: java.lang.NullPointerException: Attempt to invoke virtual method 'android.database.sqlite.SQLiteDatabase android.content.Context.openOrCreateDatabase(java.lang.String, int, android.database.sqlite.SQLiteDatabase$CursorFactory, android.database.DatabaseErrorHandler)' on a null object reference
...
at jp.co.yd.infowajg.ydjks.fragment.InboxTabFragment.onResume(InboxTabFragment.java:50)

Последняя строка (из того, что я скопировал и вставил) как раз ссылается на

getNoteData()

Обновление Добавил весь код Fragment-класса.

public class NotesTabFragment extends AbstractTabFragment{
private static final int LAYOUT = R.layout.fragment_notes;

    NotesDBHelper notesDBHelper;

    public static NotesTabFragment getInstance(Context context){
        Bundle args = new Bundle();
        NotesTabFragment fragment = new NotesTabFragment();
        fragment.setArguments(args);
        fragment.setContext(context);

        return fragment;
    }

    @Nullable
    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        view = inflater.inflate(LAYOUT, container, false);
        notesDBHelper = new NotesDBHelper(context);

        return view;
    }

    @Override
    public void onResume() {
        super.onResume();
        getNotesItems();
    }

    private void getNotesItems(){
        RecyclerView recyclerView = (RecyclerView) view.findViewById(R.id.recycleView);
        recyclerView.setLayoutManager(new LinearLayoutManager(context));
        recyclerView.setAdapter(new InboxItemsAdapter(notesDBHelper.getAllNotes()));
    }

    public void setContext(Context context) {
        this.context = context;
    }
}
5
  • У вас context null
    – ЮрийСПб
    21 дек 2016 в 7:20
  • Тогда может и notesDBHelper = new notesDBHelper(context); перенести в onResume? 21 дек 2016 в 7:36
  • Лучше покажите где и как контекст инициализируете. Или замените его на getActivity()
    – ЮрийСПб
    21 дек 2016 в 7:53
  • Как заполняется context и зачем его хранить в поое когда есть метод getContext()/getActivity() ? Приложите полный стектрейс а не отрывок.
    – temq
    21 дек 2016 в 7:57
  • temq, нет проблем, обновил поле вопроса. 21 дек 2016 в 8:09

1 ответ 1

1

Нет особого смысла хранить контекст в отдельной переменной.

К тому же вы его совершенно неправильно используете в вашем случае - задавать его фрагменту при создании экземпляра оного - абсолютно неправильно.

Получать контекст во фрагменте надо методом getActivity()

Так что просто удалите вашу переменную context и замените её использование на вызов метода getActivity()

5
  • В onCreateView сделать как Вы сказали - нет проблем, но что насчёт getInstance? Кстати, этот метод сделал по этому образцу. 21 дек 2016 в 8:15
  • @GurebuBokofu, возможно я чего-то не знаю, но такого я ни разу не видел. Склонен полагать что автор видео немного не подумал. Контекст - это связь компонента приложения с системой. Пока фрагмент не добавлен в активити у него не вызовутся методы жизненного цикла и контекст ему не нужен ибо он существует в "безвоздушном" пространстве. Повторюсь - может быть я не знаю чего-то, но лично по моему многолетнему опыту - передача контекста в getInstance() - дичь и мрак. Да и сам метод этот принято называть newInstance() ибо это не синглтон
    – ЮрийСПб
    21 дек 2016 в 8:29
  • Благодарю Вас за объяснение. А как тогда лучше инициализировать фрагмент? Можете дать какую-нибудь ссылку на пример, написанный по правилам хорошего тона? 21 дек 2016 в 8:34
  • @GurebuBokofu, достаточно просто удалить переменную context и не передавать его во фрагмент при создании.
    – ЮрийСПб
    21 дек 2016 в 8:48
  • 1
    @GurebuBokofu контекст у фрагмента появляется после метода жц onAttach() поэтому нет необходимости как то с наружи передавать контекст т.к. это будет создавать ошибки после пересоздания фрагмента. Иногда полезней читать оффициальную документацию вместо просмотра видео с сомнительным содержанием, ну или хотя бы иногда поглядывать в нее - developer.android.com/guide/components/fragments.html?hl=ru
    – temq
    21 дек 2016 в 9:11

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.