C++ 控制台输出到 cmd 重复十次后出错

Question

我的代码的重点是打印到控制台，在提供r、b、g 或 o 键盘时显示RED、BLUE、GREEN 或OFF输入。 x 键盘输入终止。

还输出了一堆 SPACE 字符，设置了背景颜色以便在终端中显示纯色 block :

int i;
for (i=0;i<21;i++) {    // for loop to print a load of empty lines
    cout << "                                             \n";
}   // end of for loop

一开始一切正常，但是在第十次更改之后(即按 r、b、g 或 o，然后按 ENTER)，实心色 block 溢出 \n 字符。

谁能告诉我为什么会这样？

我还预计 int main() 中的 else 语句将处理与预期的 r、b、g、o 或 x 不匹配的任何输入。它适用于单个字符，但如果输入了多个字符，它就会变得一团糟，不断滚动并且不会停止输入。

这是为什么？

完整代码(通过 NppExec 使用 MinGW g++.exe 编译，在 Win7 的 cmd 中运行):

#include <windows.h>
#include "iostream"
using namespace std;

/*******FUNCTION PROTOTYPES*********/
void TermClr(int ClrHeight);
void SetColor(int value);
void PrintOut(int output, int col1, int col2);

/*******MAIN PROGRAM****************/
int main() {

    int output = 0; // variable for current colour being output, 0-3 for OFF, RED, BLUE, GREEN
    char inkey[2];  // console input written to the INKEY variable
    PrintOut(0,119,7);  // calls printout function, dark grey background for the light, normal grey for text

    while(1){   //while loops forever until break
        cin.getline(inkey,2);   //waits for input, stored to INKEY
        if(!strcmp(inkey,"x")){     // string compare whether INKEY matches "x"
            cout << "\nProgram Terminated in\n3...";
            Sleep(1000);
            cout << "\n2...";
            Sleep(1000);
            cout << "\n1...";
            Sleep(1000);
            break;  //breaks if 'x' is input
        }   // end if inkey x
        else if(!strcmp(inkey,"o")){
            PrintOut(0,119,7);  // calls PrintOut function, 'output' set to 0, red background for the light, red for text
            continue;
        }   // end of if inkey o
        else if(!strcmp(inkey,"r")){
            PrintOut(1,204,12);
            continue; 
        }   // end of if inkey r
        else if(!strcmp(inkey,"b")){
            PrintOut(2,153,9);
            continue; 
        }   // end of if inkey b
        else if(!strcmp(inkey,"g")){
            PrintOut(3,170,10);
            continue; 
        }   // end of if inkey g
        else{
            TermClr(30);
            printf("Input not recognized\n(x=terminate, o=off, r=red, b=blue, g=green)");
            continue;
        }   // end of else 
    }   //end of while
return 0;
}   // end of main

/*******FUNCTIONS*******************/
// function to clear terminal - ClrHeight is the number of new rows, use enough in the function call to clear the terminal
void TermClr(int ClrHeight) {
    int i;
    for ( i = 0; i < ClrHeight; i++ ) {
    putchar('\n');
    }   // end of for loop
}   // end TermClr

// function for changing terminal font colours (run the exe in cmd prompt to see colours, doesn't work in nppexec)
void SetColor(int value){
    SetConsoleTextAttribute(GetStdHandle(STD_OUTPUT_HANDLE),  value);
}   // end SetColor

// function to print the required text to terminal
void PrintOut(int output, int col1, int col2) {     // three inputs needed, the 'output' variable 0-3, the light colour text type, and the writing text type
    TermClr(5);         // calls func to clear teminal, 5 rows
    const char *light[4] = {"OFF", "RED", "BLUE", "GREEN"};     // defines the four light colours
    SetColor(col1);
    int i;
    for (i=0;i<21;i++) {    // for loop to print a load of empty lines with the background colour 'col1'
        cout << "                                             \n";
    }   // end of for loop
    SetColor(7);
    cout << "\nColour - ";
    SetColor(col2);         // calls the function to change the console font colour (shows if run in cmd prompt, but not nppexec console)
    cout << light[output];
    SetColor(7);
    cout << " (Output " << output << ")\n(x=terminate, o=off, r=red, b=blue, g=green)";
}   //end PrintOut

