Яuбy / Говнокод #19771 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
begin
        @project = Admin::Project.find(params[:project_id])
      rescue ActiveRecord::RecordNotFound => ex
        raise ActiveRecord::RecordNotFound, "project with id = #{params[:project_id]} not found"
      end

      begin
        user = Admin::User.find(params[:user_id])
      rescue ActiveRecord::RecordNotFound => ex
        raise ActiveRecord::RecordNotFound, "user with id = #{params[:user_id]} not found"
      end

Запостил: strax strax, (Updated )

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

  • где говно? бесполезные стандартные сообщения заменяются чем то более информативным и полезным пользователю.
    Ответить
    • Ты вообще знаешь, как выглядят "бесполезные стандартные сообщения" в ActiveRecord?
      Ответить
      • не блин тут не только все рубисты, но еще и спецы по "ActiveRecord" или как оно там называется.
        Ответить
        • Стандартное сообщение об ошибке:
          ActiveRecord::RecordNotFound: Couldn't find User with 'id'=272
          Ответить
            • В версии, которая в проекте (а это ну оооочень старая версия) - делает.
              Ответить
              • наконец-таки нашел отмазку.

                я просто сильно сомневаюсь что какой говнокодер и/или капипастер от хорошей жизни код как сверху будет писать.

                знаю что сам в куче мест подобное в крестах (и жабе) делал что бы не маятся и некоторые исключения напрямую в лог сливать. потому что "array index out of bound"/этц не очень сильно помогает.
                Ответить
                  • В коде без ошибок не должно быть ошибок вообще. Только исключения.
                    Ответить
                    • Никаких исключений: В коде без ошибок не должно быть ошибок вообще.
                      Ответить

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

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

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


    8