Chi mi dice se va bene come ragionamento?

[PHP<?php
class db_con{

var $db;

/*Costruttore per inizializzare variabili connessione*/

function db_con($HOST="localhost",$USER="root",$PW="root",$ DBNOME="nome") {

$this->db[Host] = $HOST ;

$this->db[User] = $USER ;

$this->db[PW] = $PW ;

$this->db[DBNome] = $DBNOME ;

}/*chiusura costruttore, si può richiamare altra connessione tra parentisi





/*Connessione al db */

function connetti(){

$this->db[Connessione] = mysql_connect($this->db[Host],$this->db[User],$this->db[PW]) ;

if($this->db[Connessione]){

return $this->db[Connessione] ;

}else{

print "Connessione no" ;

}

}/*chiusura metodoconnessione db*/



/*seleziono db*/

function nomedb($nomedb="logosoft"){

$this->db[DBNome] = $nomedb ;

$this->db[nomedb] = mysql_select_db($this->db[DBNome],$this->db[Connessione]) ;

return $this->db[nomedb] ;

}

/*esecuzione query*/

function query1($Query1){

if($this->db[Stato] == "true"){/*se lo stato è attivo setStato*/

$this->db[Risultato] = mysql_query($Query1,$this->db[Connessione]) ;

return $this->db[Risultato] ;

//return $this->prova() ;

}else{

print "Non sono connesso query generale(query1)..." ;

}

}/*chiusura metodo query*/[/PHP]

Poi ho una classe diagnosi

Codice PHP:
<?php
class cl_gestdiagnosi{
/*mi creo un puntatore per la classe db*/
function cl_gestdiagnosi($indb){

        
$this->gestdiagnosi $indb ;
}


function 
insdiagnosi($sql){
        
$sql "insert into tb_diagnosi (nome_diagnosi)
        values('prova')

        "
;
            
$res $this->gestdiagnosi->query1($sql) ;
}


}
?>
poi nel file

Codice PHP:
<?
session_start
() ;
include(
"../lib/class_db.php") ;
include(
"../lib/class_diagnosi.php") ;





/*rich9iamo classi*/
$db = new db_con ;
/*classe con puntatore al db*/
$cldiagnosi = new cl_gestdiagnosi($db) ;

$db->connetti() ;
$db->setstatoatt("true") ;
$db->nomedb() ;
$cldiagnosi->insdiagnosi("") ;
$db->disconnetti() ;

funziona tutto però se qualche anima buona mi dice se come ragionamento va bene

 
:ciauz
?>