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

    +166

    1. 1
    2. 2
    3. 3
    function array_min_recursive($arr, &$peaks = array(), $current_key = '') {
  return array_max_recursive($arr, $peaks, $current_key, true); // Неожиданно, правда?
}

    Запостил: DIX315, 05 Июля 2014

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

    • ava guest 05.07.2014 12:24 # 0

      Когда написали array_max_recursive, выяснилось, что он делает противоположное задуманному. Но не пропадать же добру!
      Ответить

    • ava Elvenfighter 05.07.2014 14:21 # +4

      А может array_max_recursive -- функция из параллельной вселенной, где всё наоборот? А здесь мы имеем просто феноменальный прорыв: разработано API для вызова такой функции
      Ответить

      • ava kegdan 05.07.2014 14:36 # +5

        Из параллельного мозга, который в жопе
        Ответить

    • ava bormand 05.07.2014 15:24 # +5 

      function array_max_recursive($arr, &$peaks = array(), $current_key = '') {
  return array_min_recursive($arr, $peaks, $current_key, true);
}
      Ответить

    • ava Oliver_air 07.07.2014 09:21 # 0

      Есть шанс что последним параметром передается по какому принципу делать выборку.
      Для того чтобы не копипастить кусок кода в котором нужно заменить один параметр.
      Нит?
      Ответить

      • ava bormand 07.07.2014 09:38 # +1

        Ну тогда стоило бы написать array_minmax_recursive и две обертки для него. А то как-то через жопу, что min может искать максимум.

        DRY это, конечно, хорошо... но только если он не противоречит здравому смыслу.
        Ответить

        • ava Soul_re@ver 07.07.2014 16:52 # +5

          > А то как-то через жопу, что min может искать максимум.

          А в крестах, если передать std::min_element() std::greater в качестве компаратора, то получится std::max_element()
          Ответить

        • ava Oliver_air 15.07.2014 09:57 # 0

          Рефакторинг, незнакомое слово для 95% php погромистов.
          Ответить

          • ava bormand 15.07.2014 10:02 # +1

            "Некогда рефакторить, надо код писать."
            Ответить

            • ava kegdan 15.07.2014 10:16 # +2

              Не время сейчас думать! Вот сдадим проект - тогда все подумаем!
              Ответить

      • ava bormand 07.07.2014 10:50 # 0

        > Для того чтобы не копипастить кусок кода в котором нужно заменить один параметр.
        А если передать последним аргументом 42, то посчитает среднее значение? Ну не писать же лишний цикл.
        Ответить

        • ava kegdan 07.07.2014 13:15 # 0

          А если дать ему баян и лсд...
          Ответить

        • ava wvxvw 07.07.2014 13:58 # +3

          > А если передать последним аргументом 42
          nth order statistic, очевидно же.
          Ответить

        • ava DIX315 07.07.2014 22:28 # +1

          Да, последний флаг инвертирует режим.
          И где-то видел, как передаётся именно 42 в качестве флага, функция что-то рискованное с БД выполняла при передаче этого параметра.
          Давно правда, сейчас уже не вспомню где, поиск по своим исходникам результата не дал.
          Ответить

    • ava russling 09.07.2014 16:16 # 0 

      function array_max_recursive($arr, $peaks, $current_key, $delai_naoborot = false)
      Ответить

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