mysql lib

Pagine → 1

Seconda Era ↗
Vaste terre coperte di ombre e misteri, dove ogni eroe scrive la storia del Mondo di Eidel. Seconda Era, Play By chat

10/12/2008 21:06:47

joshi ha scritto: secondo me è troppo "monolitica", senza contare che dato che prende i parametri da quella classe, non puoi parametrizzare connessioni a più database.
ah , poi bello il fatto che si perda immediatamente traccia della risorsa di connessione, almeno conservalo in una proprietà dell'oggetto.

inoltre ti tocca mantere un oggetto globale con la connessione, istanziato in libreria di inclusione. certo, questo è normale , ma dato che non è parametrizzato hai un po' di overhead.

hmm, non è pensata male, solo poco controllabile, poco scalabile.
tra l'altro, con un oggetto così, non c'è alcuna gestione degli errori, ne con un sistema vecchio ( array interno o last_error ) oppure Exception Handling.
questa è una pecca abbastanza grossa.

se fossi un professore e dovessi dare un voto, sarebbe un 5.

considerate una cosa: gli oggetti sono carini, ma sono lenti.
se non avete bisogno di logica ad oggetti, EVITATE di usarli, a maggior ragione se non li sapete usare.

le librerie "wrapper" possono sembrare una cosa sofisticata e utile, peccato che dopo vi costino in forma di lavoro raddoppiato.
perchè?
perchè dovete studiare sia come funziona lo libreria di sistema ( mysql in questo caso ), sia come funziona il wrapper, e per scrivere una query, o lavorare su una risorsa, vi tocca risputtanare il codice della libreria.. che andrà automaticamente a spaccare un'altra parte del vostro applicativo.

Accetto pienamente il 5 che mi hai dato questo perchè tale codice è incompleto (rispetto a quello che volevo fare io) e poco rifinito.
Ti do perfettamente ragione per quanto riguarda la variabile globale che decaderà con l'implementazione del pattern singleton che mi permetterà maggior elasticità di utilizzo senza contare la possibilità di monitorare al meglio ciò che faccio col db all'interno di una sessione.
Per quanto riguarda invece i parametri di accesso al database sono disposti in quel modo semplicemente perchè, all'interno del progetto dal quale è stato estrapolato il codice, i file di configurazione vengono caricati dinamicamente a seconda del modulo usato in base a delle istruzioni presenti in un file chiamato header.php, modificando gli elementi di questo file è possibile far girare contemporaneamente più progetti con lo stesso codice.
La scalabilità del codice invece è un aspetto che vorrei curare di più ma sfortunatamente non riesco a dedicarci il giusto tempo.

se non avete bisogno di logica ad oggetti, EVITATE di usarli, a maggior ragione se non li sapete usare.

Mi stai dicendo che non so usare gli oggetti? 😖

Crash Bandicoot Zone ↗
Caos, libertà e risate: nel GDR ambientato nell’universo di Crash Bandicoot, ogni storia è possibile… e nulla è troppo folle!

10/12/2008 23:45:24

Grazie per i consigli, sono sempre ben accetti...

World of Warship ↗
MMO gratuito con frenetiche battaglie navali ambientate nel ventesimo secolo. Salpa con la tua nave ed affronta i nemici!

15/12/2008 18:50:04

Ecco una bozza con alcune delle modifiche apportate...
Per il momento non vi è ancora nessuna gestione degli errori senza contare che vorrei inserire il supporto per l'utilizzo della sintassi join implicita ed i trigger.

<?
/*
This file is part of Tyrien.

Tyrien is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation,version 2 of the License.

Tyrien is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.

You should have received a copy of the GNU General Public License
along with Foobar. If not, see <http://www.gnu.org/licenses/>.

*/

/*----------------------------------*
INTERFACCIA
*----------------------------------*/

interface Daemon {
public static function getInstance();
public static function start();
public static function stop();
public static function restart();
}

