LM\AuthAbstractor\Validator\PasswordValidator::hasSpecialChars()

Complexity

Conditions	4
Paths	4

Size

Total Lines	11
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function hasSpecialChars(string $password): bool
    {
        switch (preg_match('/[^a-zA-Z0-9]/', $password)) {
            case 0:
                return false;

            case 1:
                return true;

            case false:

It seems like you are loosely comparing preg_match('/[^a-zA-Z0-9]/', $password) of type integer to the boolean false. If you are specifically checking for 0, consider using something more explicit like === 0 instead.

The function implicitly returns null when this case condition does not match. This is incompatible with the type-hinted return boolean. Consider adding a default case to the switch.

                throw new Exception();

The type LM\AuthAbstractor\Validator\Exception was not found. Did you mean Exception? If so, make sure to prefix the type with \.

        }