Tu ejercicio tiene muchos errores y lo has vuelto muy complicado sin necesidad. Se puede hacer más corto, fácil y organizado.
Por qué tu clase Principal está prácticamente vacía? Deberías tener el menú aquí.
Por qué cuando haces una instancia de Datos, todos los argumentos son
null
? Esto no es una buena práctica, si todos seránnull
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.En la clase Principal tienes un método llamado
Menu()
que no hace nada? No sería mejor removerlo?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 tienesempresa ="";
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.En tu método
MostrarDatos()
se te puede presentar el errorjava.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 delfor
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.