Как убрать непонятные подчеркивания?

Question

[Neopitniy]

В классе прописан метод вывода исключений

class Test{
  /** @throws Exception */
  private function error($txt){
    throw new Exception($txt);
  }

  public function method1(){
    $this->error('текст ошибки');
  }
}

Посоветуйте, как убрать:
1. Подчеркивание аргумента $txt в скобках и подчеркивание $this->error();
2. Нормально ли вообще делать внутренний метод, оболочку? Или правильней в методе, во время ошибки, сразу писать throw new Exception('текст ошибки');

Comments

[RidgeA]

ну подчеркивается же не просто так, там что-то пишется
варианта 2
- исправить проблему
- отключить анализатор (вообще/правило)

[Neopitniy]

RidgeA,
Вызов метода error во всех методах класса — "unhandled exception"
Аргумент $txt — "argument phpdoc missing"

Не сколько парят эти уведомления, сколько сам факт их наличия. Весь код класса, я написал выше, удалил всё лишнее. Код работает. Ошибки только видит phpstorm.

[Lander]

Neopitniy,

При вызове метода вы не обрабатываете ошибку, бросаемую этим методом. Анализатор вам об этом и сообщает.
Ну и опишите параметр $txt в нотации, раз так бесят подчёркивания.

/**
 * @param string $txt
 * @throws Exception
 */

[Neopitniy]

Lander, что значит я не обрабатываю? Можете написать, в чем там именно причина?

[Lander]

Neopitniy, когда вы пишите throw new Exception(); это вы выбрасываете исключение. То есть формируете сообщение о какой то нештатной ситуации. Если же вы его выбрасываете, то вполне логично вы как то готовы его обработать и сказать приложению что делать в случае если такая ситуация произошла. Так вот вы этого НЕ делаете. :) Подробнее написано тут https://www.php.net/manual/ru/language.exceptions.php.

[Neopitniy]

Lander,
ну, там где я использую класс, я ловлю это исключение.

try{
   $a = new Test();
 }catch(Exception $e){
   echo $e->getMessage();
}

но ведь это уже другой файл. Класс отдельно ведь.

[Neopitniy]

class Test{

    /**
     * @param string $e
     * @throws Exception
     */
    public function error($e){
        throw new Exception($e);
    }

    public function run(){
        $this->error('fatal-error');
    }
}

try{
    $a = new Test();
    $a->run();
}catch(Exception $e){
    echo $e->getMessage();
}

Вот полный код скрипта. Ругается на вызов
$this-><b>error</b>('fatal-error');
Unhandled exception

Answer 1

[profesor08]

Подчеркивает потому что твой метод, как указано в нотации, принимает ноль аргументов. А ты что-то там пихаешь ему.

Comments

[Neopitniy]

Да все правильно я пихаю!!) Где там нужна пустота?

[profesor08]

Neopitniy,
/** @throws Exception */
Где тут указан принимаемый функцией аргумент? Его нету.

При этом ты вызываешь ее с аргументом
$this->error('текст ошибки');

[Neopitniy]

profesor08, то есть это ошибка лишь самого phpstorm? фактически её не существует даже?