Shorthand IF doesn't work. Why?

Question

I'm want to shorten the following if statement:

 function is_string_length_correct( $string, $min, $max = 0 ) {
    if (is_string($string)) {
        $l = mb_strlen($string);
        if ($max != 0) {
            return ($l >= $min && $l <= $max);
        } else {
            return ($l >= $min);
        }
    }
  }

Can someone explain me why the followin shorthand if doesn't work?

function is_string_length_correct( $string, $min, $max = 0 ) {
    if (is_string($string)) {
        $l = mb_strlen($string);
        return ($max != 0) ? ($l >= $min && $l <= $max) : ($l >= $min);
    }
 }

Thank you

Answer 1

The syntax is fine. The problem likely lies with the value of $max at runtime. 0 is falsy, so you are comparing it to the integer zero, it should be ($max !== 0) .

Answer 2

您也可以只使用布尔运算符来编写它：

return $l >= $min && ($l <= $max || $max == 0);