0

Допустим, имеем типичный класс:

class worker
{
  std::string name;
  std::string lastname;
  std::string dob;
public:
  worker(std::string &&n, std::string &&l, std::string &&d)
    : name(n), lastname(l), dob(d)
  {  }
  ~worker() = default;
};

Какой вариант конструктора лучше использовать, такой, или с std::move()?

worker(std::string &&n, std::string &&l, std::string &&d)
        : name(std::move(n)),
          lastname(std::move(l)),
          dob(std::move(d))
{  }
6
  • 1
    Вариант без move бесполезен, ведь ничего же не перемещается. 19 мая 2020 в 9:04
  • Но, так ведь move() приводит переменную к r-value, ничего не перемещая. Фактически перемещение происходит в конструкторе std::string(string&&)
    – KoD
    19 мая 2020 в 9:07
  • Как раз за счет приведения к r-value и будет вызываться перемещающий конструктор для std::string, а без него - вызовется копирующий. 19 мая 2020 в 9:09
  • Да. Но у нас изначально string &&, зачем ее еще раз приводить к string &&
    – KoD
    19 мая 2020 в 9:10
  • Без move ваши параметры, как имеющие имя и адрес, будут трактоваться как lvalue. Словом, move нужен.
    – Harry
    19 мая 2020 в 9:10

2 ответа 2

2

Вот живой пример -

#include <iostream>
#include <iomanip>

using namespace std;

class Test {
public:
    Test()             { cout << "Test()\n"; }
    Test(int x):val_(x){ cout << "Test(" << x << ")\n"; }
    Test(const Test& t):val_(t.val_) { cout << "Test(const Test& " << t.val_ << ")\n"; }
    Test(Test&&t)      :val_(t.val_) { cout << "Test(const Test&& " << t.val_ << ")\n"; t.val_ = 0; }
    ~Test()           { cout << "~Test(" << val_ <<")\n"; }
private:
    int val_ = 0;
};

struct Quest
{
    Test t;
    Quest(Test&& t):t(t){}
};

struct Qwest
{
    Test t;
    Qwest(Test&& t):t(move(t)){}
};

int main(int argc, const char * argv[])
{
    Test t(5), s(6);
    Quest q(move(t));
    Qwest w(move(s));
}

Убедились в необходимости move?

3
  • Т.е. std::move() нужно использовать и в таком случае: vector<mystruct> vi; vi.emplace_back(mystruct());
    – KoD
    19 мая 2020 в 9:22
  • @KoD Здесь не нужно, потому что mystruct() - уже rvalue. Можно написать проще: vi.emplace_back();. 19 мая 2020 в 9:23
  • @KoD В дубликате этот вопрос же освещается. 19 мая 2020 в 9:31
0

Ни тот ни другой. В оба конструктора нельзя передавать lvalue.

Например, если у вас есть std::string name = "Bob";, то при вызове конструктора вам придется писать std::move(name) - очень неудобно!

Вот такой конструктор больше подходит:

worker(std::string n, std::string l, std::string d)
        : name(std::move(n)),
          lastname(std::move(l)),
          dob(std::move(d))

А если параметров мало (1-2), то в качестве оптимизации можно сделать 2n конструкторов с параметрами const std::string & и std::string && (во втором случае нужен std::move, как написано в соседнем ответе). Тогда для каждой переменной у вас будет на одно перемещение меньше.

2
  • "Вот такой конструктор больше подходит" чтобы наплодить временных объектов 19 мая 2020 в 9:35
  • @user7860670 Нуу. С другой стороны, использовать везде rvalue-ссылки и на каждый чих ставить move (то есть при вызове таких конструкторов с lvalue) для меня выглядит как преждевременная оптимизация. 19 мая 2020 в 9:47

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.