Ir para conteúdo

POWERED BY:

Arquivado

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

Daniel-Lopes

Quando devo usar códigos repetidos em classes diferentes (Divisão de tarefas em Orientação a objetos - PHP)

Recommended Posts

Estou com dificuldades em entender quando um determinado trecho de código deve ser repetido em mais de uma classe afim de manter a independência da mesma, pois segundo artigos que leio sobre orientação a objetos - Um Objeto (classe) deve ser independente o suficiente para ser utilizada sem dependências. - Ter apenas uma funcionalidade, desempenhando apenas o seu papel (coesão).

Meu sistema é foi criado para vendas de comerciais, onde cada bloco/horário tem um preço diferente, calculado com base no preço médio + acréscimos - descontos.

O objetivo (funcionalidade ou papel) da classe VendasController é retornar para View Orcamentos, informações sobre produtos e valores para realização de um orçamento e posteriormente um contrato. Para retornar essas informações eu obtenho o preço médio de cada bloco e adiciono os acréscimos em alguns métodos dentro desta classe, ela tem o funcionamento independente.

Agora estou desenvolvendo outra classe que se chama DistribuicoesController, onde a sua funcionalidade é distribuir uma quantidade X de comerciais no blocos/horários disponiveis, e uma de suas opções seriam distribuir no blocos com menor custo médio.

Portanto a questão aqui é:

Tanto na Classe VendasController como na classe DistribuicoesController eu faço uso de preço médio dos blocos.

Qual seria o mais correto: eu repetir o código nas duas classes, fazer tudo dentro de uma unica classe, criar uma "super classe" e usa-la estendida dentro das classes VendasController e DistribuicoesController.

Compartilhar este post


Link para o post
Compartilhar em outros sites
1 hora atrás, Daniel-Lopes disse:

[...]Um Objeto (classe) deve ser independente o suficiente para ser utilizada sem dependências.[...]

 

Essa definição está errada e não condiz com o paradigma orientado à objetos. Ou pode apenas estar fora de contexto. Objetos podem ser independentes, entretanto, não é ruim ter dependência. Apenas, a dependência deve ser bem construída.

 

Por outro lado, esta, está correta:

1 hora atrás, Daniel-Lopes disse:

[...]Ter apenas uma funcionalidade, desempenhando apenas o seu papel (coesão)[...]

 

 

O princípio da orientação a objetos, no seu estado mais bruto, é trazer a programação o mais próximo possível do mundo real.

 

Pelo que você descreveu sobre o objeto VendaController, ele já está fazendo muito mais do que é de sua responsabilidade. Um controller, por definição, deve apenas receber uma requisição, acionar um model (caso necessário) e enviar os dados para uma view. Um controler sempre estará linkado a uma view, mas, pode nao estar linkado a um model.

 

1 hora atrás, Daniel-Lopes disse:

Qual seria o mais correto: eu repetir o código nas duas classes, fazer tudo dentro de uma unica classe, criar uma "super classe" e usa-la estendida dentro das classes VendasController e DistribuicoesController.

 

O correto é nunca repetir. Se repetir, há possibilidade de refatoração e a criação em uma classe separada.

 

Crie um objeto que seja responsável por aquilo que você está trabalhando. Se você está exibindo dados de um orçamento, provavelmente você terá alguns integrantes, como: Orçamento; Itens do Orçamento (seja produto e/ou serviços).

 

O Orçamento e Itens do Orçamento, podem estar associados a outros itens, como o produto/serviço em si, clientes, etc...

 

Quando você separa desta forma, você isola as responsabilidades e entenderá que cada objeto tem o seu papel. No seu caso, se o cálculo de preço médio dos blocos é realizado para o produto apenas quando ele está em um orçamento, bem, isso é de responsabilidade do Item do Orçamento. Por outro lado, se o preço médio do bloco deve ser calculado considerando o orçamento, o cálculo é de responsabilidade do Orçamento.

Compartilhar este post


Link para o post
Compartilhar em outros sites

 

Citar

Pelo que você descreveu sobre o objeto VendaController, ele já está fazendo muito mais do que é de sua responsabilidade. Um controller, por definição, deve apenas receber uma requisição, acionar um model (caso necessário) e enviar os dados para uma view. Um controler sempre estará linkado a uma view, mas, pode nao estar linkado a um model.

 

Entendi, no caso o Controller serve apenas para receber e enviar (a grosso modo), então neste caso as demais funcionalidades, como está de aplicar acréscimos, montar os preços para serem retornadas pelo controller para a view é feita onde em um padrão MVC ?

 

