Skip to main content
cuerpo editado
Origen Enlace
RRGT19
  • 2k
  • 10
  • 37
  • 63

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

Método BuscarDatos()

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.

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

Método BuscarDatos()

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.

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

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

Método BuscarDatos()

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.

Método nuevo agregado
Origen Enlace
RRGT19
  • 2k
  • 10
  • 37
  • 63
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;break;
            }
        }
    }

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.

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;
            }
        }
    }
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;
            }
        }
    }

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.

se añadieron 105 caracteres en el cuerpo
Origen Enlace
RRGT19
  • 2k
  • 10
  • 37
  • 63
  1. Tu ejercicio tiene muchos errores y lo has vuelto muy complicado sin necesidad. Sese 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á.

  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á.

  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á.

Origen Enlace
RRGT19
  • 2k
  • 10
  • 37
  • 63
Loading