Como melhorar meu controle

[LaerteDias 17]

Bom dia, estou conseguindo desenvolver a minha visão e o meu modelo, mas o meu controle acredito não estar otimizado o que devo fazer?

<?php
include_once 'Pegapost.php';
include 'EntidadeEvento.php';
include 'DaoEvento.php';
		// criando uma instância da classe EntidadeEvento.php
		$entidadeevento = new EntidadeEvento($nome_evento,$localizacao);
                $nome_tratado = $entidadeevento.getnome_Evento();
		$localizacao_tratado = $entidadeevento.getLocalizacao();
                //Dao seria aqui em baixo

?>

[Williams Duarte 431]

Porque a Classe de Entidades esta recendo parâmetros pelo construtor?

Outra coisa

$entidadeevento.getLocalizacao

Não é ponto (.) e sim ->

$entidadeevento->getLocalizacao();

para usar o método ou :: para métodos estáticos no PHP

[LaerteDias 17]

o Construtor recebe paremetros, pois estou "setando" as variaveis pelo mesmo, isso é incorreto?

o segundo "erro", foi meu e que acabei enviando um exemplo que estava errado e que estou acostumado ao Java e acabo me atrapalhando de vez em quanto.

[Williams Duarte 431]

o Construtor recebe paremetros, pois estou "setando" as variaveis pelo mesmo, isso é incorreto?

É uma entidade, geralmente usada para getters e setters

Então no java seria this.getNome() no php $this->getNome()

[LaerteDias 17]

Mas mesmo numa entidade não posso tipo substituir os setters por um construtor com parametro, alguns autores passam a ideia que isso é bom?

[Williams Duarte 431]

Acredito que você quebre toda a funcionalidade do teu sistema de mapeamento e dos princípios S.O.L.I.D, dois tenho certeza que estaria violando.

OCP - Open-closed principle

DIP - Dependency Inversion Principle

Saiba mais sobre os Princípios SOLID

http://eduardopires.net.br/2015/01/solid-teoria-e-pratica/


Obs.: Se você quer usar construtores, então use Fluent Interface

[LaerteDias 17]

Entendi, agora como poderei otimizar o controle e não consigo criar uma classe controle e trabalhar com os parametros, como devo fazer para receber os post em um classe controle e poder manipulá-lo?

[Williams Duarte 431]

Posta essas classes amigo

include_once 'Pegapost.php';
include 'EntidadeEvento.php';
include 'DaoEvento.php';

[Gabriel Heming 766]

Acredito que haja uma confusão ainda com as responsabilidade de cada um participante.

No seu controle, você está instanciando uma entidade. Por mais que seja apenas uma linha, não é de responsabilidade do controle. A responsabilidade das entidades de negócio, é da camada de negócio, que é o Model.

Digamos que, em um exemplo simplista, preciso criar uma entidade vazia e utilizá-la na view. O exemplo mais básico, seria.

A View é totalmente dependente da unidade, por isso está no construtor

Class View
{
    private $entity;

    public function __construct(Entity $entity)
    {
        $this->entity = $entity;
    }

    public function show()
    {
        /** código para exibição da entidade **/
    }
}

O Modelo, será responsável por criar a nossa entidade:

Class Model
{
    public function getEmptyEntity() : Entity
    {
        return new Entity();
    }
}

E o Controle, será o responsável pela interpretação da requisição (request) e que chamará os participantes necessários:

Class Controller
{
    public function index()
    {
        $model = new Model();
        $view = new View($model->getEmptyEntity());
        $view->show();
    }
}

É um exemplo um tanto "tosco", mas é bom para entender o que é, de cada qual, a sua responsabilidade.

Como, no seu exemplo, existem as variáveis $nome_tratado e $localizacao_tratado. Não é de responsabilidade do Controle tratá-los, apenas tratá-los.

Entretanto, se você está enviando valor por valor (o que não estaria errado de todo o modo), ao invés de enviar apenas a entidade, não é um problema de responsabilidades e sim de modelagem.

No link abaixo, eu falo sobre a responsabilidade da View em receber e tratar entidade:
http://forum.imasters.com.br/topic/536201-mvc-paginas-dinamicas/?p=2141430

Entretanto, como era pra ser um exemplo rápido, acabei colocando toda a criação da entidade no controle (ignore essa parte). Eu estarei alterando para evitar futuras "discrepâncias". Post do ink atualizado

[LaerteDias 17]

Posta essas classes amigo

include_once 'Pegapost.php';
include 'EntidadeEvento.php';
include 'DaoEvento.php';

//Pegapost.php  classe 
<?php

    $nome_evento = $_POST['nome-evento'];
    $localizacao = $_POST['localizacao'];  

?>

