Почему некорректно работает класс?

Question

[SVaa]

class Company
{
  public $company;

  public function __construct($company = null)
  {
    if (preg_match("/[a-zA-Z\s]+/",$company)){
      $this->conmapy = $company;}
  }

  public function checkCompany(){
    if($this->company === null){
      echo 'Имя компании введено не коректо';
    }
  }
}
$obj = new Company($_POST['company']);
$obj->checkCompany();


class Agre
{
  public $agre;

  public function __construct($ager = null)
  {
    if (!empty($agre)){
      $this->agre = $agre;}
  }

    public function checkAgre(){
      if($this->agre === null){
        echo 'Не все поля заполены';
      }
    }
  }

При вводе любых значений в инпуты класа company name выдает ошибку.
Так же класс agre подразумевает проверку наличия галочки для ползовательского соглашения но даже когда она стоит выдает ошибку. Подскажите в чем ошибка ?

Comments

[Дмитрий]

А то что у вас очипятки в слове компании
$this->conmapy = $company;}

Answer 1

[index0h]

У вас Company::__construct($company = null), $company может быть любой, и строка, и массив, и null, и объект исключения и т.д., вы ожидаете в проверке только строку, это не корректно.
С Agre - то же самое.

echo вы вообще зря используете, если что-то не так - бросайте исключение.

При вводе любых значений в инпуты класа company name выдает ошибку.

debug/var_dump в помощь

Comments

[SVaa]

Но null это единственно возможное значение типа null. Как тогда правильно нужно написать там ? Не могу понять. ООП начал учить недавно по этому еще не доконца разобрался вовосем. Если можно разжевать чучуть пожалуйста

[index0h]

SVaa, public function __construct(string $companyName)

[index0h]

SVaa, рекомендую почитать на досуге https://github.com/index0h/php-conventions

[index0h]

SVaa, в случае проверки названия компании в конструкторе - вам метод checkCompany не нужен вовсе.
Если название компании не корректное (не проходит вашу регулярку) - бросайте исключение прямо в конструкторе.

class Company
{
    /** @var string */
    private $name;

    public function __construct(string $name)
    {
        if (!preg_match("/^[a-z]+[a-z ]+[a-z]+$/i", $name)) {
            throw new \InvalidArgumentException(sprintf('Invalid company name "%s"', $name));
        }

        $this->name = $name;
    }

    public function getName(): string
    {
        return $this->name;
    }
}

[SVaa]

index0h, почему $companyName а не $company
нет не помогло

[index0h]

SVaa, потому, что $company семантически - это объект с какими-то свойствами. $company вполне может быть ее ЕГРПОУ, либо тип ООО/ИП/.... $companyName - это конкретное свойство компании.

нет не помогло

Проверяйте данные на входе. По идее в исключении должно быть описано, что не так.

[index0h]

SVaa, что бы type hint заработал необходимо добавить declare(strict_types = 1);