[eluser]boltsabre[/eluser]
Souri84,
Your code looks fine (at a casual glance), what you've done is pretty much the standard way of doing this kind of thing, and don't worry about the length of it - try doing all this (ie, your model calls, controller 'controlling' and view displaying) in the old fashioned procedural way and your code would be much much much bigger that what you have now!
Ummm, but why are you inserting anything before your doctype in your view file? this is a big no no... if you need a function, create a helper, store your function in there, load your helper in your controller, and call it in your view where you need it (but NOT BEFORE your doctype) - this way you can use your function over and over in any view)
ie: your function is called 'makeMySelectMenu($var)', so in your view call as such (just make sure you've loaded your helper file in your controller!)
</head>
<body>
<h1>Page Title</h1>
<?php makeMySelectMenu($var); ?>