C++编译器错误难以理解

Question

我有一个相当简单的任务要完成。编写一个定义时间的类。但是在其中一个函数中，我遇到了一个我无法理解的错误。

我搜索了很多解决方案，但都没有成功，最终我决定在这里发布它。

time.h

class time
{
private:
    int _hours;
    int _minutes;
    float _seconds;
    bool checkHours(int hours);
    bool checkMinutes(int minutes);
    bool checkSeconds(float seconds);

public:

time(int hours=0, int minutes=0, float seconds=0);
time(const time & tm);
~time();


void hours(int hours);
int  hours() const;
void minutes(int minutes);
int  minutes() const;
void seconds(float seconds);
float  seconds() const;


void operator=(time tm);
bool operator==(time tm);


void print();
time getTimeFromUser();
float getTimeAsFractionOfTheDay(time tm);

};

还有time.cpp

#include <iostream>
#include "time.h"

bool time::checkHours(int hours)
{
    return hours>=0 && hours<24;
}
bool time::checkMinutes(int MS)
{
    return MS>=0 && MS<60;
}
bool time::checkSeconds(float MS)
{
    return MS>=0 && MS<60;
}

//constractors
time::time(int hours, int minutes, float seconds)
{
    if(checkHours(hours) && checkMinutes(minutes) && checkSeconds(seconds))
    {
        _hours=hours;
        _minutes=minutes;
        _seconds=seconds;
    }
    else
    {
        cout<<"Error"<<endl; _hours=-1; _minutes=-1; _seconds=-1;
    }
}
time::time(const time & tm)
{
    _seconds = tm.seconds();
    _hours = tm.hours();
    _minutes=tm.minutes();
}
time::~time()
{
}

//get-set functions
void time::hours(int hours)
{
    _hours=hours;
}
int  time::hours() const
{
    return _hours;
}
void time::minutes(int minutes)
{
    _minutes=minutes;
}
int  time::minutes() const
{
    return _minutes;
}
void time::seconds(float seconds)
{
    _seconds = seconds;
}

float  time::seconds() const
{
    return _seconds;
}

//operators
void time::operator=(time tm)
{

    _hours=tm.hours();
    _minutes=tm.minutes();
    _seconds=tm.seconds();

}
bool time::operator==(time tm)
{
    return _hours=tm.hours() && _minutes==tm.minutes() && _seconds==tm.seconds();
}

//some function
void time::print()
{
    cout<<" "<<_hours<<":"<<_minutes<<":"<<_seconds<<" "<<endl;
}

time time::getTimeFromUser()
{
    time newTime;
    int userhours=-1;
    int userminutes=-1;
    float userseconds=-1;
    while (!checkHours(userhours))
    {
        cout<<"enter hours"<<endl;
        cin>>userhours;
        if(!checkHours(userhours))
        {
            cout<<"Error try again"<<endl;
        }
    }

    while (!checkMinutes(userminutes))
    {
        cout<<"enter minutes"<<endl;
        cin>>userminutes;
        if(!checkMinutes(userminutes))
        {
            cout<<"Error try again"<<endl;
        }
    }

    while (!checkSeconds(userseconds))
    {
        cout<<"enter Seconds"<<endl;
        cin>>userseconds;
        if(!checkSeconds(userseconds))
        {
            cout<<"Error try again"<<endl;
        }
    }

    newTime.seconds(userseconds);
    newTime.hours(userhours);
    newTime.minutes(userminutes);

    return newTime;
}


float time::getTimeAsFractionOfTheDay(time tm)
{

    return 0.0;
}

我遇到了以下错误：

我不明白我哪里出错了。我觉得可能是一些愚蠢的错误，但我找不到。

- cheziHoyzer

---

顺便提一下，你有一个微妙的错误：return _hours=tm.hours() && - nneonneo

错误明确指出 time 不是一种类型 - 因此在 time time::getTimeFromUser() 中引起了麻烦。 在整个声明 class time {}; 之前添加 class time;。 - Vinayak Garg

Answer 1

作为结果，你成为了一个微妙错误的受害者。 time 是一个在系统头文件 time.h 中声明的函数，通过 iostream 包含在你的程序中。当你声明 time time::getTimeFromUser() 时，编译器看到返回值并认为你指的是 函数 time！ clang 的错误信息让这一点变得明显：

time.cpp:122:1: error: must use 'class' tag to refer to type 'time' in this scope
time time::getTimeFromUser()
^
class 
/usr/include/time.h:133:8: note: class 'time' is hidden by a non-type declaration of 'time' here
time_t time(time_t *);
       ^

修复方法是将该特定函数定义为以下内容：

class time time::getTimeFromUser() {
    ...
}

如果使用C++11，

auto time::getTimeFromUser() -> time {
    ...
}

- nneonneo

Answer 2

问题在于您的头文件是time.h（已经作为标准头文件使用）。 您认为您得到了您声明的那个，但实际上您得到的是标准头文件。 因此，您没有您类的声明，而是标准头文件。 将其重命名为Test.h和Test.cpp，它就可以工作了。 并且在您的cpp文件顶部添加using namespace std。

- Gabriel

Answer 3

第一个问题：

bool time::operator==(time tm)
{
    return _hours=tm.hours() && _minutes==tm.minutes() && _seconds==tm.seconds();
//               ^
}

那里应该有一个比较运算符，而不是赋值运算符：

bool time::operator==(time tm)
{
    return _hours==tm.hours()
//               ^^
                   && _minutes==tm.minutes() && _seconds==tm.seconds();
}

第二个问题：

您正在使用属于std命名空间的cout和endl，但没有使用声明或指令。

在使用未限定名称的cout、cin和endl之前，请添加以下内容：

using std::cout;
using std::endl;
using std::cin;

或者，您可以选择在使用它们时完全限定它们的名称。例如：

    std::cout<<"Error"<<endl; _hours=-1; _minutes=-1; _seconds=-1;
//  ^^^^^

- Andy Prowl

---

虽然这些都是错误，指出它们是好的，但它们中没有一个是导致 OP 提到的编译错误的原因。 - Alok Save

+1 指出了可能被忽略的错误。 - Alok Save

@AlokSave：你说得对，我在弄清楚之前就离开了，所以...我的错 :) - Andy Prowl