O que há de errado com esse código?!

Pessoal existe algo errado com esse código em PHP, que estou tentando desenvolver?! Ela esta de acordo com a orientação a objeto?!

<?php
Class Aluno{

private $nome;
private $login;
private $senha;
private $consulta;
private $resultado;

	public function setNome($nome)
	{
 $this->nome = $nome;
	}

	public function getNome()
	{
 return $this->nome;
	}

	public function setLogin($login)
	{
 $this->login = $login;
	}

	public function getLogin()
	{
 return $this->login;
	}

	public function setSenha($senha)
	{
 $this->senha = $senha;
	}

	public function getSenha()
	{
 return $this->senha;
	}

	public function AlunoDao() {

 $novaconexao = new Banco();
 $this->consulta = $novaconexao->conectar();

	}

	public function CadastrarAluno($nome, $login, $senha) {

 $this->AlunoDao();
 $cadastrar = new Aluno;
 $cadastrar->setNome($nome);
 $cadastrar->setLogin($login);
 $cadastrar->setSenha($senha);

 if ($this->resultado = $this->consulta->prepare("INSERT into usuario(nome, login, senha) values
 (:nome, :login, :senha)")) {

 $this->resultado->bindParam(':nome', $cadastrar->getNome());
 $this->resultado->bindParam(':login', $cadastrar->getLogin());
 $this->resultado->bindParam(':senha', $cadastrar->getSenha());
 $this->resultado->execute();

 }
 //if ($this->consulta->query("INSERT into usuario(id, cpf) values ('". NULL ."','". $cadastrar->getCPF() ."')")) {
 //}
	}

 public function ListarAluno() {

 $this->AlunoDao();
 echo '<table border="1" cellspacing="2">
 <tr>
 <td>ID</td>
 <td>nome</td>
 <td>login</td>
 <td>senha</td>
 <td>atualizar</td>
 </tr>
 ';

 if ($this->resultado = $this->consulta->query("select * from usuario")) {

 foreach($this->resultado as $linha){
 //exibe o resultado
 echo '<tr>
 <td>'.$linha['id'].'</td>
 <td>'.$linha['nome'].'</td>
 <td>'.$linha['login'].'</td>
 <td>'.$linha['senha'].'</td>
 <td><a href="Modelo/AtualizarAluno.php?id='.$linha['id'].'">Atualizar</a></td>
 </tr>';
 }
 //$this->consulta->desconectar();
 }

	}
}

Desde já agradeço!