this function only adds numbers and I can not figure why?

Question

this function only adds numbers and I can not figure why ?

function calculate($num1 , $num2 , $name = "+" )
{
 
  switch ($name)
  {
    case "+" || "add" || "a";
    return $num1 + $num2 ."<br>";
    break;

    case "-" || "subtract" || "s";
    return $num1 - $num2 ."<br>";
    break;

    case "*" || "multiply" || "m";
    return $num1 * $num2 ."<br>";
    break;

    default:
    "";
  }
     
}

I also tried if statement and didnt work.

`switch` doesn’t use `||` like an `if` does. There’s multiple ways to do it but the most common is probably multiple pass through `case` items: https://3v4l.org/CkCqQ — Chris Haas, Dec 27 '22 at 04:16

score 0 · Accepted Answer · answered Jan 04 '23 at 09:02

Your syntax is not correct, do as follows:

switch ($name) {
        case "+" :
        case "add" :
        case "a":
            return $num1 + $num2 . "<br>";

        case "-" :
        case "subtract" :
        case "s":
            return $num1 - $num2 . "<br>";

        case "*" :
        case "multiply" :
        case "m":
            return $num1 * $num2 . "<br>";

        default:
            "";
    }

By the way, you don't need to break if you return something.

More information :

Switch case multiple values

Switch Documentation

this function only adds numbers and I can not figure why?

1 Answers1