class mysql{

/*----------------------------------*
INFO
*----------------------------------*/

const NAME="mysql";
const VERSION="0.7";

/*----------------------------------*
OGGETTO E PATTERN SINGLETON
*----------------------------------*/

private static $instance=null;
private $db_link=null;

private function __construct(){

}

public static function getInstance(){
if(self::$instance==null){
$c=__CLASS__;
self::$instance=new $c;
}
return self::$instance;
}

/*----------------------------------*
FUNZIONI DI CONTROLLO OGGETTO
*----------------------------------*/

/*
* La funzione start() inizializza tutti gli elementi necessari all'utilizzo della libreria.
* Prima di tutto crea l'istanza dell'oggetto mysql poi abbia la connessione al database e ne
* seleziona uno.
* Dato che la funzione start() si puòà inserire all'interno del costruttore potrebbe essere
* conveniente renderla privata, in questo modo il modulo partirà ugualmente e l'utente avrà la
* possibilità di disinserirlo quando vuole o semplicemente riavvialo.
*/
public static function start(){
self::getInstance();
self::connect();
self::dbSelect();

}

public static function stop(){
mysql_close(self::$db_link);
self::$instance=null;
}

public static function restart(){
self::stop();
self::start();
}

/*----------------------------------*
FUNZIONI DI CONTROLLO MYSQL
*----------------------------------*/

private function connect(){
self::$db_link=mysql_connect(mysqlSetting::DB_HOST.mysqlSetting::DB_PORT,mysqlSetting::DB_USER,mysqlSetting::DB_PASS)
or die('Errore di connessione al database!');
}

private function dbSelect(){
if(self::$db_link==null)die('Non è stato trovato un link valido al database!');
self::$db_link=mysql_select_db(mysqlSetting::DB_NAME,self::$db_link) or die('Database richiesto non trovato!');
}

private function mysqlError(){
if(mysql_errno()!=0)return mysql_error();
}

private function query2array($result){
// Prende in entrata il risultato odella query e ritorna un array...
if ($result){
$i=0;
do{
$riga=mysql_fetch_array($result);
$recordset[$i]=$riga;
$i++;
}while($riga);
unset($recordset[$i-1]);
return $recordset;
}else{
return FALSE;
}
}

private function sendQuery($query){
$result=mysql_query($query,self::$db_link);
return $result;
}

public function read($tabella=false,$campi="*",$condizione=false,$ordine=false,$limite=false){
if(!$tabella)die('Non è stata specificata una tabella in cui leggere');
$query="select $campi from $tabella"; // Query base per il recupero dei dati.
if ($condizione != false)$query.=' WHERE '.$condizione; // Se condizione utilizzato lo include nella query.
if ($ordine != false)$query.=' ORDER BY '.$ordine; // Ordino il risultato della richiesta se richiesto.
if ($limite != false) $query.=' LIMIT '.$limite; // Limito i risultati della richiesta se richiesto.
$result=self::query2array(self::sendQuery($query)); // In questo modo metto tutto in un array...
return $result;
}

public function write($table,$record){
/*
* Prende in entrata in array associativo contenente i dati da inserire.
*/
$query="insert into $table (";
$valori="";
foreach($record as $chiave=>$valore){
// Il campo id è determinato da mysql...
if($chiave=="id") continue;
$query.="$chiave,";
$valori.="'$valore',";
}
$query=substr($query,0,-1);
$valori=substr($valori,0,-1);
$query.=") values($valori) ";
$result=self::sendQuery($query);
if($result){
return mysql_insert_id();//Stampa l'ultimo id...
}else{
return $result;
}
}

public function update($tabella=null,$campi=null,$condizione){
if($tabella==null)die('Non è stata specificata una tabella su cui agire!');
if($campi==null)die('Bisogna specificare i campi interessati');
// Ancora da finire...
}

public function del(){

}

public function drop(){

}

public function join(){

}

}
?>

Il codice non è per nulla pronto per essere utilizzato l'ho postato semplicemente per far vedere a che punto sono arrivato (potendoci lavorare saltuariamente) per darvi la possibilità di intercedere dandomi consigli.

L'interfaccia serve solo nell'ambito del progetto tyrien quindi non è strettamente necessaria altrove.

AlterEgo ↗
Scopri AlterEgo GdR: un incredibile universo narrativo in stile medievale fantasy che cresce, muta e vive giorno dopo giorno grazie ai suoi giocatori!

19/12/2008 03:20:13 e modificato da metalj il 19/12/2008 03:21:43

Ecco una versione funzionante della libreria, ovviamente da testare.

Prima di tutto il controller, come scritto nel codice stesso non è prettamente necessario per lo script!

<?
/*
This file is part of Tyrien Framework.

Tyrien is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation,version 2 of the License.

Tyrien is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.

You should have received a copy of the GNU General Public License
along with Foobar. If not, see <http://www.gnu.org/licenses/>.
*/

Per quanto riguarda il file di configurazione è identico a quello precedente, lo trovate un post più in alto.

class mysqlController{
/*
Il controller in questo caso serve unicamente a caricare i file necessari al funzionamento del modulo.
E' ovvio ed abbastanza scontato che il programmatore potrebbe preferire la sostituzione di questo codice
con un metodo più uniforme con il sistema di destinazione.

La necessità di utilizzare questo controller si avverte solo al primo caricamento del modulo all'interno di uno script.

Allo stato attuale la singola funzione init() gestisce come unico parametro in entrata il path di un
eventuale file di configurazione alternativo('../config/mysql.php' in questo caso).

*/
public function init($path=null){
$configPath='FILE DI CONFIGURAZIONE';
if($path!=null)$configPath=$path;
require_once($configPath);
require_once('POSIZIONE LIBRERIA');
}

}
?>

