0

我的程序有这个小问题。在 Visual Studio 2012 中它运行良好，但如果我用 G++ 编译它（是的，出于上述原因，我必须使用它来编译），错误信号 11(SIGSEGV) 或 6(SIGABRT) 会根据输入触发. 这是一个编程练习，我有另一个程序（在在线服务器上）用 10 个不同的输入测试我的程序。正如我所说，该程序在使用 Visual Studio 2012 时编译并运行良好。

关于程序：它找到从起点 (x,y) 到多个出口的最短路径（出口的数量无关且不同。可能只有 1 个出口，也可能有 200 个）。输入如下：

7 12          // maze height and width
##########.#  //
#..........#  //
#.###.######  //
#..X#.#.....  // the maze blueprint
#.###.#.####  //
#..........#  //
############  //

我的程序：

#include <iostream>
#include <vector>

typedef struct _laby_t {
    int h, w;
    char **pohja; // 'pohja' is finnish and means layout
} laby_t;

typedef std::vector<int> monovector;
typedef std::vector< std::vector<int> > bivector;

laby_t *laby_allocate (int r, int c)
{
    laby_t *laby;
    int i;

    laby = new laby_t[sizeof (laby_t)];
    laby->pohja = new char *[r];
    for (i = 0; i < r; i++)
    {
        laby->pohja[i] = new char[c];
    }
    laby->h = r;
    laby->w = c;

    return laby;
}

int wander(int y, int x, laby_t *&_laby, int goals)
{
    laby_t *laby = _laby;
    int found = 0, depth = 0, min_path = 1000000;
    bool b = 0;
    bivector openList;
    monovector start; start.push_back(y); start.push_back(x);
    bivector closedList;

    openList.push_back(start);

    while(found < goals)
    {

        y = openList.back()[0]; x = openList.back()[1];
        monovector r; r.push_back(y); r.push_back(x); closedList.push_back(r);
        openList.pop_back();
        if(laby->pohja[y][x] != '*') laby->pohja[y][x] = '-';
        depth++;

        if(y == 0 || y+1 == laby->h || x == 0 || x+1 == laby->w) {
            found++;
            if(depth < min_path) min_path = depth;
            if(found >= goals) {
                std::cout << min_path << std::endl;
                break;
            }
            laby->pohja[y][x] = '-';

            goto back_track;
        }
        else
        {
            b = 0;
            if(laby->pohja[y+1][x  ] == '.') { monovector r; r.push_back(y+1); r.push_back(x); openList.push_back(r); b=1; }
            if(laby->pohja[y  ][x+1] == '.') { monovector r; r.push_back(y); r.push_back(x+1); openList.push_back(r); b=1; }
            if(laby->pohja[y-1][x  ] == '.') { monovector r; r.push_back(y-1); r.push_back(x); openList.push_back(r); b=1; }
            if(laby->pohja[y  ][x-1] == '.') { monovector r; r.push_back(y); r.push_back(x-1); openList.push_back(r); b=1; }
            if(!b)
            {
back_track:     while(closedList.size() > 0)
                {
                    //std::cout << closedList.size() << std::endl;
                    int c_y = closedList.back()[0]; int c_x = closedList.back()[1];
                    int o_y = openList.back()[0];   int o_x = openList.back()[1];

                    laby->pohja[y][x] = '*';

                    y = c_y; x = c_x;

                    laby->pohja[y][x] = '*';

                    if( (c_y+1 == o_y && c_x   == o_x) ||
                        (c_y   == o_y && c_x+1 == o_x) ||
                        (c_y-1 == o_y && c_x   == o_x) ||
                        (c_y   == o_y && c_x-1 == o_x) )
                    {
                        laby->pohja[y][x] = '-';
                        y = o_y; x = o_x;
                        closedList.pop_back();
                        depth--;
                        break;
                    }
                    else {
                        closedList.pop_back();
                        depth--;
                    }
                }
            }
        }
    }

    return min_path;
}

int main()
{
    int h, w, goals = 0;
    std::cin >> h >> w;

    laby_t *laby;
    laby = laby_allocate(h, w);

    for(int i = 0; i < laby->h; i++)
        std::cin >> laby->pohja[i];

    for(int i = 1; i < laby->h-1; i++) {
        if(laby->pohja[i][0] == '.') goals++;
        if(laby->pohja[i][laby->w-1] == '.') goals++;
    }

    for(int i = 1; i < laby->w-1; i++) {
        if(laby->pohja[0][i] == '.') goals++;
        if(laby->pohja[laby->h-1][i] == '.') goals++;
    }

    for(int i = 0; i < laby->h; i++)
        for(int j = 0; j < laby->w; j++) {
            if(laby->pohja[i][j] == 'X') {
                wander(i, j, laby, goals);
                goto _exit;
            }
        }

_exit:

    //system("pause");
    return 0;
}

我已经完成了关于错误信号的作业，如果你们不知道：http ://www.yolinux.com/TUTORIALS/C++Signals.html

提前致谢。

4

2 回答 2

5

该代码可以在带有 G++ 4.7.1 的 Mac OS X 10.7.5 上干净地编译，这很好：

g++ -g -Wall -Wextra laby.cpp -o laby

不幸的是，当结果在 下运行时valgrind，它会产生：

==15030== Invalid write of size 1
==15030==    at 0x306BE: std::basic_istream<char, std::char_traits<char> >& std::operator>><char, std::char_traits<char> >(std::basic_istream<char, std::char_traits<char> >&, char*) (in /usr/lib/libstdc++.6.0.9.dylib)
==15030==    by 0x10000117D: main (laby.cpp:113)
==15030==  Address 0x10001632c is 0 bytes after a block of size 12 alloc'd
==15030==    at 0xB823: malloc (vg_replace_malloc.c:266)
==15030==    by 0x5768D: operator new(unsigned long) (in /usr/lib/libstdc++.6.0.9.dylib)
==15030==    by 0x576DA: operator new[](unsigned long) (in /usr/lib/libstdc++.6.0.9.dylib)
==15030==    by 0x1000008C0: laby_allocate(int, int) (laby.cpp:21)
==15030==    by 0x100001146: main (laby.cpp:110)

因此，函数中的内存分配存在问题laby_allocate()。或者几个...

laby_t *laby_allocate (int r, int c)
{
    laby_t *laby;
    int i;

    laby = new laby_t[sizeof (laby_t)];

这一行分配了一个数组laby_t；它在数组中分配与 a 中的字节数一样多的元素laby_t。这不是你需要的。

    laby = new laby_t;

继续：

    laby->pohja = new char *[r];
    for (i = 0; i < r; i++)
    {
        laby->pohja[i] = new char[c];
    }

这没有为数据末尾的 null 分配足够的空间......这就是为什么“写”是 1 个字节的原因。更改c为c+1并valgrind提供干净的健康单。

    laby->h = r;
    laby->w = c;

    return laby;
}

给出的答案是15；我不相信这是正确的。

于 2012-12-29T17:40:53.920 回答
2

此行溢出您的内存分配。如果用户输入w个字符，则需要(w+1) 个字符来保存以空字符结尾的字符串。

    std::cin >> laby->pohja[i];

这条线还分配了许多laby_t对象的数组，尽管您似乎只想要一个。也许您将 C++new与 C混淆了malloc。

laby = new laby_t[sizeof (laby_t)];

你可以用这个替换它。

laby = new laby_t;

这似乎也是 C 的残余。这不是一个错误，但它不必要地用冗余符号污染了当前的命名空间。

typedef struct _laby_t { ... } laby_t;

你可以用这个替换它。

struct laby_t { ... };
于 2012-12-29T17:30:20.150 回答