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

    +159

    1. 1
    if (!is_array($array)) {$array = array($array)}

    Запостил: partizan22, 14 Апреля 2013

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

    • ava WGH 14.04.2013 16:43 # +4

      Надо было писать while.
      Ответить

    • ava Serabas 14.04.2013 19:58 # +2

      а я вот в js как то делал такое. нужно было передавать именно массив из элементов, даже если элемент один единственный (jQUery работает по тому же принципу). Придумалась такая проверка, если аргумент не массив, тогда что нам мешает обернуть его в массив и передать как ни в чем не бывало?
      Ответить

    • ava bot-minurast 14.04.2013 22:45 # 0

      Ну да, ну демоническая типизация. Пердача аргументов, хуле.
      function foo ( $array=array() ) {
      if (!is_array($array)) $array = (array)$array;
      // ...
      }
      Ответить

      • ava deep 15.04.2013 00:01 # 0 

        function foo( $arg=array() ) {
echo $arg[0];
}

$x = (object) array("a"=>1, "b"=>2);
foo($x); // Fatal error:  Cannot use object of type stdClass as array

        Хуле...
        Ответить

    • ava DrFreez 16.04.2013 03:48 # 0

      Всегда так делаю :3 В чём говнокод то?
      Ответить

      • ava Sh1tM4ker 16.04.2013 15:59 # 0

        Проверка не нужна:
        $array = (array)$array;
        Ответить

        • ava scriptin 16.04.2013 20:26 # +1

          Этот код не эквивалентен.
          $ php -r '$a = new StdClass(); $a->p = 123; var_export((array) $a);'
array (
  'p' => 123,
)
          Ответить

          • ava Sh1tM4ker 16.04.2013 21:13 # 0

            Не знал. Тогда в чём говнокод? Кроме отсутствующей точки с запятой, что в общем-то просто опечатка.
            Ответить

            • ava scriptin 16.04.2013 23:16 # +2

              Лично мне забавно видеть большое количество упоминаний слова "array" в таком маленьком отрывке кода. Ну и название $array для переменной не очень удачно выбрано. В целом больше ничего плохого не вижу.
              Ответить

    • ava guest 10.05.2013 03:12 # 0

      Все проще.
      Теперь можно функцию вызывать так:
      foo(['bar', 'baz']) или foo(['bar']) или так foo('bar')
      Ответить

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