1

Tengo dos botones dentro de mi CardView, el primero borra la CardView y los datos de mi database, este funciona correctamente. Según en que item le des al botón lo borra correctamente.

holder.myButtonDelete.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                //Query deleteQuery = databaseReference.child("personas").orderByChild("name").equalTo(personasList.get(position).toString());
                databaseReference.child("Personas").addListenerForSingleValueEvent(new ValueEventListener() {
                    @Override
                    public void onDataChange(DataSnapshot dataSnapshot) {
                        for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
                            Personas personasTemp = snapshot.getValue(Personas.class);
                            if (personas.getDatospersona().equals(personasTemp.getDatospersona())) {
                                databaseReference.child("Personas").child(snapshot.getKey().toString()).removeValue();
                                personasList.remove(position);
                                notifyDataSetChanged();
                                if (personasList.size() == 0) {
                                    VerPersonasAdministrador.textViewEmptyView.setVisibility(View.VISIBLE);
                                }
                                break;

                            }

                        }

                    }

                    @Override
                    public void onCancelled(DatabaseError databaseError) {

                    }
                });
            }
        });
    }

Y este es mi otro botón, que sirve para cambiar el valor true por false del atributo esNuevo, el problema es que le des al botón en el item que le des, siempre hace la acción en el primer item.

holder.myButtonNuevo.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                //Query deleteQuery = databaseReference.child("personas").orderByChild("name").equalTo(personasList.get(position).toString());
                databaseReference.child("Personas").addListenerForSingleValueEvent(new ValueEventListener() {
                    @Override
                    public void onDataChange(DataSnapshot dataSnapshot) {
                        for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
                            Personas personasTemp = snapshot.getValue(Personas.class);
                            if (personas.getDatospersona().equals(personasTemp.getDatospersona())) {
                                databaseReference.child("Personas").child(snapshot.getKey().toString());
                                snapshot.getRef().child("esNuevo").setValue(false);
                                notifyDataSetChanged();
                                if (personasList.size() == 0) {
                                    VerPersonasAdministrador.textViewEmptyView.setVisibility(View.VISIBLE);
                                }
                                break;

                            }

                        }

                    }

                    @Override
                    public void onCancelled(DatabaseError databaseError) {

                    }
                });
            }
        });
    }
9
  • Hola me parece que el problema es donde estas definiendo los listeners. Tienes algún mensaje en el LogCat?
    – Jorgesys
    el 1 jun. 2017 a las 16:16
  • Nop, como ya te digo, en el primer item funciona perfecto, pero si le das al boton en los demas item hacen la funcion pero en el primer item, aunque le des al 2,3,4... @Jorgesys
    – UserNameYo
    el 1 jun. 2017 a las 16:21
  • entonces el problema no es el Click, te sugiero agregues revises si en realidad se ejecuta el onClick(). @UserNameYo
    – Jorgesys
    el 1 jun. 2017 a las 16:23
  • Eso te pasa luego de hacer el delete con el botón? O antes?. Otra cosa, asegurate de que estas instanciando correctamente los botones (asegurate de que no tengan el mismo ID). el 1 jun. 2017 a las 17:50
  • Sin usar el botón delete, ya que ese funciona bien, el problema está en el segundo botón, que haga click desde el item que lo haga, siempre hace la función en el primer item. @Andrespengineer
    – UserNameYo
    el 1 jun. 2017 a las 17:52

2 respuestas 2

0

Asegura que estos listener los definas dentro del constructor de tu clase ViewHolder :

class ViewHolder extends RecyclerView.ViewHolder{

        ...
        ...
        public ViewHolder(View itemView) {
        ...
          holder.myButtonDelete.setOnClickListener(new View.OnClickListener() {
        ...
          holder.myButtonNuevo.setOnClickListener(new View.OnClickListener() {
        ...
2
  • entonces el problema no es el Click, te sugiero agregues revises si en realidad se ejecuta el onClick().
    – Jorgesys
    el 1 jun. 2017 a las 16:28
  • Si se ejecuta el onClick pero siempre hace la función en el primer item le des al item que le des @Jorgesys he editado la pregunta, me puedes ayudar?
    – UserNameYo
    el 1 jun. 2017 a las 16:40
0

Finalmente lo he conseguido solucionar.

El problema era que no obtenía getKey por lo tanto siempre se hacía el cambio en la misma Persona

Así que quitando esto que no servía

databaseReference.child("Personas").child(snapshot.getKey().toString());
snapshot.getRef().child("esNuevo").setValue(false);

Por:

databaseReference.child("Personas").child(snapshot.getKey().toString()).child("esNuevo").setValue(false);

Ya obtengo getKey y ya no tengo problemas!

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.