Trying to update Database addValueEventListener

I am trying to update my database.
Something that i am doing doesn’t work. At the first time that I call this method it updates the database but finishes the activity. At the second time that I call that method it gets into an infinite loop.

public void savePractice(){
        //This Method saves the Practice in the Database under the Athlete practices
        currentAthleteRef.addValueEventListener(new ValueEventListener() {
            @Override
            public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
                currentAthlete = dataSnapshot.getValue(Athlete.class);
                if (currentAthlete.practices == null){
                    currentAthlete.practices = new ArrayList<Practice>();
                    currentAthlete.practices.add(practice);
                }
                else {
                    currentAthlete.practices.add(practice);
                }
                currentAthleteRef.child("Practices").setValue(currentAthlete.practices);
                finish();
            }

            @Override
            public void onCancelled(@NonNull DatabaseError databaseError) {
                showProgress("Failed to find Athlete please tell us!!!!!!");
            }
        });
    }

My quick guess is that you're looking to read the data only once, and update it. That is not what your code does however: addValueEventListener adds a semi-permanent listener, that stays actively listening for the data until you remove it.

If you want to only read the value once, and then stop listening for change, you should use:

currentAthleteRef.addListenerForSingleValueEvent(...

Overall though: if the new value depends on the current value in the same database location, you should use a transaction to prevent concurrent updates from overwriting each other.