Подавить часть конструктора по умолчанию?

Question

Подавить часть конструктора по умолчанию?

0

Можно ли подавить часть инициализации конструктора по умолчанию? Мой текущий конструктор по умолчанию выглядит так:

Jd::Jd() {
    time_t utcTime = time(NULL);
    struct tm tmLocal;
    localtime_s( &tmLocal, &utcTime );

    jd_ = gregorian_to_jd( 
                        tmLocal.tm_year + 1900, 
                        tmLocal.tm_mon + 1, 
                        tmLocal.tm_mday,
                        tmLocal.tm_hour,
                        tmLocal.tm_min,
                        tmLocal.tm_sec
                          );
}

И я использую две константы для инициализации моих объектов JD: WTIMEOFDAY и NOTIMEOFDAY.

Jd const NOTIMEOFDAY;
Jd const WTIMEOFDAY;

Я хочу, чтобы NOTIMEOFDAY был инициализирован как объект, построенный по умолчанию, но с использованием только метода year, month и day метода gregorian_to_jd() вместо всего. Это возможно?

EDIT: конструкторы в классе Jd

Jd();
Jd( jdn_t jdn ) : jd_( jdn ) { } //Sets the internal datamember to whatever is passed in.
//Jd( bool includeTime );

И ошибка, которую я получаю:

error C2668: 'calendar::Jd::Jd' : ambiguous call to overloaded function
could be 'calendar::Jd::Jd(bool)
or       'calendar::Jd::Jd(calendar::jdn_t)

Delete 13 окт. 2013, в 05:03

Источник

Теги:

c++

1 ответ

Ещё вопросы

Zac Howland · Accepted Answer · 2013-10-13T01-39-00.000Z

Добавьте еще один конструктор, который принимает параметры:

Jd::Jd(int year, int month, int day, int hour, int min, int sec)
{
    jd_ = gregorian_to_jd(year, month, day, hour, min, sec);
}

При создании ваших констант:

const Jd NOTIMEOFDAY(2013, 10, 12, 0, 0, 0); // new constructor called
const Jd WTIMEOFDAY; // default constructor called

Или вы можете использовать этот метод:

Jd::Jd(bool includeTime = true) 
{
    time_t utcTime = time(NULL);
    struct tm tmLocal;
    localtime_s( &tmLocal, &utcTime );

    jd_ = gregorian_to_jd( 
                        tmLocal.tm_year + 1900, 
                        tmLocal.tm_mon + 1, 
                        tmLocal.tm_mday,
                        includeTime ? tmLocal.tm_hour : 0,
                        includeTime ? tmLocal.tm_min : 0,
                        includeTime ? tmLocal.tm_sec : 0);
}

И затем инициализируйте свои константы:

const Jd NOTIMEOFDAY((bool)false);
const Jd WTIMEOFDAY;

Или...

// this will allow your double version to work
Jd::Jd(jdn_t jdn, bool includeTime = true) 
{
    // assuming what jdn_t looks like, so you'd have to make some adjustments
    jd_ = gregorian_to_jd( 
                        jdn.tm_year + 1900, 
                        jdn.tm_mon + 1, 
                        jdn.tm_mday,
                        includeTime ? jdn.tm_hour : 0,
                        includeTime ? jdn.tm_min : 0,
                        includeTime ? jdn.tm_sec : 0);
}

Или...

// this should fix the whole issue and is probably the better solution
class Jd
{
    // other members
public:
    explicit Jd(bool includeTime); // prevent implicit conversion
    explicit Jd(jdn_t jdn); // also prevents implicit conversion
};

Jd::Jd(bool includeTime) // no default parameter
{
    time_t utcTime = time(NULL);
    struct tm tmLocal;
    localtime_s( &tmLocal, &utcTime );

    jd_ = gregorian_to_jd( 
                        tmLocal.tm_year + 1900, 
                        tmLocal.tm_mon + 1, 
                        tmLocal.tm_mday,
                        includeTime ? tmLocal.tm_hour : 0,
                        includeTime ? tmLocal.tm_min : 0,
                        includeTime ? tmLocal.tm_sec : 0);
}

Проблема в том, что я хочу, чтобы константа NOTIMEOFDAY была инициализирована до текущей даты, я не хочу, чтобы она была жестко задана.
Тогда time(NULL); localtime_s(...); звонка time(NULL); localtime_s(...); когда-нибудь, прежде чем инициализировать константу и передать эти значения. Кроме того, вы можете добавить параметр по умолчанию в ваш конструктор по умолчанию ...
Это конфликтует с другим конструктором, который у меня есть: Jd( double jdn ) : jd_( jdn ) { }
Нет конфликта между конструктором, который принимает double и конструктором, который принимает bool до тех пор, пока вы передаете фактический double и bool конструктору. Вы также можете использовать ключевое слово explicit чтобы предотвратить неправильный вызов конструкторов преобразования.
Даже с явным ключевым словом я получаю неоднозначные ошибки вызова, которые не позволяют мне компилировать
Какие еще конструкторы вы объявили и какую именно ошибку вы получите?
Опубликовано как редактирование к основному сообщению.
Если вы не пытаетесь сделать что-то вроде const Jd a(0); вместо const Jd a(false) не должно быть двусмысленного вызова (нет, когда я компилирую код в VS2012). Вы также можете принудительно привести приведение к режиму bool (что не нужно, но показано в правке).
Я не уверен, почему это говорит неоднозначный вызов. Если я не включаю логический конструктор и просто создаю константу со значением false в качестве списка параметров, он не только компилируется, но и не выдает ошибок при запуске и просто возвращает ноль. Поэтому я не понимаю, почему это проблематично.
Скорее всего, он вызывает конструктор Jd::Jd(double) (поскольку он является единственным другим, отличным от заданного по умолчанию, после удаления версии bool ) с 0 в качестве входных данных (именно поэтому вы получите 0 качестве его значения). Попробуйте это с версией bool и приведите ее явно: const Jd a((bool)false);
Хотя это работает для этого конкретного экземпляра, оно вызывает проблемы с целой кучей Jd-объектов, использующих двойной конструктор в тестовом модуле, который у меня есть. И мне не разрешено редактировать юнит-тесты, поэтому они должны оставаться как есть, поэтому я не уверен, что этот метод будет работать.
Другой вариант - добавить параметр по умолчанию в ваш double конструктор ... отредактировано.