1

Há algum tempo atrás criei a classe abaixo para fazer operações de CRUD no banco de dados.

Baseado em boas práticas de programação, o que vocês mudariam na classe? e o porque?

<?php        
    Banco::getConexao();

    abstract class Banco {
        private static $database = 'mysql';
        private static $host = 'localhost';
        private static $banconome = 'testes';
        private static $user = 'root';
        private static $pass = '';
        private static $conexao = NULL;

        public function __construct() { }

        public function __destruct(){
            self::$conexao = NULL;
        }// __destruct

        public static function getConexao(){
            $dsn = self::$database.':host='.self::$host.';dbname='.self::$banconome;
            try{
                if(is_null(self::$conexao)):
                    self::$conexao = new PDO($dsn, self::$user, self::$pass);
                    self::$conexao->setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION );
                endif;
                return self::$conexao;
            }catch(PDOException $e){
                self::erroLog(__FILE__, __FUNCTION__, $e->getMessage());
            }   
        }// getConexao

        public static function insert($tabela, $dados){
            $sql = 'INSERT INTO '.$tabela.' (';
            foreach($dados as $key => $value):
                $campos[] = $key;
                $tokens[] = '?';
                $valores[] = $value;
            endforeach;
            try{
                $sql .= implode(', ', $campos).') VALUES ('.implode(', ', $tokens).')';
                $query = self::$conexao->prepare($sql);
                $query->execute($valores);
            }catch(PDOException $e){
                self::erroLog(__FILE__, __FUNCTION__, $e->getMessage());
            }
        }// insert

        public static function selectAll($tabela, $condicao = NULL){
            $sql = 'SELECT * FROM '.$tabela;
            if(!is_null($condicao)) $sql .= ' WHERE '.$condicao;
            try{
                $query = self::$conexao->prepare($sql);
                $query->execute();
                return $query->fetchAll(PDO::FETCH_OBJ);
            }catch(PDOException $e){
                self::erroLog(__FILE__, __FUNCTION__, $e->getMessage());
            }
        }// selectAll

        public static function select($tabela, $campos, $condicao = NULL){
            $sql = 'SELECT '.$campos.' FROM '.$tabela;
            if(!is_null($condicao)) $sql .= ' WHERE '.$condicao;
            try{
                $query = self::$conexao->prepare($sql);
                $query->execute();
                echo $sql;
                return $query->fetchAll(PDO::FETCH_OBJ);
            }catch(PDOException $e){
                self::erroLog(__FILE__, __FUNCTION__, $e->getMessage());
            }
        }// select

        public static function update($tabela, $dados, $condicao = NULL){
            $sql = 'UPDATE '.$tabela.' SET ';
            foreach($dados as $key => $value):
                $campos[] = $key.'=?';
                $valores[] = $value;
            endforeach;
            $sql .= implode(', ', $campos);
            if(!is_null($condicao)) $sql .= ' WHERE '.$condicao;
            try{
                $query = self::$conexao->prepare($sql);
                $query->execute($valores);
            }catch(PDOException $e){
                self::erroLog(__FILE__, __FUNCTION__, $e->getMessage());
            }
        }// update

        public static function delete($tabela, $condicao = NULL){
            $sql = 'DELETE FROM '.$tabela;
            if(!is_null($condicao)) $sql .= ' WHERE '.$condicao;
            try{
                $query = self::$conexao->prepare($sql);
                $query->execute();
            }catch(PDOException $e){
                self::erroLog(__FILE__, __FUNCTION__, $e->getMessage());
            }
        }// delete

        public static function countResult($tabela, $campo = NULL){
            !is_null($campo) ? $campo = $campo : $campo = '*';
            $sql = 'SELECT '.$campo.' FROM '.$tabela;
            try{
                $query = self::$conexao->prepare($sql);
                $query->execute();
                echo $sql;
                return count($res = $query->fetchAll(PDO::FETCH_OBJ));
            }catch(PDOException $e){
                self::erroLog(__FILE__, __FUNCTION__, $e->getMessage());
            }
        }// countResult

        public static function erroLog($arquivo, $rotina, $mensagem){
            echo 'Aconteceu um erro com os seguintes Dados:<br />';
            echo '<b>No Arquivo = </b>'.$arquivo.'<br />';
            echo '<b>Na rotina = </b>'.$rotina.'<br />';
            echo '<b>Mensagem = </b>'.$mensagem.'<br />';
        }// erroLog

    }// Banco
?>

Como a classe é usada:

$dados = array('nome' => 'Teste', 'senha' => '123');

$dados2 = array('nome' => 'Teste2', 'senha' => '1234');

$res = Banco::select('user', 'nome');

$res = Banco::selectAll('user', "nome LIKE '%teste%'");

$res = Banco::select('user', 'nome=Teste, senha=123');
3
  • 4
    Se eu fosse voce daria uma olhada no medoo.in para ter umas ideias. Algo que eu mudaria seria esse monte de metodos estaticos e fazer uma classe dinamica orientada a objetos, alem de uma funcao de erro melhor que usa logs em vez de um simples echo
    – Olimon F.
    Commented 7/04/2015 às 23:44
  • Para o exemplo de uso da classe funcionar não falta um Banco::getConexao() ?
    – gmsantos
    Commented 8/04/2015 às 11:57
  • Não, pois o Banco::getConexao() está no inicio da classe Banco, ai não é necessário ficar chamando a conexao.
    – MeuChapeu
    Commented 8/04/2015 às 13:46

1 Resposta 1

2

Mudaria em sua classe os seguintes pontos:

  1. Remover o abstract da classe: A finalidade de uma classe abstrata é ela ser base de uma classe herdada. No seu caso ela está sendo utilizada para evitar que a classe banco seja instanciada. Mesmo não sendo muito bem visto, eu usaria Singleton em casos simples ou algum outro pattern (Dependency Injection por exemplo).

  2. Com a alteração acima eliminaria também as chamadas estáticas (static) em todos os métodos e trocaria o self:: para $this.

  3. No método countResult mudaria a ordem da comparação do if ternário:

    // De
    !is_null($campo) ? $campo = $campo : $campo = '*';
    // Para
    $campo = !is_null($campo) : $campo ? '*';
    
  4. Alteraria a visibilidade do erroLog() para private, pois esse método não é útil fora da classe

    private function erroLog($arquivo, $rotina, $mensagem)
    
  5. Outro ponto que me incomoda em sua classe é o uso do try ... catch em todos seus métodos. Qualquer erro de PDO não irá gerar um erro no PHP, mas irá exibir o mensagem em HTML, que pode ser difícil de ser percebida dependendo do layout da sua pagina e pode causar ainda mais erros. Sua classe deveria arremessar a Exception para frente e deixar para uma camada superior tratar o erro em um outro try ... catch

1
  • Nessa parte do try ... catch, eu poderia ter uma classe que so tratasse as exceptions? ai eu chamava a classe passando a exception para ela?
    – MeuChapeu
    Commented 8/04/2015 às 13:50

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .