C++ consoleプログラム受信シグナルSIGSEGV、セグメンテーションフォールト

Question

私は、私のマップであるマルチディメンション配列へのポインタを使用しようとしています。したがって、最後のコードでは、すべてのマップでコリジョンを生成する代わりにポインタを使用できます。不幸にも、地図を描いた後、矢印を押して移動すると、プログラムがクラッシュします。 デバッガは、これはマップの一つであるタイトル

にあるものだけを私に言った：

char *map2[15] = {  
"###^######^###",  
"#L   #", 
"^S   #", 
"#S   #", 
"# K  #", 
"########### #", 
"#G   #", 
"#  #### #", 
"#  #M #", 
"# @  # #", 
"## ########^##"}; 

これは初めである：

マップや移動を示す

 mainmap = false; 
     x = 2; 
     game.showing_different_maps(15, map2); // (y, somemap) 

：

void Game::showing_different_maps(int y, char **somemap) 
{ 
bool differentmap = true; 
while (differentmap) 
{ 
    system("cls"); 
    for(int i = 0; i < y; i++) // i < map2[y][] 
    { 
     cout << somemap[i] << endl; // drawing a map !THIS CAUSES CRASH! 
    } 

    system("pause>nul");  
    if(GetAsyncKeyState(VK_UP)) 
    { 
     mapki.move(-1, 0); 
    } 
    if(GetAsyncKeyState(VK_DOWN)) 
    { 
     mapki.move(1, 0); 
    } 
    if(GetAsyncKeyState(VK_RIGHT)) 
    { 
     mapki.move(0, 1); 
    } 
    if(GetAsyncKeyState(VK_LEFT)) 
    { 
     mapki.move(0, -1); 
    } 
} 

を

}

衝突を扱う

スクリプト：

void Maps::move(int V, int H) 
{ 
int y2 = y + V; 
int x2 = x + H; 
if(map2[y][x2] == ' ' 
|| map2[y][x2] == 'K' 
|| map2[y][x2] == 'W') 
{ 
    map2[y][x] = ' '; 
    x += H; 
    map2[y][x] = '@'; 
} 
if(map2[y2][x] == ' ' 
|| map2[y2][x] == 'K' 
|| map2[y2][x] == 'W') 
{ 
    map2[y][x] = ' '; 
    y += V; 
    map2[y][x] = '@'; 
}      

}

私は[] [] [] []、代わりにMAP2のsomemapどこでも書きたかったが、それが動作するかどうか、私はチェックcan`t、 "の理由プログラム受信信号SIGSEGV、セグメンテーションフォルト。クラッシュ。

Answer 1

ここでのエラーは、サイズが15であるが、初期化された要素が11個しかないポインタの配列に由来します。つまり、最後の4はNULLに初期化されます。

char *map2[15] = {  
"###^######^###",  
"#L   #", 
"^S   #", 
"#S   #", 
"# K  #", 
"########### #", 
"#G   #", 
"#  #### #", 
"#  #M #", 
"# @  # #", 
"## ########^##", 
NULL, NULL, NULL, NULL}; 

それ自体で、これは無害だろうが、あなた後で使用：

game.showing_different_maps(15, map2); // (y, somemap) 

だからあなたがそうcout << SIGSEGVにヌル文字ポインタを渡して終了実際にMAP2のためのあなたの宣言はこれに等しいです。

あなたは、少なくともここでは、11の代わりに15を初期化配列のサイズを渡す必要があります。

game.showing_different_maps(11, map2); // (y, somemap) 

をしかし、あなたがテストする必要がありますので、私は、あなたがmoveで、後に問題があります疑うこと0 < = Y + V < VMAXおよび0 < = x + H < HMAX（ここではVMAX = 11およびHMAX = 14）。そうでない場合は、文字列や文字列配列の範囲外にアクセスして、未定義のビヘイビアを呼び出すことができます。