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) {
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) {
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();
}
});
}
}