0

la consulta es la siguiente necesito hacer un metodo que me busque por ID los productos de una lista de productos. Como hago yo para recorrer dicha lista y que corte cuando encuentra el producto? Yo lo hice con un foreach, donde por cada producto de la lista guardaba el producto y comparaba el id de ese mismo con el id que me llegaba por parametro, pero claro, me dijeron que no puedo cortar ciclos foreach deben terminar el ciclo entero. Como hago esto pero con un while?

public Producto traerProdcuto(int idProducto) throws Exception{
    for(Producto p:gondola) {
        if(p.getIdProducto()==idProducto) {
            return p;
        }   
    }throw new NullPointerException("No se ha encontrado el producto");

}

Lo que intente fue esto luego fue esto:

public Producto traerProducto(int idProducto)throws Exception {
        boolean found = false;
        int vueltas  = 0;
        
        while(found==false) {
            Producto p = gondola.get(vueltas);
            if(p.getIdProducto() ==idProducto) {
                found =true;
                return p;
            }
            vueltas++;
            
        }
        
        throw new Exception("El producto con la id: "+idProducto+", no se ha encontrado");
    }

El problema que tengo es que sigo interrumpiendo un ciclo sin respetar la condicion, y si no hay ningun producto con ese ID sigue iterando luego de terminada la lista y tira una excepcion

10
  • Cuál lista de productos? Cuál foreach? Las preguntas del tipo "Tengo que hacer esto" sin mostrar el código que has escrito no suelen ser bien recibidas, ya que la respuesta será siempre adivinando y no encaja en el formato del sitio. Recomiendo que hagas el recorrido para entender el sitio y de paso ganar tu primera medalla. También estaría bien que revisaras Cómo preguntar para que recibas respuesta cuanto antes.
    – Alfabravo
    Commented el 16 sept. 2020 a las 17:57
  • Soy ignorante en java. Pero, no creo que pueda existir un lenguaje en el que los ciclos de control no puedan ser interrumpidos y alguien lo use. Con seguridad debe existir en java una instrucción como break; o similar para salirse de un bucle. Y, si no la hay, todavía puedes modificar el contador de iteraciones y hacerlo igual al límite
    – quevedo
    Commented el 16 sept. 2020 a las 17:58
  • A tu pregunta le falta: el código que tienes ahora mismo, los datos que estás intentando usar, la salida esperada, lo que intentaste para resolver lo que te piden. Por favor, usa el enlace 'editar' que está justo debajo, edítala y añade todo esto para que podamos ayudarte
    – Alfabravo
    Commented el 16 sept. 2020 a las 17:58
  • Ahi agregue lo que yo estoy haciendo pero me dijeron que es una mala practica por eso no se como resolverlo de otra manera. Gracias por los consejos! Commented el 16 sept. 2020 a las 18:17
  • 1
    no se quien te dijo que es una mala practica.. tal vez deberias preguntarle a esa persona porque es una mala practica y como resolverlo... si encontraste el item, sali con un break y listo...
    – gbianchi
    Commented el 16 sept. 2020 a las 18:34

2 respuestas 2

1

Lo que estas haciendo deberia funcionar correctamente. Es una cuestion de estilo de programacion ya que ambos daran el mismo resultado.

La forma de hacerlo sin cortar la ejecucion del resto de la funcion es aplicando instruccion break en el for; La instruccion return es final y prevendra que otras lineas despues de tu for incluidas en el metodo traerProducto se ejecuten, pero no quiere decir que este mal.

Por ejemplo, si el objectivo de la funcion traerProducto es encontrar el producto que estas buscando, no veo razon para crear variables y asignarles valores para retornarlos "por fuera del loop"

Ejemplo:

public Producto traerProdcuto(int idProducto) throws Exception {
    Producto productoEncontrado;
    for(Producto p:gondola) {
        if(p.getIdProducto()==idProducto) {
            productoEncontrado = p;
            break;
        }   
    }

    if (!productoEncontrado) {
       throw new NullPointerException("No se ha encontrado el producto");
    }
 
    return productoEncontrado;
}

En una instruccion while el "break" o instruccion de corte es evaluada por cada iteracion

public Producto traerProducto(int idProducto) throws Exception {
    int vueltas  = 0;
    Producto productoEncontrado;

    // Deberias agregar una segunda instruccion de corte en el caso en el que el producto no exista    
    //por ejemplo !productoEncontrado && vueltas >= totalProductos) 
    while(!productoEncontrado) {
        Producto p = gondola.get(vueltas);
        if(p.getIdProducto() ==idProducto) {
            found =true;
            productoEncontrado = p;
        }
        vueltas++;
    }

    return productoEncontrado;
}

Como podras ver el codigo de arriba deberia tener exactamente el mismo resultado, y pero los cambios aplicados son redundantes... Como dato adicional, nunca quieres iterar en vano

0

Imagino que tu profesor requiere que el return este fuera del while, de tal manera que la condición del while interrumpa la iteración, de tal manera que el returnes lo ultimo que se ejecutara, es cuestión de estilo, pero al final seguir las buenas practicas es lo importante, recomiendo revisar los libros de Cleancode y cleancoder

1
  • Esto no va como respuesta sino como comentario.
    – Alfabravo
    Commented el 16 sept. 2020 a las 21:49

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.