在C++中实现图形DFS，指针问题

很抱歉，有些变量名之类的名字很奇怪，因为我不是以英语为母语的人，我对编程也很陌生

所以，我在做一个关于图形拓扑排序和类似的项目，但是我不能让DFS工作，我知道我可能会因为在那里使用指针而丢失数据，但无论如何我不会在我的程序中使用这个邻接列表。问题是要得到正确的结果，在这种情况下，当顶点被输入d[]和左f[]时，但在递归中，我的指针会发疯，有时当我返回递归时，显然没有其他事情发生，我想至少因为这是我第一次使用调试函数。我在这里坐了8个小时，这已经不是我的第一个问题了，但我设法解决了一些问题，这就是为什么代码看起来很难看，我坐在这个调试器旁边，一个多小时没有取得任何进展，所以我决定问，我第一次使用这个网站，我希望你能帮助我，当我好一点的时候，我一定会报答你，下面是代码：

#include <iostream>
#include <cstdlib>
#include <time.h>
struct m_sasiedztwa
{
    int s;
    int** m;
    m_sasiedztwa(int a,float b) : s(a)
    {
        m = new int*[s];
        for(int i = 0; i < s; ++i)  m[i] = new int[s];
        for(int j=0; j<s; ++j)
            for(int k=0; k<s; ++k) if(j!=k) m[j][k]=((rand()%100)>(100*b))? 0 : 1; else m[j][k]=0;

    }
    ~m_sasiedztwa()
    {
        delete[] m;
    }
};
struct lista
{
    int key;
    lista *next;
};
struct l_nast
{
    int s;
    lista** arr;
    l_nast(int** m, int a) : s(a)
    {
        lista *curr,*prev;
        arr = new lista*[s];
        for(int i=0;i<s;++i)
        {
            arr[i] = new lista;
            curr = arr[i];
            prev=curr;
            prev->key=-1;
            for(int j=0;j<s;++j)
            {
                if(m[i][j]==1) {curr->next= new lista;curr->key=j;prev=curr;curr=curr->next;}

            }
            prev->next=nullptr;
        }

    }

~l_nast() {delete[] arr;}
};


//Here is the issue

bool *Nowy;
int c;
int* d,*f;
void DFS(int j,l_nast l_a)
{

            Nowy[j]=false;
            d[j]=c++;
            std::cout<<"Am I here yet..."<<j<<" "<<c<<std::endl;
                while((l_a.arr[j]!=nullptr)&&(l_a.arr[j]->key!=-1))
                {
                    std::cout<<"checking "<<(l_a.arr[j]->key)<<"...\n";
                    if(Nowy[l_a.arr[j]->key])
                    {

                    DFS(l_a.arr[j]->key,l_a);

                    }

                    if(l_a.arr[j]->next!=nullptr)                               //And I think this may be the problem, but I honestly don't know
                        l_a.arr[j]=l_a.arr[j]->next;
                            else break;
                }

            f[j]=c++;std::cout<<"Yohoo!"<<j<<" "<<c<<std::endl;

}




//Untill there

using namespace std;



int main()
{
    srand (time(NULL));
    for(int q=5; q<6; q+=5)
    {

        m_sasiedztwa a = m_sasiedztwa(q, 0.2);
        m_sasiedztwa b = m_sasiedztwa(q, 0.4);

        l_nast l_a = l_nast(a.m,q);
        l_nast l_b = l_nast(b.m,q);
      /*
           for(int i=0; i<q; ++i)
            {
                for(int j=0; j<q; ++j)
                {
                    cout << a.m[i][j] << " ";
                }
                cout<<endl;
            }
            cout<<endl;
*/

        Nowy = new bool [q];
        d = new int [q];
        f = new int [q];
        c=0;
    for (int i = 0; i < q; i++)
        Nowy[i] = true;
/*for(int qq=0;qq<q;qq++)
  while((l_a.arr[qq]!=nullptr))
  {
      cout<<l_a.arr[qq]->key<<endl;
      l_a.arr[qq]=l_a.arr[qq]->next;

  }
*/

        for(int j=0;j<q;j++)
        {

            if(Nowy[j]) DFS(j,l_a);

        }



        a.~m_sasiedztwa();
        b.~m_sasiedztwa();

        l_a.~l_nast();
        l_b.~l_nast();
    }

    return 0;
}

---

正如我所说，它不漂亮，抱歉打扰你了，我需要帮助的是让函数DFS正确地用D[]，这是一个表，如果它指示顶点被访问时的整数，和F[]表，当顶点取自堆栈时，只需订购1，2，3…，问题是它在中间中断，有时它像7/10，有时只是2/10，它会断裂，当然，它也会对更大的图形起作用。指针丢失，它现在试图检查[那里有一些大数字]，程序崩溃。

因此，我使用struct很糟糕，犯了很多错误，由于一些评论，我决定使用向量向量表示adj矩阵，使用正向列表向量表示adj列表。这就是改变m_sasiedztwa结构的原因

struct m_sasiedztwa
{
    int s;
    std::vector<std::vector<int>> m;
    m_sasiedztwa(int a,float b) : s(a), m(s, std::vector<int>(s,0))
    {
        for(int j=0; j<s; ++j)
            for(int k=0; k<s; ++k)
                if(j!=k)
                    m[j][k]=((rand()%100)>(100*b))? 0 : 1;
     }
};

l_nast结构：

struct l_nast
{
    int s;
    std::vector<std::forward_list<int>> arr;
    l_nast(std::vector<std::vector<int>> m, int a) : s(a), arr(s)
    {
        for(int i=0;i<s;++i)
        {
            auto it = arr[i].before_begin();
            for(int j=0;j<s;++j)
            {
                if(m[i][j]==1) {it = arr[i].emplace_after(it,j);}

            }
        }

    }

};

以及DFS：

void DFS(int j,l_nast l_a)
{

            Nowy[j]=false;
            d[j]=c++;
            std::cout<<"Visiting "<<j<<"as "<<c<<std::endl;
        auto x=l_a.arr[j].begin();

        for(auto& x: l_a.arr[j])
        {
                    if(Nowy[x])
                    {

                    DFS(x,l_a);

                    }

        }
            f[j]=c++;std::cout<<"Leaving "<<j<<"as "<<c<<std::endl;

}

---

a、 ~m_sasiedztwa；-你为什么要这样做，还有像这样的线条？这肯定是错误的。永远不要显式地调用析构函数，除非您正在执行诸如placement new之类的操作，而您肯定不会这样做；这是不够的。它只删除int**m，这是指向内部维度缓冲区的指针数组，而不删除内部维度缓冲区。这里内存大泄漏。考虑一下，制作一个简单的2D矩阵类来处理三／五的规则。开始。该结构将替换执行各种动态分配的结构。注意，根本不需要析构函数。谢谢你的建议！这确实更好，我为析构函数的错误使用感到抱歉，我甚至还没有使用向量，但它看起来好多了。编辑：我有一个问题，因为我是这个网站的新手，我应该编辑我的帖子并实现你在这里写的东西，这样其他人可以更容易地阅读代码，还是我应该保持原样？你应该保持原样。