1. PHP / Говнокод #12100

    +49

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    if($model->validate(array('code')))
{
	$model->addError('code', 'Введен неверный код купона. Купон с таким кодом уже существует.');
}
else
{
	if($model->save())
	{
		$this->redirect(array('view','id'=>$model->id));
	}
}

    Запостил: sleeper, 12 Ноября 2012

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

    • ava tirinox 12.11.2012 15:24 # +1

      "Извините, Вы не можете использовать указанный пароль. Такой пароль уже использует пользователь Misha. Пожалуйста, придумайте другой пароль."
      Ответить

      • ava bormand 12.11.2012 15:38 # +2

        Но разве с купонами это не вполне закономерно?

        Если Миша уже ввел свой код со своего купона, и получил за него какие-нибудь бонусы, а я взял и ввел такой же номер, то эта ошибка вполне резонна.

        Это же не пароль, знание которого может пригодиться в дальнейшем...
        Ответить

      • ava Fai 12.11.2012 16:14 # +4

        При регистрации вы не можете использовать следующие пароли:

        пароль1 (пользователь1)
        пароль2 (пользователь2)
        ...
        парольN (пользовательN)
        Ответить

    • ava Govnocoder#0xFF 12.11.2012 17:23 # −1

      Coupon.
      Ответить

    • ava sleeper 13.11.2012 09:02 # 0

      И никто не заметил главного: нарушения в логике.
      Ответить

      • ava roman-kashitsyn 13.11.2012 09:08 # 0

        Можно поподробнее?
        Ответить

        • ava bormand 13.11.2012 09:45 # 0

          ! в первой строке пропущено. Если только эта опечатка - то довольно уныло
          Ответить

          • ava Steve_Brown 13.11.2012 10:09 # 0

            Ну, тут еще неизвестно. Может, валидный идентификатор означает "существующий в базе", тогда все ок.
            Ответить

            • ava roman-kashitsyn 13.11.2012 10:29 # +2

              Вот и я о том же подумал. Нельзя методы называть validate(). Ну вернул он флажок, и что он значит? "Да, валидный"/"Нет, ошибок не обнаружено"?
              Гораздо разумнее назвать isValid().

              Не зная семантики этого конкретного validate(), сложно сказать, есть ошибка в логике или нет.
              Ответить

    • ava Razban_Guestov 14.11.2012 09:44 # 0

      Исходя из того, что Я вижу это Yii, значит validate возвращает return !$this->hasErrors();

      Собсно $model->addError() и будет влиять на hasErrors(), но только до валидации.

      Логика сломана только потому, что save() и так врубит валидацию, т.е. должно быть так:

      if( !$model->save() )
      {
      // Делаем чего-нибудь, например логируем
      }
      else
      {
      $this->redirect( 'view', array( 'id'=>$model->id ) );

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

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