Ir para conteúdo

POWERED BY:

Arquivado

Este tópico foi arquivado e está fechado para novas respostas.

George Feitosa Jr

Estudando sobre PHP POO - ajuda

Recommended Posts

Eai pessoal, seguinte estou estudando sobre como programar orientado a objetos no php e estava com algumas duvidas semana passada. Hoje estou tentando fazer um sistema simples de login(ainda estou terminando ele xD), mas orientado a objetos para ver se estou indo bem. Procurei a todo tempo fazer com que as classes sejam totalmente reutilizáveis. Poderiam analisar meu código e me informar se estou cometendo erros?

 

Classe que faz a conexão com o banco de dados

	class Conexao {
		private $dsn, $user, $pass;
		protected $pdo;
		public function __construct($dsn, $user, $pass){
			$this->dsn = $dsn;
			$this->user = $user;
			$this->pass = $pass;
			$this->connect();
		}
		public function connect(){
			try{
				return $this->pdo = new PDO($this->dsn, $this->user, $this->pass);
			} catch(PDOException $e){
				die('Falha na conexão com o banco de dados').$e->getMessage();
			}
		}
		
	}

Classe que resgata, edita, deleta e atualiza a tabela no banco de dados

	class CRUD extends Conexao{
		
		private $sql;
		private $table;
		private $values;
		private $condition;

		public function __construct($table){
			parent::__construct("mysql:host=localhost;dbname=sistema_compras;","root","");
			$this->table = $table;
		}
		public function readDocumment($condition = null){
			$this->condition = $condition;	
			
			if($this->condition != null){
				$this->sql = "SELECT * FROM {$this->table} WHERE {$this->condition}";
			}else{
				$this->sql = "SELECT * FROM {$this->table}";
			}
			return $this->query($this->sql)->fetchAll(PDO::FETCH_ASSOC);
		}
		public function query($sql){
			return $this->pdo->query($sql);
		}
		public function insertDocumment($values){
			$this->values = $values;
			
			$this->sql = "INSERT INTO $this->table SET $this->values";
			$this->query($this->sql);
		}
		public function updateDocumment($values, $condition){
			$this->condition = $condition;
			$this->values = $values;
			
			$this->sql = "UPDATE $this->table SET $this->values WHERE $this->condition";
			$this->query($this->sql);
		}
		public function deleteDocumment($condition){
			$this->condition = $condition;
			$this->sql = "DELETE FROM $this->table WHERE $this->condition";
			$this->query($this->sql);
		}
	}

Classe que fica responsável pelo login do usuário

	class Login extends CRUD{

		private $email, $pass;
		private $checkUser, $user;

		/*AQUI É INFORMADA A TABELA QUE VAI SER PROCURADO O USUARIO*/
		public function __construct($table){
			parent::__construct($table);
		}
		/*MÉTODO PARA PROCURAR O USUARIO NO BANCO DE DADOS*/
		public function getUser($cond){
			/*EXECUTA A QUERY DENTRO DA CLASSE CRUD PARA PROCURAR O USUARIO*/
			$this->checkUser = parent::readDocumment($cond);
			
			/*SE NAO ACHAR NADA, INFORMA QUE OS DADOS ESTAO INCORRETOS*/
			if(!$this->checkUser){
				echo ('Dados incorretos');
			}else{
				/*SE ACHAR ELE RETORNA OS ARRAYS DO USUARIO*/
				foreach($this->checkUser as $this->user){
					return $this->user;
				}
			}
		}
	}
	$login = new Login("usuario_tb");
	
	$usuario = 'usuario@email.com';
	$senha = 'senha12345';
	
	$dados = $login->getUser("email_usuario = '$usuario' AND senha_usuario = '$senha'");
	
	if($dados){
		extract($dados);
		echo 'Bem vindo - '.$nome_usuario;
	}

enfim pessoal é isso, o que vocês acham? Estou ainda tendo uma ideia muito errada sobre poo ou estou indo pelo caminho certo?  t+

Compartilhar este post


Link para o post
Compartilhar em outros sites

Sua Classe de Conexão pode implementar o Pattern Registry ou Singleton. Dessa forma você evita abrir mais de uma conexão a cada instancia da classe.

 

