Skip to main content
4 de 4
cuerpo editado
RRGT19
  • 2k
  • 10
  • 37
  • 63
  1. Tu ejercicio se puede hacer más corto, fácil y organizado usando ArrayList de tipo Datos para guardar más de una empresa y poder trabajar con ellas. Podrás agregar y remover rápidamente, una pequeña investigación sobre esto te será de ayuda.

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

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.

Método BuscarDatos()

void BuscarDatos() {
        
        boolean encontrada = false;
        
        do {
            System.out.println("\nNombre de la empresa:");
            String nombre = leer.nextLine();
            
            for (int i = 0; i < vector.length; i++) {

                // Si la empresa fue encontrada
                if (vector[i].getEmpresa().equals(nombre)) {
                    System.out.println("\nEmpresa encontrada...");
                    System.out.println("Domicilio: " + vector[i].getDomicilio() + "\n");
                    encontrada = true; // Salir del do-while
                    break; // Salir del for
                }
            }
            
            // Si la empresa no fue encontrada
            if (!encontrada) {
                System.out.println("Empresa no encontrada!\n");
            }
            
        } while (!encontrada);
    }

Con este método llamado BuscarDatos, si tienes las dos empresas podrás buscar cualquiera de ellas por su nombre pero, hay un problema, el mismo que te mencioné anteriormente sobre NullPointerException. Si solamente ingresas 1 empresa, a la hora de buscar por su nombre lo va a encontrar correctamente pero, si ingresas un nombre erróneo, el error aparecerá. Para evitar estos errores puedes comprobar antes de entrar al for si vector tiene información o no y cuánta información tiene, si es 1 empresa o tiene 2.

Como puedes ver, no te soluciono todo el problema ya que realmente arreglarlo es algo bien fácil, solo debes pensar un poco y ese será tu trabajo para que vayas desarrollando lógica.

Espero haberte ayudado.

RRGT19
  • 2k
  • 10
  • 37
  • 63