La condición i < s.Length
va a ser siempre cierta porque el valor de i
empieza por s.Length - 1
y va decreciendo hasta que al llegar a -1
genera la excepción.
Deberías cambiar el bucle para que se ejecute mientras el valor de i
sea 0 o superior:
while (i >= 0)
{
string s1 = new string(s[i], 4);
Console.WriteLine(s1);
i = i - 1;
}
Otra manera de conseguir el mismo resultado sin hacer uso de bucles sería como muestro a continuación:
// Convertimos la cadena a una matriz de caracteres
char[] sArray = s.ToCharArray();
// Invertimos el orden de los elementos (caracteres) de la matriz
Array.Reverse(sArray);
// Imprimirmos el resultado repitiendo cada carácter en una línea separada
Console.WriteLine(
string.Join(
Environment.NewLine,
Array.ConvertAll(sArray, letra => new string(letra, 4))
)
);
Para imprimir el resultado hemos iterado los elementos de la matriz con caracteres, generando una cadena de caracteres con la repetición solicitada y, por último, colocando una nueva línea entre cada carácter repetido 4 veces.
PD: Gracias a Jose A. Fernandez por sugerir la primera parte de la mejora.