'Dinamik' diğer işlevleri arayacak bir yeniden kullanılabilir PHP oluşturma işlevi Yardım

2 Cevap php


This is my first time I am trying to make something seriously with relational db in MySQL and some kind of CMS created in PHP with CodeIgniter.
I came to the part where I have to insert some data in few many-to-many related tables.
In my code everything works fine (well, with just few minutes of testing), but I need some help with creating reusable function which will help me a lot to make all relations in my tables...

Benim kod şeyi Yorumlamak için çalıştı, böylece tüm açıklamalar var vardır ...

<?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.
  }
 }

}
?>

Peki, şimdi bu kodu bakıyor, ben bu işlevi de bu biçimlendirme parçası koymak olabilir görmek, ama şu anda o kadar önemli değil ...

Bu herhangi bir yardım için size çok çok teşekkür ederim!

2 Cevap

Bunu yapmanın tercih edilen bir yolu, bir Table Data Gateway kullanmaktır. Yerine

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

Eğer veritabanındaki her tablo için sınıflar oluşturmak. Her tablo sınıfı, örneğin içine CRUD mantığını kapsüller

class Countries
{
    $protected $_db;

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

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

    // ... other methods
}

Buna ek olarak, tüm bu şeyler birlikte aittir ve muhtemelen sorgulardan biri başarısız olursa, herhangi bir veri eklemek istemiyorum çünkü bu tür işler için transactions kullanmanızı öneririm. Ben CodeIgnitor işlemleri nasıl işlediğini bilmiyorum, ama temelde, daha sonra bu şekilde yapmak gerekir:

$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
}

Bu dinamik yukarıda önerilen gibi yapıyor, (call_user_func() Bu yapabilirdi) bir işlevi çağırmak için nasıl sorunuza cevap vermez iken kod çok daha rahat hale getirir.

Sizin soru bir sırayla tüm fonksiyonlarını çalıştırmak istiyorum ya da sadece kullanıcı sunulan ne bağlı verişi istiyorsanız gibi biraz muğlak. İlk durumda, işlem yaklaşım kullanın. İkinci durumda, sadece uygun sınıf örneğini ve kaydetme yöntemini çağırır.

Tamamen sizin gereksinimleri emin, ama sonra olabileceğini sanmıyorum call_user_func:

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

$function_name = 'process';

call_user_func($function_name, $countries);

Bu şekilde, dinamik, diyelim ki, ülkelerin listesini dayalı bir fonksiyon atayabilirsiniz.

etiketler