如果这是错误的网站,请提前致歉,如果是的话,请告诉我!

我编写了一个函数来检查特定的std::map中是否存在密钥,并想知道这是否是一种好习惯,以及是否有人可以在改进方面投入任何指针。

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++ - C++ std::map这是正确的做法吗,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/18647357/

10-11 22:47
查看更多