[котвася]

ха сам додумался
лишнем было

Код:

void Date::Date(int dd, int yyyy)
{
	day = dd;
	year = yyyy;
}

и как раз назрело еще два вопроса:
1) Как перегрузить print() что бы она понимала автоматически сколько ей печатать параметров?
2) Как перегрузить Date() c типом char?

[котвася]

хо хо хо сам решил!
вот:

Код:

class Date
{
public:
	Date(int, char *, int);
	Date(int, int);
	Date(int, int, int);
	
	void print() const;
	void printChar() const;
	void setDate(int, int, int);
private:
	int day;
	int month;
	char *_month;
	int year;
};
Date::Date(int dd, char *mm, int yyyy)
{
	day = dd;
	_month = mm;
	year = yyyy;
}

[El Scorpio]

Date::Date(int dd, char *mm, int yyyy) { day = dd; _month = mm; year = yyyy; } »

Неправильно.
Представь,*что дальше в тексте программы будет команда на освобождение области памяти, на которую указывает mm. Тогда попытка обратиться к _month приведёт к ошибке.

И вообще, использовать char* нельзя. Для работы с текстом нужно объявлять объекты строковых контейнеров (string, AnsiString и т.д.)

[котвася]

пример можно?

[Glareone]

И вообще, использовать char* нельзя. »

Date::Date(int dd, char *mm, int yyyy) { day = dd; _month = mm; year = yyyy; } »

В данном примере, действительно, так лучше не делать.Но вообще, использовать char* не запрещает никто. Просто за выделением памяти mm=new... delete[] mm; самому приходится следить.

[pva]

ещё пара советов по оптимизации: исхожу из предположений
1) последовательность уточнения даты: год, месяц, день, час, минута, секунда, либо в обратном порядке. Т.е. нет смысла задавать дату днём и годом (без указания месяца).
2) обратный порядок нужен только для того, чтобы сокращённо задавать промежуток времени (например интервал 1 день 5 секунд). А его лучше задавать количеством секунд, а не датой. Поэтому обратный порядок тоже не нужен.

Код:

// Таким образом нужен один конструктор:
class Date {
public:
  Date(int year=0, int month=1, int day = 1, int hour=0, int min=0, int sec=0);
  ...
private:
  _hidden_init(); // не раскрывается inline
};

// т..к. по сути мы даём возможность заполнить структуру, то лучше вынести inline
// так мы разделим присвоение параметров (как бы inline) и дальнейшую обработку _hidden_init()
inline Date::Date(...)
{
   ...
   _hidden_init();
}

Если нужно иногда использовать время отдельно от даты, то можно сделать так:

Код:

// Таким образом нужен один конструктор:
class Time {
public: Time(int h, int m, int s);
};
class Date {
public: Date(int y, int m, int d);
};
class DateTime
{
public:
  DateTime(const Date&, const Time&);
  DateTime(const Date&);
  DateTime(const Time&);
};
// тогда путаницы не возникнет

[Drongo]

pva, Он до этого дойдёт позже. В книге Дейтелов изучение классов начинается с самых простых примеров без проверки на ошибки и только чуть-чуть погодя будут этот класс Time улучшать, добавляя функции проверки, коррекции и т.д.

[котвася]

pva, это я догоняю. Мне надо, например, перегрузить вот так: ДАТА(01 01 1900) и ДАТА( 1 января 1900). Не пойму пока как это сделать "правильно".
и вопрос:
если написать _hidden_init(); то конструктор перестает быть по умолчанию "встраиваемый" и будет "вызываться"?
и что бы он стал inline это надо явно указывать? я правильно понимаю?

[котвася]

Нашел такое решение:

Код:

#include <iostream>
class Date
{
public:
	Date(int, char, int);
	Date(int, int, int);
	
	void print() const;
	void printChar() const;
	void setDate(int, int, int);
private:
	int day;
	int month;
	int year;
};
Date::Date(int dd, int mm, int yyyy)
{
	day = dd;
	month = mm;
	year = yyyy;
}
Date::Date(int dd, char mm, int yyyy)
{
	day = dd;
	month = mm;
	year = yyyy;
}

void Date::setDate(int dd, int mm, int yyyy)
{
	day = dd;
	month = mm;
	year = yyyy;
}

void Date::print() const
{
	if(month >= 0)
		std::cout << day << "/" <<  month << "/"  << year << std::endl;
	else
		std::cout << day << "/" << year << std::endl;
}
void Date::printChar() const
{
	std::cout << day << "/" <<  char(month) << "/"  << year << std::endl;
}

int main()
{
	setlocale(LC_ALL,".1251");
	char _month[]="январь";
	Date d(2,12,2000);
	std::cout << _month[0] << std::endl;
	Date d3(1, _month[0],1991);

	d.print();
	d3.printChar();

	system("pause");
	return 0;
}

только вот одна проблема:
в Date могу передать только одну букву названия месяца.
Так что думаю тут без * никак так как надо передавать массив. Или есть другой "правильный путь"?