1

Estan dentro de un for y saca todos los nombres, como lo haría para limiatrlo a 50 o cualquier numero, los nombres los saco de un json.

public  void tarjetas() throws IOException{
   
        JSONObject json = readJsonFromUrl("http://ddragon.leagueoflegends.com/cdn/12.7.1/data/es_ES/champion.json"); 
        Set<String> nombres = json.getJSONObject("data").keySet();

        for (String  nombre: nombres ) {

            JSONObject jsondata = json.getJSONObject("data");
            JSONObject jsname = jsondata.getJSONObject(nombre);
            
            jPanelListadoCampeones.add(new Campeon( jsname.get("name").toString(), jsname.get("title").toString(), nombre));
            
    
        }
    }
2
  • Deberás mirar la documentación de la API que estás consumiendo para ver cómo gestionan la limitación de la respuesta el 3 jun. 2022 a las 7:03
  • Lo saco de un json directamente, no creo q sea eso, lo saca todos el problema es el tiempo que tarda en colocarlos, y mejor entonces limito los personajes, pero no se como limitar el bucle for el 3 jun. 2022 a las 7:13

1 respuesta 1

3

te voy a dar tres opciones, ya que en este caso se puede hacer con un for (menos correcto) o con un while, te voy explicando poco a poco:

Ciñéndonos a tu código con un for-each una opción sería esta:

public  void tarjetas() throws IOException{

    JSONObject json = readJsonFromUrl("http://ddragon.leagueoflegends.com/cdn/12.7.1/data/es_ES/champion.json"); 
    Set<String> nombres = json.getJSONObject("data").keySet();
    //Contador de vueltas que contará el número de vuelta y que se irá incrementando hasta 50 y cuando llegue a 50 rompemos el bucle
    Integer contador=1;

    for (String  nombre: nombres ) {

        JSONObject jsondata = json.getJSONObject("data");
        JSONObject jsname = jsondata.getJSONObject(nombre);
        
        jPanelListadoCampeones.add(new Campeon( jsname.get("name").toString(), jsname.get("title").toString(), nombre));
        //Si el contador llega a 50, rompemos bucle, sino... seguimos, pero incrementamos el contador de vueltas
        if(contador==50){
            break;
        }else{
            contador++;
        }   
    }
}

Pero esto no es del todo correcto, ya que si no vas a recorrer nunca el array completo, hacer un for (String nombre: nombres ) para luego romperlo al de 50 registros, no es una buena práctica, en lugar de eso, se puede usar un while, vamos a ello:

public  void tarjetas() throws IOException{

    JSONObject json = readJsonFromUrl("http://ddragon.leagueoflegends.com/cdn/12.7.1/data/es_ES/champion.json"); 
    Set<String> nombres = json.getJSONObject("data").keySet();
    //Definimos un iterador para recorrer el set
    Iterator<String> iterador= nombres.iterator();
    //Contador que se irá incrementando hasta 50 y cuando llegue a 50 rompemos el bucle
    Integer contador=1;
    String nombreOperar;
    // Mientras el iterador tenga registros que recorrer (puede que no haya 50) y el número de registros recorridos sea menos o igual  a 50, hacemos cosas..., después ya no.       
    while (iterador.hasNext() && contador<=50 ) {
        nombreOperar = iterador.next();
        JSONObject jsondata = json.getJSONObject("data");
        JSONObject jsname = jsondata.getJSONObject(nombreOperar );
        
        jPanelListadoCampeones.add(new Campeon( jsname.get("name").toString(), jsname.get("title").toString(), nombreOperar ));
        //Incrementamos el contador
        contador++;

    }
}

O de manera alternativa podemos usar un for:

public  void tarjetas() throws IOException{

    JSONObject json = readJsonFromUrl("http://ddragon.leagueoflegends.com/cdn/12.7.1/data/es_ES/champion.json"); 
    Set<String> nombres = json.getJSONObject("data").keySet();
    //Convertimos el Set a ArrayList
    List<String> nombresList= new ArrayList<>(nombres);
    String nombreOperar;
    // Buclamos hasta dar 50 vueltas, estructura del for:
    // for (declaracion de variable a buclar; condición de quedada en el bucle; operación incremental)
    for (int i =0; i<50: i++) {
        nombreOperar = nombresList.get(i)
        JSONObject jsondata = json.getJSONObject("data");
        // Operamos con el elemento del array de la posición i
        JSONObject jsname = jsondata.getJSONObject(nombreOperar );
        
        jPanelListadoCampeones.add(new Campeon( jsname.get("name").toString(), jsname.get("title").toString(), nombreOperar)); 
    }
}

Bajo mi humilde opinión, la que más me gusta es la segunda opción, ya que en la tercera hay que hacer una conversión de Set a ArrayList y eso puede ser costoso, y en la primera, es una mala práctica, directamente, la segunda me parece la más eficiente.

10
  • 1
    No has usado un for, has usado un for-each, en este caso un for no creo que sea "mala práctica"... for (int i = 0; i < 50; i++){}. Especialmente práctico si defines el límite como una constante en vez de hardcodearlo en el for, queda fácil de leer y obvio
    – Benito-B
    el 3 jun. 2022 a las 7:33
  • 1
    buena aportación, no había caído, edito respuesta
    – chikebox
    el 3 jun. 2022 a las 7:34
  • pero en la 3a no coge los nombres el 3 jun. 2022 a las 7:41
  • dame un segundo que edito la última parte, que no es correcta, tienes razón (es lo que tiene no probarlo)
    – chikebox
    el 3 jun. 2022 a las 7:42
  • Yaya no te preocupes se agradece la ayuda el 3 jun. 2022 a las 7:43

Tu Respuesta

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

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