我收到一条警告，说我已达到非空函数的结尾

Question

我有一个函数可以确定一年是否是闰年，并且我收到警告，但我不确定警告在哪里。 如果是闰年则返回 1，否则返回 0。

int isLeapYear(int yyyy) {
    if (yyyy % 4 == 0) {
        if (yyyy % 100 = 0) {
            if (yyyy % 400 == 0) {
                return 1;
            }
            else {
                return 0;
            }
        }
        else {
           return 1;
        }
    }
}

Answer 1

其他人指出了您的错误并提供了其他编码方法。 让我提供另一个。 这是一种早期返回方法，可以使代码更扁平、更短且更易于理解：

bool is_leap(int y) {
    if (y % 400 == 0) return true;
    if (y % 100 == 0) return false;
    return y % 4 == 0;
}

Answer 2

如果yyyy % 4 == 0计算结果为 false， if (yyyy % 4 == 0) { ... }主体被跳过，则您什么都不返回。 添加一个return 0; 到最后。

这是一个不同的实现：

#include <stdbool.h>

bool isLeapYear(int const year) {
  if (year % 4) return false;
  if (year % 100) return true;
  return !(year % 400);
}

Answer 3

根据您编写此函数的方式，可以退出而不return任何内容。

int isLeapYear(int yyyy) {
    if (yyyy % 4 == 0) {
        ...
    }

    <--- Nothing Is Returned Here
}

你应该做这样的事情：

bool isLeapYear(int yyyy)
{
    bool leapYear = false;

    if (yyyy % 4 == 0) {
        // If Leap Year Set leapYear To true
    }

    return leapYear;
}

这样做的好处是，只有一个退出点可以设置默认return值并且更易于阅读。