Говнокод

"PHP" / Говнокод #19676

public function getWinnerEmail($id)
{
        if (!is_numeric($id)) {
            throw new BadRequestHttpException('Ошибка:' . __FUNCTION__ . ' ' . __LINE___);
        }

        //$project = Projects::findOne(['id' => $id]);
        $project = (new \yii\db\Query())
            ->select(['id', 'win_user_id'])
            ->from(self::tableName())
            ->where(['id' => $id])
            ->one();
//        if ($project['win_user_id'] == null) {
//            return false;
//        }

        $result = User::findOne(['id' => $project['win_user_id']]);
        if ($result != null) {
            return ['id' => $result->id, 'email' => $result->email];
        }
}

Метод модели "Project"
PHP-Framework yii2

Запостил:

NotFake, 22.03.2016 (Updated 29.03.2018)

Комментарии (6) RSS

KEKC 10 лет ago # 0

Yii - это вообще говно полное. Для того, чтобы вывести </form> надо вызывать $widget->close() или что-то такое. ЗАЧЕМ?!
Ответить
- Vasiliy 10 лет ago # 0 ⇈
  
  потому что виджеты могут быть вложенными.
  Ответить
  - KEKC 10 лет ago # 0 ⇈
    
    Ну и что? Поставил </form> вместо длиннющей строчки и пошёл дальше закрывать виджеты. Никакой причины использовать для этого функцию нет.
    Ответить
    - Vasiliy 10 лет ago # 0 ⇈
      
      Это в простом случае /form если включена защита от csrf то будет еще сгенерирован токиен и добавлен инпут. Или вы это тоже хотите руками делать?
      
      А если сначала защита отключена а потом её включили предлагаете обходить все формы?
      Ответить
      - KEKC 10 лет ago # 0 ⇈
        
        Да, предлагаю нормально писать программу - обходить все формы, если надо, вместо того, чтобы нагромождать всякие вызовы методов экземпляров классов.
        Ответить
M-A-X 10 лет ago # 0

Это типичный код на PHP-фреймворках.
Ничего странного. 🙂
Ответить

Добавить комментарий Отменить ответ

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

А не использовать ли нам bbcode?