0

Доброго времени суток. Пытаюсь использовать фрагменты в приложении. Первый фрагмент отображает картинку, второй текст. При запуске устанавливается фрагмент с картинкой, при выборе в боковом меню Drawer картинка в фрагмент устанавливается, все отлично, затем нажимаю по кнопке в активити (метод onClick), фрагмент с картинкой заменяется на второй фрагмент с текстом и происходит работа там, тоже все отлично. Но вот когда затем снова кликаю по пункту в боковом меню, то получаю ошибку NPE в строке ((ImageView) frag.getView().findViewById(R.id.fragment_horoicon)).setImageResource(R.drawable.oven); (также указал комментом в коде), хотя вроде перед switchем произвел смену фрагмента с текстом на фрагмент с картинкой. Подскажите пожалуйста, где кроется ошибка? Спасибо.

P.S. для бокового меню используется библиотека MaterialDrawer

MainActivity.java

import android.app.Fragment;
import android.app.FragmentTransaction;
import android.os.AsyncTask;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.ImageView;
import android.widget.TextView;
import com.mikepenz.materialdrawer.Drawer;
import com.mikepenz.materialdrawer.DrawerBuilder;
import com.mikepenz.materialdrawer.model.DividerDrawerItem;
import com.mikepenz.materialdrawer.model.PrimaryDrawerItem;
import com.mikepenz.materialdrawer.model.SecondaryDrawerItem;
import com.mikepenz.materialdrawer.model.interfaces.IDrawerItem;
import org.jsoup.Jsoup;
import org.jsoup.nodes.Document;
import org.jsoup.nodes.Element;
import org.jsoup.select.Elements;

import java.io.IOException;

public class MainActivity extends AppCompatActivity {

    Drawer drawer;
    Fragment_image frag_image;
    Fragment_text frag_text;
    FragmentTransaction frag_trans;
    Button button_yesterday;
    String link;
    String prelink;
    Parser par;
    String text;
    String date;
    Fragment frag;

    public void onClick(View view) {

        frag_trans = getFragmentManager().beginTransaction();
        frag_trans.replace(R.id.frag_container, frag_text); // заменяем фрагмент с картинкой на фрагмент с текстом
        frag_trans.commit();

        switch (view.getId()) {
            case R.id.button_yesterday:
                link = prelink + "yesterday.html";
                break;
    }


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

        frag_image = new Fragment_image();
        frag_text = new Fragment_text();

        button_yesterday = (Button) findViewById(R.id.button_yesterday);

        frag_trans = getFragmentManager().beginTransaction();
        frag_trans.add(R.id.frag_container, frag_image); // устанавливаем первый фрагмент с картинкой
        frag_trans.commit();
        frag = getFragmentManager().findFragmentById(R.id.frag_container);

        PrimaryDrawerItem item1 = new PrimaryDrawerItem().withIdentifier(1).withName(R.string.select_sign).withSelectable(false);
        SecondaryDrawerItem item2 = (SecondaryDrawerItem) new SecondaryDrawerItem().withIdentifier(2).withName(R.string.oven);

        drawer = new DrawerBuilder()
                .withActivity(this)
                .withHeader(R.layout.drawer_header)
                .withTranslucentStatusBar(true)
                .withActionBarDrawerToggleAnimated(true)
                .withSelectedItem(-1)
                .addDrawerItems(
                        item1,
                        new DividerDrawerItem(),
                        item2
                )
                .withOnDrawerItemClickListener(new Drawer.OnDrawerItemClickListener() {
                    @Override
                    public boolean onItemClick(View view, int position, IDrawerItem drawerItem) {

                        frag_trans = getFragmentManager().beginTransaction();
                        frag_trans.replace(R.id.frag_container, frag_image); // при клике в боковом меню по пункту списка устанавливаем фрагмент с картинкой и подгружаю картинку в ImageView
                        frag_trans.commit();
                        frag = getFragmentManager().findFragmentById(R.id.frag_container);

                        switch ((int) drawerItem.getIdentifier()) {
                            case 2:
                                ((ImageView) frag.getView().findViewById(R.id.fragment_horoicon)).setImageResource(R.drawable.oven); // тут получаю NPE
                                break;
                            default:
                                break;
                        }
                        return true;
                    }
                })
                .build();
    }
}

UPD

public class MainActivity extends AppCompatActivity {

    Fragment_image frag_image;
    Fragment_text frag_text;
    FragmentTransaction frag_trans;
    Fragment frag;


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

        frag_image = new Fragment_image();
        frag_text = new Fragment_text();

