CodeIgniter helpful hints

[eluser]xwero[/eluser]
some hints :

- When you have to use an input value multiple times put it in a variable it saves typing and you only have to make one call to the input->post method.

- Put form validation and other form related actions in a separate library. It keeps your controller cleaner and you see patterns sooner

Code:

// controller
$this->load->library('formactions');
function index()
{
   if(array_key_exists('addbutton',$_POST))
   {
      list($msg,$redirect) = $this->formactions->add();
   }
   if(array_key_exists('updatebutton',$_POST))
   {
      list($msg,$redirect) = $this->formactions->update();
   }
   if(array_key_exists('deletebutton',$_POST))
   {
      list($msg,$redirect) = $this->formactions->delete();
   }
   if($redirect != '')
   {
       redirect($redirect);
   }
   else
   {
       $data['msg'] = $msg;
   }
   $this->load->view('someview',$data);
}
// library
class Formactions
{
   var $ci;
  
   function Formactions()
   {
      $this->ci =& get_instance();
   }
  
   function add()
   {
      $msg = '';
      $redirect = '';
      // validation
      if(!$this->ci->validation->run())
      {
         $msg = $this->ci->validation->errror_string;
      }
      else
      {
         // do something else (for example database action, manipulate image, write to file, ...)
         // redirect if needed
         $redirect = 'controller/success';
      }
      return array($msg,$redirect);
   }
}