Login não é CRUD, então não faz sentido herdar funcionalidades! Basta injetar através do construtor!

 

Você está injetando dados de acesso através da Classe CRUD.

 

CRUD.php

public function __construct($table){
     parent::__construct("mysql:host=localhost;dbname=sistema_compras;","root","");
     $this->table = $table;
}

O correto é ter um arquivo de configuração "config.php", desta forma você injetaria diretamente na conexão.

 

 

Onde mais você usa esses objetos?

 

private $sql; private $table; private $values; private $condition;

Há não ser no próprio escopo da função, declarando nelas uma variável local, que pode ser simplesmente usada diretamente!

Compartilhar este post


Link para o post
Compartilhar em outros sites

Existe um problema aqui:

 

Citar

public function connect(){
    try{
        return $this->pdo = new PDO($this->dsn, $this->user, $this->pass);
    } catch(PDOException $e){
        die('Falha na conexão com o banco de dados').$e->getMessage();
    }
}

 

E dois problemas aqui:

Citar

die('Falha na conexão com o banco de dados') . $e->getMessage();

 

não coloque "$e->getMessage()" concatenado com die, não é este o objetivo de die

Indico algo como:

die("Falha na conexão com o banco de dados: {$e->getMessage()}");

 

Dito isso, o problema em "connect" é que você está parando todo o sistema de dentro de uma classe sem permitir que a exceção seja tratada de fora, nesse caso, você pode criar sua própria classe que implemente "Throwable" ou extenda "Exception", pois não pode deixar que uma exceção tipo PDO seja esperada, pois esse tipo gerenciador não deve ser conhecido.

Veja um exemplo somente para o método "connect"

 

DataBaseConnectionException.php

class DataBaseConnectionException extends Exception
{}

Exemplo:

/**
* @throws DataBaseConnectionException
*/
public function connect() 
{
    try {
        return $this->pdo = new PDO($this->dsn, $this->user, $this->pass);
    } 
    catch(PDOException $e) {
        throw new DataBaseConnectionException($e->getMessage());
    }
}

 

Dessa maneira a exceção será tratada de fora

