Peter is the former President of the New Zealand Open Source Society. He is currently working on Business Workflow Automation, and is the core maintainer for Gravity Workflow a GPL workflow engine.
Hi Simon, congratulations on completing the task as written.
Take another look at the code with the eyes of a code reviewer. Ask yourself how you might improve or refactor the code base while introducing the new functionality. If you do make improvements note down what the changes are and how they make the code better, but keep them to yourself for now.
Next week when I publish my solution I plan to have a discussion about the different approaches people took.
Hi Simon, congratulations on completing the task as written.
Take another look at the code with the eyes of a code reviewer. Ask yourself how you might improve or refactor the code base while introducing the new functionality. If you do make improvements note down what the changes are and how they make the code better, but keep them to yourself for now.
Next week when I publish my solution I plan to have a discussion about the different approaches people took.
Thank you Peter.
I did see quite a few changes that could be made.
I've removed the link from my post, until other people have had a go.