调用另一个函数的函数给出错误的输出c++？

Question

仅当参数中给出的 sku 字符串与“inventory”数组的成员匹配时，incrementStock 函数才调用“addProduct”函数（该数组的类型为 *Product，大小为 50）。我在构造函数中将数组初始化为 nullptr。 “num”是增量编号。 当我测试它并输入一个有效的 sku 到 incrementStock 时，我从 addproduct 函数中得到“没有空间”。

void Supplier::addProduct(Product *p)
{
bool space = false;
int counter=0;
        while(!space && counter < inventory.size() )
        {
                if(inventory[counter] == nullptr )
                {
                        inventory[counter] = p;
                        space = true;
                }
        counter++;
        }

        if (!space)
        {
                cout << "no space" << endl;
        }

}


void Supplier::incrementStock(const string &sku, int num)
{
bool found = false;
        for( int i = 0; i < inventory.size(); i++ )
        {
                if( inventory[i] && sku == inventory[i]->getSKU())
                {
                        found=true;
                        addProduct(inventory[i]);
                        inventory[i]->setQuantity(inventory[i]->getQuantity() +num);
                }
        }

        if (found ==false)
        {
                cout << "not found" << endl;
        }
}

Answer 1

考虑这个循环：

    for( int i = 0; i < inventory.size(); i++ )

如果您在此循环中找到了 sku 匹配项，则会将该项目的一个额外副本添加到库存中。这有点奇怪，但如果您想要库存中同一指针的多个副本，那很好。

问题是，在循环的那次迭代之后，循环会继续，它也会找到我们刚刚制作的副本，并查看它是否匹配，然后再次制作另一个副本。如此重复直到阵列已满。