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代替字符，如果你只存儲字符？