Skip to main content
4 de 5
se agrego codigo
David Minaya
  • 3.2k
  • 1
  • 10
  • 19

El problema esta en que estas almacenado los valores de los items seleccionados en variables (coseguro, estado y nombre) y cada vez que seleccionas un nuevo item el valor de esa variable se actualiza. Por lo que en el for lo que haces es obtener el mismo valor de las mismas variables varias veces, lo que da la ilusión de solo obtienes el valor del ultimo item, ya que el ultimo valor que le asignas a las variables es el del ultimo item. Por lo que en tu base de datos estas actualizando el mismo valor varias veces. Es algo que lo puedes comprobar fácilmente si en el for imprimes el valor de las variables con la sentencia Log.i().

Para solucionar tu problema debes almacenar los valores de las variables en una lista y después en el for recorres la lista y obtiene los diferentes valores que esta contiene, y procedes a almacenarlos en tu base de datos.

ConfiguracionOSAdapter

public class ConfiguracionOSAdapter extends RecyclerView.Adapter<ConfiguracionOSHolder> implements Filterable {

    ...

    public List<String> coseguro = new ArrayList<>(); 
    public List<String> estado = new ArrayList<>(); 
    public List<String> nombre = new ArrayList<>();

    ...

    @Override
    public void onBindViewHolder(final ConfiguracionOSHolder holder, int position) {
        holder.nombreOS.setText(OS.get(position).getNombre());
        holder.estado.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
            @Override
            public void onItemSelected(AdapterView<?> parent, View view, int position, long id) {
                
                // Si a a lista no se ha añadido ningún elemento, el método
                // remove() lanzara una exepcion y se saltara al catch, donde
                // se añadirán los primeros elementos a lista. Si a la lista
                // ya se le había agregado un elemento, se removerá el elemento
                // con el método remove() y se agregara uno nuevo en su posición. 
                try {
                    estado.remove(position);
                    nombre.remove(position);
                    estado.add(holder.estado.getSelectedItem().toString());
                    nombre.add(holder.nombreOS.getText().toString());
                } catch (IndexOutOfBoundsException i) {
                    estado.add(holder.estado.getSelectedItem().toString());
                    nombre.add(holder.nombreOS.getText().toString());
                }

                if (position == 1) {
                    holder.coseguroOS.setEnabled(true);

                }else {
                    holder.coseguroOS.setText("");
                    holder.coseguroOS.setEnabled(false);
                }
            }

            ...
        });

        holder.coseguroOS.addTextChangedListener(new TextWatcher() {
            public void afterTextChanged(Editable s) {

                try {
                    coseguro.remove(position);
                    coseguro.add(holder.coseguroOS.getText().toString());
                } catch (IndexOutOfBoundsException i) {
                    coseguro.add(holder.coseguroOS.getText().toString());
                }
            }

            ...
        });
    }

    ...
}

En tu actividad obtienes los valores de las listas y los almacenas en tu base de datos. Pero en vez de recorrer las listas con el numero de items total del recyclerView, es mejor que lo hagas con el numero de elementos de una de las listas, ya que el numero de elementos de las lista puede ser menor al numero de items del recyclerView. Eso te ahorrara algunos errores.

ConfiguracionOS

public class ConfiguracionOS extends AppCompatActivity {

    ...

    String nombre, est, cos;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        
        ...

        actualizar.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                ControladorObraSocial controlador = new ControladorObraSocial(context);

                // Obtienes el tamaño de la lista nombre. Este es el valor 
                // que utilizaras para recorrer todas las listas
                contador = osView.getAdapter().nombre.size();

                // Obtienes los valores de las listas y los almacenas en 
                // base de datos.
                for (int i=0; i < contador; i++ ) {
                nombre = adapter.nombre.get(i);
                cos = adapter.coseguro.get(i);
                est = adapter.estado.get(i);
                controlador.setDatos(nombre, est, cos);
                controlador.cargarDatos();
               }
               finish();
            }
        });
    }
}
David Minaya
  • 3.2k
  • 1
  • 10
  • 19