¿Por qué el resultado de la condición es falso cuando comparo caracteres?
Miremos el siguiente código:
#include <stdio.h>
int main(void)
{
char A[2];
printf("Fichero con cosas, quiere escrbir algo mas o hacer uno nuevo:");
scanf("%s", A);
if(A=='a')
{
f2=fopen(archivo2,"a");
}
if(A=='w')
{
f2=fopen(archivo2,"w");
}
return 0;
}
En este ejemplo, declaro un array de 2 elementos y le pedimos al usuario una cadena, aunque en realidad el usuario solo debería de ingresar únicamente un caracter.
Entonces, sí el usuario teclea la letra a
, este caracter estará guardado en la posición 0
(aunque en realidad lo que se guarda es el código ASCII del caracter) del array.
scanf se encargará que el caracter nulo (indica el fin de la cadena) esté en la posición 1
.
Hasta ahí, no hay problemas, pero cuando nos topamos en esta condición:
if(A=='a')
Nos damos cuenta que esa condición nunca se cumplirá, porque el identificador A
, representa la dirección de memoria del primer elemento del array. Por lo tanto, estás comparando una dirección con un código ASCII (el de caracter 'a'
). Esa es la razón del porque SIEMPRE da false
la restricción.
Entonces lo que podrías hacer, es usar la función strcmp de la cabecera string.h
:
#include <stdio.h>
#include <string.h>
int main(void)
{
char A[2];
printf("Fichero con cosas, quiere escrbir algo mas o hacer uno nuevo:");
scanf("%s", A);
if(strcmp(A, "a") == 0)
{
f2=fopen(archivo2,"a");
}
if(strcmp(A, "w") == 0)
{
f2=fopen(archivo2,"w");
}
return 0;
}
De este modo, podrás comparar si dos cadenas son iguales. De igual manera, hay que evitar el desbordamiento de búfer en esta línea:
scanf("%s", A);
En el array A
solo podemos guardar un caracter + el caracter nulo, así que si el usuario ingresa más de la cuenta, provocará un comportamiento indefinido, esto significa que el programa puede funcionar como no puede funcionar, el resultado es arbitrario.
Un posible resultado sería sobrescribir la memoria del propio programa. Otro resultado sería intentar leer o escribir en una dirección que ya es usado por otro proceso, ahí en ese caso, lo normal es que el sistema mate el proceso actual.
En fin, debes evitar el desbordamiento de esta manera:
//Sí el usuario ingresa más de 1 caracter, scanf solo leerá uno.
scanf("%1s", A);
En este hilo explico con más detalle como evitar el desbordamiento cuando pides una cadena con scanf
.
Solución 2: En vez de pedir una cadena, podrías pedir únicamente un caracter:
#include <stdio.h>
#include <string.h>
int main(void)
{
//En esta variable guardamos el caracter.
char A;
printf("Fichero con cosas, quiere escrbir algo mas o hacer uno nuevo:");
//Hay que pasar la dirección de memoria de A
scanf("%c", &A);
//Ahora si podemos comparar códigos ASCII :)
if(A == 'a')
{
f2=fopen(archivo2,"a");
}
//Ahora si podemos comparar códigos ASCII :)
if(A == 'w')
{
f2=fopen(archivo2,"w");
}
return 0;
}
De este modo, nos evitamos tres cosas:
1.- No hay que preocuparse por el desbordamiento de búfer.
2.- No necesitamos usar la función strcmp
.
3.- No hay que enfocarse en dejar un espacio para el caracter nulo.