Citar

O correto é nunca repetir. Se repetir, há possibilidade de refatoração e a criação em uma classe separada.

Então é isso que está me deixando preocupado com minhas classes...

Por exemplo, eu tenho a classe Clientes, onde no momento do Orçamento eu necessito retornar para View uma listagem de clientes, dentro do controller Orcamentos eu chamo o Model Clientes para retornar essa listagem (ja que para realizar um orçamento necessita de escolher um cliente), está correto esta prática ?

 

 

Citar

Crie um objeto que seja responsável por aquilo que você está trabalhando. Se você está exibindo dados de um orçamento, provavelmente você terá alguns integrantes, como: Orçamento; Itens do Orçamento (seja produto e/ou serviços).

Tenho uma situação distinta onde uso os valores médios dos produtos em cada bloco/horario (exemplo: spot 30", testemunhal, spot 15",  no bloco das 15:00) em dois locais, que não tem ligação em si.

 

Neste caso teria que criar um objeto responsável por retornar esta tabela de preços e utiliza-la nos outros dois objetos (Controllers) distintos para serem retornados ?

Compartilhar este post


Link para o post
Compartilhar em outros sites
4 minutos atrás, Daniel-Lopes disse:

Entendi, no caso o Controller serve apenas para receber e enviar (a grosso modo), então neste caso as demais funcionalidades, como está de aplicar acréscimos, montar os preços para serem retornadas pelo controller para a view é feita onde em um padrão MVC ?

 

O padrão MVC em si define apenas que, regras de negócio devem ser tratadas na camada (layer) Model . Ou seja, esse cálculo deve estar no model. 

 

O model, por sua vez, pode ser compreendido como N camadas. Essas camadas podem ser:

  • Entidades (Entities), ex.: Orçamento, Cliente, Produto;
  • Controle de dados, ex.: DAO (Data Access Object), DataMapper, FileSystem;
  • Operacional, ex.: contábil, estoque, financeiro, etc...

Deve-se entender que o Model é um ecossistema que podem ser diversas camadas. Essas camadas não são definidas pelo MVC e sim por outros padrões. Eu costumo criar sempre as seguintes "camadas" no Model:

  • Model: os objetos que são acessados diretamente pelo controller;
  • DataMapper: conjunto de objetos específicos para o mapeamento de dados de uma base de dados para entidades;
  • Entities: entidades do sistema, são as entidades que são persistidas na base de dados;
  • Entidades de negócio: São entidades que não são persistidas, mas estão intrinsincamente ligadas as entidades e possuem regras complexas de negócio, como contabilidade, financeiro, estoque, etc..

A forma que você irá separar, vai depender do quão complexo é o seu sistema. Há também o DDD (Domain-Driven Design) que prove uma excelente abordagem para a arquitetura.

 

4 minutos atrás, Daniel-Lopes disse:

Então é isso que está me deixando preocupado com minhas classes...

Por exemplo, eu tenho a classe Clientes, onde no momento do Orçamento eu necessito retornar para View uma listagem de clientes, dentro do controller Orcamentos eu chamo o Model Clientes para retornar essa listagem (ja que para realizar um orçamento necessita de escolher um cliente), está correto esta prática ?

 

Sim, a prática está ok. Você pode verificar os seguintes tópicos sobre o assunto:

https://pt.stackoverflow.com/a/197651/5007

 

4 minutos atrás, Daniel-Lopes disse:

Tenho uma situação distinta onde uso os valores médios dos produtos em cada bloco/horario (exemplo: spot 30", testemunhal, spot 15",  no bloco das 15:00) em dois locais, que não tem ligação em si.

 

Neste caso teria que criar um objeto responsável por retornar esta tabela de preços e utiliza-la nos outros dois objetos (Controllers) distintos para serem retornados ?

 

Existe uma abordagem chamda de Anemic Model, ela define que entidades devem ter nada ou o mínimo de regras de negócio possível e que outros objetos seriam responsávels por essa regra de negócio. Essa abordagem é muito criticada e, normalmente, o recomendado é inserir a regra de negócio, especifica de uma entidade, na entidade.

 

Caso uma regra de negócio seja compartilhada, essa regra deve ser separada em outro objeto (vide a minha camada de Entidades de negócio).

 

Mas isso fica a sua escolha, ambas as abordagens existem, só Anemic Model não é recomendada.

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
×

Informação importante

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