Talk in Polish at PHPCon Poland 2016
Short URL: https://joind.in/talk/cb1e2
(QR-Code (opens in new window))
Refactoring hacked
Comments
Comments are closed.
Zostalo pokazanych bardzo malo technik refaktoryzacyjnych. Prezentacja w stylu: rafactoring jest i fajnie go mozna uzywac.
Odnioslem podobne wrazenie jak @Piotr Olaszewski. Duzo mowienia o zaletach refaktoringu i nic poza tym. Dla mnie to byl zbior oczywistych oczywistosci znanych juz nawet dla poczatkujacych programistow.
Two stars for sharing the idea of boy scouts rule and explaining that everyone can refactor on daily basis.
I find it very hard to give negative feedback, but unfortunately that was the only advantage of this lecture.
Your main sins:
1. You refactored just one example too deep, instead of selling us some tricks that are ready to use on different situations, like Fowler's book presents
2. Sadly, you made the code less readable and transparent, you broke some good practices
3. What worried me the most, there were errors and mistakes in your code. Examples: a) missing $developer declaration on second snippet. b) calling "new array()" on another one (array is not an object). c) using archaic syntax (array() instead of []) d) bad formatting
4. Some slides were very hard to read, and you also skipped some of them
I was really eager to go to your lecture, but sadly I came out disappointed.
Ma?y refaktoring ani g??bokie nurkowanie.
Refactoring != Redesign
Where's the "hacked" part? I felt like You were talking about how to name functions properly, nothing more.
I didn't like the part when You refactored "setState(SomeClass::APPLIEAD)" (a setter introduces an intention) to "applied()" (a method which name suggest that will return a bool, same like isApplied()). I would rather name the function "apply()" to still indicate an intention.
Bardzo przyjemnie si? s?ucha?o, dobry sposób prezentacji, sposób mówienia. Sam temat nie by? mo?e super porywaj?cy, ale uwa?am, ?e by?o to dobrze zrobione przypomnienie zasad refaktoryzacji. Plus za to, ?e powiedzia?e?, ?e trzeba refaktoryzowa? na co dzie?, przy ka?dej okazji.