<?php
	class EntidadeEvento{
          private $nome_evento;
	  private $localizacao;
            public function __construct(){
			
			}
            public function setnome_Evento($nome){ 	
				$this->nome_evento =  $nome;
			}
	    public function setLocalizacao($localiza){
				$this->localizacao = $localiza;
			}
            public function getnome_Evento(){
				return $this->nome_evento;
			}
	    public function getLocalizacao(){
				return $this->localizacao;
			}			
        }

?>

[LaerteDias 17]

Uma dúvida não é pelo controle que enviarei os objetos para o DAO?

[Gabriel Heming 766]

Em MVC, DAO é uma camada (tier) do Model.

[LaerteDias 17]

Mas como ficaria a estrutura da Dao e onde a chamaria?

[Gabriel Heming 766]

Mas como ficaria a estrutura da Dao[...]

A estrutura da DAO é dependente apenas do seu padrão. Não há porque mudá-la.

[...]e onde a chamaria?

Você a chamaria dentro da Model, apenas nela.

A existência, ou não, de um storage não deve ser de responsabilidade do controlle. Ele apenas tem o conhecimento de qual Model deve ser chamada para cada requisição.

O tipo de gerenciamento/mapeamento de dados do storage (DAO, DataMapper, TDG/RDG, etc..) também não é de responsabilidade do Controller, apenas do Model.

Em um exemplo simplista, para salvar dados:

Class Controller
{
    public function post()
    {
        if ($_SERVER['REQUEST_METHOD'] == 'POST')
        {
            try
            {
                $post = $_POST;
 
                /** validação/sanitização dos dados do $post omitidos **/

                $model = new Model();
                $model->save($post);
            }
            catch (RuntimeException $exception)
            {
                /** tratamento em caso de exceção **/
            }
        }

        /** demais códigos omitidos **/
    }
}

Model:

class Model
{

    private $pdo;

    public function __construct()
    {
        $this->pdo = new PDO(/** configuração para conexão **/);
        $this->pdo->setAttribute(PDO::ATTR_ERRMODE , PDO::ERRMODE_EXCEPTION); 
    }

    public function save(array $post)
    {
        $entity = $this->getEmptyEntity();
        $entity->setField1($post['field1']);
        $entity->setField2($post['field2']);
        $entity->setField3($post['field3']);
        
        $dao = new EntityDAO($this->pdo);
        $dao->save($dao);
    }

    public function getEmptyEntity() : Entity
    {
        return new Entity();
    }

}

EntityDAO:

class EntityDAO
{

    private $pdo;

    public function __construct(PDO $pdo)
    {
        $this->pdo = $pdo;
    }

    public function save(Entity $entity)
    {
        $sql = 'INSERT INTO entity (field1 , field2 , field3) VALUES (? , ? , ?)'
        $statement = $this->pdo->prepare($sql);
        $statement->execute([$entity->getField1() , $entity->getField2() , $entity->getField3()]);
    }
}

Dessa forma, todas as responsabilidades estão separadas em cada uma de suas camadas.

Mas, veja bem que, esse código acima ainda pode ser muito melhorado e, para um projeto de médio e grande porte, ele carrega alguns problemas bem comuns. Como, por exemplo, utilizar algum container para os objetos que podem ser de uso recorrente(PDO, DAO, etc..) ao invés de criar uma instância nova sempre que necessário.

Outro detalhe é o uso direto de PDO, em alguns casos, outras alternativas podem ser válidas. O padrão DAO também é um padrão simplista, ele é um objeto de acesso aos dados para um storage específico. Caso mudar o tipo de storage, provavelmente deves mudar o DAO também.

Cada padrão vai ter suas vantagens e desvantagens. Nenhum vai te entregar uma solução perfeita, apenas a mais próxima da necessidade do seu projeto.

[LaerteDias 17]

Só mais uma dúvida como no meu form referenciaria o controle para pegar os post, porque no meu projeto estava tentado de uma página inicial com formulário que envia para o controle para poder cadastrar no BD.

[Gabriel Heming 766]

Nesse caso, normalmente é utilizado um padrão denominado FrontController.

[LaerteDias 17]

<form class="form-horinzontal" id="formulario" name="formevento" method="post" action="o que deveria colocar aqui">

No action tem como enviar via post para a classe controle? Essa é minha grande dúvida.

[LaerteDias 17]

Então é no controle que vou tratar os posts, e para que serviria os getters e setters na entidade?

[Gabriel Heming 766]

Para alterar o estado de uma entidade durante o tempo execução (RunTime).

[LaerteDias 17]

Então eu recebo uma valor pelo post e preciso concatenar, anteriormente estava concatenando utilizando os metodos getters e setters, isso pelo que eu entendi está errado.