AGILE IN ACTION

We recently held a code review for a system we'll be working with intensely over the coming months. Like car parking spaces, desk moves, checkout queues and methodologies, code reviews are one of those things that tend to make normal, sane, rational people turn feral. It was important that the right tone be set so that everyone involved could engage constructively and with respect. In order to create a common understanding of what we were trying to achieve we kicked off with a discussion about what 'well-factored' code really meant, attempting to eliminate some of the subjectivity that typically surrounds this topic (checkout some of the discussion on the c2 wiki!). I threw together this mind map to give us something to use as a starting point: