在CodeIgniter中的不同文件之间重用函数的正确方法是什么？

Question

I'm new to CodeIgniter and I have this function which I use in more than one model.

// Returns TRUE if the array of values supplied
// each correspond to a column in the database table.
// Else returns the value of the bad column.
private function _columns_exist( $columns, $table ) {

    foreach( $columns as $key => $value ) {

        if( !$this->db->field_exists( $key, $table ) ) {

            return $key;

        }

    }

    return TRUE;

}

What is considered the correct way to write the function once, and reuse it from many different files?

Do I want to create a whole library just for this one function?

Answer 1

If you are using a common function across models, you can extend the core model in the first place and get all your other models to extend that. Read more on extending core classes here: http://ellislab.com/codeigniter/user-guide/general/core_classes.html

class MY_Model extends CI_Model {

  public function __construct()
  {
      parent::__construct();
  }

  public function common_func()
  {
    // content
  }
}

then you can use this function in all other models

class Some_model extends My_Model {

  public function __construct()
  {
      parent::__construct();
  }

  public function some_func()
  {
    // use the common function here
    $result = $this->common_func();
  }
}

Answer 2

The point of CodeIgniter is that there are basically two types of custom functions. Ones that live in Libraries as library methods and ones that live in Helpers as simple functions.

The difference between them is minimal and it is for you to decide where to keep them, but the best practice is that helper functions are mostly standalone and library methods are the ones that use CI libraries like database, sessions, your other models.

For your function it's best to make a library and keep it there, but you can, as well, make a helper as you're not overusing any database querying. If it was me, I would make a helper:

Make a file in the helpers directory named db_helper.php and add an element db to helper array in autoload.php file. Note that you don't need to write the full name db_helper , only db .

Put this code in db_helper.php file you've created:

<?php
if(!function_exists('_columns_exist')) {
    function _columns_exist( $columns, $table ) {
        $CI =& get_instance();
        foreach( $columns as $key => $value ) {
            if( !$CI->db->field_exists( $key, $table ) ) {
                return $key;
            }
        }
        return TRUE;
    }
}

Now you can use this function anywhere - in your controllers, models, even view files as you would if it was a simple function:

_columns_exist($columns, $table)

Note that you don't use $this in helpers or libraries, you must get an instance of CodeIgniter singleton object first, then you manipulate it. You can find more info on this in the user guide or on StackOverflow.

Also see my comments to darkdad 's answer, that is another great approach I would go for.