Question

我在确定const在特定情况下的确切应用时遇到了一些麻烦。这是我的代码：

struct Widget
{
    Widget():x(0), y(0), z(0){}

    int x, y, z;
};

struct WidgetHolder //Just a simple struct to hold four Widgets.
{
    WidgetHolder(Widget a, Widget b, Widget c, Widget d): A(a), B(b), C(c), D(d){}

    Widget& A;
    Widget& B;
    Widget& C;
    Widget& D;
};

class Test //This class uses four widgets internally, and must provide access to them externally.
{
    public:
        const WidgetHolder AccessWidgets() const
        {
            //This should return our four widgets, but I don't want anyone messing with them.
            return WidgetHolder(A, B, C, D);
        }

        WidgetHolder AccessWidgets()
        {
            //This should return our four widgets, I don't care if they get changed.
            return WidgetHolder(A, B, C, D);
        }

    private:
        Widget A, B, C, D;
};

int main()
{
    const Test unchangeable;

    unchangeable.AccessWidgets().A.x = 1; //Why does this compile, shouldn't the Widget& be const?
}

基本上，我有一个名为test的类。它在内部使用了四个小部件，我需要它来返回这些小部件，但如果测试被声明为const，我希望小部件也返回const。

有人可以向我解释为什么main（）中的代码会编译吗？

非常感谢。

Answer 1

你需要创建一个专门用于保存const Widget＆amp;的新类型。对象。即：


struct ConstWidgetHolder
{
    ConstWidgetHolder(const Widget &a, const Widget &b, const Widget &c, const Widget &d): A(a), B(b), C(c), D(d){}

    const Widget& A;
    const Widget& B;
    const Widget& C;
    const Widget& D;
};

class Test
{
public:
    ConstWidgetHolder AccessWidgets() const
    {
        return ConstWidgetHolder(A, B, C, D);
    }

现在您将收到以下错误（在gcc 4.3中）：

widget.cc: In function 'int main()':
widget.cc:51: error: assignment of data-member 'Widget::x' in read-only structure

在标准库中使用类似的习惯用法，即迭代器，即：


class vector {
    iterator begin();
    const_iterator begin() const;

Answer 2

unchangeable.AccessWidgets（）：

此时，您正在创建一个WidgetHolder类型的新对象。此对象不受const保护。

您还在WidgetHolder中创建新的小部件，而不是对Wdiget的引用。

Answer 3

您的WidgetHolder将保留无效引用（指针）。您正在将堆栈上的对象传递给构造函数，然后保持对其（临时）地址的引用。这肯定会破裂。

您应该只为与参考本身具有相同（或更长）生命周期的对象分配引用。

如果必须保留引用，则将引用传递给构造函数。更好的是，根本不要保留参考文献，只需制作副本。

Answer 4

这是编译因为虽然WidgetHolder是一个const对象，但是这个常量不会自动应用于指向WidgetHolder（由其引用）的对象。在机器级别考虑它 - 如果WidgetHolder对象本身保存在只读内存中，您仍然可以写入WidgetHolder指向的内容。

问题似乎在于这一行：

WidgetHolder(Widget a, Widget b, Widget c, Widget d): A(a), B(b), C(c), D(d){}

正如Frank所提到的，WidgetHolder类中的引用将在构造函数返回后保留无效引用。因此，您应将其更改为：

WidgetHolder(Widget &a, Widget &b, Widget &c, Widget &d): A(a), B(b), C(c), D(d){}

执行此操作后，它将无法编译，我将其作为练习留给读者来解决剩下的解决方案。

Answer 5

编辑：他删除了他的答案，让我看起来有点愚蠢：）

Flame的回答是危险的错误。他的WidgetHolder在构造函数中引用了一个值对象。一旦构造函数返回，那个传递值的对象将被销毁，因此您将保存对被销毁对象的引用。

使用他的代码的一个非常简单的示例应用程序清楚地显示了这一点：

#include <iostream>

class Widget
{
    int x;
public:
    Widget(int inX) : x(inX){}
    ~Widget() {
    std::cout << "widget " << static_cast< void*>(this) << " destroyed" << std::endl;
     }
};

struct WidgetHolder
{
    Widget& A;

public:
    WidgetHolder(Widget a): A(a) {}

    const Widget& a() const {
    std::cout << "widget " << static_cast< void*>(&A) << " used" << std::endl;
    return A;
}

};

int main(char** argv, int argc)
{
Widget test(7);
WidgetHolder  holder(test);
Widget const & test2 = holder.a();

return 0;
}

输出类似于

widget 0xbffff7f8 destroyed
widget 0xbffff7f8 used
widget 0xbffff7f4 destroyed

为避免这种情况，WidgetHolder构造函数应该引用它想要存储为参考的变量。

struct WidgetHolder
{
    Widget& A;

public:
    WidgetHolder(Widget & a): A(a) {}

  /* ... */

};

Answer 6

如果包含的类是const，原始查询是如何将WidgetHolder作为const返回。 C ++使用const作为函数签名的一部分，因此您可以使用相同函数的const和none const版本。当实例为无const时调用none const one，当实例为const时调用const one。因此，解决方案是通过函数而不是直接访问小部件持有者中的小部件。我在下面创建了一个更简单的例子，我相信它可以解答原始问题。

#include <stdio.h>

class Test
{
public:
  Test(int v){m_v = v;}
 ~Test(){printf("Destruct value = %d\n",m_v);}

 int& GetV(){printf ("None Const returning %d\n",m_v); return m_v;  }

 const int& GetV() const { printf("Const returning %d\n",m_v); return m_v;}
private:
  int m_v;
};

void main()
{
  // A none const object (or reference) calls the none const functions
  // in preference to the const
  Test one(10);
  int& x = one.GetV();
  // We can change the member variable via the reference
  x = 12;

  const Test two(20);
  // This will call the const version  
  two.GetV();

  // So the below line will not compile
  // int& xx = two.GetV();

  // Where as this will compile
  const int& xx = two.GetV();

  // And then the below line will not compile
  // xx = 3;

}

就原始代码而言，我认为将WidgetHolder作为类Test的成员更容易，然后返回const或none const引用，并使Widgets成为持有者的私有成员，并为每个Widget提供const和none const访问器。

class WidgetHolder {
...

Widget& GetA();
const Widget& GetA() const;
...
};

然后在主课上

class Test {
...
WigetHolder& AccessWidgets() { return m_Widgets;}
const WidgetHolder&AcessWidgets() const { return m_Widgets;}

private:
  WidgetHolder m_Widgets;
...
};

Const Struct＆amp;

6 个答案: