El parámetro alfabeto
en realidad no es un array, sino, un puntero.
int eso(char letra, int alfabeto[]) { ... }
Es decir, el compilador convierte el segundo parámetro a un puntero:
int eso(char letra, int* alfabeto) { ... }
Sabiendo esto, el resultado que devuelve sizeof(alfabeto)
en una máquina de 32 bits sería 4
bytes (que básicamente es el tamaño en bytes que consume el puntero) y el sizeof(*alfabeto)
devolvería la cantidad de bytes que ocupe la variable al que apunte el puntero alfabeto
, que en este caso, sería 4
bytes.
Sabiendo todo esto, el valor que recibe la variable longitud
sería 1
:
int longitud = sizeof(alfabeto) / sizeof(*alfabeto);
int longitud = 4 / 4
int longitud = 1
Ese es el problema, esa expresión no calcula la longitud del array. Evidentemente, este resultado solo sucedería en una máquina de 32 bits.
El resultado en una máquina de 64 bits sería:
int longitud = sizeof(alfabeto) / sizeof(*alfabeto);
int longitud = 8 / 4
int longitud = 2
Entendiendo lo anterior, podríamos responder a esta duda:
Este programa deberia poder diferenciar si la primera letra de mi palabra es minuscula o mayuscula, pero solo lo hace si la primera letra es una 'a' o 'A'
Claro, solo funcionará para las letras 'a'
y 'A'
porque la longitud que calculaste fue 1
y no permitirá recorrer todo el array.
La solución a este problema, es pasar la longitud del array a un parámetro de la función:
#include <stdio.h>
int ascii(char letra)
{
int valorAscii = letra;
return valorAscii;
}
int eso(char letra, int alfabeto[], int longitud)
{
for(int i = 0; i < longitud; i++)
{
if(ascii(letra) == alfabeto[i])
{
return 1;
}
}
return 0;
}
int main()
{
int may[] = {65,66,67,68,69,70,71,72,73,74,75,76,77,78,79,80,81,82,83,84,85,86,87,88,89,90};
int min[] = {97,98,99,100,101,102,103,104,105,106,107,108,109,110,111,112,113,114,115,116,117,118,119,120,121,122};
char palabra[20];
printf("INGRESE UNA PALABRA \n");
scanf("%s",&palabra);
fflush(stdin);
printf("%d \n", eso(palabra[0], may, sizeof may / sizeof *may));
printf("%d \n", eso(palabra[0], min, sizeof min / sizeof *min));
return 0;
}
Por supuesto, puedes ocultar esa expresión (el que calcula la longitud del array) en una macro:
#define ESO(letra, array) eso(letra, may, sizeof array / sizeof *array))
Y luego lo puedes usar de esta forma:
#include <stdio.h>
#define ESO(letra, array) eso(letra, may, sizeof array / sizeof *array)
int ascii(char letra)
{
int valorAscii = letra;
return valorAscii;
}
int eso(char letra, int alfabeto[], int longitud)
{
for(int i = 0; i < longitud; i++)
{
if(ascii(letra) == alfabeto[i])
{
return 1;
}
}
return 0;
}
int main()
{
int may[] = {65,66,67,68,69,70,71,72,73,74,75,76,77,78,79,80,81,82,83,84,85,86,87,88,89,90};
int min[] = {97,98,99,100,101,102,103,104,105,106,107,108,109,110,111,112,113,114,115,116,117,118,119,120,121,122};
char palabra[20];
printf("INGRESE UNA PALABRA \n");
scanf("%19s", palabra);
printf("%d \n", ESO(palabra[0], may));
printf("%d \n", ESO(palabra[0], min));
return 0;
}
Y de esta forma ya no necesitas pasar explicitamente la longitud del array, sino, la macro lo hace por ti.
De igual manera, esta solución no es tan eficiente, ya que necesita tener el alfabeto en memoria y de paso, debe realizar una búsqueda lineal.
Una solución más simple sería:
#include <stdio.h>
#define esMay(letra) (letra >= 'A' && letra <= 'Z')
#define esMin(letra) (letra >= 'a' && letra <= 'z')
int main()
{
char palabra[20];
printf("INGRESE UNA PALABRA \n");
scanf("%19s", palabra);
printf("%d \n", esMay(palabra[0]));
printf("%d \n", esMin(palabra[0]));
return 0;
}
Observaciones:
1.- En este caso no necesitas limpiar el búfer del teclado y si fuera necesario, no uses la función fflush, ya que esta rutina es expresamente para vaciar el búfer stdout y no para el stdin.
Una forma estándar de limpiar el búfer stdin sería:
int ch;
while((ch = getchar()) != '\n' && ch != EOF);
2.- Sí vas a usar la función scanf para leer cadenas, acostumbra especificar el tamaño del array en el primer parámetro, así evitas un posible desbordamiento de búfer.
//No se coloca 20, porque se debe dejar un espacio para el caracter nulo.
scanf("%19s", palabra);
3.- No hace falta anteponer el ampersand en este caso:
scanf("%s", &palabra);
Ya que el identificador palabra
es un alias de la dirección base del array. Por ende, en ambos casos darán el mismo resultado:
printf("%p = %p\n", &palabra, palabra);
int longitud=sizeof(alfabeto)/sizeof(*alfabeto)
en el sitio en donde lo usas. Y podemos seguir hablando de cual era tu problema :-)