Question

我对此很新，我已经尝试了一切，但是我无法让它工作，它只显示“无效的数学运算”，无论我改变什么。请帮我解决这个问题。

#include <stdio.h>
#include <math.h>

int fnCalculate (int fn, int sn);

int main()
{
    int fn, sn;
    char op;
    printf("Enter 2 interger values and math operation [+] [-] [*] : ");
    scanf("%d %d %c", &fn, &sn, &op);
    printf("%d", fnCalculate(fn,sn));
    getch();
    return 0;

}
int fnCalculate (int fn, int sn)
{
char op;
switch(op){

case '+' : printf("%d + %d = %d", fn, sn, fn+sn);break;
case '-' : printf("%d - %d = %d", fn, sn, fn-sn);break;
case '*' : printf("%d * %d = %d", fn, sn, fn*sn);break;
default : printf("Invalid Math Operation");break;

}
}

Answer 1

您必须将操作字符发送到该功能。函数中的op未初始化并保存垃圾值。

void fnCalculate (int fn, int sn, char op)
{
    switch(op){

    case '+' : printf("%d + %d = %d", fn, sn, fn+sn);break;
    case '-' : printf("%d - %d = %d", fn, sn, fn-sn);break;
    case '*' : printf("%d * %d = %d", fn, sn, fn*sn);break;
    default : printf("Invalid Math Operation");break;
    }
}

称之为：

fnCalculate(fn,sn,op);

此外，它应返回int值。它没有返回任何东西，所以那里还有另一个问题。或者将签名更改为void，然后只需调用该函数，而不是printf。

Answer 2

您还需要更改函数定义以接受op的值，并且在调用时，您需要传递op的值。否则，在您当前的功能中，

char op;
switch(op){

正在尝试读取未初始化的自动局部变量，该变量调用未定义的行为。

功能未显示计算答案

2 个答案: