Семантика перемещения и оценка порядка функций

Используйте инициализацию списка для построения B. Затем элементы гарантированно оцениваются слева направо.

C(std::unique_ptr<A> a) : B{a->x, std::move(a)} {}
//                         ^                  ^ - braces

Из §8.5.4/4 [dcl.init.list]

В initializer-list списка braced-init-list initializer-clauses, включая все, что является результатом расширений пакета (14.5.3 ), оцениваются в том порядке, в котором они появляются. То есть каждое вычисление значения и побочный эффект, связанные с данным предложением-инициализатором, располагаются перед каждым вычислением значения и побочным эффектом, связанным с любым предложением-инициализатором, которое следует за ним в разделенный запятыми список initializer-list.

В качестве альтернативы ответу Praetorian вы можете использовать делегат конструктора:

class C : public B {
public:
    C(std::unique_ptr<A> a) :
        C(a->x, std::move(a)) // this move doesn't nullify a.
    {}

private:
    C(int x, std::unique_ptr<A>&& a) :
        B(x, std::move(a)) // this one does, but we already have copied x
    {}
};

Предложение Praetorian об использовании инициализации списка, кажется, работает, но у него есть несколько проблем:

1. Если аргумент unique_ptr указан первым, нам не повезло.
2. Для клиентов B слишком легко случайно забыть использовать {} вместо (). Разработчики интерфейса B навязали нам эту потенциальную ошибку.

Если бы мы могли изменить B, то, возможно, лучшим решением для конструкторов было бы всегда передавать unique_ptr по ссылке rvalue, а не по значению.

struct A { int x; };

class B {
  B(std::unique_ptr<A>&& a, int x) : _x(x), _a(std::move(a)) {}
};

Теперь мы можем безопасно использовать std::move().

B b(std::move(a), a->x);
B b{std::move(a), a->x};