so habe ich ist ususally ein Modell gesehen Darstellung einer Tabelle in der Datenbank wie für Benutzer-Tabelle geht es so etwas wie:ein einziges crud Modell für alle Steuerungen statt separater Modelle für jeden Controller Mit
class user_model {
public $id ;
public $username ;
public $password ;
public $email ;
public function save(){
$db->query(" insert into `users` (username , email , password) values ('$this->username' , '$this->email' , '$this->password') ");
}
public function delete(){
$db->query(" delete from users where id = $this->id ");
}
}
aber diese proccess ist ziemlich langsam und die meisten Modelle, was zu tun ist einfach CRUD Betrieb ... so verwende ich ein einziges crud Modell für fast alle meine Controller wie:
class crud_model {
public function save($tbl , $data){
$db->query(" insert into $tbl (".explode(',' , array_keys($data)).") values (".explode(',' , $data).") ");
}
public function delete( $tbl , $data){
$db->query(" delete from $tbl where $data['column'] = $data['val'] ");
}
}
Bitten beachten Sie, dies ist eine sehr vereinfachte Version mein Modell und im Grunde ist es nichts wie die Originalcode (verwende aktiven Datensatz im Originalcode und kann komplexe Szenarios verarbeiten), also ignoriere Syntax und technische Fehler
Also möchte ich wissen, ob es ein Problem mit diesem Ansatz gibt? vermisse ich etwas?
was ist der Punkt von vielen Modellen, die, wenn Sie von einem CRUD Modell bekommen kann .... es scheint nur, wie wast Zeit
Trotz der Tatsache, dass Sie vorbereitete Anweisungen verwenden sollten, um einen Sicherheitsgewinn zu erzielen, ist Ihr Code nicht falsch. –
@HenriqueBarcelos eigentlich benutze aktiven Datensatz für die Abfrage, ich wollte nur den Code so einfach wie möglich halten – max
Anstatt eine einzige Klasse zu verwenden, könnten Sie eine gemeinsame Basisklasse erben, um die 'save' und' delete' zu schreiben Methoden nur einmal. – user3409662