ループループは永遠にネストされています

Question

初めてのヘビーなポスターです！私はC + +とコーディングのような不正な練習するいくつかのランダムウォークアルゴリズムをコーディングしようとしたが、私の関数は永遠にループし、実際に何もしません。<の前に< < endl; cin.getとループ内のもう1つはエラーチェックのためのもので、エラーがhループのためにあるたびに、私のコアを100％クロックして何もしません。 ？誰かが私を助けてくださいすることができます

#include <iostream> 
#include <cstdlib> 
#include <ctime> 

#define MAP_WIDTH  64 
#define MAP_HEIGHT  24 
#define WALK_DIRECTION 4 
#define TILE_FLOOR  0 
#define TILE_WALL  1 

using namespace std; 

int MapArray[MAP_HEIGHT][MAP_WIDTH]; 
int StepCount = 0; 
const int MaxStepCount = 60; 
void SetWalls(void); 
int GenerateRandom(int a); 
void DrunkardsWalk(void); 

int main(void){ 

    SetWalls(); 
    DrunkardsWalk(); 

    for(int h = 0; h < MAP_HEIGHT; h++){ 
     cout << endl; 
     for(int w = 0; w < MAP_WIDTH; w++){ 
      switch(MapArray[h][w]){ 
       case TILE_WALL: 
       cout << "#"; 
       break; 

       case TILE_FLOOR: 
       cout << "."; 
       break; 
      } 
     } 
    } 
    return 0; 
} 

void SetWalls(void){ 

    for(int h = 0; h < MAP_HEIGHT; h++){ 
     cout << endl; 
     for(int w = 0; w < MAP_WIDTH; w++){ 
      MapArray[h][w] = TILE_WALL; 
     } 
    } 
} 

int GenerateRandom(int a){ 

    int b; 
    srand(time(NULL)); 
    b = rand() % a; 
    return b; 
} 

void DrunkardsWalk(void){ 

    int RandHeight = GenerateRandom(MAP_HEIGHT); 
    int RandWidth = GenerateRandom(MAP_WIDTH); 
    int RandDirection = GenerateRandom(WALK_DIRECTION); 

    cout << "Before while" << endl; 
    cin.get(); 
    while(StepCount != MaxStepCount){ 
     cout << "After while" << endl; 
     cin.get(); 
     for(int h = 0; h < RandHeight; h++){ 
      cout << "For h" << endl; 
      cin.get(); 
      for(int w = 0; w < RandWidth; w++){ 
       cout << "For w" << endl; 
       cin.get(); 
       if(RandDirection == 1){ 
        MapArray[h+1][w] = TILE_FLOOR; 
        StepCount++; 
       } 
       else if(RandDirection == 2){ 
        MapArray[h-1][w] = TILE_FLOOR; 
        StepCount++; 
       } 
       else if(RandDirection == 3){ 
        MapArray[h][w+1] = TILE_FLOOR; 
        StepCount++; 
       } 
       else if(RandDirection == 4){ 
        MapArray[h][w-1] = TILE_FLOOR; 
        StepCount++; 
       } 
      } 
     } 
    } 
} 

Answer 1

をあなたは、全体の実行のために一度だけsrand(time(NULL));を呼び出す必要がありますので、あなたの関数の外に移動し、このようなmain()にそれを置くこともできます。

// .. 

int main(void){ 
    srand(time(NULL)); 
    //.. 

    return 0; 
} 

// .. 

int GenerateRandom(int a){ 
    int b; 
    b = rand() % a; 
    return b; 
} 

Answer 2

ループ条件

while(StepCount != MaxStepCount) 

StepCountがループの繰り返しごとに複数増分されると、それは良い考えではありません。

あなたは

while(StepCount < MaxStepCount) 

ことによってそれを交換した場合それを終了する必要があります。

cin.get（）でブレークポイントを使用する代わりに、gdbのようなデバッガを使用することを検討してください。デバッグの方が効率的で、各ループのカウンタの値を尋ねることができます。