что то не так с компилятором?

@grrrrr · Регистрация: 21.04.2009

Author24 — интернет-сервис помощи студентам

есть класс, который переписан с книги Лафоре Р.

C++

#include <cstdlib>
#include <iostream>
#include <conio.h>
 
class time
{
      private:
              int hrs, mins, secs;
      public:
             time() : hrs(0), mins(0), secs(0)
             {}
             time(int h, int m, int s) : hrs(h), mins(m), secs(s)
             {}
             void display();
             void add_time(time t1, time t2);             
};
 
void time::display()
{
     std::cout << hrs << ":" << mins << ":" << secs << "\n";
}
void time::add_time(time t1, time t2)
{
     secs = t1.secs + t2.secs;
     if(secs > 59)
     {
             secs = secs - 60;
             mins++;
     }
     mins = t1.mins + t2.mins;
     if(mins > 59)
     {
             mins = mins - 60;
             hrs++;
     }
     hrs = t1.hrs + t2.hrs;
}
 
int main(int argc, char *argv[])
{
    time time1(5, 59, 59);
    time time2(4,30,30);
    time time3;
    time3.add_time(time1, time2);
    std::cout << "time3 = " << time3.display;
    
    system("PAUSE");
    return EXIT_SUCCESS;
}

ошибки:

