0

Como dice el titulo, tengo un problema al tratar de modificar un atributo de un objeto al momento de ser recorrido; lo que hago es crear un ciclo para recorrer un listado, dentro de este ciclo creo un objeto el cual voy a llenar desde una base de datos SQLite, y al momento de modificar uno de los atributos me sale un error de NullPointerException, lo curioso es que sale el error tras haber hecho ya un ciclo, y si no modifico el objeto los objetos se llenan bien.

selected = "0"+String.valueOf(i+1);
        Log.e("onItemSelected",selected);
        aList = sqLiteAccumulated.getAllAccumulated(selected);

        sqLiteItems = new SQLiteItems(ItemActivity.this);
        pList = new ArrayList<ProductModel>();
        Log.e("aList size", aList.size()+"");
        for (int j = 0; j<aList.size(); j++){
            productModel = new ProductModel();
            productModel = sqLiteItems.readItem(aList.get(j).getMI2_ARTIC());
            productModel.setMI_ACT(Float.parseFloat(aList.get(i).getMI2_ACT()));
            if (productModel!=null){
                pList.add(productModel);
            }
        }

y el log me muestra esto

java.lang.NullPointerException: Attempt to invoke virtual method 'void co.dibso.gomezvelasquezvendedores.Modelo.ProductModel.setMI_ACT(float)' on a null object reference

pero esto ocurre después de haber pasado ya un ciclo.

3
  • Deberias de verificar si el metodo readItem returna null antes de ejecutar el metodo setMI_ACT y asi te evitarias el error.
    – Einer
    Commented el 12 oct. 2017 a las 2:39
  • Mira Cómo preguntar para que tu pregunta sea mejor recibida. También, aprovecha y haz el recorrido de bienvenida para entender mejor cómo funcionamos y de paso obtener tu primera medalla!
    – gbianchi
    Commented el 12 oct. 2017 a las 3:32
  • size no devuelve un elemento mas de lo que tiene el array en realidad?? cuando devuelve un 1.. el elemento es en realidad el 0, el unico que hay.
    – gbianchi
    Commented el 12 oct. 2017 a las 3:33

1 respuesta 1

0

El metodo readItem esta retornando null y aun asi ejecutas el metodo setMI_ACT sin validar que no este null.

Intenta verificar primero si esta null y luego ejecutas el metodo:

    //...
    for (int j = 0; j<aList.size(); j++){
        productModel = new ProductModel();
        productModel = sqLiteItems.readItem(aList.get(j).getMI2_ARTIC());
        if (productModel!=null){
            productModel
            .setMI_ACT(Float.parseFloat(aList.get(i).getMI2_ACT()));
            pList.add(productModel);
        }
    }
2
  • Eso era, se me paso por alto ese punto aun cuando ya estaba validando el producto como tal para agregarlo a la lista, gracias.
    – mlaguirre
    Commented el 12 oct. 2017 a las 13:44
  • Puedes aceptar la respuesta si era lo que esperabas.
    – Einer
    Commented el 12 oct. 2017 a las 16:50

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.