Адаптер Firebase RecylcerView не получает точную позицию просмотра

Я использую пользовательский интерфейс Firebase и столкнулся с уникальной проблемой. Когда я загружаю RecyclerView, проблем нет. Я щелкаю элемент в RecyclerView и получаю значение .push() элемента, по которому был сделан щелчок. Затем он берет данные из этого значения Firebase .push(), которое хранится в базе данных Firebase, и отправляет их другому действию. Смотрите изображения ниже:

введите здесь описание изображения введите здесь описание изображения

Второй образ — это активность, которая получает данные через намерение. Когда я возвращаюсь к RecyclerView, я снова нажимаю «8» и загружаю другой номер (сейчас он выглядит как 1).

В конечном счете, я обеспокоен тем, что у меня неправильно настроен RecyclerView. Вот мой код:

Создайте объект опроса (POJO) и напишите в Firebase:

        mSubmitPollCreation.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {

            //TODO: Need to check if poll requirements are added, i.e. Question, Answer, ......
            //check if image has been loaded first
            if (resultImageURL == null) {
                Toast.makeText(getApplicationContext(), getResources().getString(R.string.no_image_selected), Toast.LENGTH_LONG).show();
                return;
            }

            //capture answers
            if (mNumberOfPollAnswersCreatedByUser > 5) {
                Toast.makeText(getApplicationContext(), getResources().getText(R.string.poll_answers_greater_than_five), Toast.LENGTH_LONG).show();
                mNumberOfPollAnswersCreatedByUser = 5;
            }
            for (int i = 0; i < mNumberOfPollAnswersCreatedByUser; i++) {
                EditText editText = (EditText) mEditTextAnswerLayout.findViewWithTag(getResources().getString(R.string.created_answer_editText_id) + String.valueOf(i + 1));
                String editTextInputForAnswer = String.valueOf(editText.getText());
                mPollAnswers.add(0, editTextInputForAnswer);                 Toast.makeText(getApplicationContext(),editTextInputForAnswer,Toast.LENGTH_SHORT).show();
            }


            Poll poll = new Poll(mCreatePollQuestion.getText().toString(), resultImageURL, mPollAnswers);
            Map<String, Object> pollMap = poll.toMap();
            final String key = mBaseRef.child("Polls").push().getKey();
            Map<String, Object> childUpdates = new HashMap<String, Object>();
            childUpdates.put("/Polls/" + key, pollMap);

            mBaseRef.updateChildren(childUpdates);
            Collections.reverse(mPollAnswers);
            for (int i = 0; i < mPollAnswers.size(); i++) {
                mBaseRef.child("Polls").child(key).child("answers").child(String.valueOf(i + 1)).updateChildren(poll.answerConvert(mPollAnswers, i));
            }

            Intent toHomeActivity = new Intent(CreateActivity.this, HomeActivity.class);
            toHomeActivity.putExtra("viewpager_position", 2);
            startActivity(toHomeActivity);
        }
    });

Где я читаю из Firebase и заполняю RecyclerView:

public class LiveFragment extends Fragment {
// TODO: Rename parameter arguments, choose names that match
// the fragment initialization parameters, e.g. ARG_ITEM_NUMBER
private static final String ARG_PARAM1 = "param1";
private static final String ARG_PARAM2 = "param2";

private RecyclerView mRecyclerview;
private DatabaseReference mBaseRef;
private DatabaseReference mPollsRef;
private LinearLayoutManager mLayoutManager;
private FloatingActionButton mFloatingActionAdd;

private FirebaseRecyclerAdapter<Poll, PollHolder> mFireAdapter;

private RecyclerView.ItemAnimator mItemAnimator;


// TODO: Rename and change types of parameters
private String mParam1;
private String mParam2;

private OnFragmentInteractionListener mListener;

public LiveFragment() {
    // Required empty public constructor
}

/**
 * Use this factory method to create a new instance of
 * this fragment using the provided parameters.
 *
 * @param param1 Parameter 1.
 * @param param2 Parameter 2.
 * @return A new instance of fragment LiveFragment.
 */
// TODO: Rename and change types and number of parameters
public static LiveFragment newInstance(String param1, String param2) {
    LiveFragment fragment = new LiveFragment();
    Bundle args = new Bundle();
    args.putString(ARG_PARAM1, param1);
    args.putString(ARG_PARAM2, param2);
    fragment.setArguments(args);
    return fragment;
}

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    mBaseRef = FirebaseDatabase.getInstance().getReference();
    mPollsRef = mBaseRef.child("Polls");

    if (getArguments() != null) {
        mParam1 = getArguments().getString(ARG_PARAM1);
        mParam2 = getArguments().getString(ARG_PARAM2);

    }
}

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
                         Bundle savedInstanceState) {
    // Inflate the layout for this fragment
    final View v = inflater.inflate(R.layout.fragment_new, container, false);
    Log.v("TAG", "ON CREATE CALLED FROM NEW");

    mRecyclerview = (RecyclerView) v.findViewById(R.id.list);
    mRecyclerview.setItemAnimator(new SlideInLeftAnimator());


    mLayoutManager = new LinearLayoutManager(getContext());
    mLayoutManager.setOrientation(LinearLayoutManager.VERTICAL);
    mLayoutManager.setReverseLayout(true);
    mLayoutManager.setStackFromEnd(true);

    mFloatingActionAdd = (FloatingActionButton) getActivity().findViewById(R.id.myFAB);
    mFloatingActionAdd.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            Intent I = new Intent(getActivity().getApplicationContext(), CreateActivity.class);
            startActivity(I);
        }
    });


    mRecyclerview.setLayoutManager(mLayoutManager);
    return v;
}

