[OOP] Classe Db

<?

class dbClass
{
    var $_connect;
    var $_result;
    
    function dbConnect($_host, $_user, $_pass)
    {
        $this->_connect = @mysql_connect($_host, $_user, $_pass) or die("Errore: Connessione non effettuata");
    }
    
    function dbSelect($_dbname)
    {
        @mysql_select_db($_dbname, $this->_connect) or die("Errore: Db non selezionato");
    }
    
    function dbDisconnect()
    {
        @mysql_close($this->_connect);
        $this->_connect = false;
    }
    
    function dbQuery($query)
    {
        return @mysql_db_query($this->_connect, $query) or die(mysql_errno());
    }
    
    function dbFetchArray($result)
    {
        return @mysql_fetch_array($result) or die(mysql_errno());
    }
    
    function dbFetchObject($result)
    {
        return @mysql_fetch_object($result) or die(mysql_errno());
    }
    
    function dbNumRows($result)
    {
        return @mysql_num_rows($result) or die(mysql_errno());
    }
}    
?>

Questa è una classe Db che sto scrivendo a tempo perso per creare un CMS, unico problema è che non funziona

Cioè non mi torna nessun errore ed è questo quello che mi fa incavolare... il codice che la usa è questo:

<?
$db = new dbClass();
$db->dbConnect("localhost","root","");
$db->dbSelect("my_core_class");

$result = $db->dbQuery("SELECT id,nome FROM test");
$result = $db->dbFetchArray($result);
print_r($result);

$db->dbDisconnect();
?>

La connessione e la selezione del db viene effettuata con successo. La tabella di prova che utilizzo esiste e ha 2 record.

