0

ボグル ボードのすべての位置を再帰的にカバーする必要があるコードがいくつかあります。しかし、何らかの理由で、たった 2 つの単語を見つけた後にセグメンテーション違反がスローされるのですか? なぜそれが起こるのでしょうか?

おそらくエラーをスローしているコード:

//++++++++++++ Find Word ++++++++++++++//
void findword(vector<vector<tile> > board, unsigned int row, unsigned int col, set<string> &results, unsigned int board_width, string word, set<string> dictionary)
{               
    if(checkbound(row, col, board_width) == false)          // If tile is outside of the boundary than do nothing
        return;
    if(board[row][col].getused() == true)               // If tile has already been used than do nothing
        return;
    word = word + board[row][col].getvalue();           // Adds letter to word

    //Check Prefixes
    set<string>::iterator pos;
    pos = dictionary.lower_bound(word);             // Creates an iterator at position
    if(pos== dictionary.end())                  // If it reaches the end without finding word than do nothing
    {
        return;

    }
    else if(word == pos->substr(0,word.length()))
    {
        cout<<"word: " <<word<<endl;
        cout<<"dict: " <<*pos<<endl;
        if(word == *pos)                    // If word = word at prefix
        {       
            cout<< word<<" word inserted"<<endl;    
            results.insert(word);               // Add words to results set
        }
    }
    else
        return;

    //set to used
    board[row][col].setused(true);                  // set tile to used




    findword(board, row-1, col-1, results, board_width, word, dictionary);  // Checks all tiles around every tile
    findword(board, row-1, col, results, board_width, word, dictionary);
    findword(board, row-1, col+1, results, board_width, word, dictionary);
    findword(board, row, col-1, results, board_width, word, dictionary);
    findword(board, row, col+1, results, board_width, word, dictionary);
    findword(board, row+1, col-1, results, board_width, word, dictionary);
    findword(board, row+1, col, results, board_width, word, dictionary);
    findword(board, row+1, col+1, results, board_width, word, dictionary);

    board[row][col].setused(false);                 // set tile to not-used
}

与えられたエラー:

word: r
dict: riot
word: ro
dict: robot
word: rob
dict: robot
word: robo
dict: robot
word: robot
dict: robot
robot word inserted
word: roo
dict: root
word: root
dict: root
root word inserted
Segmentation fault (core dumped)

Valgrind の主なエラー コード:

==4629== Invalid read of size 1
==4629==    at 0x407C2E: tile::getused() (tile.cpp:33)
==4629==    by 0x401ACE: findword(std::vector<std::vector<tile, std::allocator<tile> >, std::allocator<std::vector<tile, std::allocator<tile> > > >, unsigned int, unsigned int, std::set<std::string, std::less<std::string>, std::allocator<std::string> >&, unsigned int, std::string, std::set<std::string, std::less<std::string>, std::allocator<std::string> >) (main.cpp:58)
==4629==    by 0x4020EC: findword(std::vector<std::vector<tile, std::allocator<tile> >, std::allocator<std::vector<tile, std::allocator<tile> > > >, unsigned int, unsigned int, std::set<std::string, std::less<std::string>, std::allocator<std::string> >&, unsigned int, std::string, std::set<std::string, std::less<std::string>, std::allocator<std::string> >) (main.cpp:93)
==4629==    by 0x4020EC: findword(std::vector<std::vector<tile, std::allocator<tile> >, std::allocator<std::vector<tile, std::allocator<tile> > > >, unsigned int, unsigned int, std::set<std::string, std::less<std::string>, std::allocator<std::string> >&, unsigned int, std::string, std::set<std::string, std::less<std::string>, std::allocator<std::string> >) (main.cpp:93)
==4629==    by 0x401F78: findword(std::vector<std::vector<tile, std::allocator<tile> >, std::allocator<std::vector<tile, std::allocator<tile> > > >, unsigned int, unsigned int, std::set<std::string, std::less<std::string>, std::allocator<std::string> >&, unsigned int, std::string, std::set<std::string, std::less<std::string>, std::allocator<std::string> >) (main.cpp:91)
==4629==    by 0x402264: findword(std::vector<std::vector<tile, std::allocator<tile> >, std::allocator<std::vector<tile, std::allocator<tile> > > >, unsigned int, unsigned int, std::set<std::string, std::less<std::string>, std::allocator<std::string> >&, unsigned int, std::string, std::set<std::string, std::less<std::string>, std::allocator<std::string> >) (main.cpp:95)
==4629==    by 0x402BF0: main (main.cpp:185)
==4629==  Address 0x4c3b178 is 8 bytes after a block of size 48 alloc'd

チェックバウンド関数:

//+++++++++++ Check Bounds ++++++++++++//
bool checkbound(unsigned int row, unsigned int col, unsigned int board_width)
{
    if(row < 0 || row > board_width || col < 0 || col > board_width)
        return false;
    else
        return true;
}

ボード:

r b o
o i t
r o h
4

2 に答える 2

2

board_width の値は? これが 3 の場合、インデックス付けが 0 から始まり、利用可能な最大インデックスが 2 であることを忘れているため、checkbound 関数の条件は次のようにする必要があります。

    if(row < 0 || row >= board_width || col < 0 || col >= board_width)
于 2013-07-10T06:04:25.397 に答える