Well… you could do that, but there’s a number of reasons why not to. Firstly, by using a snippet like the one above in the template.php, you are providing a solution to all your node types. Secondly, in the example above, you’re not using standard Drupal functions such as l() and t(), which you really ought to for reasons of coding standards, availability for translation and server security.

Just so you know…