Back to Phpstan

booleanAnd.leftNotBoolean

website/errors/booleanAnd.leftNotBoolean.md

2.2.11.2 KB
Original Source

Code example

php
<?php declare(strict_types = 1);

$string = 'hello';
$flag = true;

if ($string && $flag) {
	// ...
}

Why is it reported?

The left side of the && (boolean AND) expression is not a boolean value. PHP will implicitly cast the non-boolean value to bool before evaluating the expression. This implicit type coercion can lead to unexpected behaviour depending on PHP's type juggling rules.

This rule is part of phpstan-strict-rules and enforces that only boolean values are used with the && operator, making the code's intent explicit.

In the example above, $string is of type string, not bool, so using it on the left side of && relies on PHP's loose type coercion.

How to fix it

Use an explicit comparison to produce a boolean value:

diff-php
 <?php declare(strict_types = 1);

 $string = 'hello';
 $flag = true;

-if ($string && $flag) {
+if ($string !== '' && $flag) {
 	// ...
 }

Or convert the value to boolean with a meaningful comparison:

diff-php
 <?php declare(strict_types = 1);

 $string = 'hello';
 $flag = true;

-if ($string && $flag) {
+if (strlen($string) > 0 && $flag) {
 	// ...
 }