返回 (\r) 中断 cout

Question

std::cout 在我的 keyPressed 字符串中打印额外的字符，可能是因为“\r”，例如，如果 keyPressed = “右箭头”，当我按下向上箭头时，它会打印“keyPressed = Up arrowoww”，然后，当我再次按右箭头时，它会再次正常打印“keyPressed = 右箭头”，但如果我按“右箭头”以外的任何箭头键，它会在最后打印一些不需要的额外字符

Error example

源码：

游戏.cpp

#include "engine.h"
#include <iomanip>

Engine eng;

int main() {
    while (eng.isRunning) {
        eng.getInput();
        std::cout << std::setw(5);
        std::cout << "\r X = " << eng.playerX;
        std::cout << "| Y = " << eng.playerY;
        std::cout << "| KEY = " << eng.keyPressed;
        Sleep(100);
    }
    return 0;
}

engine.h

#ifndef ENGINE_H
#define ENGINE_H

#include <iostream>
#include <Windows.h>
#include <string>

class Engine {
public:
    // Game
    bool isRunning = true;
    bool gettingInput = true;

    // Player
    int playerX = 1;
    int playerY = 1;
    char playerModel = 'P';

    // Test / Debug
    std::string keyPressed;

    // Functions
    char getInput() {
        // Gets arrow keys states
        while (this->gettingInput) {
            this->keyPressed = "";
            if (GetAsyncKeyState(VK_RIGHT)) {
                // Right arrow key
                this->playerX++;
                this->keyPressed = "Right arrow";
                break;
            }
            else if (GetAsyncKeyState(VK_LEFT)) {
                // Left arrow key
                this->playerX--;
                this->keyPressed = "Left arrow";
                break;
            }
            else if (GetAsyncKeyState(VK_UP)) {
                // Up arrow key
                this->playerY++;
                this->keyPressed = "Up arrow";
                break;
            }
            else if (GetAsyncKeyState(VK_DOWN)) {
                // Down arrow key
                this->playerY--;
                this->keyPressed = "Down arrow";
                break;
            }
            else if (GetAsyncKeyState(VK_END)) {
                exit(0);
            }
            Sleep(255);
        }
    }
};

#endif

解决此问题的最佳/最简单方法？ 我搜索并测试了3天，但没有找到任何东西，请帮助我。

Answer 1

由于您要覆盖先前的输出，因此当您打印较短的字符串时，仍会显示先前输出中的多余字符。将 \r 替换为 \n 以查看实际输出的内容。

你可以在你的键名后面输出一些空格来用空格覆盖那些多余的字符并删除它们。

Answer 2

查看您提供的代码后，我确实发现了一些与代码设计有关的问题或顾虑：我将对其进行分解并解释一些我认为可以提高代码质量的事情。我将从您的 main.cpp 开始，然后转到您的 Engine 类。

你最初有这个：

#include "engine.h"
#include <iomanip>

Engine eng;

int main() {
    while (eng.isRunning) {
        eng.getInput();
        std::cout << std::setw(5);
        std::cout << "\r X = " << eng.playerX;
        std::cout << "| Y = " << eng.playerY;
        std::cout << "| KEY = " << eng.keyPressed;
        Sleep(100);
    }
    return 0;
}

我看到的第一个主要问题是您在全局级别声明了Engine eng。我们可以解决这个问题

#include "engine.h"
#include <iostream>
#include <iomanip>

int main() {
    Engine eng; // declare it here as the first object in main; now it has local
                // scope within main's function and is now in Automatic Storage 
                // instead of Global Storage.
    while( ... ) {
        // ....
    }
    return 0;
};

---

下一个问题从 main 函数中的 while 循环的条件表达式开始。 您目前拥有：

