c++ 警告：枚举值未在开关 [-Wswitch] 中处理

Question

我正在尝试在没有警告的情况下编译以下代码：

    while (window.pollEvent(event))
    {
        switch (event.type) {
            case sf::Event::Closed:
                window.close(); break;
            case sf::Event::KeyPressed:
                if(event.key.code == sf::Keyboard::Escape )
                    window.close();
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::Space ) )
                    particleSystem.fuel( 200/* * window.getFrameTime() */);
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::A ) )
                    particleSystem.setPosition( --xpos, ypos );
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::D ) )
                    particleSystem.setPosition( ++xpos, ypos );
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::W ) )
                    particleSystem.setPosition( xpos, --ypos );
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::S ) )
                    particleSystem.setPosition( xpos, ++ypos );
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::Left ) )
                    particleSystem.setGravity( --xgrv * 0.1f, ygrv * 0.1f);
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::Right ) )
                    particleSystem.setGravity( ++xgrv * 0.1f, ygrv * 0.1f );
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::Up ) )
                    particleSystem.setGravity( xgrv * 0.1f, --ygrv * 0.1f );
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::Down ) )
                    particleSystem.setGravity( xgrv * 0.1f, ++ygrv * 0.1f );
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::G ) )
                    particleSystem.setGravity( 0.0f, 0.0f );
                if( sf::Keyboard::isKeyPressed( sf::Keyboard::P ) )
                    particleSystem.setPosition( 320.0f, 240.0f );
                break;
    }

但是，我收到了很多警告：

/home/bluszcz/private/repo/deerportal/game.cpp:444: warning: enumeration value 'LostFocus' not handled in switch [-Wswitch]

这对我来说不是问题，因为我不需要处理所有类型的事件。

添加

default:
    break;

我的代码删除了警告，但这是解决此问题的最佳方法吗？

Answer 1

明确

这取决于您要实现的目标。管理规则是

It is better to be explicit.

省略大小写只会让您看起来好像忘记了一些。明确地向您的代码的后续读者保证您打算对某些事件不做任何事情。

鉴于此，您有两个选择：

选项 1 - 添加默认值

default:
  break;

这会抑制警告，并明确表示您不打算在此处处理其他事件类型。

选项 2 - 列出每个值

列出每个事件类型，后跟 break。这也是明确的，并且有额外的好处，如果您添加事件类型，编译器将再次警告您您的 switch 不完整。当您有许多 switch 语句时，这可能很有价值，其中一些需要修改以在添加枚举值时执行新操作。

一系列 if 语句呢？

我不建议在这里使用一系列 if 语句。 switch 更清晰，减少了输入量，并且（如您所见）可以为您省略的情况生成更好的编译器警告。

c++ 警告：枚举值未在开关 [-Wswitch] 中处理

c++ warning: enumeration value not handled in switch [-Wswitch]

c++

warnings

compiler-warnings

switch-statement

明确

选项 1 - 添加默认值

选项 2 - 列出每个值

一系列 if 语句呢？