question à propos de la validation d'un formulaire

question à propos de la validation d'un formulaire - Javascript/Node.js - Programmation

Marsh Posté le 28-02-2023 à 10:17:08    

Bonjour,
 
je suis entrain de suivre une formation typescript en ligne et je ne comprends pas le choix du formateur concernant une expression booleenne.
 
 

Code :
  1. function validate(validatableInput: Validatable) {
  2.   let isValid = true;
  4.   if (validatableInput.required) {
  5.     isValid = isValid && validatableInput.value.toString().trim().length !== 0;
  6.   }
  8.   if (
  9.     validatableInput.minLength != null &&
  10.     typeof validatableInput.value === "string"
  11.   ) {
  12.     isValid =
  13.       isValid && validatableInput.value.length >= validatableInput.minLength;
  14.   }
  16.   if (
  17.     validatableInput.maxLength != null &&
  18.     typeof validatableInput.value === "string"
  19.   ) {
  20.     isValid =
  21.       isValid && validatableInput.value.length <= validatableInput.maxLength;
  22.   }
  24.   if (
  25.     validatableInput.min != null &&
  26.     typeof validatableInput.value === "number"
  27.   ) {
  28.     isValid = isValid && validatableInput.value >= validatableInput.min;
  29.   }
  31.   if (
  32.     validatableInput.max != null &&
  33.     typeof validatableInput.value === "number"
  34.   ) {
  35.     isValid = isValid && validatableInput.value <= validatableInput.max;
  36.   }
  38.   return isValid;
  39. }


 
Moi inuitivement j'aurais fait tout simplement ca:
 

Code :
  1. function validate(validatableInput: Validatable) {
  2.   let isValid = true;
  4.   if (validatableInput.required) {
  5.     isValid = validatableInput.value.toString().trim().length !== 0;
  6.   }
  8.   if (
  9.     validatableInput.minLength != null &&
  10.     typeof validatableInput.value === "string"
  11.   ) {
  12.     isValid = validatableInput.value.length >= validatableInput.minLength;
  13.   }
  15.   if (
  16.     validatableInput.maxLength != null &&
  17.     typeof validatableInput.value === "string"
  18.   ) {
  19.     isValid =validatableInput.value.length <= validatableInput.maxLength;
  20.   }
  22.   if (
  23.     validatableInput.min != null &&
  24.     typeof validatableInput.value === "number"
  25.   ) {
  26.     isValid = validatableInput.value >= validatableInput.min;
  27.   }
  29.   if (
  30.     validatableInput.max != null &&
  31.     typeof validatableInput.value === "number"
  32.   ) {
  33.     isValid = validatableInput.value <= validatableInput.max;
  34.   }
  36.   return isValid;
  37. }


 
Du coup j'aimerais savoir l'interet de l'approche du formateur.  :heink:  
 
Une idée ?

Message cité 1 fois

---------------
collectionneur de pâtes thermiques
Reply

Marsh Posté le 28-02-2023 à 10:17:08   

Reply

Marsh Posté le 02-03-2023 à 09:31:35    

Je ne vois pas de différence entre ce que t'as mis pour le formateur et ta vision des choses :/
 
Edit : ah oui, j'avais pas vu le isValid &&  :sleep:


Message édité par rufo le 02-03-2023 à 13:26:58

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta
Reply

Marsh Posté le 02-03-2023 à 10:24:20    

tompouss a écrit :

Bonjour,
 
je suis entrain de suivre une formation typescript en ligne et je ne comprends pas le choix du formateur concernant une expression booleenne.
 
 

