vector <bool> iterator not dereferencable </bool>

Question

我正在尝试在编写程序后调试我的程序并遇到此错误：

这是我的代码：

#include<fstream>
#include <iostream>
#include <vector>
#include <string>
#define MAXINT 2147483647

using namespace std;

struct Edge
{
int id, weight;

Edge(int y, int w)
{
    id = y;
    weight = w;
}
};

struct Node
{
vector <Edge *> Edges;
};

struct Graph
{
vector < Node *> vertices;
vector < int > indices;

void Resize(int x)
{
    if (vertices.capacity() < x)
        vertices.resize(x);
}

void InsertEdge(int x, int y, int weight)
{
    Resize(((x > y) ? x : y) + 1);
    InsertVertex(x);
    InsertVertex(y);
    vertices[x]->Edges.push_back(new Edge(y, weight));
}

void InsertVertex(int x)
{
    if (vertices[x] == NULL)
    {
        Node *t = new Node;
        vertices[x] = t;
        indices.push_back(x);
    }
}
};



void Dij(Graph const &g, int start)
{
Node *temp;
vector<bool> check;
vector<int>  distance, prev;
int v, w, weight, dist;

for (int i = 0; i <= g.indices.size(); i++)
{
    check.push_back(false);
    distance.push_back(MAXINT);
    prev.push_back(-1);
}

v = start;
distance[v] = 0;

while (!check[v])
{
    check[v] = true;
    temp = g.vertices[v];


    for (int i = 0; i < temp->Edges.size(); i++)
    {
        w = temp->Edges[i]->id;
        weight = temp->Edges[i]->weight;

        if (distance[w] > (distance[v] + weight))
        {
            distance[w] = distance[v] + weight;
            prev[w] = v;
        }
    }

    v = 1;
    dist = MAXINT;

    for (int x = 0; x < g.indices.size(); x++)
    {
        int i = g.indices[x];

        if (!check[i] && dist > distance[i])
        {
            dist = distance[i];
            v = i;
        }
    }
}
}

int main()
{
int startNode, nodeOne, nodeTwo, number;
Graph g;
ifstream myReadFile;
myReadFile.open("P:\\Documents\\New Folder\\Test\\src\\Read.txt");
while (!myReadFile.eof()) 
{
    myReadFile >> nodeOne;
    myReadFile >> nodeTwo;
    myReadFile >> number;
    g.InsertEdge(nodeOne, nodeTwo, number);
}

cout<< "Enter the starting node: ";
cin >> startNode;

Dij(g, startNode);

return 0;
}

我为烦人的格式化道歉= /。它在dij方法的最后一个for循环期间中断。有人知道我可能会省略什么吗？

Answer 1

帕迪是对的！

但作为建议，请勿使用vector<bool> ...

请参阅，C ++众神想要创建一个节省空间的存储结构来存储bools。为了提高空间效率，他们使用了bits。因为C ++中没有bits单位：他们被迫使用chars。但是一个字符是8位!! C ++众神提出了一个独特的解决方案：他们制作了一个特殊的成员类型：reference来访问bools。您无法以任何其他方式访问boolean值。从技术上讲，vector<bool>甚至不是一个容器：由于元素是chars，因此迭代器无法解除引用。

存储位的更好，更清晰的方法是使用bitset类。

Answer 2

我认为你在这些向量中预先填充了错误数量的元素。您正在迭代g.indices.size()，它应该是g.vertices.size()。

您的其他代码知道indices可能比vertices短。您可以使用从check中提取的值为distance，prev和indices向量编制索引。您获得的运行时错误可能是由于调试模式下的迭代器边界检查。

这可以解决您的问题：

for (int i = 0; i <= g.vertices.size(); i++)  // <-- notice the change here
{
    check.push_back(false);
    distance.push_back(MAXINT);
    prev.push_back(-1);
}