Помогите с созданием повторно используемой функции PHP, которая будет «динамически» вызывать другие функции

Question

Это мой первый раз, когда я пытаюсь сделать что-то серьезно с помощью реляционных БД в MySQL и какой-то CMS, созданной в PHP с CodeIgniter.
Я пришел к той части, где мне нужно вставить некоторые данные в несколько связанных между собой таблиц.
В моем коде все работает нормально (ну, всего за несколько минут тестирования), но мне нужна помощь в создании функции многократного использования, которая очень поможет мне установить все отношения в моих таблицах ...

Я попытался прокомментировать все в своем коде, поэтому все объяснения там есть ...

<?php
function add(){

 // This is what user posted in form.
 // There are two input fields:
 // name is always only one record
 // country can be a single record or array separated with " | " characters
 // I use CodeIgniter's $this->input->post instead of $_POST[]
 $name = $this->input->post('name');
 $countries = $this->input->post('country');

 // Inserting data to first table
 $data = array('firstName' => htmlentities($name)); // preparing array for inserting
 $insert_name = $this->db->insert('names', $data); // inserting with CodeIgniter's help
 $last_inserted_ID = $this->db->insert_id(); // getting last inserted ID

 // Inserting data to second table

 // Formatting of posted string of countries
 // Users can post strings similar to this:
 // "Austria"
 // "Austria |"
 // "Austria | "
 // "Austria | Australia"
 // "Austria | Australia |"
 // "Austria | Australia | "
 // and similar variations
 // What I need here is clear array with country names
 $separator = strpos($countries,"|"); // check for "|" character
 if ($separator === FALSE){ // if there is no "|" character in string
  $countries_array[] = $countries; // array is only one value (only one country)
 } else {
  $countries_array = explode(" | ", $countries); // explode my array
  if (end($countries_array) == ""){ // if last item in array is ""
   array_pop($countries_array); // eliminate last (empty) item
  }
 }

 // Now, this is the part I think I will use lots of times.
 // I would like to make this a separate function so I could use it in many places :)
 // I would pass to that function few values and I would use one of them
 // to call different functions in this same class.
 // I guess I should pass data ($countries_array) and function names I wish to call?????? This is problematic part for my brain :))
 // Check the comments below...
 for ($i = 0; $i < sizeof($countries_array); $i++){
  $insertIDS = array(); // this will be an array of IDs of all countries
  $tempdata = $this->get_countries($countries_array[$i]); // query which looks if there is a country with specific name
                // Right here, instead of calling $this->get_countries
                // I would like to call different functions, for example
                // $this->get_links($links_array[$i])
                // or $this->get_categories($categories_array[$i])
                // etc.
  if(sizeof($tempdata) != 0){ // so, if a record already exists
   foreach ($tempdata as $k => $v){
    $insertIDS[] = $k; // insert those IDs in our array
   }
  } else { // and if a record does not exist in db
   $this->add_country($countries_array[$i]); // add it as a new record...
               // This is also one of the places where I would call different functions
               // for example $this->add_link($links_array[$i])
               // or $this->add_categories($categories_array[$i])
               // etc.
   $insertIDS[] = $this->db->insert_id(); // ...get its ID and add it to array
  }

  // Finally, insert all IDs into junction table!
  foreach ($insertIDS as $idKey => $idValue){
   $this->add_names_countries($last_inserted_ID, $idValue); // Another place for calling different functions
                  // example $this->add_names_links($last_inserted_ID, $idValue)
                  // etc.
  }
 }

}
?>

Что ж, глядя на этот код сейчас, я вижу, что могу добавить эту часть форматирования и в эту функцию, но сейчас это не так важно ...

Большое спасибо за любую помощь в этом !!

Answer 1

Предпочтительный способ сделать это - использовать Шлюз табличных данных . Вместо

$this->db->insert('countries', $data);

вы создаете классы для каждой таблицы в вашей базе данных. Каждая таблица инкапсулирует логику CRUD в класс, например,

class Countries
{
    $protected $_db;

    public function __construct($db)
    {
        $this->_db = $db;
    }

    public function save(array $countries)
    {
        $this->db->insert('countries', $countries);
    }

    // ... other methods
}

Кроме того, я предлагаю использовать транзакций для такого рода работы, потому что все эти вещи принадлежат друг другу, и вы, вероятно, не захотите вставлять какие-либо данные, если один из запросов завершится неудачно. Я не знаю, как CodeIgnitor обрабатывает транзакции, но в основном вы должны сделать это следующим образом:

$this->db->startTransaction();          // like try/catch for databases
$countries = new Countries($this->db);
$countries->save($countryData);
$links = new Links($this->db);
$links->save($linkData);
// ...
if($this->db->commit() === false) {     // returns true when no errors occured
    $this->db->rollback();              // undos in case something went wrong
}

Хотя это не отвечает на ваш вопрос о том, как динамически вызывать функцию (call_user_func() может сделать это), выполнение этого, как предложено выше, делает ваш код гораздо более понятным.

Ваш вопрос немного расплывчат в том, хотите ли вы запускать все функции в последовательности или просто хотите обмениваться в зависимости от того, что отправил пользователь. Для первого случая используйте транзакционный подход. Во втором случае вы просто создадите экземпляр соответствующего класса и вызовете метод save.

Answer 2

Не совсем уверен в ваших требованиях, но я думаю, что вы могли бы после call_user_func:

function process($countries) {
// do stuff
}

$function_name = 'process';

call_user_func($function_name, $countries);

Таким образом, вы можете динамически назначать функцию на основе, скажем, списка стран.