feedback de este código (muy muy simple)

H

hioks #1 Dic '20

Hola
simplemente me gustaría saber hasta qué punto está bien este código, la otra opción fácil y sencilla en vez de un nest es crear otra función, lo cual personalmente veo innecesario.

es una tonteria de codigo y de duda, lo sé, pero me vendría bien una respuesta.

bueno, el codigo es muy sencillito, pero basicamente lo que hago es recorrer un array y hacer un print de una letra según el valor de los indices, por lo tanto si
tengo un array [2, 5], el programa printea AA y AAAAA.

qué pensáis?

_ _

i = 0;

 while (i < array.length) {
  for (int w = 1; w <= array[i]; w++) {
   System.out.print("A");
            }
            i++;
        }

Hipnos #2 Dic '20 Dungeon Master

Te falta el retorno de carro porque eso solo va a escribir un chorro de As seguidas.

daniiee_93 #3 Dic '20

Así por encima el código esta bien, lo que comenta Hipnos te van a salir todos los prints en la misma linea si no haces println.

Para simplificarlo a la hora de entender el código podrías hacer algo así y te queda super resumido y entendible aunque por detrás seguramente haga un bucle For:

int[] array = {2,5};

Arrays.stream(array).forEach(number -> {
    System.out.println(StringHelper.repeat('A',number));
});

JuAn4k4 #4 Dic '20

Por qué se llaman array y w las variables?
Puedes dividir en dos métodos para evitar el doble loop.

array.foreach(x => printA(x)) 

fn printA(times: int) {
  for 0..times { print("A") }
  println("") 
}

feedback de este código (muy muy simple)

Usuarios habituales

Tags