Ecco invece la libreria mysql riscritta:

<?
/*
This file is part of Tyrien.

Tyrien is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation,version 2 of the License.

Tyrien is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.

You should have received a copy of the GNU General Public License
along with Foobar. If not, see <http://www.gnu.org/licenses/>.

*/

/*----------------------------------*
INTERFACCIA
*----------------------------------*/

interface Daemon {
public static function getInstance();
public function start();
public function stop();
public function restart();
}

class mysql{

/*----------------------------------*
INFO
*----------------------------------*/

const NAME="mysql";
const VERSION="0.7";

/*----------------------------------*
OGGETTO E PATTERN SINGLETON
*----------------------------------*/

private static $instance=null;
public $db_link=null;

private function __construct(){
$this->start();
}

public static function getInstance(){
if(self::$instance==null){
$c=__CLASS__;
self::$instance=new $c;
}
return self::$instance;
}

/*----------------------------------*
FUNZIONI DI CONTROLLO OGGETTO
*----------------------------------*/

/*
* La funzione start() inizializza tutti gli elementi necessari all'utilizzo della libreria.
* Prima di tutto crea l'istanza dell'oggetto mysql poi abbia la connessione al database e ne
* seleziona uno.
* Dato che la funzione start() si puòà inserire all'interno del costruttore potrebbe essere
* conveniente renderla privata, in questo modo il modulo partirà ugualmente e l'utente avrà la
* possibilità di disinserirlo quando vuole o semplicemente riavvialo.
*/
public function start(){
//self::getInstance();
$this->connect();
$this->dbSelect();

}

public function stop(){
mysql_close($this->db_link);
$this->instance=null;
}

public function restart(){
$this->stop();
$this->start();
}

/*----------------------------------*
FUNZIONI DI CONTROLLO MYSQL
*----------------------------------*/

public function connect(){
$this->db_link=mysql_connect(mysqlSetting::DB_HOST.mysqlSetting::DB_PORT,mysqlSetting::DB_USER,mysqlSetting::DB_PASS)
or die('Errore di connessione al database!');
}

private function dbSelect(){
if($this->db_link==null)die('Non è stato trovato un link valido al database!');
mysql_select_db(mysqlSetting::DB_NAME,$this->db_link) or die('Database richiesto non trovato!');
}

private function mysqlError(){
if(mysql_errno()!=0)return mysql_error();
}

private function query2array($result){
// Prende in entrata il risultato odella query e ritorna un array...
if ($result){
$i=0;
do{
$riga=mysql_fetch_array($result);
$recordset[$i]=$riga;
$i++;
}while($riga);
unset($recordset[$i-1]);
return $recordset;
}else{
return FALSE;
}
}

private function sendQuery($query){
$result=mysql_query($query,$this->db_link);
return $result;
}

public function read($tabella=false,$campi="*",$condizione=false,$ordine=false,$limite=false){
if(!$tabella)die('Non è stata specificata una tabella in cui leggere');
$query="select $campi from $tabella"; // Query base per il recupero dei dati.
if ($condizione != false)$query.=' WHERE '.$condizione; // Se condizione utilizzato lo include nella query.
if ($ordine != false)$query.=' ORDER BY '.$ordine; // Ordino il risultato della richiesta se richiesto.
if ($limite != false) $query.=' LIMIT '.$limite; // Limito i risultati della richiesta se richiesto.
$result=$this->query2array($this->sendQuery($query)); // In questo modo metto tutto in un array...
return $result;
}

public function write($table,$record){
/*
* Prende in entrata in array associativo contenente i dati da inserire.
*/
$query="insert into $table (";
$valori="";
foreach($record as $chiave=>$valore){
// Il campo id è determinato da mysql...
if($chiave=="id") continue;
$query.="$chiave,";
$valori.="'$valore',";
}
$query=substr($query,0,-1);
$valori=substr($valori,0,-1);
$query.=") values($valori) ";
$result=$this->sendQuery($query);
if($result){
return mysql_insert_id();//Stampa l'ultimo id...
}else{
return $result;
}
}

public function update($tabella=null,$campi=null,$condizione){
if($tabella==null)die('Non è stata specificata una tabella su cui agire!');
if($campi==null)die('Bisogna specificare i campi interessati');
// Ancora da finire...
}

public function del(){

}

public function drop(){

}

public function join(){

}

}
?>

I file di configurazione è riportato qualche post più in su...