// TODO: Rename method, update argument and hook method into UI event
public void onButtonPressed(Uri uri) {
    if (mListener != null) {
        mListener.onFragmentInteraction(uri);
    }
    scrollToPosition();
}

@Override
public void onStart() {
    super.onStart();

    mFireAdapter = new FirebaseRecyclerAdapter<Poll, PollHolder>(Poll.class, R.layout.latest_item, PollHolder.class, mBaseRef.child("Polls")) {
        @Override
        protected void populateViewHolder(PollHolder viewHolder, final Poll model, final int position) {
            viewHolder.mPollQuestion.setText(model.getQuestion());
            Picasso.with(getActivity().getApplicationContext())
                    .load(model.getImage_URL())
                    .fit()
                    .into(viewHolder.mPollImage);
            viewHolder.mView.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View view) {
                    Log.v("TAG", model.toString());
                    Log.v("TAG","ONCLICKED");
                    Intent toPoll = new Intent(getActivity(), PollHostActivity.class);
                    toPoll.putExtra("POLL_ID", mFireAdapter.getRef(position).getKey());
                    startActivity(toPoll);

                }
            });
            Log.v("TAG", mFireAdapter.getRef(position).getKey());
            Log.v("TAG", String.valueOf(position));
            Log.v("TAG", model.getQuestion());
            Log.v("TAG", model.getImage_URL());
        }
    };

    mRecyclerview.setAdapter(mFireAdapter);
    scrollToPosition();



}

@Override
public void onStop() {
    super.onStop();
    if (mFireAdapter != null) {
        mFireAdapter.cleanup();
    }
}


@Override
public void onDetach() {
    super.onDetach();
    mListener = null;
}


public static class PollHolder extends RecyclerView.ViewHolder implements View.OnClickListener {

    TextView mPollQuestion;
    ImageView mPollImage;
    View mView;


    public PollHolder(View itemView) {
        super(itemView);

        mPollQuestion = (TextView) itemView.findViewById(R.id.latest_item_question);
        mPollImage = (ImageView) itemView.findViewById(R.id.pollThumbNailImage);
        this.mView = itemView;
        itemView.setOnClickListener(this);

    }

    @Override
    public void onClick(View view) {
        Log.v("TAG", "CLICKED");
    }
}


private void scrollToPosition(){
    mFireAdapter.registerAdapterDataObserver(new RecyclerView.AdapterDataObserver() {
        @Override
        public void onItemRangeInserted(int positionStart, int itemCount) {
            super.onItemRangeInserted(positionStart, itemCount);
            int pollCount = mFireAdapter.getItemCount();
            int lastVisiblePosition = mLayoutManager.findLastCompletelyVisibleItemPosition();

            // If the recycler view is initially being loaded or the user is at the bottom of the list, scroll
            // to the bottom of the list to show the newly added message.
            if (lastVisiblePosition == -1 ||
                    (positionStart >= (pollCount - 1) && lastVisiblePosition == (positionStart - 1))) {
                mRecyclerview.scrollToPosition(positionStart);
            }
        }
    });
   }
}

Мои мысли:

  1. я вызываю mFireAdapter.cleanup() в правильном месте?

  2. Проблема с нажатием «8» и просмотром «1» не появляется при первой загрузке RecyclerView, это происходит только тогда, когда я перехожу от действия (второе изображение) и возвращаюсь к RecyclerView.

  3. Я вызываю .setStackFromEnd(true) и .setReverseLayout(true), чтобы изменить данные, хранящиеся в FirebaseDatabase, что может способствовать возникновению проблемы.

  4. Я настроил .setOnClickListener() в методе .populateViewHolder() и использую следующее, чтобы передать ключ .push() в намерении для следующего действия:

      toPoll.putExtra("POLL_ID", mFireAdapter.getRef(position).getKey());
    

Этот метод может получить неправильный ключ.

Наконец, база данных Firebase:

введите здесь описание изображения

Любая помощь будет оценена по достоинству!


person tccpg288    schedule 29.12.2016    source источник
comment
попробуйте заменить position на holder.getAdapterPosition()   -  person koceeng    schedule 29.12.2016
comment
Хотите сделать как ответ? Вроде пока работает...   -  person tccpg288    schedule 29.12.2016


Ответы (1)


Вы не должны использовать position вне populateViewHolder, так как его значение может отличаться. Вместо этого сделайте viewHolder окончательным и используйте viewHolder.getAdapterPosition() для его замены.

person koceeng    schedule 29.12.2016
comment
Где / как я могу сделать это окончательным? - person tccpg288; 29.12.2016
comment
в своем protected void populateViewHolder(PollHolder viewHolder, final Poll model, final int position) сделай ...(final PollHolder viewHolder, ... - person koceeng; 29.12.2016
comment
Понял, не знал, что могу изменить код пользовательского интерфейса Firebase. - person tccpg288; 29.12.2016
comment
На самом деле, я думаю, что в пользовательском интерфейсе Firebase есть какая-то функция, позволяющая напрямую получить ваш пользовательский объект или его позицию. - person koceeng; 29.12.2016
comment
Имеется в виду, что есть что-то еще, что я могу добавить к своему коду? - person tccpg288; 29.12.2016
comment
Я думаю, что это достаточно хорошо, однако, если вам интересно, взгляните на его источник здесь github.com/firebase/FirebaseUI-Android/tree/master/database/src/ - person koceeng; 29.12.2016
comment
Давайте продолжим обсуждение в чате. - person koceeng; 29.12.2016
comment
Это не имело никакого значения вообще - person AlwaysConfused; 16.04.2017
comment
@StuckBetweenTrees, это решение работает для этого сообщения. Если вы пробовали это для своей проблемы, и это не сработало, возможно, вы столкнулись с другой проблемой. (Я видел ваш вопрос, надеюсь, вы скоро получите ответ) - person koceeng; 16.04.2017