1. Куча / Говнокод #11679

    +131

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (http_status_code == 200 || http_status_code == 500) {
  .... //success!!
} else {
  .... // error!!
}

    Было замечено в одном андроид приложении.

    Запостил: mr.The, 30 Августа 2012

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

    • ava eth0 30.08.2012 20:01 # +1

      Ну, в какой-то степени это можно считать успехом.
      Ответить

      • ava 3.14159265 30.08.2012 20:06 # +2

        >http_status_code == 500
        К успеху шёл - не получилось, не фартануло
        Ответить

      • ava roman-kashitsyn 31.08.2012 10:11 # +1 

        if (http_status_code == 500) {
  // LOL: I've killed the server ;D
}
        Ответить

        • ava mr.The 31.08.2012 11:08 # 0

          Самое интересное, что сервер на рельсах и он отдаёт 500 ошибку, когда происходит эксепшн. Причём, текст ошибки отдаётся в коде страницы.
          Ответить

          • ava roman-kashitsyn 31.08.2012 11:10 # +1

            Так многие сервера по умолчанию поступают, тот же Tomcat
            Ответить

          • ava bormand 31.08.2012 12:52 # −2

            > текст ошибки отдаётся в коде страницы
            IIS с ASP.NET тоже этим грешит. Да и в PHP такая фишка включается в конфиге...

            Но на самом деле очень плохая идея в продакшене показывать экцепшены юзеру... Имхо лучше писать в лог, а в ответе выдать банальное "На сервере произошла ошибка. Порядковый номер ошибки #321.".
            Ответить

            • ava mr.The 31.08.2012 13:36 # 0

              Эксепшн выглядит нормально, типа "Validation failed: Username has already been taken", так что с этим всё хорошо. Кроме того, до продакшена, пока что, ещё дошло.
              Ответить

    • ava TarasB 31.08.2012 09:34 # +4

      на ондрюшке это нормально, хуй найдёшь, как правильно делать, нихуя в апи и документации непонятно, и делают всё каждый всё по-своему, на сосунгах некоторые функции не так работают например
      Ответить

      • ava guest 31.08.2012 11:53 # +8

        Хм. Поразительно: запятые вроде есть, но все равно нихрена не понятно.

        >На ондрюшке это нормально. Хуй найдешь, как правильно делать: нихуя в апи и документации непонятно - и каждый делает всё по-своему. На сосунгах некоторые функции не так работают, например.

        Синтаксис - великая сила. Правильный синтаксис - великая сила Добра.
        Ответить

      • ava LispGovno 31.08.2012 11:57 # +5

        Тебе нужно больше языков выучить, чтобы правильно составлять предложения.
        Ответить

    • ava Lure Of Chaos 31.08.2012 16:44 # +2 

      if (http_status_code == 200 || http_status_code == 500) {
  error = "Всё хорошо";
} else {
  error = "Всё хорошо";
}
      Ответить

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