Я не могу правильно передать номер элемента во фрагмент через ViewPager - PullRequest
0 голосов
/ 02 апреля 2020

Код ViewPager:

public class CrimePagerActivity extends AppCompatActivity {

    private static final String EXTRA_CRIME_ID =
            "com.bignerdranch.android.criminalintent.crime_id";

    private static ViewPager mViewPager;
    public static List<Crime> mCrimes;
    public static int indexPager;

    private Button startButton;
    private Button finishButton;

    public static Intent newIntent(Context packageContext, UUID crimeId) {
        Intent intent = new Intent(packageContext, CrimePagerActivity.class);
        intent.putExtra(EXTRA_CRIME_ID, crimeId);
        return intent;
    }

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

        UUID crimeId = (UUID) getIntent()
                .getSerializableExtra(EXTRA_CRIME_ID);

        mViewPager = (ViewPager) findViewById(R.id.crime_view_pager);
        mViewPager.setClipToPadding(false);
        mViewPager.setPadding(20, 0, 20, 0);
        mViewPager.setPageMargin(20);
        mCrimes = CrimeLab.get(this).getCrimes();
        FragmentManager fragmentManager = getSupportFragmentManager();
        mViewPager.setAdapter(new FragmentStatePagerAdapter(fragmentManager) {

            @Override
            public Fragment getItem(int position) {  // !!!
                Crime crime = mCrimes.get(position);
                indexPager = position;
                return CrimeFragment.newInstance(crime.getId());
            }

            @Override
            public int getCount() {
                return mCrimes.size(); //!!!!!!!!
            }
        });

        for (int i = 0; i < mCrimes.size(); i++) {
            if (mCrimes.get(i).getId().equals(crimeId)) {
                mViewPager.setCurrentItem(i);
                indexPager = i;
                break;
            }
        }
    }

    public static void razmeshcheniyePager(int a){
        mViewPager.setCurrentItem(a);
    }
}

И код фрагмента:

 public class CrimeFragment extends Fragment {

    private static final String ARG_CRIME_ID = "crime_id";

    private Crime mCrime;
    private EditText mTitleField;
    private Button mDateButton;
    private CheckBox mSolvedCheckBox;

    private Button startButton;
    private Button finishButton;

    private CrimePagerActivity mPagerActivity;

    public static CrimeFragment newInstance(UUID crimeId) {
        Bundle args = new Bundle();
        args.putSerializable(ARG_CRIME_ID, crimeId);
        CrimeFragment fragment = new CrimeFragment();
        fragment.setArguments(args);
        return fragment;
    }

        @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
            UUID crimeId = (UUID) getArguments().getSerializable(ARG_CRIME_ID);
            mCrime = CrimeLab.get(getActivity()).getCrime(crimeId);
    }

    @Nullable
    @Override
    public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) {
        View v = inflater.inflate(R.layout.fragment_crime, container, false); // уточнить, что происходит здесь
        mTitleField = (EditText) v.findViewById(R.id.crime_title);


        startButton = (Button) v.findViewById(R.id.start_button); 
        if(CrimePagerActivity.indexPager == 2) startButton.setEnabled(false);
        startButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                mPagerActivity.razmeshcheniyePager(0);
            }
        });

        finishButton = (Button) v.findViewById(R.id.fifsh_button); 
        if (CrimePagerActivity.indexPager == CrimePagerActivity.mCrimes.size()-3) finishButton.setEnabled(false);
        finishButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                mPagerActivity.razmeshcheniyePager(CrimePagerActivity.mCrimes.size()-1);
            }
        });

        mTitleField.setText(mCrime.getTitle());
        mTitleField.addTextChangedListener(new TextWatcher() {
            @Override
            public void beforeTextChanged(CharSequence s, int start, int count, int after) {

            }

            @Override
            public void onTextChanged(CharSequence s, int start, int before, int count) {
                mCrime.setTitle(s.toString()); // возвращает строку, которая используется для задания заголовка Crime
            }

            @Override
            public void afterTextChanged(Editable s) {

            }
        });

        mDateButton = (Button) v.findViewById(R.id.crime_date); // кнопка во фрагменте создаётся немного не так, как в активности
        mDateButton.setText(mCrime.getDate().toString()); // вывод даты на кнопке
        mDateButton.setEnabled(false); // блокироание возможности нажатия кнопки

        mSolvedCheckBox = (CheckBox)v.findViewById(R.id.crime_solved);
        mSolvedCheckBox.setChecked(mCrime.isSolved()); // метод setChecked меняет булеву значение булевой переменной на противоположное
        mSolvedCheckBox.setOnCheckedChangeListener(new OnCheckedChangeListener() {
            @Override
            public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                mCrime.setSolved(isChecked);
            }
        });
        return v;
    }
}

Мне нужно заблокировать кнопку, чтобы перейти к началу или концу списка, если пользователь уже в начале или конце списка. Для этого я пытаюсь передать позицию элемента через переменную stati c. Проблема в том, что положение элемента передается неправильно (то есть, когда должно быть передано число 0, передано 2, когда должно быть передано 99, передано 97). И даже если вы учтете это в коде фрагмента, блокировка кнопок все равно не будет работать правильно (кнопки могут даже быть заблокированы на нескольких элементах, расположенных поблизости). Я попытался найти проблему с помощью отладки, столкнулся с еще одной неясной функцией. Код перед блокировкой кнопок можно вызывать несколько раз по очереди (он может не вызываться вообще). Код некоторых методов в анонимном классе new FragmentStatePagerAdapter (который входит в активность CrimePagerActivity) можно вызывать огромное количество раз. Пожалуйста, объясните, что происходит и что с этим делать.

PS Пример кода, взятый из книги "Android Программирование: Руководство по ранчо для больших ботаников" (3-е издание) (Руководства по ранчо для больших ботаников). Билл Филлипс, Крис Стюарт, Кристин Марсикано. глава 11

1 Ответ

0 голосов
/ 02 апреля 2020

Вы устанавливаете indexPager в getItem(). Это не правильно. getItem() может быть вызван в любое время по любой причине и не обязательно отражает текущий просмотренный или выбранный элемент. Вам нужно найти другой способ установки текущего «выбранного» элемента.

См. Документацию для FragmentStatePagerAdapter. В этой документации есть хороший пример того, как это сделать.

...