列出迭代器而不是dereferencable

Question

所以我正在写这个道路网络类​​。它包含一个地图，用于保持一个顶点和一组与之相连的顶点。

struct vertex {
      double lat;  //latitude
      double longit;  //longitude
      vertex(double lat, double longit) :lat(lat), longit(longit) {}
};
struct hash_vertex { //hash function for map and set
      unsigned operator()(const vertex& v) const {
          string s(to_string(v.lat) + to_string(v.longit));
          hash<string> hash;
          return hash(s);
      }
};
struct equal_vertex {  //equal function for map and set
      bool operator()(const vertex &v1, const vertex &v2) const {
            return abs(v1.lat - v2.lat) + abs(v1.longit - v2.longit) < error;
      }
};
class road_network {
  private:
      unordered_map<vertex, unordered_set<vertex,hash_vertex,equal_vertex>, hash_vertex, equal_vertex> road;
  public:
      void addedge(const vertex &u, const vertex &v) {
          auto it = *road.find(u);
          auto it2 = *road.find(v);
          it.second.insert(v);
          it2.second.insert(u);
}
};

它编译。但每当我尝试使用函数addge时，程序会抛出一个运行时错误：列出迭代器而不是dereferencable？

有人能告诉我这段代码有什么问题吗？ 提前致谢！

Answer 1

你应该在解除引用之前检查find的结果：

auto it = road.find(u);
if (it != road.end()) {  auto x = *it;}

如果find找不到该元素，则返回end迭代器和解除引用的未定义行为。

Answer 2

您取消引用find()的迭代器结果，而不测试有效结果。 像这样改变你的代码：

      auto it = road.find(u);
      auto it2 = road.find(v);
      if(it != road.end() && it2 != road.end()) {
          it->second.insert(v);
          it2->second.insert(u);
      }