0

次のスニペットがあります。

protected function sendEmail($email)
{
    extract($email);

    $this->transmail->locale($locale)
                    ->timezone($timezone)
                    ->template($template)
                    ->subject($subject)
                    ->send($header, $params);
}

このコードは完全に機能します (完全なソース コードはこちら)。ただし、外出先でいくつかの優れた慣行に従うことを確認したいと思います。私は現在 [いくつかの CodeClimate 警告] (PHPMD) を取得しています ( https://codeclimate.com/github/timegridio/timegrid/app/Listeners/SendBookingNotification.php ):

  • 「$locale」などの未使用のローカル変数は避けてください。
  • 「$timezone」などの未使用のローカル変数は避けてください。
  • 「$template」などの未使用のローカル変数は避けてください。
  • 「$subject」などの未使用のローカル変数は避けてください。
  • 「$header」などの未使用のローカル変数は避けてください。
  • 「$params」などの未使用のローカル変数は避けてください。

それを行うためのエレガントな方法はどれですか?

list()またはそのようなもので変数を明示的に宣言する必要がありますか?

前もって感謝します

4

1 に答える 1