Ir para conteúdo

Arquivado

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

targa

PHP Uso dos métodos __get e do __set

Recommended Posts

Boa tarde!

 

Desculpe, caso seja um tema básico, pois estou entrando no mundo do PHP OO agora.

Criei uma classe que teria umas quantidade grande de atributos e com os gets e sets, o netbeans começou alertar que a classe deveria ter no máximo 200 linha.

Em uma consulta com o objetivo de diminuir, encontrei os métodos __get e __set e surgiu algumas dúvidas:

  • Apesar de ter toda a estrutura, isto seria mesmo Orientação a Objeto? Não seria o mesmo que tornar os atributos publico, uma vez que você chama "$classe->atributo"?
  • O melhor caminho não seria, refatorar a classe em classes menores ou esta é um boa pratica de programação?
  • O que acontece com os recursos de autocompletar, tem com ativá-los?

 

Obrigado.

Compartilhar este post


Link para o post
Compartilhar em outros sites

 

Citar

 netbeans começou alertar que a classe deveria ter no máximo 200 linha.

Oush.

Tem sentido isso?

Já vi alguns argumento a respeito de JavaScript por deixar no cache do navegador, mas linguagem php desconheço.

Será q tem a ver com esse padrão?!

https://www.youtube.com/watch?v=gB5Gej0O400&t=121s

(Recomendo fortemente esse canal)

 

 

 

Citar

O melhor caminho não seria, refatorar a classe em classes menores ou esta é um boa pratica de programação?

O tamanho da classe não interefere em boa ou má pratica e sim como está estruturado o seu projeto.

 

Alguns principios pro exemplo SOLID:

http://forum.imasters.com.br/topic/510435-solid-principio-da-responsabilidade-unica/

 

https://pt.wikipedia.org/wiki/Injeção_de_dependência

 

Da uma lida tmb nesse post, parece bastante com seu contexto.

https://forum.imasters.com.br/topic/545070-como-implementar-solid-sem-violar-seus-principios/

 

 

Se quiser posta seu código, a galera com mais tempo de vivência em O.O. pode dar um help

 

 

Hoje quando alguém cita qualidade de código lembro dos Design Patterns, 


 - Factory Method
 - Abstract Factory
 - Simple Factory
 - Bridge
 - Builder
 - Composite
 - Adapter
 - Chain Of Responsability
 - Strategy
 - Proxy
 - Dependency Injection
 - Singleton

Compartilhar este post


Link para o post
Compartilhar em outros sites

Nossa, quanta informação boa!

Repensei bastante nas varias possibilidades de refatorar o código.

Algumas coisas, não conhecia e tive estudar outras ainda estou abstraindo?

 

Estou postando parte do Código para caso queiram olhar ou dar algumas sugestões.

Este código foi gerado com base no livro PHP - Programando com Orientação a Objeto.

Se tiverem algumas dicas de outros livros, agradeço.

<?php
class Maquinas extends Classe {
    private $id;
    // equipamento
    private $nome;
    private $dataCadastro;
    private $dataInstalacao;
    private $responsavel;
    private $destinacao;
    // hardware 0/5
    private $especificacao;
    private $marca;
    private $modelo;
    private $serialNumber;
    private $patrimonio;
    // software 0/2
    private $so;
    private $software;
    private $softwarePrincipal;
    private $softwareAdicionais;
    // Rede 0/5
    private $ip;
    private $nomeServidor;
    private $localLogico;
    private $alias;
    private $monitoramentoWhatsup;
    // Infra 
    private $tensao;
    private $amper;
    // Aquisição/Lotação/Convenio
    private $especificacao;
    // Assistencia Técnica 0/7
    private $processo;
    private $empresa;
    private $telefone;
    private $situacao;
    private $inicioVigencia;
    private $fimVigencia;
    private $valor;
    // informações adicionais 0/1
    private $infoAdicionais;
    // não agrupados
    private $switch;
    private $contrato;
    private $assinatura;
    private $observacao;

    public function __construct() {
        parent::__construct();
    }
	
	// método que faz o mapeamento para auxiliar na geração de SQL baseado no livro
  	// foram feitas alterações para auxiliar na geração de formulários
  	// e para que consiga obter os atributos do objetos com nome diferente da coluna da tabela.
  	// cada chamada de método aqui, penso em criar um objeto para criar uma agregação/composição não sei ainda.
    protected function defineCampos() {
        $this->getCampos()->addCampo("id", "codigo", true, "cod");
        $this->addCamposEquipamentos();
        $this->addCamposHardware();
        $this->addCamposSoftware();
        $this->addCamposRedes();
        $this->addCamposInfra();
        $this->addCamposAssistencia();
        $this->getCampos()->addCampo("infoAdicionais", "Informações Adicionais", true, "outrasi");
        $this->addCamposNaoAgrupados();
    }

