如何修复不起作用的复制构造函数？[已结束]-解网

问：

这个问题是由一个错别字或一个无法再重现的问题引起的。虽然类似的问题可能在这里成为主题，但这个问题的解决方式不太可能帮助未来的读者。

去年关闭。

改进此问题

我正在尝试简化我的代码。为什么我不能使用带注释的构造函数而不是以前的（未注释的）构造函数？

struct Direction {
    const double x, y, z;
    Direction(double _X, double _Y, double _Z) : x(_X), y(_Y), z(_Z) {}
    Direction(Direction& _D) : x(_D.x), y(_D.y), z(_D.z) {}
}

class Movement {
private:
    const double v;
    const Direction d;
public:
    Movement(double _V, double _X, double _Y, double _Z) : v(_V), d(_X, _Y, _Z) {}
    Movement(double _V, Direction _D) : v(_V), d(_D) {}
    Movement(Movement& _M) : v(_M.v), d(_M.d.x, _M.d.y, _M.d.z) {} // this works
 // Movement(Movement& _M) : v(_M.v), d(_M.d) {}                   // this doesn't work
}

关于如何让它工作的任何想法？谢谢。

C++ 类复制构造函数

<source>: In copy constructor 'Movement::Movement(Movement&)':
<source>:15:44: error: binding reference of type 'Direction&' to 'const Direction' discards qualifiers
   15 |     Movement(Movement& _M) : v(_M.v), d(_M.d) {}                   // this doesn't work
      |                                         ~~~^
<source>:4:26: note:   initializing argument 1 of 'Direction::Direction(Direction&)'
    4 |     Direction(Direction& _D) : x(_D.x), y(_D.y), z(_D.z) {}
      |               ~~~~~~~~~~~^~

该成员是，但构造函数需要一个非常量引用。dconstDirection

如果构造函数采用常量引用，则代码将进行编译，因为构造函数不会修改参数：

struct Direction {
    const double x, y, z;
    Direction(double _X, double _Y, double _Z) : x(_X), y(_Y), z(_Z) {}
    Direction(const Direction& _D) : x(_D.x), y(_D.y), z(_D.z) {}
};

class Movement {
private:
    const double v;
    const Direction d;
public:
    Movement(double _V, double _X, double _Y, double _Z) : v(_V), d(_X, _Y, _Z) {}
    Movement(double _V, Direction _D) : v(_V), d(_D) {}
    //Movement(Movement& _M) : v(_M.v), d(_M.d.x, _M.d.y, _M.d.z) {} // this works
    Movement(Movement& _M) : v(_M.v), d(_M.d) {}                   // this doesn't work
};

int main() {
    Movement x(0.1,0.2,0.3,0.4);
    Movement y(x);
}

然而，成员是棘手的，很少是正确的方式。成员是私有的，因此即使在非常量上也没有必要制作它们。用户无论如何都无法修改它们。我建议从成员中删除所有参数，并添加到所有可以是 .特别是复制构造函数应将其参数为 .constconstMovementconstconstconstconst &

struct Direction {
    double x, y, z;
    Direction(double _X, double _Y, double _Z) : x(_X), y(_Y), z(_Z) {}
    Direction(const Direction& _D) : x(_D.x), y(_D.y), z(_D.z) {}
};

class Movement {
private:
    double v;
    Direction d;
public:
    Movement(double _V, double _X, double _Y, double _Z) : v(_V), d(_X, _Y, _Z) {}
    Movement(double _V, const Direction& _D) : v(_V), d(_D) {}
    Movement(const Movement& _M) : v(_M.v), d(_M.d) {}                   // this doesn't work
};

int main() {
    Movement x(0.1,0.2,0.3,0.4);
    Movement y(x);
}

如果 s 成员在构造后也不应修改，则使它们 .创建一个类成员足以防止其修改，而成员则阻止一些有用的操作（赋值等）。Directionprivateprivateconst

上一个：移动构造函数实际上不是在 Microsoft 的示例中调用的？

下一个：非类类型的成员的错误请求 - 在复制的对象上使用方法时出现问题

如何修复不起作用的复制构造函数？[已结束]

How to fix a copy constructor that doesn't work? [closed]

评论