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

    +159

    1. 1
    2. 2
    3. 3
    4. 4
    foreach($db_top_list as $key=>$info) {
    // много кода...
    $top_list[$key] = $info;
}

    Зачем плодить переменные?

    Запостил: scorpil, 18 Августа 2010

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

    • ava sectus 18.08.2010 12:48 # 0

      Хрен его знает, может для того, чтобы получить на выходе один нетронутый массив и один производный от него массив?
      Ответить

    • ava pasha 18.08.2010 13:27 # +3

      // много кода... - это что?
      ......
      $key = function_key($key);
      $info = function_info($info);
      ......
      Ответить

    • ava paranoid 18.08.2010 13:52 # 0

      $db_top_list[$key] - было бы странно, а здесь может быть вполне нормальная ситуация. Зависит от того, что не показано
      Ответить

    • ava scorpil 18.08.2010 14:18 # −2

      Уже разобрался, там в куче кода сначала идет использование переменных только в качестве аргументов, но есть небольшая функция их изменяющая. Так что жаль что здесь посты не удаляются. :(
      Ответить

      • ava sectus 19.08.2010 04:17 # 0

        Предположение моё оказалось верным, но тем не менее плюсов за этот код накинули.
        Ответить

    • ava Lure Of Chaos 18.08.2010 17:50 # 0

      вообще, предполагается, что $key и $info только для чтения. может не говнокод, но плохой стиль
      Ответить

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