PHP Translating a ENUM in switch case not working

Question

        echo $state;  //debug
        switch($state){
            case "Sleeping":
                $label = "Is Sleeping";
            case "Running":
                $label = "Is Running";
            default: 
                $label = "Could not retrieve state";
        }

$state is filled from an SQL database (Enum type), the echo message prints "Sleeping", but the $label is filled with the default value

You need to include break sentences in each case. The execution is "falling" from the first case until the default. https://www.php.net/manual/en/control-structures.switch.php You've got a detailed explanation of what is happening in your code in the link. — RubioRic, Jun 04 '21 at 07:45

XMehdi01 · Answer 1 · 2021-06-04T08:46:36.633

8

In your case choose match (PHP 8) expression is significantly shorter and it doesn't require a break statement.

$state = "Running";
$label = match ($state) {
    "Sleeping"=> "Is Sleeping",
    "Running" => "Is Running",
    default => "Could not retrieve state"
};
echo $label; // "Is Running"

edited Jun 04 '21 at 08:46

answered Jun 04 '21 at 07:57

XMehdi01

5,538
2
10
34

1

With a side note that this is a PHP 8 feature. – El_Vanja Jun 04 '21 at 08:23
Yes, I added a reference to it! – XMehdi01 Jun 04 '21 at 08:47
Is there a way to run two commands on a successful match? – FontFamily Jun 27 '23 at 17:31

score 4 · Accepted Answer · answered Jun 04 '21 at 07:47

PHP switch statements continue executing the remainder of the switch block after finding a match, until it finds a break; statement. You'll want to add break; to the end of each case:

        echo $state;  //debug
        switch($state){
            case "Sleeping":
                $label = "Is Sleeping";
                break;
            case "Running":
                $label = "Is Running";
                break;
            default: 
                $label = "Could not retrieve state";
        }

score 4 · Answer 3 · answered Jun 04 '21 at 18:58

4

For completeness:

$states = [
    'Sleeping' => 'Is Sleeping',
    'Running' => 'Is Running',
];
echo $states[$state] ?? 'Could not retrieve state';

answered Jun 04 '21 at 18:58

Álvaro González

142,137
41
261
360

PHP Translating a ENUM in switch case not working

3 Answers3