<?xml version="1.0"?>
<rss version="2.0">
  <channel>
    <title>PHPDeveloper.org</title>
    <link>http://www.phpdeveloper.org</link>
    <description>Up-to-the Minute PHP News, views and community</description>
    <language>en-us</language>
    <pubDate>Mon, 20 May 2013 23:17:47 -0500</pubDate>
    <ttl>30</ttl>
    <item>
      <title><![CDATA[PHPMaster.com: The Importance of Code Review]]></title>
      <guid>http://www.phpdeveloper.org/news/18431</guid>
      <link>http://www.phpdeveloper.org/news/18431</link>
      <description><![CDATA[<p>
PHPMaster.com has a new article about a practice that's becoming more popular in recent years to help increase the quality of code that comes out of development - <a href="http://phpmaster.com/the-importance-of-code-review/">code reviews</a>.
</p>
<blockquote>
Every developer knows the pain of banal mistakes. A wrong attribute here, a misspelled property there, an accidentally duplicated line of code which you missed because of the coffee-fueled 16 hour hackathon you've been on. [...] Code review is simply the act of having someone else look at your code to find the mistakes you missed.
</blockquote>
<p>
The tutorial talks about the types of code reviews (three of them with varying levels of involvement) as well as some best practices to follow in your reviews like: 
</p>
<ul>
<li>Know your common mistakes and actively fight them.
<li>Peer code review means being reviewed by someone of equal or greater skill.
<li>Collect metrics.
<li>Be mindful of the social aspect - finding bugs is good, not bad!
</ul>]]></description>
      <pubDate>Fri, 31 Aug 2012 12:08:50 -0500</pubDate>
    </item>
    <item>
      <title><![CDATA[Chris Roane's Blog: PHP Programming: The Benefits of Peer Pressure]]></title>
      <guid>http://www.phpdeveloper.org/news/14314</guid>
      <link>http://www.phpdeveloper.org/news/14314</link>
      <description><![CDATA[<p>
In the latest post to his blog <i>Chris Roane</i> looks at something that, while not generally considered a good thing, can help to make your development and work better - <a href="http://www.montanaprogrammer.com/web-project-management/peer-pressure/">peer pressure</a>.
</p>
<blockquote>
When you are held responsible to other people and when you are in an environment that has competition, that not only helps the company as a whole, but it allows for more individual growth. In the typical corporate atmosphere, accountability and competition can be lost because no one really knows anyone else. But the other side of this, in extra small companies, you can have too much accountability...where every move is constantly watched.
</blockquote>
<p>
He notes that, without the pressure of peers - ones that you really know and that really know you and your work - the quality of the applications written degrades in most situations. Sure, there'll be the one or two developers that are extra motivated, but without the real interaction with other skilled developers, innovation can quickly fade.
</p>]]></description>
      <pubDate>Wed, 07 Apr 2010 13:12:04 -0500</pubDate>
    </item>
    <item>
      <title><![CDATA[Brandon Savage's Blog: Peer Review: You Have Not Because You Ask Not (Requests & Responses)]]></title>
      <guid>http://www.phpdeveloper.org/news/13255</guid>
      <link>http://www.phpdeveloper.org/news/13255</link>
      <description><![CDATA[<p>
<i>Brandon Savage</i>'s <a href="http://www.brandonsavage.net/peer-review-you-have-not-because-you-ask-not-requests-responses/">latest post</a> in his "Peer Review" series has been added to his blog today. This time he focuses on the requests and responses - modifying the sample code to make it more testable on how things are quested and the responses they give back.
</p>
<blockquote>
There is one last area that I want to address, and this has everything to do with object-oriented principles and code reusability. For those who are familiar with OO programming, they realize that the use of classes does not make something object oriented by nature. In this final part of the series, we'll move one step closer to being object-oriented, by introducing the concepts of request and response objects.
</blockquote>
<p>
He changes up the Twitter object to take in a HTTPRequest object (instead of creating one by itself) and to use exceptions and a "Twitter response object" to encapsulate any responses from the service making it much easier to work with and understand than just a true/false return.
</p>]]></description>
      <pubDate>Mon, 21 Sep 2009 10:07:46 -0500</pubDate>
    </item>
    <item>
      <title><![CDATA[Brandon Savage's Blog: Peer Review: Testable Code And Architecture]]></title>
      <guid>http://www.phpdeveloper.org/news/13221</guid>
      <link>http://www.phpdeveloper.org/news/13221</link>
      <description><![CDATA[<p>
In this <a href="http://www.brandonsavage.net/peer-review-testable-code-and-architecture/">latest post</a> in his "Peer Review" series <i>Brandon Savage</i> looks at things that can make the sample code he's been working with easier to test.
</p>
<blockquote>
Now that we've worked out the abstraction issues and the logic questions, we should take a moment to focus our attention on a few of the issues relating to the architecture and testability of the class we've worked out.
</blockquote>
<p>
He mentions changes to a call to <a href="http://php.net/get_included_files">get_included_files</a>, a bit of code requiring magic quotes, and changes to the code to make writing using tests simpler (like injecting objects rather than creating them and writing a unit test every time a bug is found so it doesn't happen again).
</p>]]></description>
      <pubDate>Tue, 15 Sep 2009 07:50:55 -0500</pubDate>
    </item>
    <item>
      <title><![CDATA[Brandon Savage's Blog: Peer Review: Improving The Business Logic]]></title>
      <guid>http://www.phpdeveloper.org/news/13182</guid>
      <link>http://www.phpdeveloper.org/news/13182</link>
      <description><![CDATA[<p>
<i>Brandon Savage</i> has posted the <a href="http://www.brandonsavage.net/peer-review-improving-the-business-logic/">fifth part</a> of his "Peer Review" series where he's taken a sample application and worked it over - refactoring, updated to meeting coding standards and abstracting out interfaces to simplify the code. In this latest article he looks at a method to improve the business logic behind the scenes.
</p>
<blockquote>
So far, we've done quite a bit of work on our Twitter class, making it better. There's still work to be done, though, especially improving the logic. The Twitter class we have now has a number of logical flaws in it that we need to address. Additionally, there are some logical flaws that we started with that I want to highlight, even though we've already fixed them. Let's get started with those.
</blockquote>
<p>
He looks at a few things like setting the host name as a property, using the return of the HTTP request and handling exceptions (like when Twitter's down). The resulting code is included.
</p>]]></description>
      <pubDate>Tue, 08 Sep 2009 10:04:07 -0500</pubDate>
    </item>
  </channel>
</rss>