    private function addCamposEquipamentos() {
        $this->getCampos()->addCampo("id", "Grupo", true, "grupo");
        $this->getCampos()->addCampo("nome", "id", true, "nome");
        $this->getCampos()->addCampo("dataCadastro", "Data de Recebimento", true, "datac");
        $this->getCampos()->addCampo("dataInstalacao", "Data de Instalação", true, "datai ");
        $this->getCampos()->addCampo("responsavel", "Responsável", true, "resp");
        $this->getCampos()->addCampo("destinacao", "Destinação", true, "destina");
    }

    private function addCamposHardware() {
        // ...
    }

    private function addCamposSoftware() {
        // ..
    }

    private function addCamposRedes() {
        // ..
    }

    private function addCamposInfra() {
        // ..
    }

    private function addCamposAssistencia() {
        // ..
    }

    private function addCamposNaoAgrupados() {
        // ..
    }

    // gets and sets

}

 

Compartilhar este post


Link para o post
Compartilhar em outros sites

A classe acima de cara já viola o 1º principio do S.O.L.I.D. que é a SRP " Single responsibility principle", divida sua classe em domínios.

É melhor ter varias classes desacoplando seu objetos, do que uma tentando fazer tudo.

SOLID – Teoria e Prática – Demo + Vídeo
http://www.eduardopires.net.br/2015/01/solid-teoria-e-pratica/

 

18 horas atrás, targa disse:

Classe deveria ter no máximo 200 linha.

Meus código não passa disso, porem não utilizo Getters And Setters

De uma olhada aqui no post

PHP “deep in”: visibilidade, relevância de atributos e meta programação

Compartilhar este post


Link para o post
Compartilhar em outros sites
Em 22/03/2017 at 18:03, targa disse:
  • Apesar de ter toda a estrutura, isto seria mesmo Orientação a Objeto? Não seria o mesmo que tornar os atributos publico, uma vez que você chama "$classe->atributo"?
 

Continua sendo Orientaçao a Objetos (considerado, por alguns, obscurecimento da POO) . Entretanto, isso faz parte da meta programação.

http://pt.stackoverflow.com/questions/22398/o-que-é-metaprogramação

 

Em 22/03/2017 at 18:03, targa disse:
  • O melhor caminho não seria, refatorar a classe em classes menores ou esta é um boa pratica de programação?
 

Sempre deve ser refatorado em classes menores. O S.O.L.I.D., que já foi mencionado, define várias regras para isso. Outro, que gosto de lembrar, são sobre Object Calisthenics. O conceito é vindouro do Java (do livro The ThoughtWorks Anthology) e foi migrado para o PHP pelo Guilher Branco e Rafael Dohms:

https://pt.slideshare.net/guilhermeblanco/php-para-adultos-clean-code-e-object-calisthenics

 

Em 22/03/2017 at 18:03, targa disse:
  • O que acontece com os recursos de autocompletar, tem com ativá-los?
 

Não há como ativar, pois, do ponto de vista da meta programação, são irrelevantes.

 

Entretanto, apenas dizer que um classe não pode ter setters e getters é muito abrangente. A regra que, na minha opnião, melhor define quando uma classe deve ou não possuir tais métodos é a Tell, don't ask. A qual você pode ler na regra número 9 de Object Calisthenics  ou nos links abaixo:

http://wiki.c2.com/?TellDontAsk

https://pragprog.com/articles/tell-dont-ask

 

Outro ponto, sobre o limite de linhas, acaba sendo uma irrelevante, pois uma vez que tenha em mente como deve-se abstrair uma classe, raramente ela irá ficar "inchada". Com exceções para classes de regras de negócio, que essas podem sim ignorar certas regras devido a alta complexidade.

 

Assim como a meta programação, você irá encontrar N paradigmas com seus prós e contras. Ao meu ver, a meta programação gera agilidade e redução de repetição de código. Entretanto, é obscura quanto ao que um participante pode ou não fazer. Como, por exemplo, documentação a nível de código (PHPDocumentor, IDEs, etc...), correspondência de modelagem (a modelagem é apenas "similar" ao que é implementado).

 

Outro pronto, é sobre como o desenvolvedor espera que o código se comporte e como o sistema pode se comportar. Isso você pode verificar no exemplo abaixo (básico, mas ainda válido):

 

 

Compartilhar este post


