Lo primero sería que de verdad trabajes con un carácter mas bien que con una cadena para la sentencia siguiente:
letra = Console.ReadLine(); // esto te devuelve una cadena que tal vez contenga mas de una letra.
Para cambiar el tipo de la variable letra
a char
en vez de string
, pudieras hacer la conversión algo así como:
letra = char.Parse(Console.ReadLine()); // esto te devuelve un char si de verdad la cadena contiene un solo carácter.
Puedes usar char.TryParse()
también si deseas verificar la cadena antes de convertirla a un char
, pero esa es la idea general.
Una vez que tienes a letra
de tipo char
, ahora puedes comparar con cualquier letra que forma parte de la cadena palabra
usando un índice. Por ejemplo:
if (letra == palabra[0])
{
Console.WriteLine("La primera letra de la palabra corresponde");
}
if (letra == palabra[1])
{
Console.WriteLine("La segunda letra de la palabra corresponde");
}
// ...
Esta idea se puede usar en un bucle también si lo deseas. Por ejemplo:
for (int i = 0; i < palabra.Length; i++)
{
if (letra == palabra[i])
{
Console.WriteLine($"La posición #{i+1} en la palabra corresponde a la letra");
}
}
La ventaja de trabajar usando el índice de la cadena mas bien que usar un Substring
es que te evitas de crear cadenas temporarias innecesarias, una operación relativamente costosa. No creo que en tu programa sea un problema que vayas a notar, pero en programas mas grandes, el crear cadenas temporarias innecesariamente puede afectar el rendimiento de forma notable.