C++ std::unordered_map用const char*作键时查找已存在key失败

std::unordered_map<const char*, ImporterFn>查找命中失败的核心问题是：const char*作为键时，默认的哈希计算、相等判断都是针对指针本身的内存地址，而非指针指向的字符串内容。

• 调用addImporter("png", ...)时，传入的"png"是位于只读常量区的字符串字面量，传入的指针指向常量区内存
• 调用loadFile时，你从路径中截取得到的ext是局部std::string对象，ext.c_str()返回的指针指向这个局部字符串内部管理的字符数组，和常量区"png"的内存地址完全不同
• 两者指向的字符串内容完全一致，但内存地址不同，默认判断逻辑会直接判定为键不匹配，自然查找失败。

直接将unordered_map的键类型替换为std::string即可，不需要自定义哈希或比较谓词——标准库已经为std::string实现了基于字符串内容的哈希计算和相等判断，逻辑可靠且没有额外的生命周期管理风险。
需要修改的核心代码段如下：

1. 修改私有成员的map定义：

   private:
       // 键类型从const char*替换为std::string
       std::unordered_map<std::string, ImporterFn> importers{};
   

2. 对应调整addImporter的参数与内部逻辑：

   void addImporter(const std::string& extension, ImporterFn fn) {
       // 重复注册判断直接基于字符串内容匹配
       if(importers.contains(extension))
           return;
       std::cout << "Adding importer for " << extension << std::endl;
       importers.emplace(extension, fn);
   }
   

3. 调整loadFile中的查找逻辑，顺手补上无扩展名文件的边界判断：

   void loadFile(const std::string& path) {
       std::size_t extDot = path.find_last_of(".");
       if(extDot == std::string::npos) {
           std::cout << "Invalid file: no extension found for " << path << std::endl;
           return;
       }
       std::string ext = path.substr(extDot+1);
       // 直接传入std::string类型的扩展名查找，无需转c_str()
       auto it = importers.find(ext);
       if(it != importers.end()) {
           std::cout << "Importer found for " << ext << std::endl;
           it->second(path);
       }
       else {
           std::cout << "No importer found for " << ext << std::endl;
       }
   }
   

不推荐为const char*自定义哈希和比较谓词的方案：这种写法要求你手动保证所有传入的const char*指针生命周期有效，极易出现悬空指针、内存非法访问的问题，用std::string做键会自动管理内存，无额外心智负担。

修复后重新编译运行，输出符合预期：

Adding importer for png
Importer found for png
Loading png image
No importer found for jpeg

内容的提问来源于stack exchange，提问作者louvtt