在映射中插入一对时，C++ 是否需要额外的代码？

Question

你好伟大的 StackOverflow 社区！我正在制作一些 C++，但在std::map.

这是 2 个映射，将类似 ID 存储unsigned int为键，将另一个对象存储为值：

    std::map<unsigned int, FIFO> _fifos;
    std::map<unsigned int, Kitchen> _kitchens;

两个地图都private在一个类中，我在该类的public方法中插入这样的：

    FIFO newFIFO(_internalCount);
    Kitchen newKitchen(_args, newFIFO);

    _kitchens.insert(std::make_pair(_internalCount, newKitchen));
    _fifos.insert(std::make_pair(_internalCount, newFIFO));

---

这里开始麻烦。
我的编辑器（VSCode）和编译器（g++）似乎都接受_fifos.insert()但不接受_kitchens.insert().

VSCode 说：

no instance of overloaded function "std::map<_Key, _Tp, _Compare, _Alloc>::insert [with _Key=unsigned int, _Tp=Kitchen, _Compare=std::less<unsigned int>, _Alloc=std::allocator<std::pair<const unsigned int, Kitchen>>]" matches the argument list..."

虽然 g++ 首先显示了这一点，但在列出了 C++ 深处的一堆错误之后（特别是在 stl_pair.h 中）：

error: no matching function for call to ‘std::pair<unsigned int, Kitchen>::pair(unsigned int&, Kitchen&)’
   66 |     _kitchens.insert(std::pair<unsigned int, Kitchen>(_internalCount, newKitchen));
      |                                                                                 ^

我已经尝试过其他std::pair定义，就像在另一个问题中一样，但没有成功。
考虑到我对 C++ 的深度了解不足，这里是否存在类型/语法问题或std::pair?
预先感谢您的支持！

---

编辑：
感谢您的建议和有关代码示例/复制的文章的链接。
添加一点上下文，这是一个学生项目，旨在学习和生成并发代码。
主要思想是创建一个比萨店，其中一个对象Reception代表主进程，Kitchen对象代表分叉进程，并且Chefs往往是管理单个、分离的对象std::thread，抽象类Pizza将在一定时间内被烹饪。

我真的不想打扰你软件设计，很明显有一些奇怪或糟糕的选择。为了让您一目了然，每个FIFO对象都处理一个系统 FIFO 管道，并且它们全局管理着Reception多个对象之间的 IPCKitchens因为这里的内存不是共享的。

施工时，Reception拥有自己的管道（旨在接收任何人的确认Kitchens）。这个目前没有实现，目前肯定不是很有用

Reception必须向 发送订单Kitchens，这就是为什么Reception创建一个管道并将其传递给新的Kitchens，FIFO 和厨房都由 标识unsigned int _internalCount，每个新厨房都会增加。
std::map<unsigned int, Kitchen> _kitchens将增长以存储任何新创建的 Kitchen 并std::map<unsigned int, FIFO> _fifo存储用于接收的每个通道，并使用它们发送数据。
请注意_internalCountis 附加到管道名称，其结果类似于'./pipes/kitchen_1', './pipes/kitchen_2', etc....
我不知道 StackOverflow 中所有简化和明确的最佳实践，但下面是 Reception、Kitchen 和 FIFO 类。

no instance of overloaded function "std::map<_Key, _Tp, _Compare, _Alloc>::insert [with _Key=unsigned int, _Tp=Kitchen, _Compare=std::less<unsigned int>, _Alloc=std::allocator<std::pair<const unsigned int, Kitchen>>]" matches the argument list..."

如果您需要什么或想要更多详细信息，请告诉我:)

---

编辑 2：
我在第一行之后添加了一些来自 g++ 的额外错误/注释（显示在上面的初始问题中）

error: no matching function for call to ‘std::pair<unsigned int, Kitchen>::pair(unsigned int&, Kitchen&)’
   66 |     _kitchens.insert(std::pair<unsigned int, Kitchen>(_internalCount, newKitchen));
      |                                                                                 ^

与往常一样，提前感谢您的时间和建议！

---

编辑 3：
我现在开始将我大脑中的一切联系起来，谢谢大家。事情变得棘手，但我肯定会从你的所有建议中学习。我会再次仔细阅读@reinstate-monica 的答案，并在我身边做进一步的研究，我有解决这个问题并让它变得更好的关键。
抱歉，我的第一个问题不清晰，我没想到我的第一个问题会如此棘手，我希望它更清楚。
我的下一个问题将更清楚，我将尝试从一开始就制作最少的可复制代码，我将仔细查看 StackOverflow @ted-lyngmo 的良好实践！
感谢大家的支持和时间，照顾好自己！

Answer 1

首先，您应该放置而不是插入 - 这避免了副本：

_kitchens.emplace(_internalCount, std::move(newKitchen));
_fifos.emplace(_internalCount, std::move(newFIFO));

其次，Kitchen可能是不可复制和不可移动的，这就是你的麻烦所在。确保它可以被复制或至少移动。如果是，你必须展示一个最小的例子。我可以写一个，但它会起作用，一个非工作变体将是微不足道的，没有帮助。所以首先向我们展示你的作品:)

第三：我不知道您的 FIFO 对象的设计，但将其临时实例传递Kitchen给构造函数可能只不过是一个错误。当您完成所有这些工作的函数退出时，Kitchen将有一个悬空引用。因此，您真正想要的是以下内容：

auto fifo_it = _fifos.emplace(std::piecewise_construct, {_internalCount}, {_internalCount}).first;
if (fifo_it.second)
  // if the new fifo was actually inserted
  _kitchens.emplace(std::piecewise_construct, {_internalCount}, {_args, fifo_it.first->second});

这样，厨房将有一个对至少有一个飞行机会活得足够长以供使用的先进先出装置的参考。

我非常怀疑传递_internalCount给地图和地图中包含的对象。这种信息重复通常是一种糟糕的设计味道。

您应该告诉我们您想要达到的目标，这样可能会出现更好的设计。