Leap Year Basic C ++ Boolean始终返回true

Question

我编写此代码以确定输入的年份是否为闰年。意思是，那些被4和400整除的是闰年，而100或其他东西则不是。

但是，我的程序总是为布尔值返回true，因此输出将相信每年都是闰年。

到目前为止，这是我的代码：

#include <iostream>
#include <cmath>
#include <string>
#include <iomanip>

using namespace std;
 bool leap_year(int year);
int main()
{
    int year; 
     bool leap_year(int year);
    cout << "Please input the year in question: ";
    cin >> year;
    if (leap_year == false)
    {
        cout << "The year is not a leap year.  ";
    }
    else
    {
        cout << "The year is a leap year. ";
    }
    return 0;
}
  bool leap_year(int year)
{

    if (year % 4 == 0)
    {
        bool leap_year = true;
    }
    else if (year % 400 == 0)
    {
        bool leap_year = true;
    }
    else if (year % 100 == 0)
    {
        bool leap_year = false;
    }
    else
    {
        bool leap_year = false;
    }

    if (bool leap_year = false)
    {
        return false;
    }
    else
    {
        return true;
    }
}

Answer 1

您正在声明一堆局部变量

else if (year % 100 == 0)
{
    bool leap_year = false;
}

一旦}括号，此变量超出范围，您存储的值也是如此。

if (bool leap_year = false)
{
    return false;
}

这是定义一个变量leap_year并将其赋值为false。 if依次会评估为false，因此它总会进入其他条件。

我冒昧地重写了一部分程序。您现在可以看到如何对函数进行调用。同样在函数中，局部变量is_leap_year用于存储返回值，最后返回。我还纠正了逻辑，因为之前的第一个％4检查是正确的，并且没有其他if语句将被执行，这不是你想要的。

#include <iostream>
#include <cmath>
#include <string>
#include <iomanip>

using namespace std;
bool leap_year(int year);
int main()
{
    int year; 
    cout << "Please input the year in question: ";
    cin >> year;
    if (leap_year(year) == false)  //Call the function and check if return is false
    {
        cout << "The year is not a leap year.  ";
    }
    else
    {
        cout << "The year is a leap year. ";
    }
    return 0;
}
  bool leap_year(int year)
{
    bool is_leap_year = false;
    if (year % 4 == 0)
    {
       is_leap_year = true;
    }
    if (year % 100 == 0)
    {
        is_leap_year = false;
    }
    if (year % 400 == 0)
    {
        is_leap_year = true;
    }
    return is_leap_year;
}

Answer 2

应该是  IF（（年％4 == 0＆amp;年％100！= 0）||（年％400 == 0））      返回ISLEAPYEAR。

Answer 3

您的问题是，是否退回true或false的条件是：

if (bool leap_year = false)
{
    return false;
}
else
{
    return true;
}

您要将值false分配给在条件语句中初始化的变量leap_year。赋值运算符已定义为T& T::operator =(const T2& b)，这意味着您只是在评估条件中的false。

解决这个问题的一种方法是在leap_year的顶部声明bool leap_year(int year)而不是每次使用它（这是毫无意义的行为）。所以你的函数应该是这样的：

bool leap_year(int year) {
    bool leap_year = false;

    if (year % 4 == 0) {
        leap_year = true;
    } else if (year % 400 == 0) {
        leap_year = true;
    } else if (year % 100 == 0) {
        leap_year = false;
    }

    if (leap_year == false) {
        return false;
    } else {
        return true;
    }
}

但更好的解决方案是使用the functionality that C++ already provides：

bool leap_year(int year) {
    tm bar = { 0, 0, 0, 29, 1, year - 1900 };

    return static_cast<time_t>(-1) != mktime(&bar) && bar.tm_mday == 29 && bar.tm_mon == 1 && bar.tm_year == year - 1900;
}

[Live Example]

Answer 4

//You can simplify it like this:
bool leap_year(int year)
{
    bool leap_year  = false;
    if ((year % 4 == 0 && year%100 !=0) || year % 400 == 0)
    {
        bool leap_year = true;
    }
    return leap_year; 
}

为避免这种情况：

if (leap_year = false)
    {
        return false;
    }

您可以像这样检查布尔值

if (leap_year)
 {
        // code to execute when leap_year is true
 }