Есть ли неповторяющийся способ позволить программисту выбирать между копированием и перемещением семантики для инициализации члена?
Я хочу иметь возможность инициализировать каждое поле класса либо с помощью переместить семантику или скопировать семантику. Все конструкторы будут использовать по существу один и тот же код для конструирования, например так:
LogRecord::LogRecord(const Logger &logger, LogLevel level, const std::wstring &message)
: level(level), logger_name(logger.GetName()), message(message), sequence_number(LogRecord::record_count++), source_class_name(), source_method_name(), time(std::chrono::system_clock::now()) {
}
LogRecord::LogRecord(const Logger &logger, LogLevel level, std::wstring &&message)
: level(level), logger_name(logger.GetName()), message(message), sequence_number(LogRecord::record_count++), source_class_name(), source_method_name(), time(std::chrono::system_clock::now()) {
}
LogRecord::LogRecord(const Logger &logger, LogLevel level, const std::wstring &message, const std::wstring &source_class_name, const std::wstring &source_method_name)
: level(level), logger_name(logger.GetName()), message(message), sequence_number(LogRecord::record_count++), source_class_name(source_class_name), source_method_name(source_method_name), time(std::chrono::system_clock::now()) {
}
LogRecord::LogRecord(const Logger &logger, LogLevel level, std::wstring &&message, const std::wstring &source_class_name, const std::wstring &source_method_name)
: level(level), logger_name(logger.GetName()), message(message), sequence_number(LogRecord::record_count++), source_class_name(source_class_name), source_method_name(source_method_name), time(std::chrono::system_clock::now()) {
}
LogRecord::LogRecord(const Logger &logger, LogLevel level, const std::wstring &message, std::wstring &&source_class_name, const std::wstring &source_method_name)
: level(level), logger_name(logger.GetName()), message(message), sequence_number(LogRecord::record_count++), source_class_name(source_class_name), source_method_name(source_method_name), time(std::chrono::system_clock::now()) {
}
LogRecord::LogRecord(const Logger &logger, LogLevel level, std::wstring &&message, std::wstring &&source_class_name, const std::wstring &source_method_name)
: level(level), logger_name(logger.GetName()), message(message), sequence_number(LogRecord::record_count++), source_class_name(source_class_name), source_method_name(source_method_name), time(std::chrono::system_clock::now()) {
}
и т.п.
Есть ли лучший способ сделать это, чем просто объявить конструктор для каждой возможной комбинации, как это?
class LogRecord {
public:
LogRecord(const Logger &logger, LogLevel level, const std::wstring &message);
LogRecord(const Logger &logger, LogLevel level, std::wstring &&message);
LogRecord(const Logger &logger, LogLevel level, const std::wstring &message, const std::wstring &source_class_name, const std::wstring &source_method_name);
LogRecord(const Logger &logger, LogLevel level, std::wstring &&message, const std::wstring &source_class_name, const std::wstring &source_method_name);
LogRecord(const Logger &logger, LogLevel level, const std::wstring &message, std::wstring &&source_class_name, const std::wstring &source_method_name);
LogRecord(const Logger &logger, LogLevel level, std::wstring &&message, std::wstring &&source_class_name, const std::wstring &source_method_name);
LogRecord(const Logger &logger, LogLevel level, const std::wstring &message, const std::wstring &source_class_name, std::wstring &&source_method_name);
LogRecord(const Logger &logger, LogLevel level, std::wstring &&message, const std::wstring &source_class_name, std::wstring &&source_method_name);
LogRecord(const Logger &logger, LogLevel level, const std::wstring &message, std::wstring &&source_class_name, std::wstring &&source_method_name);
LogRecord(const Logger &logger, LogLevel level, std::wstring &&message, std::wstring &&source_class_name, std::wstring &&source_method_name);
...
private:
std::wstring message, source_class_name, source_method_name;
...
};
Вот упрощенная форма, чтобы ее было легче читать.
Object это класс с членами, и Member это точное имя членов. Member В типе определены как конструктор копирования, так и конструктор перемещения.
По сути, мой вопрос состоял в том, как я мог бы сделать следующее с меньшим количеством дублирования кода:
class Object {
public:
Object(const Member &x, const Member &y, const Member &z) : x(x), y(y), z(z) {}
Object(Member &&x, const Member &y, const Member &z) : x(x), y(y), z(z) {}
Object(const Member &x, Member &&y, const Member &z) : x(x), y(y), z(z) {}
Object(Member &&x, Member &&y, const Member &z) : x(x), y(y), z(z) {}
Object(const Member &x, const Member &y, Member &&z) : x(x), y(y), z(z) {}
Object(Member &&x, const Member &y, Member &&z) : x(x), y(y), z(z) {}
Object(const Member &x, Member &&y, Member &&z) : x(x), y(y), z(z) {}
Object(Member &&x, Member &&y, Member &&z) : x(x), y(y), z(z) {}
private:
Member x, y, z;
}
Решение
Я не стал бы беспокоиться обо всех этих перегрузках. Всегда бери std::wstring аргументы по значению и std::move их в mem-инициализаторе. Тогда вам нужно только 3 определения конструктора. Предостережение заключается в том, что вы получаете дополнительную конструкцию ходов в тех случаях, когда вам передают значение, но вы, скорее всего, можете с этим жить.
LogRecord(const Logger &logger, LogLevel level, std::wstring message)
: level(level), logger_name(logger.GetName()), message(std::move(message)), ...
{}
Обратите внимание, что конструкция перемещения может быть На) для небольших значений N из-за небольшой оптимизации строки.
Другой вариант — идеальная пересылка, как указано в комментариях. Вы могли бы сделать что-то вроде
template<typename Message>
LogRecord(const Logger &logger, LogLevel level, Message&& message)
: level(level), logger_name(logger.GetName()), message(std::forward<Message>(message)), ...
{}
Возможно добавлю static_asserts, чтобы напечатать лучшие сообщения об ошибках, которые Message является или преобразуется в, std::wstring,
Другие решения