Ejercicio 13

[ijaureguialzo]

Observaciones

(1)

Para controlar el bucle do-while, en vez de una variable numérica usa un boolean, queda más clara su función. Una variable numérica da a entender que vas a contar algo.

(2)

En vez de escribir esto:

if (new String(x).equals("par")==true) {

Escribe:

if (x.equals("par")) { // .equals() ya devuelve true o false

Queda mucho más claro.

(3)

En este bloque:

if (new String(x).equals("par")==true) {
                    System.out.println("Los 100 primeros números pares son:");
                    int vueltas = 99;
                    int par = 0;
                    while (vueltas > 0) {
                        System.out.print(par+", ");
                        par = par + 2;
                        --vueltas;
                    }
                    System.out.println("198.");
                    ++end;
                } else {
                    if (new String(x).equals("impar")==true) {
                        System.out.println("Los 100 primeros números impares son:");
                        int vueltas = 99;
                        int impar = 1;
                        while (vueltas > 0) {
                            System.out.print(impar+", ");
                            impar = impar + 2;
                            --vueltas;
                        }
                        System.out.println("199.");
                        ++end;
                    } else {

¿No hay mucha repetición? Hay muy poco que cambie de uno a otro si te fijas.

Mejoras necesarias

• Reescribe el bloque (3) usando solo un if y un for.

[danieltamargo]

Lo acabo de rehacer. A ver que pinta tiene ahora.
Como me gusta mantener las distintas versiones para ver los cambios en un momento dado lo he resubido sin eliminar el anterior, la carpeta del ejercicio "mejorado" se llama "Ejercicio13-100ParImparV2" y la carpeta del ejercicio obsoleto "version1confallos".

[ijaureguialzo]

No hace falta que subas nuevas copias, puedes ver como estaba el archivo .java en cualquier versión anterior en GitKraken usando File History.

[ijaureguialzo]

El for también admite esto:

 for (vuelta = 1; vuelta < 100; vuelta += 2) {

Y avanza de dos en dos.

El if del final es un poco raro, si cambiamos el límite del bucle, hay que calcular a mano los números. ¿No hay otro modo de conseguir esto?

[danieltamargo]

Lo he vuelto a resubir, tiene mejor pinta ahora