在地图中插入一对时，C++需要附加代码吗？
你好，伟大的StackOverflow社区！我正在制作一些C++，在std::map中插入元素时遇到了一个问题。
下面是两个映射，将类似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定义，比如在this another question中，但是没有成功。
考虑到我对C++深度的贫乏了解，这里是否存在类型/语法问题，或者std::pair“缺少”了什么？
提前感谢您的支持！
编辑：
感谢您的建议和关于代码示例/复制的文章的链接。
为了添加一点上下文，这是一个学生项目，旨在学习并生成并发代码。
其主要思想是创建一个披萨店，其中一个对象Reception表示主进程，Kitchen对象表示分叉进程，Chefs倾向于管理一个单独的std::thread对象，其中抽象类Pizza将在一定时间内被煮熟。
我真的不想在软件设计上打扰你，很明显，有一些奇怪的或错误的选择。为了让您了解一下，每个FIFO对象都处理一个系统FIFO管道，它们在全局上管理Reception和multiples Kitchens之间的IPC，因为这里的内存是不共享的。
在构建时，Reception有自己的管道(用于接收来自任何Kitchens的确认)。这个方案目前还没有实现，而且目前肯定没有多大用处。
Reception必须向Kitchens发送订单，这就是为什么Reception创建了一个管道并将其传递给新的Kitchens，并且FIFO和厨房都由一个unsigned int _internalCount标识，每一个新厨房都会增加。
std::map<unsigned int, Kitchen> _kitchens将存储任何新创建的厨房，std::map<unsigned int, FIFO> _fifo将存储接收的每个频道，并使用它们发送数据。
请注意，_internalCount被附加到管道名称，这是类似于'./pipes/kitchen_1', './pipes/kitchen_2', etc...的结果。
我不知道所有的最好的做法在StackOverflow简化和明确，但以下是接待，厨房和先进先出课程。
#include <fstream>
#include <map>
#include <queue>
#include <string>
#include <vector>
class FIFO {
public:
FIFO() = delete;
FIFO(int channel);
FIFO(const FIFO& copy);
~FIFO();
public:
void operator>>(std::string& container);
void operator<<(const std::string& data);
void operator<<(const char* data);
FIFO& operator=(const FIFO& copy);
void operator()();
public:
std::string readFromChannel();
void sendToChannel(const std::string& data);
void flushChannel();
public:
std::string getFIFOname() const;
private:
void createDataChannel(int type);
private:
std::string _fifoPath;
std::fstream _fifo;
};
class Kitchen {
public:
Kitchen() = delete;
Kitchen(Args args, FIFO newFifo);
~Kitchen();
public:
FIFO getFifo() const;
bool getStatus() const;
private:
void goCooking();
void dispatchOrders();
bool isFree();
private:
Stock _stock;
Time _time;
int _maxChefs;
std::vector<Chef> _chefs;
std::queue<std::string> _orders;
int _currentAssign;
bool _status;
bool _isOpen;
FIFO _channel;
};
class Reception {
public:
Reception() = delete;
Reception(int argc, char** argv);
~Reception();
public:
void openPlazza();
void closePlazza();
public:
void createKitchen(std::string order);
void sendRequest(std::string order, int kitchenNb);
void receiveConfirmation();
unsigned int findKitchenAvailable();
private:
bool _open; /*
OrderManager _checker; ** Encapsulate other
Args _args; ** aspects of the project
Shell _shell; **
Process _process; */
FIFO _mainPipe; // Not very useful here
std::map<unsigned int, FIFO> _fifos;
std::map<unsigned int, Kitchen> _kitchens;
unsigned int _internalCount;
};
int main() {}如果您需要什么或需要更多详细信息，请通知我:)
编辑2:
我在这里添加了一些额外的错误/注意事项，在第一行之后开始从g++开始(显示在上面的初始问题中)
/usr/include/c++/9/bits/stl_pair.h:436:9: note: candidate: ‘template<class ... _Args1, long unsigned int ..._Indexes1, class ... _Args2, long unsigned int ..._Indexes2> std::pair<_T1, _T2>::pair(std::tuple<_Args1 ...>&, std::tuple<_Args2 ...>&, std::_Index_tuple<_Indexes1 ...>, std::_Index_tuple<_Indexes2 ...>)’
436 | pair(tuple<_Args1...>&, tuple<_Args2...>&,
/usr/include/c++/9/bits/stl_pair.h:436:9: note: template argument deduction/substitution failed:
/usr/include/c++/9/bits/stl_pair.h:529:14: note: mismatched types ‘std::tuple<_Tps ...>’ and ‘unsigned int’
529 | return __pair_type(std::forward<_T1>(__x), std::forward<_T2>(__y));一如既往，提前感谢您的时间和建议！
编辑3:
我现在开始把我脑子里的一切都联系起来了，谢谢你们。事情变得棘手，但我肯定从你的所有建议。我将仔细阅读复述-莫妮卡的答案，并在我身边做进一步的研究，我有解决这个问题的钥匙，使它更好。
不好意思，我不清楚，我没想到我的第一个问题是如此棘手，我希望它更清楚。
我的下一个问题会更清楚，我将尝试从一开始就尽量减少可复制代码，我将仔细观察StackOverflow @ted的良好效果！
谢谢大家的支持和时间，保重自己！
回答 2
Stack Overflow用户
发布于 2020-06-10 18:31:21
首先，您应该放置，而不是插入--这样可以避免复制：
_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传递给映射和包含在映射中的对象。这种重复的信息通常是一种糟糕的设计气味。
你应该告诉我们你想要实现什么，这样就会有更好的设计。
Stack Overflow用户
发布于 2020-06-11 16:28:02
class Kitchen {
public:
Kitchen() = delete;
Kitchen(Args args, FIFO newFifo);
~Kitchen();由于您定义了析构函数，所以C++足够聪明地实现默认的复制和移动构造函数和赋值几乎肯定是错误的，因此它不会为您生成缺省值。因此，不可能复制一个Kitchen。
Kitchen newKitchen(_args, newFIFO);
_kitchens.insert(std::make_pair(_internalCount, newKitchen));insert取一对，并尝试将其复制到地图中。如果没有复制构造函数，它就无法做到这一点。有两种解决办法。显而易见的一点是使Kitchen可复制，但这可能是很棘手的。更好的解决方案是不要复制厨房，而是让地图创建一个新的Kitchen，就像在Reinstate Monica's answer中演示的那样。
https://stackoverflow.com/questions/62310584
复制相似问题
腾讯云开发者