Link para o post
Compartilhar em outros sites

  • Conteúdo Similar

    • Por ILR master
      Fala galera.
      Espero que todos estejam bem.
      Seguinte: Tenho um arquivo xml onde alguns campos estão com : (dois pontos), como o exemplo abaixo:
       
      <item>
      <title>
      d sa dsad sad sadasdas
      </title>
      <link>
      dsadas dsa sad asd as dsada
      </link>
      <pubDate>sadasdasdsa as</pubDate>
      <dc:creator>
      d sad sad sa ad as das
      </dc:creator>
      </item>
       
      Meu código:
       
      $link = "noticias.xml"; 
      $xml = simplexml_load_file($link); 
      foreach($xml -> channel as $ite) {     
           $titulo = $ite -> item->title;
           $urltitulo = $ite -> item->link;
           print $urltitulo = $ite -> item->dc:creator;
      } //fim do foreach
      ?>
       
      Esse campo dc:creator eu não consigo ler. Como faço?
       
      Agradeço quem puder me ajudar.
       
      Abs
       
       
    • Por First
      Olá a todos!
       
      Eu estou criando um sistema do zero mas estou encontnrando algumas dificuldades e não estou sabendo resolver, então vim recorrer ajuda de vocês.
      Aqui está todo o meu código: https://github.com/PauloJagata/aprendizado/
       
      Eu fiz um sistema de rotas mas só mostra o conteúdo da '/' não sei porque, quando eu tento acessar o register nada muda.
      E eu também quero que se não estiver liberado na rota mostra o erro de 404, mas quando eu tento acessar um link inválido, nada acontece.
      Alguém pode me ajudar com isso? E se tiver algumas sugestão para melhoria do código também estou aceitando.
       
       
      Desde já, obrigado.
    • Por landerbadi
      Olá pessoal, boa tarde
       
      Tenho uma tabela chamada "produtos" com os seguintes campos (id, produto) e outra tabela chamada "itens" com os seguintes campos (id, prod_01, prod_02, prod_03, prod_04).
       
      Na tabela produtos eu tenho cadastrado os seguintes produtos: laranja, maçã, uva, goiaba, arroz, feijão, macarrão, etc.
       
      Na tabela itens eu tenho cadastrado os itens da seguinte maneira:
       
      1, laranja, uva, arroz, feijão;
      2, maçã, macarrão, goiaba, uva;
      3, arroz, feijão, maçã, azeite
       
      Meu problema é o seguinte: 
      Eu escolho um produto da tabela "produtos", por exemplo "uva".  Preciso fazer uma consulta na tabela "itens" para ser listado todos os registros que contenham o produto "uva" e que todos os demais produtos estejam cadastrados na tabela "produtos".
       
      No exemplo acima seria listado apenas dois registros, pois o terceiro registro não contém o produto "uva". 
       
      Alguém pode me ajudar? Pois estou quebrando a cabeça a vários dias e não consigo achar uma solução.
    • Por landerbadi
      Boa tarde pessoal. Estou tentado fazer uma consulta no banco de dados porém estou tendo dificuldades. Tenho uma tabela chamada "itens" com os seguintes campos: id, item, plural, ativo. Nela tem cadastrado vários itens e seu respectivo plural. No campo ativo eu coloco a letra "S" para informar que esta palavra está ativa no sistema. Por exemplo: 1, casa, casas, S 2, mesa, mesas, S 3, cama, camas, S 4, moto, motos, S 5, rádio, rádios O quinto registro "radio" não está ativo no sistema pois não tem um "S" no campo ativo. E outra tabela chamada "variações" com os seguintes campos (id, item1, item2, item3) com os seguintes registros: 1, casa, camas, moto 2, mesas, casas, radio 3, rádio, cama, mesa Eu preciso fazer uma busca na tabela variações da seguinte maneira: Eu escolho um registro na tabela "itens", por exemplo "casa". Preciso fazer com que o php me liste todos os registros da tabela "variações" que contenham a palavra "casa". Porém se tiver algum registro com a palavra "casas" também tem que ser listado. Neste caso ele irá encontrar dois registros. Agora eu preciso que o php verifique os demais itens e faça a listagem apenas dos item que estão ativos (que contenham um "S" no campo ativo. Neste caso ele irá encontrar apenas um registro, pois o segundo registro contém a palavra "rádio". E "rádio" não está ativo na tabela itens. Como faço isso?
    • Por First
      Olá a todos!
       
      Quando eu tento fazer o login me mostra esse erro "Could not log you in."; Alguém sabe me ajudar a resolver esse problema no meu código?
      <?php require_once("core/init.php"); if (Input::exists()) { if (Token::check(Input::get("token"))) { $validate = new Validate(); $validation = $validate->check($_POST, array( "username" => array("required" => true), "password" => array("required" => true) )); if ($validation->passed()) { $user = new User(); $remember = (Input::get("remember")) === "on" ? true : false; $login = $user->login(Input::get("username"), Input::get("password"), $remember); if ($login) { Session::flash("home", "Welcome back!"); Redirect::to("index.php"); } else { echo "Could not log you in."; } } else { foreach ($validation->errors() as $error) { echo $error."<BR>"; } } } } ?> <form action="" method="POST"> <div class="field"> <label for="username">Username</label> <input type="text" name="username" id="username"> </div> <div class="field"> <label for="password">Password</label> <input type="password" name="password" id="password"> </div> <div class="field"> <label for="remember"> <input type="checkbox" name="remember" id="remember"> Remember me </label> </div> <input type="hidden" name="token" value="<?php echo Token::generate(); ?>"> <input type="submit" value="Log in"> </form>  
       
      Desde já obrigado.
×

Informação importante

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