How can I get rid of this logic from the view?

[eluser]Aken[/eluser]
If you're going to separate the logic that generates the views into a view_things() method, you should make it private or put an underscore on the front, otherwise it can be called through the browser.

Code:

private function view_things() {}
// OR
function _view_things() {}

Otherwise that's a perfect starting point. Basically, if you find yourself doing logic, like comparisons or checking for empties, in your view, you can do it in your controller instead if you want to keep the code clean. Doing some logic is okay, it can just get really complicated to read if you have a lot. That's where template scripts like Smarty and Twig come in handy. Or, if you're just echoing variables, you can use CI's Parser library instead of doing a ton of opening/closing PHP tags.