[SOLUCIONADO] Problema con cadena String aleatoria

[sevillano665]

Buenas, mi problema es que tengo que hacer un método que me devuelva un String con una letra aleatoria en mayúscula repetida "n" veces. Como paramétros tiene que recibir tanto un Random como un integer para la longitud de la cadena, pero cuando lo testeo no me devuelve nada. Este es el código:

Código:

public static String cadenaAleatoria(Random rnd, Integer n){
		char[] elementos={'A','B','C','D','E','F','G','H','I','J','K','L','M','N','O','P',
				'Q','R','S','T','U','V','W','X','Y','Z'};

		char[] conjunto = new char[n];
		String cadena = "";

			int r = rnd.nextInt(elementos.length);
			conjunto[0] = elementos[r];
			String letra = conjunto.toString();
			
			while(conjunto.length < n){
				cadena = cadena + letra;
				
			}			
		
		return cadena;
		}

Saludos!

[chuidiang]

Este bucle no termina nunca

Código Java:

while(conjunto.length < n){
                cadena = cadena + letra;
}
 

ya que o bien no entra nunca, o bien entra y no sale ya que dentro no se altera ni conjunto.length ni n.

Se bueno.

[sevillano665]

Tienes razón, lo he modificado pero el resultado es el mismo

Código Java:

public static String cadenaAleatoria(Random rnd, Integer n){
        char[] elementos={'A','B','C','D','E','F','G','H','I','J','K','L','M','N','O','P',
                'Q','R','S','T','U','V','W','X','Y','Z'};
 
        char[] conjunto = new char[n];
        String cadena = "";
 
            int r = rnd.nextInt(elementos.length);
            conjunto[0] = elementos[r]; ////contiene el caracter aleatorio
            String letra = conjunto.toString(); //contiene el caracter aleatorio en String
            
            while(conjunto.length < n){
                cadena = cadena + letra; //suma a la cadena el caracter repetido
                for(int i=1; i < n; i++){ 
                    conjunto[i] = 'A'; //suma 1 la longitud del array
                }
                
            }           
        
        return cadena;
        }
 

Saludos!

[chuidiang]

Y con la modificación, dentro del while ¿donde modificas cadena.length o n para que el while deje de cumplirse?

Esto

conjunto[i] = 'A'; //suma 1 la longitud del array

no modifica la longitud del array, sólo estás rellenándolo de 'A' y la longitud sigue siendo igual.

Se bueno.

[sevillano665]

Ya lo solucioné, el "for" no tenía sentido y en el while tenía que poner la longitud de la "cadena" y no del conjunto.