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

    +160.3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    $arr = getArr1();
$arr2 = getArr2();
foreach ($arr2 as $key => $value){
	$arr[$key] = $value;
}

    вот такой вот способ суммировать массивы

    Запостил: paranoid, 26 Октября 2009

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

    • ava guest 26.10.2009 10:45 # 0

      Я тут вижу копирование массива, а не суммирование. Странно, а нельзя было сделать так?
      $arr = $arr2;
      Ответить

      • ava striker 26.10.2009 11:18 # 0

        тут именно суммирование.
        элементы из массива arr2 будут добавлены в arr1, причем, если у них есть общие ключи, то они будут заменены на соответствующие значения arr2.
        Аналог array_merge, в общем
        Ответить

        • ava cheef 26.10.2009 11:26 # +1

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

          • ava guest 02.11.2009 17:19 # 0

            http://ru.php.net/array_merge_recursive (PHP 4 >= 4.0.1, PHP 5)
            ты про это? =))

            профи такие профи =)
            Ответить

        • ava ivmaroz 26.10.2009 11:29 # +1

          array_merge сбрасывает числовые ключи
          Ответить

        • ava paranoid 26.10.2009 12:01 # +1

          array_merge работает не так. Он ещё и цисловые индексы переупорядочивает
          а здесь аналог $arr = $arr + $arr2;
          Ответить

        • ava guest 26.10.2009 12:13 # +1

          array_merge работает не так.
          Ответить

      • ava paranoid 26.10.2009 12:16 # 0

        тут будет $arr = $arr + $arr2;
        Ответить

        • ava ivmaroz 26.10.2009 12:29 # 0

          Только наоборот надо переставить переменные
          $arr = $arr2 + $arr;
          Ответить

          • ava paranoid 26.10.2009 13:36 # 0

            значения из $arr заменяются значениями из $arr2 - так что $arr + $arr2
            Ответить

            • ava ivmaroz 26.10.2009 13:44 # 0

              Все правильно, поэтому $arr2 + $arr
              Т.е. если есть ключ и в первом и во втором слагаемом, то значение не заменяется из второго слагаемого. Если нет, то просто добавляется.
              Ответить

              • ava paranoid 26.10.2009 13:56 # 0

                если ты внимательно посмотришь, то окажется, что в цикле происходит именно замена. а значит $arr + $arr2
                Ответить

                • ava ivmaroz 26.10.2009 14:26 # 0

                  И я про то же самое говорю, именно про замену. Второй аргумент не заменяет значения из первого с одинаковыми ключами. Поэтому надо поменять местами слагаемые. Вот же какой упертый, взял бы и проверил
                  <?

$arr = array(
	'a' => 'arr',
);
$arr2 = array(
	'a' => 'arr2',
);

print_r($arr + $arr2); // Будет arr (и где замена?)
print_r($arr2 + $arr); // Будет arr2 (а вот здесь есть замена)
?>
                  Ответить

                  • ava paranoid 26.10.2009 14:48 # +1

                    мда...
                    Замечание: Общие ключи будут перезаписаны по принципу "первый пришел - первый обработан".
                    и пойми тут что в мануале этим хотели сказать.
                    Ответить

                  • ava interested 26.10.2009 18:47 # 0

                    Вот это да!
                    На протяжении пяти комментариев выясняется, как же нужно написать: так или эдак, чтобы говнокод разрешить...
                    Сила!
                    Ответить

                    • ava ivmaroz 26.10.2009 20:15 # 0

                      А что делать, если не верит, как должно быть правильно :)
                      Ответить

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