PHP Code Review

Stefan Priebsch, Arne Blankerts, Sebastian Bergmann (24.Sep.2009)
Talk at CodeWorks 2009 (Los Angeles) (English - US)

Rating: 3 of 5

In this workshop, three PHP experts with different software engineering focuses (testing, architecture, and security) willperform an interactive code review together with the audience.Using examples from Open Source projects, attendees of thisworkshop learn how experts look at code, what tools they use during code reviews, what good code and bad code looks like,and how to avoid the most common gotchas. They are invited to bring their own code for an anonymous code review for anincreased benefit from the workshop.Each of the three PHP experts involved in this workshop will present an in-depth workshopfor their particular area of expertise in the afternoon that builds upon this one.

Who are you?

Claim talk

By clicking this button you are declaring that you are the speaker responsible for it and a claim request will be sent to the administrator of the event.

If the claim is approved you will be able to edit the information for this talk.

Are you sure?

 
Comments closed.

Comments

Rating: 3 of 5

26.Sep.2009 at 20:54 by Oleg Baranovsky

This was an interesting exercise in reviewing some of the "pearls" in various open source projects. It would be much more interesting and useful, though, if presenters would give a little bit more insight in the train of thought that goes into the code review process and maybe present some examples that can show audience some traits to avoid (and why -- performance, security, reliability etc.) and some other traits to acquire.

Would make it much more fruitful experience, so members of audience can leave the presentation having learned something instead of just looking at what stupid things other people have done. For that we all have enough humiliation reviewing our own code from some time ago :-)

Rating: 3 of 5

27.Sep.2009 at 04:48 by Joe Devon

On the positive side, I enjoyed seeing Sebastian Bergmann's phploc tool. It really showed at a glance the cost as well as the audaciousness of the CakePHP project to try and make up for PHP 4's lack of OOP.

80mg of memory to say Hello World is a lot, and I wouldn't start to use CakePHP until they support php 5.3+, especially after seeing this code review... but bravo to Nate for starting such an interesting project and on the success they've had.

On the downside, it was kind of like having Wayne Gretzky, Roger Federer & Babe Ruth on stage cherry picking to criticize, the worst games that high school players of their sport played...

Federer on how Rafa Nadal can improve = fascinating. But Wayne Gretzky on how a minor leaguer with bad technique sucks, not so much.

As Oleg was pretty much saying, I'd like to have seen this top notch team of coders take mediocre code or even good code and show how they would approach it to make it awesome code.

Rating: 3 of 5

06.Oct.2009 at 14:12 by Keith Casey

Picking apart examples is pretty easy but I agree that looking at the *process* and strategy behind a Code Review would be incredibly powerful. Further, when the trio were reworking group-provided live code, this is an even better presentation.

Cloud server hosting by Combell Combell      © Joind.in 2012