Какие есть PHP Refactoring Tools или Ваши идеи по доработке существующих или созданию?

Question

[PO6OT]

Приблизительно работать должно хотя-бы так:

<?php

 function ping($host, $action=false){
  $http=@fsockopen($host, 80, $errno, $errstr, 1);
  if($http){
   fwrite($http, "GET /".$action." HTTP/1.0\r\nAccept: */*\r\nHost: ".$host."\r\n\r\n");
   fclose($http);
   return true;
  }else{
   return false;
  }
 }

?>

преобразовывается в

<?php function ping($host, $action=false) { return ($http=@fsockopen($host, 80, $errno, $errstr, 1)) ? fwrite($http, "GET /".$action." HTTP/1.0\r\nAccept: */*\r\nHost: ".$host."\r\n\r\n").fclose($http) : false; }

Чтоб хоть чуть понимал говнокод и использовал фичи системных функций для замены кастомных.
Может у Вас есть свои идеи оптимального рефакторинга? Подскажите - буду рад их воплотить в жизнь в виде онлайн-инструмента, если не найду готового.

Рефакторер желательно чтоб работал на Windows или онлайн.

Comments

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

Вам нужен обфускатор?

[PO6OT]

Сергей Протько: кажется обфускатор это рефакторер с диким инопланетным кодом на выходе, непонятным образом продолжающим работать так-же как и оригинал. Нет, мне нужен рефакторинг с остатком смысла в коде.

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

PO6OT: нет.

[PO6OT]

Сергей Протько: разве обфускатор не делает код нечитаемым?

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

PO6OT: ну так в вашем примере вы худо бедно читабельный код превратили в нечитабельное мессиво. Это обфускация. А рефакторинг - это когда мы берем код и делаем его лучше.

[PO6OT]

Сергей Протько: у каждого своё понимание прекрасного..

Answer 1

[Максим]

То что приведено в вопросе это никакой не рефакторинг, и не обфускация. А скорее говнокодинг. Такую оптимизацию Вам поможет произвести парочка пьяных индусов за десяток баксов.

Теперь более сурьезно. То что Вы делаете - это мягко говоря плохо. То что в коде стало на 2 пробела меньше и условие сменилось на другое не сделает Ваш код легче, быстрее, круче. То что было сделанно - это убийство читаемости и поддержки кода. Если есть желание оптимизировать код, то воспользуйтесь "акселераторами" и оставьте свой код в покое, чтобы потом можно было спокойно поправить.

Answer 2

[entermix]

Что вы подразумеваете под словом "Рефакторинг" ?

Рефа́кторинг (англ. refactoring), или реорганизация кода — процесс изменения внутренней структуры программы, не затрагивающий её внешнего поведения и имеющий целью облегчить понимание её работы

https://ru.wikipedia.org/wiki/%D0%A0%D0%B5%D1%84%D...

Чем Ваш код на выходе стал понятнее? Тем, что Вы добавили тернарный оператор? Но зачем Вы поместили все в одну строку?

У Вас на выходе скорее обфускация

Comments

[PO6OT]

Полная переработка кода с оптимизацией, сокращением излишних структур без изменения сути кода ну и форматирование на дорожку, хотя для последнего есть отдельные инструменты.

[PO6OT]

Не надо тыкать цитатами из вики, представляя меня найнизшим нубом. Я вполне понимаю что спрашиваю

[PO6OT]

Никакая это не обфускация. Тот код который в примере на выходе - вполне читаемый.

[entermix]

PO6OT: пример кода с вопроса (в одну строку), который Вы хотите получить в результате не является кодом, над которым провели удачный рефакторинг.

[PO6OT]

Рефакторинг в моём примере имеет целю сокращение кода, увеличение производительности (хоть и на доли миллисекунд), сохраняя более-менее читабельность, не приводя к полному непониманию.

[PO6OT]

entermix: Да, я кажется не правильно назвал. Мне нужен оптимизатор.

[PO6OT]

entermix: Писать оптимизированный код я умею, но есть старый код, который лень переписывать, вот думаю, может закинуть в машинку, само всё произойдёт.

[PO6OT]

entermix: ну авторефакторер хотя-бы базового говнокода, вроде замены кучи условий на один норм. свитч или удаление объявления одноразовых переменных, впихивая одно выражение внутрь другого. что-то такое должно быть. надоело переписывать 3х летний завалявшийся код

[entermix]

PO6OT: такого не встречал, для приведения к стандартам можно использовать PHP Coding Standards Fixer

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

Код, в первую очередь, должен быть понятным.

Если у Вас много кода и все эти усилия с оптимизацией только ради производительности - включите акселератор.

[Максим]

entermix: блин, не глянул обновление, и продублировал Ваш ответ. Даже ссылку поставил такую же.

[PO6OT]

entermix: спасибо, вам +