clean code?

[php_b34st]

Is there a shorter way of doing this:

 

        if($position == '1') { $style = 'GK'; }
if($position == '2') { $style = 'DEF'; }
if($position == '4') { $style = 'MID'; }
if($position == '6') { $style = 'STR'; }

[mbeals]

<?php

switch($positoin){

   case 1: $style = 'GK';
                break;
   case 2: $style = 'DEF';
                break;
   case 4: $style = 'MID';
                break;
   case 6: $style = 'STR';
                break;
}
?>

[proggR]

That was going to be my suggestion too

Can you tell me though, with the case what if you have a multi line block of code to run in each case do you just put it in parenthesis with the break inside it? I can't remember and haven't done it in a while (but I'm going to switch most of my site over the the switch, case, default method of comparisons soon).

[papaface]

That was going to be my suggestion too

Can you tell me though, with the case what if you have a multi line block of code to run in each case do you just put it in parenthesis with the break inside it? I can't remember and haven't done it in a while (but I'm going to switch most of my site over the the switch, case, default method of comparisons soon).

Would be:

<?php

switch($positoin){

   case 1: $style = 'A lot
of code';
   echo 'A lot of code';
   break;

   case 1: $style = 'A lot
of code';
   echo 'A lot of code';
   break;

   case 1: $style = 'A lot
of code';
   echo 'A lot of code';
   break;

   case 1: $style = 'A lot
of code';
   echo 'A lot of code';
   break;
}
?>

No { } needed

[proggR]

Okay thanks. I haven't used switch in years and couldn't remember.

[php_b34st]

Thanks I will use that instead