Code :
  1. function validate(validatableInput: Validatable) {
  2.   let isValid = true;
  4.   if (validatableInput.required) {
  5.     isValid = isValid && validatableInput.value.toString().trim().length !== 0;
  6.   }
  8.   if (
  9.     validatableInput.minLength != null &&
  10.     typeof validatableInput.value === "string"
  11.   ) {
  12.     isValid =
  13.       isValid && validatableInput.value.length >= validatableInput.minLength;
  14.   }
  16.   if (
  17.     validatableInput.maxLength != null &&
  18.     typeof validatableInput.value === "string"
  19.   ) {
  20.     isValid =
  21.       isValid && validatableInput.value.length <= validatableInput.maxLength;
  22.   }
  24.   if (
  25.     validatableInput.min != null &&
  26.     typeof validatableInput.value === "number"
  27.   ) {
  28.     isValid = isValid && validatableInput.value >= validatableInput.min;
  29.   }
  31.   if (
  32.     validatableInput.max != null &&
  33.     typeof validatableInput.value === "number"
  34.   ) {
  35.     isValid = isValid && validatableInput.value <= validatableInput.max;
  36.   }
  38.   return isValid;
  39. }


 
Moi inuitivement j'aurais fait tout simplement ca:
 

Code :
  1. function validate(validatableInput: Validatable) {
  2.   let isValid = true;
  4.   if (validatableInput.required) {
  5.     isValid = validatableInput.value.toString().trim().length !== 0;
  6.   }
  8.   if (
  9.     validatableInput.minLength != null &&
  10.     typeof validatableInput.value === "string"
  11.   ) {
  12.     isValid = validatableInput.value.length >= validatableInput.minLength;
  13.   }
  15.   if (
  16.     validatableInput.maxLength != null &&
  17.     typeof validatableInput.value === "string"
  18.   ) {
  19.     isValid =validatableInput.value.length <= validatableInput.maxLength;
  20.   }
  22.   if (
  23.     validatableInput.min != null &&
  24.     typeof validatableInput.value === "number"
  25.   ) {
  26.     isValid = validatableInput.value >= validatableInput.min;
  27.   }
  29.   if (
  30.     validatableInput.max != null &&
  31.     typeof validatableInput.value === "number"
  32.   ) {
  33.     isValid = validatableInput.value <= validatableInput.max;
  34.   }
  36.   return isValid;
  37. }


 
Du coup j'aimerais savoir l'interet de l'approche du formateur.  :heink:  
 
Une idée ?


 
À la fin de ta fonction isValid sera égal au dernier if dans lequel il sera passé. Si tous tes tests renvoient false, mais le dernier true ton isValid vaudra true.
Celui du formateur opère un et logique entre le true défini à l'init de isValid et le retour de chaque test joué dans chacun des if et donc à partir du moment où y'a un false dans les tests, isValid sera à false au final.


---------------
There's more to life than the boy in that mirror.
Reply

Marsh Posté le 02-03-2023 à 11:55:18    

+1
 
Perso j'aurais mis le isValid dans le if plutôt qu'avec un && dans l'affectation, je trouves ça plus lisible et évite un test non nécessaire, mais chacun voit midi à sa porte...
 
Y'a aussi l'option de mettre un return false à chaque test, plutôt que continuer une série de test qui de toute façon devra renvoyer false à la fin.


---------------
D3
Reply

Marsh Posté le 02-03-2023 à 12:34:06    

Merci pour vos réponses :)
j'aime bien savoir le pq d'un choix, c'est important surtout s'il ya une raison précise.
 
Mais je me suis rendu compte que la question avait déjà été posée dans la rubrique  question & réponse de la formation, mais j'ai pas encore l'habitude de cette plateforme.
 
J'aime bien l'idée de @mechkrut de retourner false à chaque test, ce serait à priori plus lisible, donc je vais tester ca


---------------
collectionneur de pâtes thermiques
Reply

Marsh Posté le 02-03-2023 à 21:02:21    

Moi pour ce genre de tests je préfère avoir un truc dégueulasse qui effectue tous les tests et retiens le résultat de chaque test. Car typiquement c'est la saisie d'un mot de passe où tu tentes une fois, ça ne va pas, tu corriges, tu passes le premier test, ça ne va toujours pas car cette fois tu ne passes pas le second test, etc.
Très pénible, même si normalement en affichant un message d'erreur rappelant tous les tests qui seront effectués pallie souvent d'une manière simple au problème.


---------------
C'est en écrivant n'importe quoi qu'on devient n'importe qui.
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed