如果这是错误的站点,请提前致歉,如果是,请告诉我!
我编写了一个函数来检查特定 std::map
中是否存在 key 并想知道这是否是一个好的做法,并且,是否有人可以提出任何改进建议。
std::map
允许为值接受多种数据类型。
union Variants {
int asInt;
char* asStr;
Variants(int in) { asInt = in; }
Variants() { asInt = 0;}
Variants(char* in) { asStr = in; }
operator int() { return asInt; }
operator char*() { return asStr; }
};
template<typename T, typename Y>
bool in_map(T value, std::map<T, Y> &map)
{
if(map.find(value) == map.end()) {
return false;
}else{
return true;
}
}
然后我可以主要使用以下内容:
std::map<string, Variants> attributes;
attributes["value1"] = 101;
attributes["value2"] = "Hello, world";
if(in_map<std::string, Variants>("value1", attributes))
{
std::cout << "Yes, exists!";
}
如有任何帮助或建议,我们将不胜感激。抱歉,如果这不符合规则或标准。谢谢!
最佳答案
我看到您的函数最大的问题是您丢弃了生成的迭代器。
当您检查映射中是否存在某个键时,大多数时候您希望在此之后检索/使用关联的值。在这种情况下使用您的函数会迫使您以性能为代价进行双重查找。我会完全避免使用该函数,直接编写测试,保留迭代器以备后用,以避免无用的查找:
auto it = map_object.find("key");
if (it != map_object.end())
use(it->second);
else
std::cout << "not found" << std::endl;
当然,如果您只是检查一个键是否存在而不关心关联的值,那么您的功能就可以了(考虑到其他人在评论中告诉您的内容),但我认为它的用例非常有限并不值得额外的功能。你可以这样做:
if (map_object.find("key") != map_object.end())
std::cout << "found, but I don't care about the value" << std::endl;
关于C++ std::map 这是正确的做法吗,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/18647357/