Ошибка с массивами в C++

Question

Проблема в том, что я хочу «o», а не повторять ... Я пытался найти, но я провалился! Также объясните мне, как это происходит. Я новый программист!

#include <iostream> 
#include <string> 
#include <stdio.h> 
#include <stdlib.h> 
#include <conio.h> 
#include <time.h> 

using namespace std; 

int main() 
{ 
    //all in-game variables here 
    string o = "o"; 
    int oH = 0; 
    int oW = 0; 
    //variables ending point... 

    bool run = true; 
    bool frameShow = true; 
    char input; 
    int width = 53; 
    int height = 22; 

    string px[width][height]; 

    while (run) 
    { 
     for (int xEmp=0; xEmp<height; xEmp++){ 
      for (int yEmp=0; yEmp<width; yEmp++) 
      { 
       px[xEmp][yEmp]= " "; 
      } 

      if (frameShow) 
      { 
       clrscr();   // Must be at start 

       px[oH][oW] = o; 

       for (int x=0; x<height; x++) 
       { 
        for (int y=0; y<width; y++) 
        { 
         cout << px[x][y]; 
        } 
        cout << "\n"; 

        frameShow = false; // Must be at end 
       } 
      } 

      if (kbhit()) 
      { 
       input = getch(); 

       // Most Used ones are: 
       // char 119 for "w" 
       // char 97 for "a" 
       // char 115 for "s" 
       // char 100 for "d" 
       // char 32 for "space" 
       // char 27 for ESC 

       if (input == 119) 
       { 
        oH--; 
        frameShow = true; 
       } 
       else if (input == 115) 
       { 
        oH++; 
        frameShow = true; 
       } 
       else if (input == 97) 
       { 
        oW--; 
        frameShow = true; 
       } 
       else if (input == 100) 
       { 
        oW++; 
        frameShow = true; 
       } 
       else if (input == 27) 
       { 
        // Game exits... 

        // To terminate, use: 
        run = false; 
       } 

       if(oH > height) 
       { 
        oH = height; 
       } 
       else if(oH < 0) 
       { 
        oH = 0; 
       } 

       if(oW > width - 1) 
       { 
        oW = width - 1; 
       } 
       else if(oW < 0) 
       { 
        oW = 0; 
       } 

      } 
     } 
    } 

    // Output for confiming program termination 
    clrscr(); 
    cout << "\n - Terminated! - \n"; 
    return 0; 
} 

Answer 1

В начале, width и height использование изменяется. px[width][height];, px[xEmp][yEmp], xEmp<height, yEmp<width. Один раз вы используете его как px[width][height];, затем px[height][width];. держите свой код согласованным!

Также if(oH > height) { oH = height; } неправ. вычесть один из высоты.

Это также, вероятно, не делать то, что вы хотите:

for (int x=0; x<height; x++) 
    for (int y=0; y<width; y++) 
    { 
     cout << px[x][y]; 
    } 
    cout << "\n"; 

Используйте скобки правильно, если вы не знаете, как использовать их, положить их ВСЕГДА!

Опять же, я думаю, что вы не используете скобки правильно:

for (int xEmp=0; xEmp<height; xEmp++){ 
    for (int yEmp=0; yEmp<width; yEmp++) 
    { 
     px[xEmp][yEmp]= " "; 
    } 
... // do other things 
} 

Я думаю, что вы хотите, чтобы закрыть его сразу установить все обратно в космос и их делать другие работы. Как и сейчас, он вернет одну строку в пробелы, распечатает все и запустит ваш код и только после пустой следующей строки.

for (int xEmp=0; xEmp<height; xEmp++){ 
    for (int yEmp=0; yEmp<width; yEmp++) 
    { 
     px[xEmp][yEmp]= " "; 
    } 
} 
... // do other things 

Ps: clrscr() не является стандартной функцией, работает только в Windows, я думаю, для Linux использовать system('clr');

Ps2: Почему вы используете std::string вместо полукокса, если вы только хранить символьные?