Question

以下代码是一本书中的示例＆＃34; Design Patterns Explained Simply＆＃34;。我尝试使用其他问题的建议方式，但结果不好。我怎么能弄清楚这个问题：

commands[0] = &SimpleCommand(&object, &Number::dubble);

＆＃34;警告：接受临时＆＃34;？

的地址

#include <iostream>
#include <vector>
using namespace std;

class Number
{
  public:
    void dubble(int &value)
    {
        value *= 2;
    }
};

class Command
{
  public:
    virtual void execute(int &) = 0;
};

class SimpleCommand: public Command
{
    typedef void(Number:: *Action)(int &);
    Number *receiver;
    Action action;
  public:
    SimpleCommand(Number *rec, Action act)
    {
        receiver = rec;
        action = act;
    }
     /*virtual*/void execute(int &num)
    {
        (receiver->*action)(num);
    }
};

class MacroCommand: public Command
{
    vector < Command * > list;
  public:
    void add(Command *cmd)
    {
        list.push_back(cmd);
    }
     /*virtual*/void execute(int &num)
    {
        for (unsigned int i = 0; i < list.size(); i++)
          list[i]->execute(num);
    }
};

int main()
{
  Number object;
  Command *commands[3];
  commands[0] = &SimpleCommand(&object, &Number::dubble); // "warning: taking address of temporary"

  MacroCommand two;
  two.add(commands[0]);
  two.add(commands[0]);
  commands[1] = &two;

  MacroCommand four;
  four.add(&two);
  four.add(&two);
  commands[2] = &four;

  int num, index;
  while (true)
  {
    cout << "Enter number selection (0=2x 1=4x 2=16x): ";
    cin >> num >> index;
    commands[index]->execute(num);
    cout << "   " << num << '\n';
  }
}

Answer 1

SimpleCommand(&object, &Number::dubble)

创建名为 rvalue 的内容。这是一个临时值，将在语句结束时销毁，因此您不应该继续引用它。

Answer 2

违规行是第三行。

Number object;
Command *commands[3];
commands[0] = &SimpleCommand(&object, &Number::dubble); // "warning: taking address of temporary"

在此，SimpleCommand(&object, &Number::dubble)构造一个临时的，它将在语句的末尾不再存在，&获取其地址。因此警告 - 指针将悬空（指向不再存在的对象）。任何解除引用该指针都将导致未定义的行为。编译器不需要诊断这个，但是你的帮助对你不利。

只需像对待其他对象一样：构造对象然后存储其地址。

SimpleCommand simple(&object, &Number::dubble);
commands[0] = &simple;

请注意，如果在command[0]不再存在后使用simple，则会遇到同样的问题。更真实的代码（例如，不是玩具main()中的所有内容，如＃34中所述;评论中无用）可以很容易地将问题commands[0]继续存在 - 并且在对象之后使用 - 点不再存在。这也会导致未定义的行为 - 但编译器不太可能识别并发出警告。

警告：获取临时地址 - C ++

2 个答案: