Skip to main content
1 de 4
RRGT19
  • 2k
  • 10
  • 37
  • 63
  1. Tu ejercicio tiene muchos errores y lo has vuelto muy complicado sin necesidad. Se puede hacer más corto, fácil y organizado.

  2. Por qué tu clase Principal está prácticamente vacía? Deberías tener el menú aquí.

  3. Por qué cuando haces una instancia de Datos, todos los argumentos son null? Esto no es una buena práctica, si todos serán null entonces, simplemente no los pongas y para permitir la instancia de la clase Datos, construyes un constructor vacío en dicha clase. Seguro podrás investigar qué es un constructor vacío.

  4. En la clase Principal tienes un método llamado Menu() que no hace nada? No sería mejor removerlo?

  5. En el constructor que tienes en la clase Datos no estás guardando los valores que tienen tus variables. Cuando ingresas los datos de la empresa y haces esto new Datos(empresa, domicilio, .....), estás haciendo una llamada al constructor de la clase. Luego dentro del constructor tienes empresa =""; el cual no está haciendo nada, simplemente limpiando la variable, es decir, no estás asignando los datos de la empresa que solicitaste en ningún lado.

  6. En tu método MostrarDatos() se te puede presentar el error java.lang.NullPointerException. Qué pasaría si ingresas solamente 1 empresa y tratas de Listar Empresas? Aquí aparecerá el error mencionado, te muestro a continuación el por qué. Para solucionar esto hay que tener una forma de salir del for una vez la primera empresa se haya mostrado, de lo contrario, el error aparecerá y tu programa fallará.

Método MostrarDatos() incorrecto para mostrar más de 1 empresa

void MostrarDatos() {

// Si solo tenemos 1 empresa, la variable contador = 1

        for (int i = 0; i < vector.length; i++) {

            System.out.println("Empresa " + i + ": " + vector[i].getEmpresa());
            ..
            ...
            ....
            .....
            // La primera iteración será exitosa pero...
            // En la segunda iteración, es decir, dentro de vector[i + 1]
            // no habrá datos para mostrar y se presentará el error
        }
    }

Método MostrarDatos() correcto para mostrar 2 empresas (una forma rápida de hacerlo)

void MostrarDatos() {

        for (int i = 0; i < vector.length; i++) {

            System.out.println("Empresa " + i + ": " + vector[i].getEmpresa());
            System.out.println("Domicilio " + i + ":" + vector[i].getDomicilio());
            System.out.println("Codigo Postal " + i + ":" + vector[i].getCod_postal());
            System.out.println("Localidad " + i + ":" + vector[i].getLocalidad());
            System.out.println("Pais " + i + ":" + vector[i].getPais());

            // Si solo hay 1 empresa
            // Salir del for para evitar el error "java.lang.NullPointerException"
            if (contador == 1) {
                return;
            }
        }
    }

Constructor arreglado para tu clase Datos:

public Datos(String empresa, String domicilio, String cod_postal, String localidad, String pais) {
        super();
        this.empresa = empresa;
        this.domicilio = domicilio;
        this.cod_postal = cod_postal;
        this.localidad = localidad;
        this.pais = pais;
    }

Con este cambio en el constructor, se mostrará correctamente los datos de las empresas que ingreses.

Espero haberte ayudado.

RRGT19
  • 2k
  • 10
  • 37
  • 63