“如何修复”；错误：控件到达非无效函数的末尾[-Werror=返回类型]”；在HackerRank？

好的，已经提供了很多答案，但我似乎真的找不到解决方案

我们的任务是找到四个整数中最大的一个。

当我运行以下代码时：

#include <stdio.h>

/*
int max_of_four(int a, int b, int c, int d)
{
    if(a>b && a>c && a>d)
    return a;
    
    else if(b>a && b>c && b>d)
    return b;
    
    else if(c>a && c>b && c>d)
    return c;
    
    else
    return d;
}
*/

int max_of_four(int a, int b, int c, int d)
{
    if(a>b)
    {
        if(a>c)
        {
            if(a>d)
            return a;
        }
    }
    
    else if(b>c)
    {
        if(b>d)
        return b;
    }
    
    else if(c>d)
    return c;
    
    else
    return d;
}


int main() {
    int a, b, c, d;
    scanf("%d %d %d %d", &a, &b, &c, &d);
    int ans = max_of_four(a, b, c, d);
    printf("%d", ans);
    
    return 0;
}

但是上面的代码在Dev-C++5.11上运行时完成了它的任务

如果我们在注释中使用函数

max\u of_four

，并注释掉后一个，那么工作也会完成

如果有人能让我知道，如何调整

max\u of_four

以使代码正常工作，我将不胜感激

是什么导致了这个错误

---

这是错误还是警告？如果是这样，为什么警告会被视为错误？

当函数不是无效的，并且您使用

If

语句返回值时，必须检查每个情况是否都得到了真正的处理。在这种情况下，最后一个else不能保证这一点，因为您使用的嵌套语句没有

else

以下是一个固定版本：

int max_of_four(int a, int b, int c, int d)
{
    if(a>b)
    {
        if(a>c)
        {
            if(a>d)
            return a;
        }
    }
    
    if(b>c) //<====== I removed the else here to fix the implicit
            //        else of the previous form that was not handled
    {
        if(b>d)
        return b;
    }
    
    if(c>d) //<===== Idem
    return c;
    
    else
    return d;
}

最后，编写以下代码也会更具可重用性：

int max_of_two(int a, int b) { return a > b ? a : b; }

因为这样您就可以将其用于任意数量的参数：

int max_of_three(int a, int b, int c) { 
    return max_of_two(max_of_two(a, b), c);
}
int max_of_four(int a, int b, int c, int d) { 
    return max_of_two(max_of_two(a, b), max_of_two(c, d));
}
//And so on

已经为以上问题提供了最佳答案

对于即将到来的读者，可以从下面的代码中理解他们的评论：

int max_of_four(int a, int b, int c, int d)
{
    if(a>b)
    {
        if(a>c)
        {
            if(a>d)
                return a;            
            else
                return d;
        }
        else if(c>d)
            return c;
        else
            return d;
    }
    else if(b>c)
    {
        if(b>d)
            return b;
        else
            return d;
    }
    else if(c>d)
    {
        return c;
    }
    else
        return d;
}

代码在HackerRank上运行平稳，没有给出任何错误

这可能就是“数组上的循环”的含义：

int max_of_four(int a, int b, int c, int d)
{
    int i;
    int arr[4]={a,b,c,d};
    int max=arr[0];
    for(i=1;i<4;i++)
    {
        if(max<arr[i])
            max=arr[i];
    }
    return max;
}

intmax\u of四（inta，intb，intc，intd）
{
int i；
int arr[4]={a，b，c，d}；
int max=arr[0]；
对于（i=1；iIf
a
小于
c
或
d
，则不返回任何内容。但此算法并不理想。将数字放入临时数组，然后在数组上循环要简单得多。好吧，@Carcigenicate。我得到了“在数组上循环”的方法。它看起来确实更简单。如果
a
小于
c
或
d
，不返回任何内容。但光标移到下一个else if语句，这听起来是使用if语句的重点。我不明白算法为何不理想，请您详细说明一下？
else if（b>c）
仅与
if（a>b）关联
。内部检查需要自己的
else if
s，或者外部的
else if
s应该更改为
if
s，这样不管结果如何，都可以运行。这就是回答者所做的，尽管循环仍然容易得多。假设您稍后添加了第五个要检查的数字。调整此代码会很混乱。@carci天才哦……好的，明白了。谢谢：）谢谢，@dspr。编译器显示完全相同的错误。我似乎没有指出您在代码中所做的更改，您能指定吗？编辑：我计算了更改。看看我的答案，它现在更完整、更具体了。好的，明白了。这个方法似乎很简单，也很容易修改。谢谢您。：）
int max_of_four(int a, int b, int c, int d)
{
    if(a>b)
    {
        if(a>c)
        {
            if(a>d)
                return a;            
            else
                return d;
        }
        else if(c>d)
            return c;
        else
            return d;
    }
    else if(b>c)
    {
        if(b>d)
            return b;
        else
            return d;
    }
    else if(c>d)
    {
        return c;
    }
    else
        return d;
}

int max_of_four(int a, int b, int c, int d)
{
    int i;
    int arr[4]={a,b,c,d};
    int max=arr[0];
    for(i=1;i<4;i++)
    {
        if(max<arr[i])
            max=arr[i];
    }
    return max;
}