while( engine.isRunning ) { //... }

这没关系，但这更多是您的Engine class's 设计的问题。在这里，您提供了一个任何人都可以访问的public member。所以让我们看看你的类声明/定义；你目前有：

#ifndef ENGINE_H
#define ENGINE_H

#include <iostream>
#include <Windows.h>
#include <string>

class Engine {
public:
    // Game
    bool isRunning = true;
    bool gettingInput = true;

    // Player
    int playerX = 1;
    int playerY = 1;
    char playerModel = 'P';

    // Test / Debug
    std::string keyPressed;

    // Functions
    char getInput() { // ... }
};

#endif

在这里，您应该保护您的数据成员并对它们具有访问修饰符功能：

#ifndef ENGINE_H
#define ENGINE_H

#include <iostream>
#include <Windows.h>
#include <string>

class Engine {
private:
    bool isRunning;
    bool gettingInput;

    // Player
    int playerX;
    int playerY;
    char playerModel;

    // Test / Debug
    std::string keyPressed;

 public:
    Engine() : 
      isRunning( false ),
      isGettingInput( false ),
      playerX( 1 ),
      playerY( 1 ),
      playerModel( 'P' ) 
    {}

    void run() { isRunning = true; // set or call other things here... }

    // Since we protected our members variables by making them private,
    // we now need some access functions to retrieve and modify them.
    bool isActive() const { return isRunning; } // make this const so it doesn't change anything
    void toggleIsActive() { isRunning = !isRunning; }

    bool retrievingInput() const { return isGettingInput; }
    void toggleRetrievingInput() { isGettingInput = !isGettingInput; } 

    int getPlayerX() const { return playerX; }
    void setPlayerX( int newX ) { playerX = newX; }

    int getPlayerY() const { return playerY; }
    void setPlayerY( int newY ) { playerY = newY; }

    // set both in one function call
    void setPlayerPosition( int newX, int newY ) {
        playerX = newX;
        playerY = newY;
    }

    char getPlayerModel() const { return playerModel; }
    // don't know if you want to change this: uncomment if you do
    // void setPlayerModel( char c ) { playerModel = c; }

    std::string& getPressedKey() const { return keyPressed; }

    char getInput() { // ... }
};    

这应该会修复你的类的界面设计。这里唯一的主要区别是我默认将Boolean 成员变量设置为false，因为通常当您第一次启动Engine 时，它当前尚未运行。所以为了解决这个问题，我们可以调用一个公共运行函数来触发它。所以 main 看起来像这样：

int main () {
    Engine eng;
    eng.run(); // this now starts the engine sets the flag to true

    while (...) { //... }

    return 0;
}

---

不过，我在你的Engine's getInput() 函数中也看到了一些问题，所以让我们来看看吧。

char getInput() {
    // Gets arrow keys states
    while (this->gettingInput) {
        this->keyPressed = "";
        if (GetAsyncKeyState(VK_RIGHT)) {
            // Right arrow key
            this->playerX++;
            this->keyPressed = "Right arrow";
            break;
        }
        else if (GetAsyncKeyState(VK_LEFT)) {
            // Left arrow key
            this->playerX--;
            this->keyPressed = "Left arrow";
            break;
        }
        else if (GetAsyncKeyState(VK_UP)) {
            // Up arrow key
            this->playerY++;
            this->keyPressed = "Up arrow";
            break;
        }
        else if (GetAsyncKeyState(VK_DOWN)) {
            // Down arrow key
            this->playerY--;
            this->keyPressed = "Down arrow";
            break;
        }
        else if (GetAsyncKeyState(VK_END)) {
            exit(0);
        }
        Sleep(255);
    }
}

第一部分是while loop's 条件语句和您的班级成员。最初，默认情况下您将此设置为true，但我在代码中的任何位置都没有看到此值正在更新。我们不需要更改它，但修复很简单，因为我们可以通过公共接口调用更改此成员。因为我默认你的isGettingInputfalse；您现在可以在进入 while 循环之前在此函数中设置它。我看到的唯一最后一个问题是，当在main's while 循环中回调此函数时；此函数从不返回值，并且从不使用返回值。

---

至于您对cout 用户的错误的实际问题：1201programalarm 几乎已经为您回答了这个问题。只是想我会用你的代码帮助你更多。