на ребенка добавлен метод работы несколько раз - PullRequest
0 голосов
/ 15 октября 2018

Я делаю вызов между двумя пользователями в моем приложении, используя базу данных Firebase.в соревновании я сохраняю player1Uid и player2Uid (player1 - игрок, который начал соревнование, а два игрока - игрок, который примет или откажется от соревнования).Я хочу сделать историю испытаний, поэтому, если задача завершена, я сохраняю ее в списке с именем «completeChallengesList», а если задача еще не завершена, я сохраняю ее в списке, называемом «незавершенный список задач». Код иногда работает нормальноно в других случаях он выводит данные более одного раза в список.

это мой код, который выводит данные:

    public void startAsynkTask() {
    //TODO : search for a solution to this error
    AsyncTask asyncTask = new AsyncTask() {
        @Override
        protected Boolean doInBackground(Object[] objects) {
            try {
                Socket sock = new Socket();
                sock.connect(new InetSocketAddress("8.8.8.8", 53), 1500);
                sock.close();
                return true;
            } catch (IOException e) {
                return false;
            }
        }

        @Override
        protected void onPostExecute(Object o) {
            if ((boolean) o) {
                clearLists();
                ChildEventListener generalChallengesListener = new ChildEventListener() {
                    @Override
                    public void onChildAdded(DataSnapshot dataSnapshot, String s) {

                        String challengeState = dataSnapshot.child("state").getValue().toString();
                        Log.v("Logging2", "onChildAdded");
                        if (challengeState.equals(uncompletedChallengeText)) {
                            getChallengeData(dataSnapshot, "onChildAdded");
                            view.onDataFound();
                        }
                    }

                    @Override
                    public void onChildChanged(DataSnapshot dataSnapshot, String s) {
                        //TODO : note that the only changing handled is when challenge moves from uncompleted to completed state
                        getChallengeData(dataSnapshot, "onChildChanged");
                        for (int i = 0; i < uncompletedChallengesList.size(); i++) {
                            if (uncompletedChallengesList.get(i).getId().equals(dataSnapshot.getKey())) {
                                uncompletedChallengesList.remove(i);
                                view.notifyAdapters(completedChallengesList.size(), uncompletedChallengesList.size());
                                break;
                            }
                        }
                        checkListsSizeAndAdjustViews();
                        view.hideProgressBar();
                    }

                    @Override
                    public void onChildRemoved(DataSnapshot dataSnapshot) {
                        startAsynkTask();
                    }

                    @Override
                    public void onChildMoved(DataSnapshot dataSnapshot, String s) {

                    }

                    @Override
                    public void onCancelled(DatabaseError databaseError) {
                        //Toast.makeText(getActivity(), "فشل تحميل البيانات من فضلك تأكد من الاتصال بالانترنت", Toast.LENGTH_SHORT).show();
                        view.hideProgressBar();
                        Log.v("Logging", "error loading data : " + databaseError);
                    }
                };

                //this code gives data where current user is player 1
                player1Listener = challengesReference.orderByChild("player1Uid").equalTo(currentUserUid).addChildEventListener(generalChallengesListener);
                //this code gives data where current user is player 2
                player2Listener = challengesReference.orderByChild("player2Uid").equalTo(currentUserUid).addChildEventListener(generalChallengesListener);

                challengesReference.addListenerForSingleValueEvent(new ValueEventListener() {
                    @Override
                    public void onDataChange(DataSnapshot dataSnapshot) {
                        onInitialDataLoaded();
                        Log.v("ChallengesFragPresenter", "completed list size :" + completedChallengesList + " , uncompleted list size : " + uncompletedChallengesList);
                        challengesReference.removeEventListener(this);
                    }


                    @Override
                    public void onCancelled(DatabaseError databaseError) {

                    }
                });
            } else {
                view.onNoInternetConnection();
            }
        }
    };

    asyncTask.execute();
}