\projects\1\new.cpp expected `;' before "time1"
\projects\1\new.cpp [Warning] statement is a reference, not call, to function `time'
\projects\1\new.cpp expected `;' before "time2"
.
.
.
что не так??

@co6ak · 24.06.2011, 11:53

....
а код ?

@grrrrr · 24.06.2011, 12:04 **[ТС]**

co6ak, см. выше

@co6ak · 24.06.2011, 12:09

C++

1 2	std::cout << "time3 = "; time3.display();

работает

Добавлено через 42 секунды
получалось
cout << "time3 = " << cout <<.....

@grrrrr · 24.06.2011, 12:13 **[ТС]**

то же самое, а вообще мне кажется что у меня проблемы с конструктором - т.к. почему то ругается на первую строку после main()

@co6ak · 24.06.2011, 12:15

хз.
попробуй конструктор по умолчанию задать как
time( int =0, int =0, int =0);
без скобок фигурных всяких. наф они не нужны там.

@grrrrr · 24.06.2011, 12:23 **[ТС]**

в книге автор рекомендует именно со скобочками

, хотя после рекомендации он почему то не аргументировал такое решение.

oxotnik · 24.06.2011, 12:28

в отдельные файлы определение и реализацию класса вынеси

Jupiter · 24.06.2011, 12:32

grrrrr, ошибка в 45-й строке

Сообщение от grrrrr

в книге автор рекомендует именно со скобочками

и правильно делает, если конструктор пустой, чтобы сразу было видно что он пустой

Добавлено через 24 секунды
да и conio.h здесь нафиг не нужен

@grrrrr · 24.06.2011, 12:55 **[ТС]**

и еще вопрос - по классам. При объявлении класса, авторы книг иногда описывают сначала закрытые, а потом открытые элементы или наоборот. Я так понимаю , что разницы нет?

@ForEveR · 24.06.2011, 13:19

grrrrr, нету

@grizlik78 · 24.06.2011, 13:23

grrrrr, разницы нет, но в начале класса можно не писать private: а в начале структуры можно не писать public: так как они подразумеваются по умолчанию.

@grrrrr · 24.06.2011, 13:28 **[ТС]**

ForEveR, ну я так и думал.

а теперь скажите, почему у меня эта прога не компилируется? Только что, разбил на файлы как посоветовал охотник, компилировал в wxDev-cpp.

1.cpp

C++

#include <cstdlib>
#include <iostream>
 
int main(int argc, char *argv[])
{    
    time time1(5, 59, 59);
    time time2(4,30,30);
    time time3;
    time3.add_time(time1, time2);
    std::cout << "time3 = " << time3.display;
    
    system("PAUSE");
    return EXIT_SUCCESS;
}

time1.h

C++

#ifndef TIME1_H
#define TIME1_H
 
class time
{
      private:
              int hrs, mins, secs;
      public:
            
             time() : hrs(0), mins(0), secs(0)
             {}
             time(int h, int m, int s) : hrs(h), mins(m), secs(s)
             {}
             void display();
             void add_time(time t1, time t2);             
};
 
#endif

time1.cpp

C++

#include <iostream>
#include "time1.h"
 
void time::display()
{
     std::cout << hrs << ":" << mins << ":" << secs << "\n";
}
void time::add_time(time t1, time t2)
{
     secs = t1.secs + t2.secs;
     if(secs > 59)
     {
             secs = secs - 60;
             mins++;
     }
     mins = t1.mins + t2.mins;
     if(mins > 59)
     {
             mins = mins - 60;
             hrs++;
     }
     hrs = t1.hrs + t2.hrs;
}

может кто подскажет? Ну или у кого компилируется, напишите какой у вас компилятор.
ЗЫ. Пробовал компилировать другие проекты с классами- все норм.

@grizlik78 · 24.06.2011, 13:33

time1.h надо подключать во всех модулях .cpp, где используется класс time

Добавлено через 1 минуту
Ну и ошибка в строке 10 файла 1.cpp так и не была исправлена

@ForEveR · 24.06.2011, 13:34

Не советую я так класс называть. Или в namespace хоть завертывайте.

@grrrrr · 24.06.2011, 13:41 **[ТС]**

ForEveR, точно!!! теперь вот ошибка одна ошибка:

47 C:\Dev-Cpp\projects\1\new.cpp statement cannot resolve address of overloaded function

ругается на 2 строку

C++

1 2	std::cout << "time3 = "; time3.display;

@ForEveR · 24.06.2011, 13:45

grrrrr, Ну дык это ж вызов функции все таки. Скобки забыли.

@grrrrr · 24.06.2011, 13:49 **[ТС]**

Все работает! Спасибо!

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	24.06.2011, 13:33	14
	time1.h надо подключать во всех модулях .cpp, где используется класс time Добавлено через 1 минуту Ну и ошибка в строке 10 файла 1.cpp так и не была исправлена 1

@ForEveR В астрале 8049 / 4806 / 655 Регистрация: 24.06.2010 Сообщений: 10,562
	24.06.2011, 13:45	17
	grrrrr, Ну дык это ж вызов функции все таки. Скобки забыли. 1

@grrrrr 49 / 49 / 13 Регистрация: 21.04.2009 Сообщений: 265
	24.06.2011, 13:49 [ТС]	18
	Все работает! Спасибо! 0

@co6ak Кошковед 521 / 509 / 63 Регистрация: 12.04.2010 Сообщений: 1,390
	24.06.2011, 11:53	2
	.... а код ? 0

@grrrrr 49 / 49 / 13 Регистрация: 21.04.2009 Сообщений: 265
	24.06.2011, 12:04 [ТС]	3
	co6ak, см. выше 0

@grrrrr 49 / 49 / 13 Регистрация: 21.04.2009 Сообщений: 265
	24.06.2011, 12:13 [ТС]	5
	то же самое, а вообще мне кажется что у меня проблемы с конструктором - т.к. почему то ругается на первую строку после main() 0

@co6ak Кошковед 521 / 509 / 63 Регистрация: 12.04.2010 Сообщений: 1,390
	24.06.2011, 12:15	6
	хз. попробуй конструктор по умолчанию задать как time( int =0, int =0, int =0); без скобок фигурных всяких. наф они не нужны там. 0

@grrrrr 49 / 49 / 13 Регистрация: 21.04.2009 Сообщений: 265
	24.06.2011, 12:23 [ТС]	7
	в книге автор рекомендует именно со скобочками , хотя после рекомендации он почему то не аргументировал такое решение. 0

oxotnik 1665 / 1134 / 80 Регистрация: 21.08.2008 Сообщений: 4,734 Записей в блоге: 1
	24.06.2011, 12:28	8
	в отдельные файлы определение и реализацию класса вынеси 0

Jupiter Каратель 6609 / 4028 / 401 Регистрация: 26.03.2010 Сообщений: 9,273 Записей в блоге: 1
	24.06.2011, 12:32	9
	grrrrr, ошибка в 45-й строке Сообщение от grrrrr в книге автор рекомендует именно со скобочками и правильно делает, если конструктор пустой, чтобы сразу было видно что он пустой Добавлено через 24 секунды да и conio.h здесь нафиг не нужен 1

@grrrrr 49 / 49 / 13 Регистрация: 21.04.2009 Сообщений: 265
	24.06.2011, 12:55 [ТС]	10
	и еще вопрос - по классам. При объявлении класса, авторы книг иногда описывают сначала закрытые, а потом открытые элементы или наоборот. Я так понимаю , что разницы нет? 0

@ForEveR В астрале 8049 / 4806 / 655 Регистрация: 24.06.2010 Сообщений: 10,562
	24.06.2011, 13:19	11
	grrrrr, нету 0

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	24.06.2011, 13:23	12
	grrrrr, разницы нет, но в начале класса можно не писать private: а в начале структуры можно не писать public: так как они подразумеваются по умолчанию. 0

@ForEveR В астрале 8049 / 4806 / 655 Регистрация: 24.06.2010 Сообщений: 10,562
	24.06.2011, 13:34	15
	Не советую я так класс называть. Или в namespace хоть завертывайте. 2