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

    +159

    1. 1
    $this->last_error = $success;

    кстати о названиях переменных :))

    Запостил: youROCK, 22 Февраля 2011

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

    • ava Sulik78 22.02.2011 18:47 # −1

      хде ГК???
      Ответить

    • ava Kirinyale 22.02.2011 18:54 # +4

      В виндовом апи тоже есть константа ERROR_SUCCESS. И чё?
      Ответить

      • ava youROCK 22.02.2011 19:07 # 0

        И что эта константа означает?
        Ответить

        • ava Kirinyale 22.02.2011 19:08 # 0

          "операция завершена успешно"
          Ответить

          • ava youROCK 22.02.2011 19:23 # −2

            Ну как бы вот и ответ на ваш вопрос :). Это не ошибка, какого хрена называть переменную, в которую записана ошибка, $success :)?
            Ответить

            • ava Kirinyale 22.02.2011 19:35 # 0

              Без полного контекста вполне можно предположить, что в переменной $success лежит код ошибки, означающий отсутствие ошибки. Правда, в этом случае логичнее было бы сделать её константой.
              Ответить

            • ava Sulik78 22.02.2011 19:38 # 0

              хахаха
              ну вы и шутник... оО
              наверное потому что success содержит либо ОК либо ERROR и пишет это в last_error?
              Ответить

              • ava youROCK 22.02.2011 19:40 # 0

                На самом деле, туда записывается либо 1 либо строка с ошибкой :)
                Ответить

            • ava Sulik78 23.02.2011 06:16 # +2

              > Это не ошибка, какого хрена называть переменную, в которую записана ошибка, $success
              а как надо то ?
              $this->last_error = $error;
              $this->last_success = $success;
              так ?
              Ответить

    • ava Lure Of Chaos 22.02.2011 21:44 # +1

      сам подход - говно
      Ответить

    • ava sectus 23.02.2011 05:24 # 0

      Может быть success заменить на result и не будет противоречия?
      Ответить

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