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

    +144

    1. 1
    $orig_id = isset($_POST["orig_id"]) ? (int) $_POST["orig_id"] : (int) $_GET["orig_id"];

    https://github.com/uisky/notabenoid/blob/6edfee9f33b8466be9164cd3ba6be90b8f3a5770/protected/controllers/MyCommentsController.php#L123

    Запостил: Stallman, 14 Мая 2015

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

    • ava crastinus 14.05.2015 12:27 # 0

      README.md больше радует. Где notabenoid.org, а где notabenoid.com.
      notabenoid.org вполне себе жив.
      Ответить

    • ava inkanus-gray 14.05.2015 12:57 # 0

      Если не передали параметр ни постом, ни гетом, то вываливаем бедному юзеру сообщение, что ключ "orig_id" отсутствует в массиве $_GET? Годно.

      Кстати, а Yii не умеет разбирать принятые значения (как это делает ZF)?
      Ответить

      • ava Fike 14.05.2015 16:13 # 0

        с zf не сидел, но минимально умеет, public function actionPetooh($id, array $filter) поймет. Другое дело, что якак-то в эту ситсему пытался залезть, и там ПИЗДЕЦ
        Ответить

      • ava Vasiliy 14.05.2015 16:37 # 0

        умеет. и в экшены передает
        Ответить

      • ava guest 16.05.2015 17:36 # +1

        > Если не передали параметр ни постом, ни гетом, то вываливаем бедному юзеру сообщение, что ключ "orig_id" отсутствует в массиве $_GET?

        То... отключаем E_NOTICE.
        Ответить

        • ava inkanus-gray 16.05.2015 19:12 # 0

          И ещё навалить собачек... Чтобы наверняка.
          Ответить

          • ava Stallman 16.05.2015 19:23 # +2

            Да вообще ставим error_reporting = 0, а то мало ли, что там ещё где вылезет.
            Ответить

    • ava Fike 14.05.2015 16:12 # 0

      пошто на уйского гоните
      Ответить

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