他の関数を「動的に」呼び出す再利用可能な PHP 関数の作成を支援します。
-
20-09-2019 - |
質問
MySQL のリレーショナル データベースと、CodeIgniter を使用して PHP で作成されたある種の CMS を使用して、本格的に何かを作成しようとしているのはこれが初めてです。
いくつかの多対多の関連テーブルにデータを挿入する必要がある部分に来ました。
私のコードではすべてが正常に機能します (まあ、ほんの数分のテストで) が、テーブル内のすべてのリレーションを作成するのに非常に役立つ再利用可能な関数を作成するのに少し助けが必要です...
コード内のすべてをコメント化しようとしたので、すべての説明がそこにあります...
<?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.
}
}
}
?>
さて、このコードを見ると、その書式設定部分をその関数にも入れることができることがわかりますが、それは現時点ではそれほど重要ではありません...
この件に関してご協力いただき、誠にありがとうございました!!
解決
これを行うための推奨される方法は、 テーブルデータゲートウェイ. 。の代わりに
$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
}
さらに、使用することをお勧めします トランザクション この種の作業では、すべてのものが一緒に属しており、クエリの 1 つが失敗した場合にはデータを挿入したくない可能性が高いためです。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()
これを行うこともできます)、上で提案したように行うと、コードがはるかに保守しやすくなります。
あなたの質問は、すべての関数を順番に実行したいのか、それともユーザーが送信した内容に応じて単に交換したいのか、少し曖昧です。最初のケースでは、トランザクション アプローチを使用します。2 番目のケースでは、適切なクラスをインスタンス化し、save メソッドを呼び出すだけです。
他のヒント
あなたの要件の完全わからないが、私はあなたがcall_user_func
後かもしれないと思う。
function process($countries) {
// do stuff
}
$function_name = 'process';
call_user_func($function_name, $countries);
この方法で、あなたは動的に国のリスト、たとえば、に基づいて機能を割り当てることができます。