0

如何比较两个小时？我尝试使用下面的代码，但它给了我两次true，但它应该给false和true：

#include <iostream>
#include <cstdlib>
#include <cmath>
using namespace std;

bool earlierEqual(int h1, int m1, int s1, int h2, int m2, int s2)
{
    if(h1 <= h2)
    {
        return true;
    }
    else
    {
        if(m1 <= m2)
        {
            return true;
        }
        else
        {
            if(s1 <= s2)
            {
                return true;
            }
            else
                return false;
        }
    }
}

bool laterEqual(int h1, int m1, int s1, int h2, int m2, int s2)
{
    if(h1 >= h2)
    {
        return true;
    }
    else
    {
        if(m1 >= m2)
        {
            return true;
        }
        else
        {
            if(s1 >= s2)
            {
                return true;
            }
            else
                return false;
        }
    }
}

int main()
{
    int h1 = 12, m1 = 4, s1 = 29;
    int h2 = 11, m2 = 12, s2 = 1;

    // false
    cout << earlierEqual(h1, m1, s1, h2, m2, s2) << "\n";
    // true
    cout << laterEqual(h1, m1, s1, h2, m2, s2) << "\n";


    return 0;
}
4

6 回答 6

3

else只有当时间相等时，您的分支才应该被激活。否则，分钟的比较将决定小时h1是否大于小时h2。您应该将代码更改为以下内容：

bool earlierEqual(int h1, int m1, int s1, int h2, int m2, int s2)
{
    if (h1 < h2)
    {
        return true;
    }
    else if (h1 == h2)
    {
        if (m1 < m2)
        {
            return true;
        }
        else if (m1 == m2)
        {
            if (s1 < s2)
            {
                return true;
            }
        }
    }

    return false;
}
于 2013-01-29T16:38:47.060 回答
1

如果小时数相等，则必须检查分钟数，如果小时数相等，则必须检查秒数。只有当有less条件时，才能立即返回true。同样适用于第二个功能：只有更大的情况下，您才能提前返回。

于 2013-01-29T16:37:08.097 回答
1

在进行比较之前，在几秒钟内转换所有内容会更容易。

于 2013-01-29T16:37:21.373 回答
1

我就是这样做的。它更具可读性且不易出错。转换为秒，然后进行比较。

#include <iostream>
#include <cstdlib>
#include <cmath>
using namespace std;

bool earlierEqual(int h1, int m1, int s1, int h2, int m2, int s2)
{
    int totalSeconds1 = getTotalSeconds(h1, m1, s1);
    int totalSeconds2 = getTotalSeconds(h2, m2, s2);

    if(totalSeconds1 <= totalSeconds2)
    {
        return true;
    }
    else
    {
        return false;
    }
}

bool laterEqual(int h1, int m1, int s1, int h2, int m2, int s2)
{
    int totalSeconds1 = getTotalSeconds(h1, m1, s1);
    int totalSeconds2 = getTotalSeconds(h2, m2, s2);

    if(totalSeconds1 >= totalSeconds2)
    {
        return true;
    }
    else
    {
        return false;
    }
}

bool getTotalSeconds(int h1, int m1, int s1)
{
   return h1 * 3600 + m1 * 60 + s1;
}
于 2013-01-29T16:50:09.073 回答
1

使用std::tie：

#include <tuple>
bool earlierEqual(int h1, int m1, int s1, int h2, int m2, int s2) {
  return std::tie(h1, m1, s1) <= std::tie(h2, m2, s2);
}
bool laterEqual(int h1, int m1, int s1, int h2, int m2, int s2) {
  return std::tie(h1, m1, s1) >= std::tie(h2, m2, s2);
}
于 2013-01-29T17:20:33.467 回答
0

只是一个建议：您可以用一个 32 位整数表示小时+分钟+秒。（小时是 0-24 - 5 位，秒 0-60：6 位分钟 0-60：6 位）

一旦你将两个数字都放在两个整数中，你基本上把逻辑放在这个块中（你需要位掩码来提取小时、分钟、秒）

伪码如下：

bool compare(val1,val2) { 

if(h1 < h2) return true; 
if( h1 ==  h2 && m1 < m2 )  return true;
if( h1 == h2 && m1 == m2 && s1 <s2) return true;
return false  ;
}
于 2013-01-29T17:20:54.130 回答