Usar o try/catch dentro do construtor já não parece bom e ainda mais usar return
nele, isso não irá funcionar, é melhor que use o try/catch fora da classe, pois assim ao menos terá um uso sensato para eles, já que usar um try/catch apenas para checar o PDO e não parar o resto do que deveria fazer parte do "mesmo" bloco não tem muito sentido
Então pra explicar melhor, creio que o problema esta ocorrendo na conexão e como usou o try/catch não emite nada e nem finaliza o script e como eu disse o return
"não funciona" no __construct
, então em seguida você tenta usar o método consulta
, mas a variável $this->conexao
não passou pelo try
por alguma falha na chamada do new PDO
, então ela ainda estava NULL
.
você passou também o bindParam
na conexão ao invés de usar no prepare
e no execute
:
$this->conexao->prepare("SELECT :campos FROM :tabela"); //erro nessa linha
$this->conexao->BindParam(':campos', $campos);
$this->conexao->BindParam(':tabela', $tabela);
Deveria ser isto:
$prepare = $this->conexao->prepare("SELECT :campos FROM :tabela"); //erro nessa linha
$prepare->BindParam(...);
$prepare->BindParam(...);
Outro problema que o Rray me atentou foi o uso dos parametros para o FROM
e colunas, não tem como isto funcionar da maneira que você fez, o bindParam
e bindValue
funcionam como escapadores de caracteres, na verdade os parametros depois de processados ficariam algo como:
SELECT 'id, nome, senha' FROM 'minhatabela'
Ou seja query não vai procurar minhatabela
no banco, ele vai procurar 'minhatabela'
com os apóstrofos incluídos e ao invés de procurar a 3 colunas ele vai procurar gerar uma coluna com o nome id, nome, senha
, como se fosse tudo uma coisa só.
Sugiro mudar para isto:
<?php
class model {
private $conexao;
public function __construct ($db, $host, $dbname, $charset, $usuario, $senha){
$this->conexao = new PDO ("$db:host=$host; dbname=$dbname; charset=$charset","$usuario","$senha");
}
public function consulta (array $campos, $tabela){
$prepare = $this->conexao->prepare("SELECT ' . $campos . ' FROM " . $tabela);
$prepare->execute();
$resultado = $prepare->fetchAll(PDO::FETCH_ASSOC);
print_r($resultado);
}
}
E na chamada:
require_once 'model.php';
try{
$conn = new model("mysql","localhost", "teocratico", "UTF8","root","");
$conn->consulta("descricao","desafios");
} catch (PDOException $erro){
echo $erro->getmessage();
}
Claro que não deve passar acesso ao controle de $campos
e $tabela
ao usuário final por GET e POST, de resto uso o bindParam
lembrando que ele sempre "escapa" os caracteres e "adiciona os apóstrofos" no processamento
echo $campos." - ".$tabela; die();
Veja o que traz em campos e tabela.