---

编辑添加:
image showing the issues
我试过使用 std::endl 代替\n，这没有区别

进一步编辑:
如果我减少 for 循环中纯色的迭代次数，即

int i;
for (i=0;i<3;i++) {    // for loop to print a load of empty lines
    cout << "                                             \n";
}   // end of for loop

因此每次通过 while 循环打印的总行数较少，在输出溢出之前我得到了很多更改，但在足够多的时间后它仍然会发生。

编辑 3:
评论中的 Mark Ransom 已将 cmd 窗口确定为第一个问题的罪魁祸首。增加缓冲区高度允许在出现不良行为之前有更多的程序周期。 (ALT+SPACE、属性、布局选项卡)
James Whyte 的回答解决了多个键盘输入的问题，尽管我确实需要做一些工作以便忽略错误的输入而不是单独解释每个字符

编辑 4:
如果我使用 system("CLS") 而不是添加新行来清除屏幕，则缓冲区大小不再重要(猜测每次调用时它都会被清除)。我知道这是不好的做法，但是见鬼，它确实有效!任何人都有一个不错的替代方案吗？

Answer 1

[编辑]!- 至少这是解决了两个字符永远循环的问题。

我认为出现问题是因为使用了 char 数组而不是单个 char。我将您的代码纳入了一个新项目并毫无问题地复制了您的问题(笑话)，并决定做一些修复程序来消除错误。

当您试图将行放入 char 数组时，该数组中已经有一个未清除的值，就会出现问题。它只是决定越过它，以永无止境的循环情况结束。

/*******MAIN PROGRAM****************/
int main() {

int output = 0; // variable for current colour being output, 0-3 for OFF, RED, BLUE, GREEN
char inkey;  // console input written to the INKEY variable
PrintOut(0, 119, 7);  // calls printout function, dark grey background for the light, normal grey for text

while (1) {   //while loops forever until break
    cin >> inkey;   //waits for input, stored to INKEY
    if (inkey == 'x') {     // string compare whether INKEY matches "x"
        cout << "\nProgram Terminated in\n3...";
        Sleep(1000);
        cout << "\n2...";
        Sleep(1000);
        cout << "\n1...";
        Sleep(1000);
        break;  //breaks if 'x' is input
    }   // end if inkey x
    else if (inkey == 'o') {
        PrintOut(0, 119, 7);  // calls PrintOut function, 'output' set to 0, red background for the light, red for text
        continue;
    }   // end of if inkey o
    else if (inkey == 'r') {
        PrintOut(1, 204, 12);
        continue;
    }   // end of if inkey r
    else if (inkey == 'b') {
        PrintOut(2, 153, 9);
        continue;
    }   // end of if inkey b
    else if (inkey == 'g') {
        PrintOut(3, 170, 10);
        continue;
    }   // end of if inkey g
    else {
        TermClr(30);
        printf("Input not recognized\n(x=terminate, o=off, r=red, b=blue, g=green)");
        continue;
    }   // end of else 
}   //end of while
return 0;
}   // end of main

最后，如果您只需要在任何时候读入单个字符，请不要使用字符数组和字符串比较。像上面那样比较单个字符，或者更好的是，使用类似于以下内容的 switch case 来提高可读性。

switch (inkey)
{
case 'r': PrintOut(1, 204, 12); continue; // Print red square.
case 'g': PrintOut(2, 153, 9); continue; // Print green square.
case 'b': PrintOut(3, 170, 10); continue; // Print blue square.
case 'o': PrintOut(0, 119, 7); continue; //Print a grey block for inactive.
case 'x':
    cout << "\nProgram Terminated in\n3...";
    Sleep(1000);
    cout << "\n2...";
    Sleep(1000);
    cout << "\n1...";
    Sleep(1000);
    break;  //breaks if 'x' is input
default: 
    TermClr(30);
    printf("Input not recognized\n(x=terminate, o=off, r=red, b=blue, g=green)");
    continue;
} // end switch