Foro de elhacker.net

Buenas compañeros, el code es el siguiente:

import java.util.*;
public class array0301
{
    private double[] alturaAlumnos;
    private double alturaMedia;
    private int indexArray;
    private Scanner entradaTeclado;
 
    public array0301()
    {
        this.alturaMedia = 0;
        this.indexArray = 0;
        this.entradaTeclado = new Scanner(System.in);
    }
 
    public void setAlturas()
    {
        while(entradaTeclado.hasNext())
        {
            if(entradaTeclado.hasNextDouble())
            {
                alturaAlumnos[indexArray] = entradaTeclado.nextDouble();
                indexArray += 1;
            }
        }
    }   
    public void getAlturas()
    {
        indexArray = 0;
        while(indexArray <= alturaAlumnos.length)
        {
            alturaMedia = alturaMedia + alturaAlumnos[indexArray];
            indexArray += 1;
        }
        System.out.format("La altura media de %d alumnos es de %.3f metros",alturaAlumnos.length,(alturaMedia / alturaAlumnos.length));
    }    
    public static void main(String args[])
    {
        array0301 p1 = new array0301();
        p1.setAlturas();
        p1.getAlturas();
    }     
}

El error que lanza es NullPointerException en las lineas 22 y 40 y no consigo saber por qué se produce.

Un saludo

Error de capa 8 :P

NullPointerException  es ocasionado porque tratas de ingresar a un indice que no existe en el arreglo.

Saludos.

Jaja si error mio sé que es, pero no entiendo porque no existe el índice si la variable alturaAlumnos la declaro como un array de tamaño indefinido y en la pedida de datos le señalo el índice con la variable indexArray que tiene valor 0 en la primera vuelta del bucle.

Saludos y gracias Darhius :)

PD: Ya está, he probado a definir un tamaño en la definición del array, así si funciona.

Codigo nuevo:

import java.util.*;
public class array0301
{
    private double[] alturaAlumnos;
    private double alturaMedia,alturaCero;
    private int indexArray,contadorEntradas;
    private Scanner entradaTeclado;
 
    public array0301()
    {
        this.contadorEntradas = 0;
        this.alturaCero = 0;
        this.alturaAlumnos = new double[1000];
        this.alturaMedia = 0;
        this.indexArray = 0;
        this.entradaTeclado = new Scanner(System.in).useLocale(Locale.US);
    }
 
    public void setAlturas()
    {
        while(entradaTeclado.hasNext())
        {
            if(entradaTeclado.hasNextDouble())
            {
                alturaAlumnos[indexArray] = entradaTeclado.nextDouble();
                indexArray += 1;
                contadorEntradas += 1;
            }
            else break;
        }   
 
    }   
    public void getAlturas()
    {
        indexArray = 0;
        if(contadorEntradas == 0)
        {
            System.out.format(Locale.US,"La altura media de %d alumnos es de %.3f metros.",contadorEntradas,alturaCero);
        }
        else
        {
            while(indexArray <= contadorEntradas)
            {
                alturaMedia = alturaMedia + alturaAlumnos[indexArray];
                indexArray += 1;
            }     
            System.out.format(Locale.US,"La altura media de %d alumnos es de %.3f metros.",contadorEntradas,(alturaMedia / contadorEntradas));
        }
    }    
    public static void main(String args[])
    {
        array0301 p1 = new array0301();
        p1.setAlturas();
        p1.getAlturas();
    }     
}
 

En realidad el NullPointerException te daba por que definías la referencia al objeto pero no lo creabas , por lo tanto la referencia era nula , no hace falta que inicialices el array en el constructor , igual te interesa inicializarlo en otro punto, lo que siempre debes tener en cuenta es que antes de usarlo , debes asegurarte de que has creado el objeto, que no estás usando una referencia nula.


Saludos