        frag_trans = getFragmentManager().beginTransaction();
        frag_trans.add(R.id.frag_container, frag_image);
        frag_trans.commit();
        getFragmentManager().executePendingTransactions();
        frag = getFragmentManager().findFragmentById(R.id.frag_container);
        Handler handler = new Handler();
        handler.postDelayed(new Runnable() {
            public void run() {
                ((ImageView)frag.getView().findViewById(R.id.fragment_horoicon)).setImageResource(R.drawable.choose_sign);
            }
        }, 100);
    }
}
9
  • Аохоже что там у вас не тот фрагмент, который вы ожидаете. Проверьте конкретный класс фрагмента в момнт нажатия и ищите ошибку в логике
    – ЮрийСПб
    15 сен 2016 в 8:54
  • @ЮрийСПб, при запуске фрагмент картинки нормально ведь отрабатывает, на фрагмент текста тоже меняется нормально, а вот обратно на фрагмент с картинкой выбивает ошибку. Хотя ведь перед свитчем в Drawer меняю фрагмент frag_trans.replace(R.id.frag_container, frag_image); Если первый раз картинка отрабатывает, значит с самим классом фрагмента все нормально. Ранее с фрагментами не работал, поэтому подозрения, что где то ошибку допускаю в принципах смены фрагментов.
    – Pollux
    15 сен 2016 в 9:10
  • Commit работает не синхронно. По сему фоагмент моет не успеть замениться. Вообще ваш способ обращения к разметке фрагмента не верен. Но попробуйте вызвать commitNow вместо просто коммиь
    – ЮрийСПб
    15 сен 2016 в 9:13
  • @ЮрийСПб commitNow нет такого метода... есть только .commitAllowingStateLoss(). Использую android.app.Fragment. По поводу обращения к разметке, делал по этому уроку
    – Pollux
    15 сен 2016 в 9:22
  • 1
    @ЮрийСПб, и в очередной раз Вы меня выручаете) Спасибо Вам огромное! Добавил после коммита getFragmentManager().executePendingTransactions(); и все заработало. Пожалуйста, напишите как ответ, отмечу.
    – Pollux
    15 сен 2016 в 22:22

1 ответ 1

2

Попробуйте вызвать getFragmentManager().executePendingTransactions(); сразу после commit(). У вас проблема в том, что, операции с фрагментами происходят не синхронно. И разметки ещё нет на момент обращения к ней. А этот метод вызовет немедленное, синхронное исполнение всех текущих транзакций

10
  • Не хотел создавать такую же тему, но очень хочу спросить у Вас. Рассматриваемый кусок кода обновил в первом посте после UPD. Вот понадобилось мне при запуске приложения установить картинку в контейнер. Сделал вроде все как положено, добавил getFragmentManager().executePendingTransactions();, но опять же на строке ((ImageView)frag.getView().findViewById(R.id.fragment_horoicon)).setImageResource(R.drawable.choose_sign); начало крашится с NPE. Думаю надо добавить паузу, чтоб контейнер успел прогрузиться.. Добавил паузу с помощью Handler и тогда заработало.
    – Pollux
    17 сен 2016 в 20:34
  • Но вот интересный момент: если убираю строку getFragmentManager().executePendingTransactions(); и ставлю паузу хоть даже 5 секунд в Handler (ну думаю чтоб наверняка контейнер успел загрузиться) и в итоге по прошествии 5 секунд опять тот же краш... Теперь вообще для меня непонятица почему оно все таки крашится, если времени прошло куча... И почему же тут не прокатывает getFragmentManager().executePendingTransactions();...
    – Pollux
    17 сен 2016 в 20:34
  • @Pollux, думаю, если не вызывать синхронное исполнение транзакции, то вы, ища фрагмент по ID получаете старый фрагмент, а в нём вашего ImageView нет. Попробуйте искать фрагмент также в отложенной задаче, а не перед ней
    – ЮрийСПб
    18 сен 2016 в 0:37
  • Выше постом Вы говорили, что я неправильно обращаюсь к разметке. Может я использую устаревшие методы и оттого и эти ошибки. Может подскажете как это правильно делать, чтобы небыло таких ошибок или знаете где почитать об этом?
    – Pollux
    18 сен 2016 в 10:43
  • @Pollux, ну... Тут долго рассказывать. Я бы сейчас это через MVP сделал бы. Так надо было бы менять лишь состояние презентера, указывая ему какую картинку он показывать должен. А фрагмент при создании/активации из презентера бы брал инфу. Так вы непосредственно фрагментом бы не манипулировали и эта проблема бы не возникла
    – ЮрийСПб
    18 сен 2016 в 10:55

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

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