Нормально ли, что в методе определены функции?

Question

[LittleFatNinja]

увидел такую структуру
даже не знаю, это очень плохо и так никогда не делать? или в некоторых случаях допустимо?

public static function foo($str) {
  //код
  $arr = ['bar1' 'bar2' 'ba3'];

  if ( func_exist($arr['bar1']) .... )  {
     function bar1() {
        //код
     }

     //код

     function bar2() {
        //код
     }

     //код
     function bar3() {
        //код
     }
   }

  preg_raplace_callback( ...., $str, 'bar1');
}

Answer 1

[trevoga_su]

нет, не нормально, это жуткий говнокод

функции some нужно/можно было просто сделать приватными

ощущение, что человек "боялся" внедрять новые классы, для этого он типа инкапсулировал функции в одном методе

короче не пиши так никогда

Comments

[LittleFatNinja]

спасибо, возьму на заметку

Answer 2

[Виталий Инчин ☢]

Если мне не изменяет память, то так делать вообще нельзя - при втором вызове, PHP станет плеваться ошибками про повторное определение существующей функции.

Но, всегда можно использовать замыкания:

public static function() {
  //код

 $funct = function() {
     //код
  }
}

Comments

[LittleFatNinja]

я такое нашел в коде
такую ахинею я бы не стал писать, но думаю может это писал человек, который понимал что делает, просто я не слишком опытный

[Сергей Протько]

LittleFatNinja: если там именованные функции - то писал очень плохой человек.

[Сергей Протько]

LittleFatNinja: либо там есть проверка что бы этот метод не вызывался дважды.

[LittleFatNinja]

Сергей Протько: да, именованые
обновил код в вопросе

[LittleFatNinja]

Сергей Протько: да, есть проверка
массив с именами функций
и все 3 фенкции в ифе проверяются function_exists

[LittleFatNinja]

но ВСЯ эта кухня какая то очень стремная, не так ли?

[Сергей Протько]

LittleFatNinja: более чем. Скорее всего автор таким образом попытался реализовать автозагрузку функций. Вместо этого лучше в composer.json прописать что бы в любом случае подключался файлик с функциями. Это при включенном opcache ничего не стоит, и не надо извращений.

[Виталий Инчин ☢]

LittleFatNinja: ну тогда это уже другое дело. Но все же, лучше так не делать....смысла не имеет, если только при первом вызове не формируются какое-то чудо данные, на которые функция в дальнейшем опирается.

[LittleFatNinja]

Виталий Инчин: хм... интересное замечание
есть что то похожее на это
обновил код

[LittleFatNinja]

в функцию прихордит строка, по регулярке что то заменяется и вызывается колбек функция

[LittleFatNinja]

по доке функции preg_raplace_callback()
"Вызываемая callback-функция, которой будет передан массив совпавших элементов из строки subject"

[Алексей Уколов]

Судя по всему, гражданин, который это писал, просто не умеет формировать callable внутри метода класса. Говнокод 80 левела.