c - 选择最大数并对其进行平方或立方时结果不正确

Question

已关闭。此问题需要 debugging details 。目前不接受答案。

---

编辑问题以包含 desired behavior, a specific problem or error, and the shortest code necessary to reproduce the problem 。这将有助于其他人回答问题。

已关闭 3 年前。

Improve this question

我希望这个程序选择最大的数字，然后根据用户的输入对其进行平方或将其放在立方体上。我知道可以使用 pow 数学函数，但我正在尝试尽可能多地使用函数来掌握它。我已经制作了一个程序，可以正确显示最大的数字，并且效果很好。我不想复制它，所以我从头开始编写它，现在我正在检查所有内容，它几乎相同，但即使对于 MAX 也显示错误的结果。我需要修复什么？

int max(int NUM1, int NUM2, int NUM3; )
{
    int NUM1, NUM2, NUM3;
    int X;
    if (NUM1 >= NUM2) {
        X = NUM1;
    } else {
        X = NUM2;
    }
    ;
    if (NUM3 >= X) {
        X = NUM3;
    }
    ;
    return X;
}

int square(int MAX)
{
    return MAX * MAX * MAX;
}

int cube(int MAX)
{
    return MAX * MAX;
}

int main()
{
    int NUM1, NUM2, NUM3, MAX;
    char A;
    printf("ENTER 3 NUMBERS: ");
    scanf("%d %d %d", &NUM1, &NUM2, &NUM3);
    MAX = max(NUM1, NUM2, NUM3);
    printf("LARGEST NUMBER: %d", MAX);
    printf("DO YOU WANT IT SQUARED OR CUBED? (S/C) ");
    scanf("%c", &A);
    if (A = "S") {
        printf("SQUARE: %d", square(MAX));
    } else {
        printf("CUBE: %d", cube(MAX));
    }
    return 0;
}

Answer 1

您的源代码中存在一些错误和问题。如果您将警告级别设置为最高，您会发现其中的大多数。特别是作为初学者，您应该这样做，例如对于 GCC，请使用选项 -Wall -Wextra如果你喜欢-pedantic .

1. 没有 #include <stdio.h>其中声明 printf()和scanf() .

2. max()的参数列表结尾有一个分号(至少)会激怒 GCC。 Clang 对此的提示是正确的。必须将其删除。

3. 函数max()具有参数和局部变量 NUM1 , NUM2 ，和NUM3 。后者掩盖了前者，换句话说，您无法访问参数。此外，局部变量的值未定义。

4. ; 后面有空语句(单个分号 if )位于max() 。他们不作恶，但会打扰普通读者。

5. square()的公式和cube()交换了。

6. 在结果输出后添加换行符，如下所示:"LARGEST NUMBER: %d\n"以视觉方式分离输出和跟随提示。

7. scanf()使用起来非常棘手，最好避免使用。对于读取单个字符尤其如此。我必须在“%c”之前插入一个空格来跳过数字输入行的换行符。

8. 表达式 (A = "S")是指针的赋值const char *到一个角色char 。但是你想比较两个字符，所以它必须是 (A == 'S') 。 C 区分字 rune 字和字符数组文字。

9. 为函数和参数和变量选择小写名称。正如其他人提到的，所有大写名称都根据约定保留给常量。

10. 请适应良好的代码风格(如果你问你最喜欢的搜索引擎，会有很多这样的代码风格)并坚持下去。