中止使用的memcpy

Question

#include <iostream> 
#include <cstring> 

using namespace std; 

const char* level1[23] = 
{ 
"XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX", 
"X                X", 
"X XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX  X", 
"X X K       X   D    X X", 
"X XXXXXXXXXXXXXXXXXXXXXX XXXXXXXXX XXXXXXXXXXXXXXXXXXXXX X X", 
"X X X  X     X     XXXXX X X", 
"X X XXXXX X XXXXXXXXXXXXXX XXXXXXXX XXXXXXXXXXX  X X", 
"X X  X X  D  X X  X X  X XXXXX X", 
"X X XXXX X X XXXXXXXXXX X  X   X XXXX  X X", 
"X  X  X X E X XXXXXXXXX XXXXXXXX X XXXXX X X", 
"X XXXXXXXXXXXXXXXX D X XXK  X  X X X  KX X", 
"X X  X  X KX XXXXXX XXXXXX XXXXXXXX X X X XXXXXXDX", 
"X X X X X X X   XXX X   X X X D X X", 
"X X X X X X XXXXXXXXXXXXXXX XXXXXXXXXXXXXX X XXXXX X X", 
"X X X X X X X X    X   KX X  X X", 
"X X X X X X X X XXXXXXXXXXX X XXXXXXXXXXX X XXXXX X", 
"X X X X X X X X X  X KX X X   X  X X", 
"X  X X X X X  X X  XX X X  XXXXXXXXXXXXX X X", 
"X X X X X X X X X XXXXXXXX X X  X  X  X X", 
"X X X  X  X X X   X X X X  X   X X", 
"X X XXXXXXXXXXXXX X XXXXXXXX X X X XXXXXXXXXXXXXXXXXX X", 
"XS X               X", 
"XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX" 
}; 

// Player structure 
struct playerinfo 
{ 
    int x;  // X position 
    int y;  // Y position 
    int keys; // Number of keys the player has 
}; 

// Current game info structure 
struct gameinfo 
{ 
    char** board;  // The maze array 
    int maxx;   // Max size of the maze array X coordinate 
    int maxy;   // Max size of the maze array X coordinate 
    int level;   // The level of the maze 
    playerinfo player;  // Link to player data struct 
}; 

// Initializes the dynamic array that stores the maze 
char** initboard(gameinfo game) 
{ 
    char** matrix = new char*[game.maxx]; 

    for (int i=0; i<game.maxx; i++) 
     matrix[i] = new char[game.maxy]; 

    return matrix; 

} 

void delboard(gameinfo game) 
{ 
    for (int i=0;i<game.maxx;i++) 
     delete [] game.board[i]; 
    delete [] game.board; 
} 

int main() 
{ 
    gameinfo game; 

    game.maxx=65; 
    game.maxy=23; 

    game.board = initboard(game); 
    memcpy(game.board,level1,sizeof(level1)); 

    delboard(game); 
} 

核心转储填充之后删除动态二维数组时（核心转储）发生在的“delete [] game.board[i];”的第一步...当使用memcpy()线这只发生。我想要做的是将c-string常量的内容复制到动态二维数组中。问题是，游戏中会有多个可能不同大小的游戏。我最初尝试game.board[0] = "contents of line 0" ... game.board[23] = "contents of line 23"，但这给了编译器警告。奇怪的是，这完美地在Windows下使用MinGW，但我需要这个也可以在Linux上使用。

Answer 1

这条线：

memcpy(game.board,level1,sizeof(level1)); 

是错误的，因为：

1. 它覆盖的game.board的内容，这是被分配由level1内容持有的char*阵列。

2. 你正在写内存，你没有分配。

你可以通过改变initboard到level1内容复制到合适的存储位置，并删除上面的一行解决问题。

char** initboard(gameinfo game) 
{ 
    char** matrix = new char*[game.maxx]; 

    for (int i=0; i<game.maxx; i++) 
    { 
     matrix[i] = new char[game.maxy]; 

     // Copy the contents of `level1[i]` to `matrix[i]`. 
     strpcy(matrix[i], level1[i]); 
    } 

    return matrix; 
} 

更新（感谢@WhozCraig的目光敏锐）

您使用game.maxx和game.maxy值是颠倒的。您需要：

game.maxx=23; // Not 65; 
game.maxy=65 // Not 23; 

然后，你需要在initboard分配game.maxy+1字符。

 matrix[i] = new char[game.maxy+1]; 

否则，就不会有足够的空间来容纳终止空字符matrix[i]。

Answer 2

您通过不调用new来分配堆栈，因此不需要在最后实际删除该数组。当代码运行完毕时，它将被处理！您不能在堆栈上删除，或者您将从您无权访问的地址删除，因此中止。

Answer 3

将char** board;更改为vector<string> board;。摆脱initboard和delboard，并在main()去：

game.board = vector<string>(begin(level1), end(level1)); 

您可能需要包括vector，string，iterator。您也可以删除maxx和maxy，并直接从board中读取它们。

See it working ..看到这一切是多么简单？