public String getChallengeData(DataSnapshot dataSnapshot, String tag) {
    Log.v("ChallengesFragPresenter", "get challenge data called");
    Log.v("ChallengesFragPresenter", "completedChallengesList : " + completedChallengesList.size()
            + ", uncompletedChallengesList " + uncompletedChallengesList.size());
    view.startCompletedChallengesAdapter(completedChallengesList);
    view.startUnCompletedChallengesAdapter(uncompletedChallengesList);

    GenericTypeIndicator<List<Question>> t = new GenericTypeIndicator<List<Question>>() {
    };
    challenge = new Challenge();
    String challengeDate = dataSnapshot.child("date").getValue().toString();
    String challengeSubject = dataSnapshot.child("subject").getValue().toString();
    String challengeState = dataSnapshot.child("state").getValue().toString();
    String challengeId = dataSnapshot.getKey();
    ArrayList challengeQuestionsList = (ArrayList) dataSnapshot.child("questionsList").getValue(t);
    long player1Score = (long) dataSnapshot.child("player1score").getValue();
    long player2Score = (long) dataSnapshot.child("player2score").getValue();

    String player1Name = dataSnapshot.child("player1Name").getValue().toString();
    String player1Image = dataSnapshot.child("player1Image").getValue().toString();
    String player1Uid = dataSnapshot.child("player1Uid").getValue().toString();
    String player2Name = dataSnapshot.child("player2Name").getValue().toString();
    String player2Image = dataSnapshot.child("player2Image").getValue().toString();
    String player2Uid = dataSnapshot.child("player2Uid").getValue().toString();

    String challengerName, challengerImage;

    if (player1Uid.equals(currentUserUid)) {
        currentPlayer = 1;
        challengerName = player2Name;
        challengerImage = player2Image;
        challenge.setSecondChallengerUid(player2Uid);//second means that it is not the player who starts the challenge
        if (tag.equals("onChildAdded")) {
            player1childrenCount++;
        }
    } else {
        currentPlayer = 2;
        challengerName = player1Name;
        challengerImage = player1Image;
        challenge.setSecondChallengerUid(player1Uid);//second means that it is not the player who starts the challenge
        if (tag.equals("onChildAdded")) {
            player2childrenCount++;
        }
    }
    challenge.setCurrentPlayer(currentPlayer);
    challenge.setChallengerName(challengerName);
    challenge.setDate(challengeDate);
    challenge.setImage(challengerImage);
    challenge.setSubject(challengeSubject);
    challenge.setState(challengeState);
    challenge.setId(challengeId);
    challenge.setQuestionsList(challengeQuestionsList);
    String score;
    if (currentPlayer == 1) {
        score = player2Score + " : " + player1Score;
    } else {
        score = player1Score + " : " + player2Score;
    }
    challenge.setScore(score);
    if (challenge.getState().equals("اكتمل")) {
        view.showCompletedChallengesTv();
        if(!completedChallengesList.contains(challenge)) {
            completedChallengesList.add(0, challenge);
        }
        view.notifyAdapters(completedChallengesList.size(), uncompletedChallengesList.size());
    } else if (challenge.getState().equals(refusedChallengeText)) {
        view.showCompletedChallengesTv();
        if(!completedChallengesList.contains(challenge)) {
            completedChallengesList.add(0, challenge);
        }
        view.notifyAdapters(completedChallengesList.size(), uncompletedChallengesList.size());
    } else if (challenge.getState().equals(uncompletedChallengeText)) {
        view.showUncompletedChallengesTv();
        if(!uncompletedChallengesList.contains(challenge)) {
            uncompletedChallengesList.add(0, challenge);
        }
        view.notifyAdapters(completedChallengesList.size(), uncompletedChallengesList.size());
    }

    return player1Uid;
}

после долгого времени отладки я обнаружил, что метод getChallengeData() вызывается более одного раза, когда добавляется новый дочерний элемент.Я много пытался понять, почему, но не смог.

ПРИМЕЧАНИЕ. Я использую задачу asynk, чтобы проверить, есть ли подключение к Интернету или нет, перед началом загрузки данных.

1 Ответ

0 голосов
/ 15 октября 2018

Я думаю, что ваша проблема в том, что вы не удаляете свой listner "generalChallengesListener", поэтому будет создано несколько ссылок, поэтому onChildAdded будет несколько раз вызываться как число созданных вами списков.Так что добавьте ссылку на свой список и знайте, когда вы должны удалить его.Надеюсь, это поможет вам =)

...