Сразу скажу, что только бегло просмотрел код. Ряд тривиальных рекомендаций:
1) Нет необходимости явно использовать this. Компайлер за Вас сам это сделает.
2) Вместо struct date я бы использовал тип time_t (что есть просто typedef unsigned long time_t). Во-первых, в 9 раз меньше места занимает, чем у Вас, а во-вторых, имеется полный набор CRT ф-ций для манипуляции с данными этого типа. Если уж так хочется, то можно написать класс-обертку, методы к-рого позволяют а) проиницилизировать переменную time_t и б) получить все нужные вам значения как-то year, month, date etc. Реализация может быть примерно такой:
| Код |
class CWTime // W - for wrapper
{
time_t m_time;
public:
CWTime()
{
m_time= time(0); // текущее сист. время
}
int Year () { return (GetLocalTm()->tm_year + 1900); }
int Month() { return (GetLocalTm()->tm_mon + 1); }
int Day ();
. . .
bool IsDst();
private:
// хэлпер-метод
tm * GetLocalTm() { return localtime( &m_time); }
};
|
Хотя есть ещё более простое решение: в БД хранить time_t, а для вместо указанного выше CWTime - использовать MFС/ATL класс - CTime. Для VC6 это класс MFC, а для VC7 - ещё и ATL. Один из конструкторов CTime принимает time_t, а набор его методов более, чем достаточный.
ЗЫ: по поводу классов для работы с БД, почему Вам CDaoRecordset не подходит? |
