1

Num backend que possui funções de validação separadas para verificar a validade dos dados, caso esses dados sejam inválidos o procedimento (retornar um 400 ou imprimir algo na tela, por exemplo) deve ocorrer nessa parte de validação ou é uma tarefa para o controller?

No frontend onde tenho uma parte que cuida das requisições ao servidor (serviços), quando é recebido um erro de uma dessas requisições, notificar o usuário é tarefa do próprio serviço ou do controlador?

De quem é a responsabilidade?

exemplo simples com php (a pergunta não é focada em nenhuma linguagem, apenas para exemplo)

main.php:

//Forma 1
isValidPassword("123", "321");

//Forma 2
isValidPassword("123", "321") or die("senhas diferente");

validation.php:

//Forma 1
function isValidPassword($password, $confirm) {
    $password === $confirm or die("senhas diferente");
}

//Forma 2
function isValidPassword($password, $confirm) {
    return $password === $confirm;
}

No frontend a ideia é a mesma, executar um alert() no controller ou na função de validação?

10
  • 1
    Guilherme, não consegui entender muito bem sua pergunta. Mas pelo que eu entendi, todas as funções de validação como campo requerido, valor máximo e minimo, comparar campos, devem ser feito no front-end, devido a velocidade e envio desnecessário ao seu backend, seu backend tem que ser responsavel por todo o processamento da sua regra de negocio. Commented 25/07/2018 às 14:12
  • 1
    Não concordo com o que disse, isso deve ser feito tanto no front (por questão de ux e não performance) como no back, por questão de segurança, alguém poderia facilmente criar requisições com dados inválidos ou mal formatados e o back não pode salvar esses dados
    – Costamilam
    Commented 25/07/2018 às 19:30
  • Voce pode aplicar seguranca no seu backend, que apenas seu front consiga comunicacao com ele, existe diversas maneiras fazer isso. Porem concordo com voce referente a nao deixar salvar dados invalidos, mas se uma seguranca bem construida dificilmente uma requisicao pode ser passada de forma indesejada Commented 25/07/2018 às 19:58
  • Desculpe, mas o @PauloAlexandre está completamente equivocado. Não há como saber de onde vem a requisição, isso é facilmente forjado. Qualquer usuário poderá enviar qualquer valor. Você mesmo pode testar isto usando o cURL, por exemplo, assim poderá enviar qualquer informação. A analise do conteúdo, se está de acordo com o que espera, deve ser feito no servidor. Claro, que como mencionado, pode também fazer no lado do cliente, apenas por questões de otimizar o experiencia do usuário.
    – Inkeliz
    Commented 27/07/2018 às 12:49
  • 1
    Obrigado @Inkeliz, o código é bem básico, só de exemplo mesmo
    – Costamilam
    Commented 30/07/2018 às 20:34

1 Resposta 1

3
+25

Ponto de Vista da Qualidade do Código

Do ponto de vista da qualidade do código, ele deveria utilizar a forma 2. A razão decorre de duas perguntas que eu, programador que herdou o seu código para manutenção, por exemplo, me farei considerando somente o nome da sua função (sem precisar olhar o código dela):

  1. O que eu espero que a sua função isValidPassord faça?
  2. O que eu espero que a sua função isValidPassord retorne?

Bom, pelo nome dela, eu espero que ela verifique se a senha é válida e retorne verdadeiro caso seja válida e falso caso contrário. Na forma 1, a função faz mais do que isso. Ela encerra o script caso a senha não seja válida, e isso pode ser um comportamento considerado inesperado por alguém "lendo" o código.

Tá errado? Não. É algo desejável? Também não, porque esse tipo de coisa (principalmente acumulada em grandes projetos) vai dificultar o trabalho e a saída de férias de colegas ou de você mesmo (a experiência demonstra isso, meu caro).

Ponto de Vista da UX

Do ponto de vista da UX as validações devem ser feitas no FrontEnd sempre que possível. A razão é porque assim um erro do usuário pode ser comunicado muito mais rapidamente, sem ele precisar esperar a comunicação com o servidor. Não costuma ser o caso da validação de senha ou de outras coisas que requerem consulta a um banco de dados, mas é o caso de validações de CPF, formatos de dados, etc. Veja que eu disse "sempre que possível".

Ponto de Vista da Arquitetura

É uma boa ideia separar a validação de qualquer ação diferente dela (regras do negócio, por exemplo) porque você torna os serviços (e eu estou usando de forma geral, a validação também pode ser um serviço) mais atomizados e reutilizáveis. A atomização, aliás, facilita os testes e a garantia de que uma alteração numa função X que use a validação V não vai quebrar essa uma outra função Y que eventualmente também use essa validação V.

3
  • Se alterasse o nome da função para algo mais claro, teria algum problema em relação a qualidade de código? Tem alguma sugestão de nome que não seja muito grande?
    – Costamilam
    Commented 27/07/2018 às 0:37
  • 1
    No seu exemplo você alteraria a função para que? Que tal isValidPasswordOrCommunicateUser? Não soa nada ok né? Talvez algo como ForcePasswordCheck ficasse melhor, mas veja que ao fazer isso você só deixa mais visível o baixo reuso da sua função. De novo: tá errado? Não tá. É bom? A minha experiência diz que não, mas realmente é um tanto discutível... Com o tempo vc melhora o seu entendimento a respeito. :) Commented 27/07/2018 às 0:47
  • 1
    Esqueci de dar a recompensa, mals
    – Costamilam
    Commented 2/08/2018 às 0:25

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 .