在 std::map 中保存向量指针的问题是什么

Question

我正在尝试通过记忆找到最佳总和，但是当将矢量指针保存在地图中时，值不断附加在矢量内并得到错误的矢量。 如果我注释掉地图插入，它可以正常工作。 如果尝试通过引用将矢量保存在地图中，则无法保存 nullptr。

std::vector<int> *bestSumV(int target, int nums[], int size) {

  static std::map<int, std::vector<int> *> memo;

  if (memo.find(target) != memo.end())
    return memo.at(target);

  if (target == 0)
    return new std::vector<int>();

  if (target < 0)
    return NULL;

  std::vector<int> *bestCom = nullptr;

  for (int i = 0; i < size; i++) {

    int reminder = target - nums[i];

    std::vector<int> *reminderResult = bestSumV(reminder, nums, size);

    if (reminderResult != NULL) {

      reminderResult->push_back(nums[i]);

      if (bestCom == nullptr || reminderResult->size() < bestCom->size()) {
        bestCom = static_cast<std::vector<int> *>(reminderResult);
      }
    }
  }
  // if i commented out the map insertion i am getting the correct value
  // and getting a vector of 5 items

  memo.insert(std::make_pair(target, std::move(bestCom)));

  return bestCom;
}

void runHowbestTest() {

  int testArray[] = {5, 4, 2};

  std::vector<int> *bestSum25 = bestSumV(25, testArray, 3);

  for (int i = 0; i < bestSum25->size(); i++) {
    std::cout << "the items  " << bestSum25->at(i) << std::endl;
  }
}

Answer 1

bestCom 是std::vector<int> * 不要std::move 它。这毫无意义，并且使代码难以阅读。

reminderResult 已经是std::vector<int> *，不需要static_cast<std::vector<int> *>。

在这条线上

memo.insert(std::make_pair(target, std::move(bestCom)));

bestCom 可能为空。发生这种情况时，

if (memo.find(target) != memo.end())
    return memo.at(target);

将返回并绕过函数逻辑。你需要：

if (memo.find(target) != memo.end() && memo.at(target))
    return memo.at(target);

而且，最有可能的是真正的问题：

// returns a memoized value
std::vector<int> *reminderResult = bestSumV(reminder, nums, size); 
...
  // modifies it.
  reminderResult->push_back(nums[i]);

您不能修改记忆值并期望它有效。

使用对象而不是指针解决了这个问题：https://godbolt.org/z/v35oT4。但我没有声明它对性能的影响。