Перегрузка конструкторов (C++)

[Kernel ops]

Изучаю c++, дошел до перегрузки конструкторов объектов. В этой теме все понятно, с перегрузкой вопросов нет. Вопрос вот в чем:

Код: [Выделить]

/* Перегрузка конструкторов */

using namespace std;

#include <iostream>
#include <ctime>
#include <cstdlib>

class timer {
int seconds;
public:
timer(char *t) {seconds=atoi(t);}
timer(int t) {seconds=t;}
timer(int min, int sec) {seconds=min*60+sec;}
void run();
};

void timer::run() {
cout << "Таймер запущен!";

int timeStart=time('\0');

do {} while(time('\0')<seconds+timeStart);

cout << "Таймер завершен!";
}

int main() {

timer runner(5);

runner.run();

return 0;
}

при этом, сначала проходит 5 секунд, и только потом друг за другом мгновенно появляются надписи "Таймер запущен!Таймер завершен!". Цикл менял на for и while - не помогло, менял названия переменных (может зарезервированные использую) - не помогло выносил cout в main - не помогло. В чем может быть дело?

[SergeyIT]

После
cout << "Таймер запущен!";
добавь
cout.flush();

[aSmile]

Можно добавить flush(), как сказали ранее, а можно дописать каждому cout в конце  << endl. Так и читаться лучше будет.

Код: (c++) [Выделить]

cout << "Таймер запущен!" << endl;

Про std::endl

[Kernel ops]

да, оба варианта работают, даже такой:

Код: [Выделить]

cout << "Таймер запущен!\n";как я понял, flush/endl очищают поток (stream) в stdout?

[Lifewalker]

да, оба варианта работают, даже такой:

Код: [Выделить]

cout << "Таймер запущен!\n";

Совершенно не по Си++сному. Самый художественно-выдержанный вариант безусловно cout << "Sometext" << endl;

[Kernel ops]

Lifewalker,
только из-за его наглядности

[Yurror]

Код: [Выделить]

time('\0')Теперь осталось только выяснить какое больное сознание родило этот шедевр

Ну хрен с ними с любителями писать '\0' вместо 0 там где требуется просто нулевой символ.
Ну хрен с ними с любителями писать 0 там где про руководству требуется писать NULL
Ну я понимаю что в итоге в функцию передастся нулевой указатель. Но это же еще понимать надо. А читать этот бред невозможно.

(Нажмите, чтобы показать/скрыть)

$ man 2 time

[quiet_readonly]

Есть ещё одна претензия к качеству кода, кроме передачи "\0": при передаче в конструктор переменной типа const char * будет вызван конструктор timer(int), а не timer(char *). Именно поэтому лучше вообще не баловаться перегрузкой конструкторов без необходимости, а если пришлось - надо проверить, не возникнет ли казусов из-за преобразования типов.

[Kernel ops]

quiet_readonly,
я перевел на clock():

Код: [Выделить]

/* Перегрузка конструкторов */

using namespace std;

#include <iostream>
#include <ctime>
#include <cstdlib>

class timer {
int seconds;
public:
timer(char *t) {seconds=atoi(t); cout << "CHK\n";}
timer(int t) {seconds=t;}
timer(int min, int sec) {seconds=min*60+sec;}
void run();
};

void timer::run() {
clock_t start=clock();

for (;start+seconds*CLOCKS_PER_SEC>clock(););
}

int main() {
timer runner("7"), checker(3);

cout << "Приготовьтесь!\n";
checker.run();

cout << "Таймер запущен!\n";
runner.run();
cout << "Таймер завершен!";
return 0;
}
и при отправке строки отправляется ссылка на неё. но вот если поставить одинарные кавычки - отправляемой переменной присваивается номер символьной константы и она отправляется в int конструктор. обошлось без NULL
не забывайте, что это только упражнение на перегрузку, естественно, куда попало ее никто пихать не будет

[Lifewalker]

Именно поэтому лучше вообще не баловаться перегрузкой конструкторов без необходимости,

(Нажмите, чтобы показать/скрыть)

Именно поэтому я постоянно на каждом углу как Катон говорю, что Си++ нужно закопать