Вопросы по оформлению кода...
Есть такой метод:
/**
* Activate module from ZIP archive
* @param $archiveFilePath
* @return bool
*/
public function activate($archiveFilePath)
{
if( file_exists($archiveFilePath) ) {
$moduleDirectory = $this->extractArchive($archiveFilePath);
if( $moduleDirectory ) {
$moduleConfig = $this->getConfig($moduleDirectory);
if( $moduleConfig ) {
} else {
\Yii::$app->session->setFlash(self::ERRORS_KEY, self::ERROR_INSTALL_CONFIG_NOT_EXIST);
return false;
}
} else {
\Yii::$app->session->setFlash(self::ERRORS_KEY, self::ERROR_ARCHIVE_EXTRACT);
return false;
}
} else {
\Yii::$app->session->setFlash(self::ERRORS_KEY, self::ERROR_ARCHIVE_NOT_EXIST);
return false;
}
}
Видим множество if
. Мне постоянно необходимо проверять значения, которые возвращают другие методы класса и выводить ошибки, если что-то не так.
Вопрос 1. Код получается ужасный (это я только начал, там много таких проверок должно еще быть). Как в таких случаях поступать правильно?
Вопрос 2. Например, у меня есть проверка if( $moduleDirectory )
. В $moduleDirectory
может быть string
или false
. Стоит ли так писать? Или лучше делать так: if( $moduleDirectory !== false )
?