Se la provate vedrete che il print_r non stampa nulla. In un primo momento ho pensato che print_r non facesse quello che serviva a me (cioè stampare l'array che ristituisce la funzione) e ho provato ad usare print $result['id'], ma anche così nada... Non trovo dove sia il problema

l'unico problema ke ho trovato è ke se selezioni il db nn serve ke usi mysql_db_query e poi i parametri di questa funzione sono: (DB_NAME, QUERY, CONNECTION)

ciao ^__^

Umm, non ci avevo pensato. Ho modificato la classe così:

<?
class dbClass
{
    function dbConnect($_host, $_user, $_pass)
    {
        @mysql_connect($_host, $_user, $_pass) or die("Errore: Connessione non effettuata");
    }
    
    function dbSelect($_dbname)
    {
        @mysql_select_db($_dbname) or die("Errore: Db non selezionato");
    }
    
    function dbDisconnect()
    {
        @mysql_close();
    }
    
    function dbQuery($query)
    {
        return @mysql_query($query) or die(mysql_errno());
    }
    
    function dbFetchArray($result)
    {
        return @mysql_fetch_array($result) or die(mysql_errno());
    }
    
    function dbFetchObject($result)
    {
        return @mysql_fetch_object($result) or die(mysql_errno());
    }
    
    function dbNumRows($result)
    {
        return @mysql_num_rows($result) or die(mysql_errno());
    }
}    
?>

ma comunque non stampa nulla a video

[heracleum]

Uhm, dunque,
il problema principale sta nel fatto che non ti stampa l'errore giusto?
E' quello che ti stupisce.

Se non erro, pur essendo php molto versatile con i tipi di dati, stranamente proprio in questo caso il problema sta nel fatto che la funzione die() vuole una stringa come parametro per il messaggio, mentre tu usi un int mysql_errno().
Quindi prova a sostituire in tutte le occorrenze di questo tipo con:
... or die("Errore n. ".mysql_errno());

Cioè basta che concateni l'int ad una stringa.
Dovrebbe funzionare, fammi sapere.
ciao

ora è così

<?
class dbClass
{
    function dbConnect($_host, $_user, $_pass)
    {
        @mysql_connect($_host, $_user, $_pass) or die("Errore: Connessione non effettuata");
    }
    
    function dbSelect($_dbname)
    {
        @mysql_select_db($_dbname) or die("Errore: Db non selezionato");
    }
    
    function dbDisconnect()
    {
        @mysql_close();
    }
    
    function dbQuery($query)
    {
        return @mysql_query($query) or die("Errore Mysql: ".mysql_errno());
    }
    
    function dbFetchArray($result)
    {
        return mysql_fetch_array($result) or die("Errore Mysql: ".mysql_errno());
    }
    
    function dbFetchObject($result)
    {
        return @mysql_fetch_object($result) or die("Errore Mysql: ".mysql_errno());
    }
    
    function dbNumRows($result)
    {
        return @mysql_num_rows($result) or die("Errore Mysql: ".mysql_errno());
    }
}    
?>

ho fatto come hai detto heracleum, ma comunque non mi stampa nulla. Ho provato togliendo il silence e mi stampa l'errore ma quello canonico di mysql

mysql_fetch_array(): supplied argument is not a valid MySQL result resource in d:\www\core_class\classDb.php on line 34

ora mettendo da parte il problema mysql, come mai non mi stampa l'errore con il die?

Grazie per l'aiuto

[heracleum]

Forse perché non è un errore ma un Warning.

per stampare l'errore presumo tu debba usare una cosa del tipo

echo "mysql_errno()"; 

per stampare l'errore presumo tu debba usare una cosa del tipo

echo "mysql_errno()"; 

potrebbe andare ma a me non serve quello, a me serve che se c'è un errore, blocchi lo script e mi dica dove sta

Giusto come tentativo, prova a non utilizzare la sintassi "return + or"; il problema potrebbe derivare da un interpretazione sbagliata di tale costrutto. ad es.

 function dbQuery($query)
 {
     $result = @mysql_query($query);
     if ($result == false)
     {     
         die("Errore Mysql: ".mysql_errno());
     }
     return $result;
} 

fai questa prova per tutte le funzioni che ritornano qualcosa, riscontri gli stessi problemi?

mavericck

<?
class dbClass
{
    function dbConnect($_host, $_user, $_pass)
    {
        @mysql_connect($_host, $_user, $_pass) or die("Errore: Connessione non effettuata");
    }
    
    function dbSelect($_dbname)
    {
        @mysql_select_db($_dbname) or die("Errore: Db non selezionato");
    }
    
    function dbDisconnect()
    {
        @mysql_close();
    }
    
    function dbQuery($query)
    {
        $this->_result = @mysql_query($query) or die("Errore Mysql: ".mysql_error());
        return $this->_result;
    }
    
    function dbFetchArray($result)
    {
        return @mysql_fetch_array($result) or die("Errore Mysql: ".mysql_error());
    }
    
    function dbFetchObject($result)
    {
        return @mysql_fetch_object($result) or die("Errore Mysql: ".mysql_error());
    }
    
    function dbNumRows($result)
    {
        return @mysql_num_rows($result) or die("Errore Mysql: ".mysql_error());
    }
}    
?>

Ora mi torna questo

1

Ora non mi da più errore, ma comunque non mi stampa l'array del risultato.

Come mai?

Ma perchè usi mysql_errno?? Molto meglio mysq_error
Cmq dovresti cercare di determinare quale funzione ti restituisce quel supposto errore.


Ciaoooo!!!

nada?

Nada??? Nada che???
Che risposta è?? Non funziona? Cosa hai provato a fare? Hai letto i suggerimenti??
Stiamo parlando di una cosa tua, ma non sembra che ti interessi molto....


Ciao!

Nada??? Nada che???
Che risposta è?? Non funziona? Cosa hai provato a fare? Hai letto i suggerimenti??
Stiamo parlando di una cosa tua, ma non sembra che ti interessi molto....


Ciao!

Ho appena risolto, il problema stava nel return. Per qualche strano motivo il return non inviava i dati se era usato così:

return @mysql_fetch_array($result) or die("Errore Mysql: ".mysql_error()); 

Ho risolto modificandolo così:

$this->_result = @mysql_fetch_array($result) or die("Errore Mysql: ".mysql_error());
return $this->_result; 

Debug, non ho postato altro semplicemente perchè non c'era nulla di nuovo ed ero fermo anche io con il test in quel punto.

Ora funziona, recupera tutti i record e li stampa correttamente. Non ci sono errori (sicuro di questo perchè ho tolto i silence per controllare) ma comunque mi stampa la stringa

Errore Mysql:

ma senza inserire l'errore, che quindi non sussiste. Non capisco perchè lancia la parte or die(..); della funzione.

La classe è questa

<?
class dbClass
{
    function dbConnect($_host, $_user, $_pass)
    {
        @mysql_connect($_host, $_user, $_pass) or die("Errore: Connessione non effettuata");
    }
    
    function dbSelect($_dbname)
    {
        @mysql_select_db($_dbname) or die("Errore: Db non selezionato");
    }
    
    function dbDisconnect()
    {
        @mysql_close();
    }
    
    function dbQuery($query)
    {
        $this->_result = mysql_query($query) or die("Errore Mysql: ".mysql_error());
        return $this->_result;
    }
    
    function dbFetchArray($result)
    {
        $this->_result = mysql_fetch_array($result) or die("Errore Mysql: ".mysql_error());
        return $this->_result;
    }
    
    function dbFetchObject($result)
    {
        return @mysql_fetch_object($result) or die("Errore Mysql: ".mysql_error());
    }
    
    function dbNumRows($result)
    {
        return @mysql_num_rows($result) or die("Errore Mysql: ".mysql_error());
    }
}    
?>

e il codice che la utilizza è questo

<?
$db = new dbClass();
$db->dbConnect("localhost","root","");
$db->dbSelect("my_core_class");

$aa = $db->dbQuery("SELECT id,nome FROM test");
while ($result = $db->dbFetchArray($aa))
{
    print $result['id']."-".$result['nome'];
}

$db->dbDisconnect();
?>

Tutte a me ogni volta devono succedere!

mavericck insegna

mavericck insegna

prova così:

    function dbQuery($query)
{
        if ($result = mysql_query($query)) {
        return $result;
    } else {
        die('Errore Mysql: '.mysql_error());
    }
} 

cmq è sempre bene implementare un debugger mysql usando la sintassi EXPLAIN di mysql

<?
class dbClass
{
    function dbConnect($_host, $_user, $_pass)
    {
        @mysql_connect($_host, $_user, $_pass) or die("Errore: Connessione non effettuata");
    }
    
    function dbSelect($_dbname)
    {
        @mysql_select_db($_dbname) or die("Errore: Db non selezionato");
    }
    
    function dbDisconnect()
    {
        @mysql_close();
    }
    
    function dbQuery($query)
    {
        $this->_result = @mysql_query($query) or die ("Errore Mysql: ".mysql_error());
        if ($this->_result)
        {
            return $this->_result;
        }
        else
        {
            die("Errore Mysql: ".mysql_error());
        }
    }
    
    function dbFetchArray($result)
    {
        $this->_result = @mysql_fetch_array($result);
        if ($this->_result)
        {
            return $this->_result;
        }
        else
        {
            die("Errore Mysql: ".mysql_error());
        }
    }
    
    function dbFetchObject($result)
    {
        $this->_result = @mysql_fetch_object($result);
        if ($this->_result)
        {
            return $this->_result;
        }
        else
        {
            die("Errore Mysql: ".mysql_error());
        }
    }
    
    function dbNumRows($result)
    {
        $this->_result = @mysql_num_rows($result);
        if ($this->_result)
        {
            return $this->_result;
        }
        else
        {
            die("Errore Mysql: ".mysql_error());
        }
    }
}    
?>

E la uso così

<?
$db = new dbClass();
$db->dbConnect("localhost","root","");
$db->dbSelect("my_core_class");

$aa = $db->dbQuery("SELECT id,nome FROM test");
while ($result = $db->dbFetchArray($aa))
{
    print $result['id']."-".$result['nome']."<br />";
}

$db->dbDisconnect();
?>

Comunque mi dice:

Errore Mysql:

Ho pensato che mi tornava errore, perchè il risultato di mysql_fetch_array è appunto una array e quindi non tornava true o false. Ho provato con un controllo is_array, ma anche così mi torna errore. La cosa che mi fa imbestialire, e che se tolgo il silence NON mi da errore.

La riscrivo io che facciamo prima

<?php
class dbClass
{
    function dbConnect($_host, $_user, $_pass)
    {
        mysql_connect($_host, $_user, $_pass) or die("Errore: Connessione non effettuata");
    }
    
    function dbSelect($_dbname)
    {
        mysql_select_db($_dbname) or die("Errore: Db non selezionato");
    }
    
    function dbDisconnect()
    {
        mysql_close();
    }
    
    function dbQuery($query)
    {
       
        if ($result = mysql_query($query))
        {
            return $result;
        }
        else
        {
            die("Errore Mysql: ".mysql_error());
        }
    }
    
    function dbFetchArray($result)
    {
        return mysql_fetch_array($result, MYSQL_ASSOC);
    }
}    
?>

errori :
1) $this->_result è prima una risorsa mysql e poi (dopo il primo fetch_array) un array ....
2) mysql_fatch_array() ritorna false sia in caso di errore sia in caso non ci siano più righe, quindi togli pure il die()
3) manca la costante MYSQL_ASSOC

Grazie, ora funziona.
Comunque la costante MYSQL_ASSOC è impostata di default, quindi non è obbligatoria.

il default è MYSQL_BOTH, se usi gli associativi sprechi solo memoria assegnandoli entrambi
perchè non impariamo ad ottimizzare il codice da subito?

cmq non è un errore

il default è MYSQL_BOTH, se usi gli associativi sprechi solo memoria assegnandoli entrambi
perchè non impariamo ad ottimizzare il codice da subito?

cmq non è un errore

Sospettavo sprecasse più risorse, ma sinceramente non ho mai controllato quanto

Comunque grazie per l'aiuto