Tutorial switch help?[SOLVED]

[Sanction]

I'm sure this is a no brainer for some..

I'm getting a syntax error with: case sf::Event::KeyPressed: 'A';
Error says: case label value has already appeared in the switch

Is there a way to change the switch around to have multiple cases?
Thanks in advance!

CODE:

(click to show/hide)

while (window.pollEvent(event))
                {
                        //Event List
                        switch (event.type)
                        {
                                // window closed
                        case sf::Event::Closed:
                                window.close();
                                break;

                                // key pressed
                        case sf::Event::KeyPressed: 'W';
                                std::cout << "Key: W Pressed." << std::endl;
                                X++;
                                break;

                        case sf::Event::KeyPressed: 'A';
                                std::cout << "Key: W Pressed." << std::endl;
                                X++;
                                break;

                                // we don't process other types of events
                        default:
                                break;
                        }
                }
 

[G.]

Your switch has 2 times the same case...:
case sf::Event::KeyPressed:

Your 'A' and 'W' don't mean anything and are useless.

[Sanction]

Yeah I get that..  I'm trying to find a solution and I just got a brain block when it comes to this switch statement. I can work it witch a if statement but its ugly

[Jesper Juhl]

switch (event.type) {
  ...
  case sf::Event::KeyPressed:
    switch (event.key.code) {
      ...
      case sf::Keyboard::Escape:
        // Escape was pressed
        break;
      ...
    }
    break;
  ...
}
 

[Hapax]

If an event has the type sf::Event::KeyPressed, it means that a key has been pressed; it could be any key!
To find out which key was pressed, you then need to check the event's field called key and its field called code, as shown in Jesper's rather clear example.

For more information on events, please read the tutorial about events 


EDIT: To address the specific question,

I'm getting a syntax error with: case sf::Event::KeyPressed: 'A';

This is incorrect.
'A'; is a separate statement (which does nothing) and is not evaluated as part of the case, as if you'd written:

case sf::Event::KeyPressed:
    'A';

This should help you visualise why it is saying you used the same case twice.

[Sanction]

Thank you guys! now I need to work in smoother movement.

CODE:

(click to show/hide)

Code: [Select]

while (window.pollEvent(event))
{
//Event List
switch (event.type)
{
// window closed
case sf::Event::Closed:
window.close();
break;

// key pressed
case sf::Event::KeyPressed:
cout << "Key was Pressed" << endl;
// what key
switch (event.key.code)
{
case sf::Keyboard::W:
cout << "W" << endl;
Y--;
break;
case sf::Keyboard::A:
cout << "A" << endl;
X--;
break;
case sf::Keyboard::S:
cout << "S" << endl;
Y++;
break;
case sf::Keyboard::D:
cout << "D" << endl;
X++;
break;
}

break;
}
}


[Hapax]

Thank you guys!

You're welcome.

now I need to work in smoother movement.

Good luck!