try {
    $conexao->connect(/*[...]*/
} 
catch(DataBaseConnectionException $e) {
    // Faça algo com $e
}

 

Compartilhar este post


Link para o post
Compartilhar em outros sites

Tirando o erro de concatenação, a solução sugerida em criar uma Exception personalizada, é a mesma coisa de dar a volta e parar no mesmo lugar.

 

Pois ambas as Exceptions, tanto PDOException e  DataBaseConnectionException, vai estender métodos da classe nativa Exception. Se quer capturar o erro da exceção em Runtime, use Monolog, muito mais simples.

Compartilhar este post


Link para o post
Compartilhar em outros sites

  • Conteúdo Similar

    • Por violin101
      Caros amigos, saudações.
       
      Por favor, me permita tirar uma dúvida com os amigos.

      Tenho um Formulário onde o Usuário digita todos os Dados necessários.

      Minha dúvida:
      --> como faço após o usuário digitar os dados e salvar, o Sistema chamar uma Modal ou mensagem perguntando se deseja imprimir agora ?

      Grato,
       
      Cesar
    • Por Carcleo
      Tenho uma abela de usuarios e uma tabela de administradores e clientes.
      Gostaria de uma ajuda para implementar um cadastro
       
      users -> name, login, passord (pronta) admins -> user_id, registratiom, etc.. client -> user_id, registratiom, etc...
      Queria ajuda para extender de user as classes Admin e Client
      Olhem como estáAdmin
      <?php namespace App\Models; use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Model; class Admin extends User {     use HasFactory;            protected $fillable = [         'name',         'email',         'password',         'registration'     ];      private string $registration;     public function create(         string $name,          string $email,          string $password,         string $registration     )     {         //parent::create(['name'=>$name, 'email'=>$email, 'password'=>$password]);         parent::$name = $name;         parent::$email = $email;         parent::$password = $password;         $this->registration = $registration;     } } User
      <?php namespace App\Models; // use Illuminate\Contracts\Auth\MustVerifyEmail; use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Foundation\Auth\User as Authenticatable; use Illuminate\Notifications\Notifiable; use Illuminate\Database\Eloquent\Relations\BelongsToMany; class User extends Authenticatable {     /** @use HasFactory<\Database\Factories\UserFactory> */     use HasFactory, Notifiable;     static string $name;     static string $email;     static string $password;     /**      * The attributes that are mass assignable.      *      * @var list<string>      */     protected $fillable = [         'name',         'email',         'password',     ];          /**      * The attributes that should be hidden for serialization.      *      * @var list<string>      */     protected $hidden = [         'remember_token',     ];     /**      * Get the attributes that should be cast.      *      * @return array<string, string>      */     protected function casts(): array     {         return [             'email_verified_at' => 'datetime',             'password' => 'hashed',         ];     }          public function roles() : BelongsToMany {         return $this->belongsToMany(Role::class);     }       public function hasHole(Array $roleName): bool     {                 foreach ($this->roles as $role) {             if ($role->name === $roleName) {                 return true;             }         }         return false;     }         public function hasHoles(Array $rolesName): bool     {                 foreach ($this->roles as $role) {             foreach ($rolesName as $rolee) {             if ($role->name === $rolee) {                 return true;             }          }         }         return false;     }         public function hasAbility(string $ability): bool     {         foreach ($this->roles as $role) {             if ($role->abilities->contains('name', $ability)) {                 return true;             }         }         return false;     }     } Como gravar um Admin na tabela admins sendo que ele é um User por extensão?
      Tentei assim mas é claro que está errado...
      public function store(Request $request, Admin $adminModel) {         $dados = $request->validate([             "name" => "required",             "email" => "required|email",             "password" => "required",             "registration" => "required"         ]);         $dados["password"] =  Hash::make($dados["password"]);                  $admin = Admin::where("registration",  $dados["registration"])->first();                  if ($admin)              return                    redirect()->route("admin.new")                             ->withErrors([                                 'fail' => 'Administrador já cadastrados<br>, favor verificar!'                   ]);                            $newAdmin = $adminModel->create(                                    $dados['name'],                                    $dados['email'],                                    $dados['password'],                                    $dados['registration']                                 );         dd($newAdmin);         $adminModel->save();         //$adminModel::create($admin);                  return redirect()->route("admin.new")->with("success",'Cadastrado com sucesso');     }  
    • Por violin101
      Caros amigos, saudações.
       
      Gostaria de tirar uma dúvida com os amigos, referente a PDV.
       
      Estou escrevendo um Sistema com Ponto de Vendas, a minha dúvida é o seguinte, referente ao procedimento mais correto.

      Conforme o caixa vai efetuando a venda, o Sistema de PDV já realiza:
      a baixa direto dos produtos no estoque
      ou
      somente após concretizar a venda o sistema baixa os produtos do estoque ?
       
      Grato,
       
      Cesar
       
    • Por violin101
      Caros amigos do grupo, saudações e um feliz 2025.
       
      Estou com uma pequena dúvida referente a Teclas de Atalho.

      Quando o Caps Lock está ativado o Comando da Tecla de Atalho não funciona.
      ou seja:
      se estiver para letra minúscula ====> funciona
      se estiver para letra maiúscula ====> não funciona
       
      Como consigo evitar essa falha, tanto para Letra Maiúscula quanto Minúscula ?

      o Código está assim:
      document.addEventListener( 'keydown', evt => { if (!evt.ctrlKey || evt.key !== 'r' ) return;// Não é Ctrl+r, portanto interrompemos o script evt.preventDefault(); });  
      Grato,
       
      Cesar
    • Por violin101
      Caros amigos, saudações.
       
      Por favor, poderiam me ajudar.

      Estou com a seguinte dúvida:
      --> como faço para para implementar o input código do produto, para quando o usuário digitar o ID o sistema espera de 1s a 2s, sem ter que pressionar a tecla ENTER.

      exemplo:
      código   ----   descrição
           1       -----   produto_A
       
      Grato,
       
      Cesar
×

Informação importante

Ao usar o fórum, você concorda com nossos Termos e condições.