我的编程逻辑在这里正确吗?

const char IsPressed = 1; // 1
const char WasHeldDown = 2; // 10
const char IsFirstPress = 4; // 100

char* keystates[256];

Class::CalculateKeyStates()
{
    for(int i = 0; i < 256; ++i)
    {
        if(this->IsDown(i))
        {
            keystates[i] |= IsPressed; // turn on
            if(keystates[i] & WasHeldDown)
            {
                //keystates[i] |= IsFirstPress;
                keystates[i] &= ~IsFirstPress; // turn off
            }
            else
            {
                keystates[i] |= WasHeldDown + IsFirstPress; // Turn on
            }
        }
        else
        {
            keystates[i] = 0; // Turn ALL off
        }
    }
}

此函数将是类、类的成员函数。另一个成员函数is down将在相关键关闭时返回true,否则返回false。

你能看到进一步改善这个功能的方法吗?

谢谢

编辑:

我会详细解释一下为什么要这样做。这是对一段代码的修改,该代码通过一个数组keystates(由三个bools组成的结构)工作,对所有键的设置都显示为false。然后再次将ispressed设置为this的值->isdown,然后第三次循环检查是否已持有密钥,如果已持有,则不再是第一次按,因此将其设置为false。如果未按下,则将第一次按下设置为“真”,并保持为“真”,因此下次将标记为“已按下”。

编辑2:

在代码中添加了一些注释并更正了一行