Code : Which show only one error message on each time.
function isValidFields() {
if (!$("#x").val()) {
utilsService.setErrorClass('x');
$scope.validationMessage.xErrorMessage = true;
} else if (!$("#a").val()) {
utilsService.setErrorClass('a');
$scope.validationMessage.yErrorMessage = true;
} else if (!$("#b").val()) {
utilsService.setErrorClass('b');
$scope.validationMessage.zgErrorMessage = true;
} else if (!$("#c").val()) {
utilsService.setErrorClass('c');
$scope.validationMessage.cgErrorMessage = true;
} else if (!$("#d").val()) {
utilsService.setErrorClass('d');
$scope.validationMessage.dgErrorMessage = true;
} else if (!$("#e").val()) {
utilsService.setErrorClass('e');
$scope.validationMessage.egErrorMessage = true;
} else if (!$("#f").val()) {
utilsService.setErrorClass('f');
$scope.validationMessage.fgErrorMessage = true;
} else {
$scope.validationMessage = {};
return true;
}
}
What are best possible ways of making above code more readable and avoid many if else statement and avoid code duplications.
Aucun commentaire:
Enregistrer un commentaire