装饰器设计模式、功能bug

Question

这是作业...我不是在寻求答案，我只是有一个错误，我不知道该怎么办。谢谢！

所讨论的错误可能与作业本身无关，但无论如何，这里是作业描述：

我正在处理一个作业（在 C++ 中），旨在通过以下方式教授装饰器设计模式的使用：带有配料的披萨的经典例子。 （我的教授也可能直接从 http 中提取它://simplestcodings.com/2010/12/26/decorator-design-pattern-example-ni-c/）。我遇到了一个小问题，我想知道是否有人可以帮助我。

我有一个主菜单（比萨饼店）对象，它接受用户的输入并对比萨饼执行所需的操作。用户从基本的披萨开始，然后可以添加配料，直到完成。因此，我的“newPizza”函数所做的第一件事就是将新的 Pizza 声明为 Plain，它是抽象类 Pizza 的子类。

然后他们可以输入自己选择的配料。每次，指向同一 Pizza 对象的指针都会发送到 addToppings() 函数，添加新的装饰，然后返回该指针。每个装饰都继承自一个价格类别，该价格类别又继承自 pizzaToppings，而后者又继承自 Pizza。

这是主订单函数的相关部分：

Pizza* Menu::newPizza()
{
cout << "\nNew Pizza";

//accept the next choice
int choose = 0;

//create the new pizza
Plain * currentPizza = new Plain();

//until they choose to end the order
while (choose != 3)
{
    //accept the choice
    cin >> choose;

    switch (choose)
    {
        //if they want to add a new topping
    case 1:
        {
            //add topping to current pizza
           //and this is where the problem is spotted by the compiler
            addTopping(currentPizza);
            break;
        }

问题是，当我尝试将指针 currentPizza 发送到函数 addTopping() 时，我得到 “运行时检查失败#3 - 变量‘currentPizza’正在使用但未初始化。”

我不是刚刚在第 7 行初始化了吗？

如果我点击“继续”，程序将继续运行并工作，但每次调用该函数时都会出现相同的错误。这只是某个地方的语法错误，还是我这里有一些实际问题？

谢谢！！

[编辑：]

addTopping() 函数：

Pizza* Menu::addTopping(Pizza* thisPizza)
{
cout << "\nAdd topping";

//declare choose int
int choose = 0;

//accept number of topping
cin >> choose;

//decide which one to add
switch (choose)
{

//mozzarella
case 1:
    {
        thisPizza = new Mozzarella(thisPizza);
        break;
    }
//mushrooms
case 2:
    {
        thisPizza = new Mushrooms(thisPizza);
        break;
    }

//another 13 possible toppings, won't bore you with the details ;)

}

cout << "\nEnd add topping\n";

return thisPizza;
}

Answer 1

您是否也将 currentPizza 声明为 Pizza 类的字段，并且您在其他地方使用它？如果是这样，您在 newPizza 中更新的 currentPizza 是特定于该方法的，您只需执行 currentPizza = new Plain();而不是在方法范围内声明一个新的 currentPizza 变量。

另外，在 addTopping 方法中，您仅更新参数 thisPizza，它是指针的副本 当前披萨。

你需要做：

currentPizza = addTopping(currentPizza);

Answer 2

如果您按值传递指针（这就是您正在做的事情），它将采用该指针值并将新的披萨分配给它。该值与上面第 7 行中找到的值不同。例如：

int bar = new int(3);
void  doSomething(int *foo){ foo = new int(5); } //memory leak here
doSomething(bar);

bar 仍然是 3。这实际上就是您正在做的事情。

您想通过引用传递指针：

void doSomething(int **foo){ delete *foo; *foo = new int(5); }

更新：

看到您想要一个嵌套类结构，其中 Child 类以多态方式保留类 Base 的记录......

void doSomething(MyClass **foo){ *foo = new MyChildClass(*foo); }

我希望作为子类中定义的一部分，您已确保正确处理资源（即指针）的释放。我建议考虑合并一个智能指针，但这可能超出了您完成此任务所需的范围。

Answer 3

一个错误是在 Menu::newPizza() 中你没有这样做：
currentPizza = addTopping(currentPizza);

此外，您还遇到了内存泄漏，因为您在堆上创建了新对象，而没有删除旧对象。

顺便说一句，听起来从 addTopping 方法返回新披萨的设计很糟糕。