LM\AuthAbstractor\Validator\PasswordValidator

Complexity

Total Complexity

4

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	hasSpecialChars()	0	11	4

<?php

declare(strict_types=1);

namespace LM\AuthAbstractor\Validator;

/**
 * This class contains methods to validate passwords.
 */
class PasswordValidator
{
    /**
     * Validates that a password contains special characters.
     *
     * @param string $password The password we want to validate.
     * @return bool Whether the password is valid or not.
     */
    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 \.

        }
    }
}