以下代码有什么错误？

Question

我仍然是C ++的初学者，应该在以下代码中找到错误。

1 class Thing
2 {
3    public:
4       char c;
5       int *p;
6       float& f;
7       Thing(char ch, float x) { c = ch; p = &x; f = x; }
9 };

我知道在第六行有一个错误：引用f需要初始化。 但是我对第七行感到困惑。 它看起来像一个构造函数，但我不能确保p =＆x;。 是正确的？ 另外，如果我想更正引用初始化的错误，该怎么办？

Answer 1

找出是否有错误的最佳方法就是简单地对其进行编译^（1） 。

如果这样做，至少会发现两个问题：

• 参考资料应初始化； 和
• 您不能将浮点指针分配给int指针。

---

^（1）根据这份笔录：

$ g++ -c -o prog.o prog.cpp
prog.cpp: In constructor ‘Thing::Thing(char, float)’:
prog.cpp:7:7: error: uninitialized reference member in ‘float&’ [-fpermissive]
       Thing(char ch, float x) { c = ch; p = &x; f = x; }
       ^
prog.cpp:6:14: note: ‘float& Thing::f’ should be initialized
       float& f;
              ^
prog.cpp:7:43: error: cannot convert ‘float*’ to ‘int*’ in assignment
       Thing(char ch, float x) { c = ch; p = &x; f = x; }
                                           ^

Answer 2

p = &x;

是不正确的，因为p是int*类型， &x是float*类型。

f = x;

很可能不是您想要的。 您可能希望f成为x的引用。 上面的行不这样做。 它将x的值分配给f引用的对象。

如果要让f成为x的引用，则需要将其初始化为：

Thing(char ch, float& x) : f(x) { ... }
               //  ^^^ different from your signature

使用

Thing(char ch, float x) : f(x) { ... }

这是有问题的，因为一旦函数返回， f将成为悬挂的引用。