2

Como fazer o if do controller retornar TRUE ou FALSE de acordo com o check() do Validator?

Atualmente ele retorna booleano para o validate();, mas eu queria que ele continuasse até o fim do código, para então retornar o booleano.

<?php
# relatorio venda controller
class RelatorioVendaController {
    public function __construct() {
        $form['nome'] = 'Ricardo';
        $form['idade'] = 24;

        if(RelatorioVenda::validate($form)){
            print 'Nenhum erro.';
        }
    }
}

# relatorio venda model
class RelatorioVendaModel extends RelatorioModel {
    public static function validate($form){
        $validators = array(
            "nome" => "required",
            "idade" => "required"
        );
        parent::rules($form, $validators);
    }
}

# relatorio model
class RelatorioModel {
    public static function rules($form, $validators){
        // junta os arrays foreach e executa a validacao
        Validator::check($field, $map['field'], $map['validator']);
    }
}

class Validator {
    public static function check($field, $value, $validator){
        // execute validator(field, value)

        if(no erros)
            return true;
    }
    public function required($field, $value){
        // validator
    }
}
?>
7
  • verifique sua função check, há um erro de sintaxe em if(no erros) Editado: a função rules da classe RelatorioModel precisa de um return também
    – gmsantos
    28/02/2014 às 18:47
  • acho que o correto seria if(no errors) 28/02/2014 às 18:50
  • é apenas marcação, essa classe é conceitual.
    – Ricardo
    28/02/2014 às 18:54
  • 3
    Voce pode esclarecer melhor o que voce quer que o codigo faça? Eu nao entendi a parte de ir até o fim do código. Outra coisa, "orientação a objetos" não significa só ter classes e herança mas também encapsulamento, o que é perdido com as quantas funções estáticas (static). Assim é praticamente como criar funções globais. Você deveria re-organizar o código para se livrar das funções estáticas se possível. 28/02/2014 às 19:55
  • De onde vem esse $form no construtor do Controller? O.o 3/03/2014 às 19:09

2 Respostas 2

1

Você pode usar uma variável!

    $booleano = RelatorioVenda::validate($form);
    if($booleano){
        print 'Nenhum erro.';
    }
    return $booleano;

Não esqueça de usar return para que as funções retornem algum resultado...

... mas lembre-se que a chamada indireta a um construtor via operador new retorna sempre uma nova instância de uma classe, e nunca um booleano ou qualquer outro tipo de resultado.

Portanto você deve mover essa parte do seu código para um outro método da classe.

3
  • 1
    Na verdade, __construct não retorna nada. Não deve haver uma declaração return no corpo do construtor. 3/03/2014 às 19:06
  • Está errado dizer que __construct não retorna nada: stackoverflow.com/a/11904285/370290 - embora, evidentemente esse uso seja completamente inadequado.
    – J. Bruni
    4/03/2014 às 17:37
  • De qualquer maneira, melhorei o enunciado na minha resposta baseado no seu feedback. Obrigado.
    – J. Bruni
    4/03/2014 às 17:42
0

Lembre-se que construtores NÃO POSSUEM RETORNO.

O melhor jeito de fazer o que você pretende é através de exceções:

# relatorio venda controller
class RelatorioVendaController {
    public function __construct($form) { //editei essa linha pra fazer sentido
        $form['nome'] = 'Ricardo';
        $form['idade'] = 24;

        if(RelatorioVenda::validate($form)){
            print 'Nenhum erro.';
        } else {
            throw new Exception('Houve algum erro de validação');
        }
    }
}

Na hora de usar, você faz:

try {
    $controller = new RelatorioVendaController($form); // seja lá de onde vem esse $form;
} catch (Exception $e) {
    // tome as providências necessárias
}
3

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .