El programa tiene varias fallas:
1. Esta línea de código:
scanf("%s", &F);
La variable F
es de tipo char
y usas en el scanf
como especificador de formato un %s
, cuando en realidad debería ser : %c
y esto se debe, porque lo que necesitas pedir es un caracter, no una cadena.
Por lo tanto, el código corregido quedaría así:
//Limpiamos el búfer del teclado antes de pedir el caracter.
while(getchar() != '\n');
printf("¿Que desea hacer con los numeros insertados?");
scanf("%c", &F);
2. Esta expresión:
F == D
Esta mal, debido a que, la variable F
es de tipo char
(guarda el código ASCII de un caracter) y D
es un puntero implícito que apunta hacia al primer elemento del arreglo. Es decir, es evidente que la condición NUNCA dará true
, porque un código ASCII no será igual a una dirección de memoria. Por lo tanto, el programa siempre mandará este mensaje por consola: Ni sumo ni multiplico los numeros
.
Posible solución:
La condición debería ser así:
//La comparación es válida, porque comparamos código ASCII de caracteres.
if(F == 'm')
{
//code
}
if(F == 's')
{
//code
}
else
{
printf("Ni sumo ni multiplico los numeros");
}
Obviamente deberíamos eliminar la declaración de las variable D
y E
, ya que no se necesitan.
3. En la segunda condición faltó anteponer la palabra clave else
.
Es decir, debería ser así:
if(F == 'm')
{
//code
}
else if(F == 's')
{
//code
}
else
{
printf("Ni sumo ni multiplico los numeros");
}
Si no lo agregas, tendrás un error lógico, debido a que, si la primera condición se cumple, la segunda no se cumplirá, pero si el caso contrario y por ende, imprimirá por pantalla el mensaje de: Ni sumo ni multiplico los numeros
.
Otra solución es dejando las declaraciones D
y E
como cadenas, pero necesitarás usar la función strcmp para comparar con la cadena que ingrese el usuario. Adicionalmente, deberás cambiar la declaración de F
, ya que no puede ser una variable de tipo char
, debido a que, solo almacenaría un caracter. Así que para poder guardar múltiples caracteres, necesitamos declararlo como un arreglo de caracteres.
Propuesta:
#include <stdio.h>
#include <conio.h>
int main()
{
float A, B, C;
//Le sumo +1 porque faltaba el espacio para el caracter nulo
char D[11+1] = "multiplicar";
char E[5+1] = "sumar";
char F[12];
printf("Dame un numero A");
scanf("%f", &A);
printf("Dame un numero B");
scanf("%f", &B);
printf("¿Que desea hacer con los numeros insertados?");
scanf("%12s", &F);
if(strcmp(F, D) == 0)
{
C = A * B;
printf("El resultado de la multiplicacion es %f", C);
}
else if(strcmp(F, E) == 0)
{
C = A + B;
printf("El resultado de la suma es %f", C);
}
else
{
printf("Ni sumo ni multiplico los numeros");
}
getche();
return 0;
}
Recomendaciones:
1. Si optamos por la segunda solución, no sería necesario declarar los arreglos D
y E
, debido a que, se puede pasar la cadena literal directamente al segundo parámetro de la función strcmp
.
2do Propuesta:
#include <stdio.h>
#include <conio.h>
int main()
{
float A, B, C;
char F[12];
printf("Dame un numero A");
scanf("%f", &A);
printf("Dame un numero B");
scanf("%f", &B);
printf("¿Que desea hacer con los numeros insertados?");
scanf("%12s", &F);
if(strcmp(F, "multiplicar") == 0)
{
C = A * B;
printf("El resultado de la multiplicacion es %f", C);
}
else if(strcmp(F, "sumar") == 0)
{
C = A + B;
printf("El resultado de la suma es %f", C);
}
else
{
printf("Ni sumo ni multiplico los numeros");
}
getche();
return 0;
}
2. No es recomendable usar la cabecera conio.h, debido a que, no pertenece a la biblioteca estándar de C, por lo tanto, si te interesa la portabilidad (puede que el compilador no lo reconozca y haya errores de compilación), no lo deberías usar.
En vez de usar getche()
, podrías hacer uso de la función getchar() para pausar el programa, debido a que, esta rutina si pertenece a la biblioteca estándar de C, por lo tanto, en cualquier compilador de C debería estar disponible.
Ejemplo:
int x;
printf("Ingrese un numero:");
scanf("%d", &x);
//Limpiamos el búfer del teclado
while(getchar() != '\n');
//Pausamos el programa
getchar();