Merge pull request #39 from Capstone-C1SE04-TI/OPTIMIZE-VALIDATION-LOGIC

Optimize Validation Logic

Author: huynhducthanhtuan

src/validators/admin.js

@@ -4,9 +4,8 @@ import {
 	returnValidationResult
 } from "./index.js";
 
-const validateSignInBody = async (req, res, next) => {
-	await validateUsername(req);
-	await validatePassword(req);
+const validateSignInBody = async (req) => {
+	await Promise.all([validateUsername(req), validatePassword(req)]);
 	return returnValidationResult(req);
 };
 

src/validators/user.js

@@ -16,53 +16,59 @@ import {
 	returnValidationResult
 } from "./index.js";
 
-const validateSignUpBody = async (req, res, next) => {
-	await validateUsername(req);
-	await validateEmail(req);
-	await validatePhoneNumber(req);
-	await validatePassword(req);
-	await validateConfirmPassword(req);
+const validateSignUpBody = async (req) => {
+	await Promise.all([
+		validateUsername(req),
+		validateEmail(req),
+		validatePhoneNumber(req),
+		validatePassword(req),
+		validateConfirmPassword(req)
+	]);
 	return returnValidationResult(req);
 };
 
-const validateSignInBody = async (req, res, next) => {
-	await validateUsername(req);
-	await validatePassword(req);
+const validateSignInBody = async (req) => {
+	await Promise.all([validateUsername(req), validatePassword(req)]);
 	return returnValidationResult(req);
 };
 
-const validateSubmitEmailBody = async (req, res, next) => {
+const validateSubmitEmailBody = async (req) => {
 	await validateEmail(req);
 	return returnValidationResult(req);
 };
 
-const validateSubmitCodeBody = async (req, res, next) => {
-	await validateEmail(req);
-	await validateCode(req);
+const validateSubmitCodeBody = async (req) => {
+	await Promise.all([validateEmail(req), validateCode(req)]);
 	return returnValidationResult(req);
 };
 
-const validateCreateNewPasswordBody = async (req, res, next) => {
-	await validateEmail(req);
-	await validatePassword(req);
-	await validateConfirmPassword(req);
+const validateCreateNewPasswordBody = async (req) => {
+	await Promise.all([
+		validateEmail(req),
+		validatePassword(req),
+		validateConfirmPassword(req)
+	]);
 	return returnValidationResult(req);
 };
 
-const validateChangePasswordBody = async (req, res, next) => {
-	await validateEmail(req);
-	await validateOldPassword(req);
-	await validateNewPassword(req);
-	await validateNewConfirmPassword(req);
+const validateChangePasswordBody = async (req) => {
+	await Promise.all([
+		validateEmail(req),
+		validateOldPassword(req),
+		validateNewPassword(req),
+		validateNewConfirmPassword(req)
+	]);
 	return returnValidationResult(req);
 };
 
-const validateUpdateProfileBody = async (req, res, next) => {
-	// await validateFullNameOptional(req);
-	await validateEmailOptional(req);
-	await validatePhoneNumberOptional(req);
-	await validateWebsiteOptional(req);
-	await validateAvatarOptional(req);
+const validateUpdateProfileBody = async (req) => {
+	await Promise.all([
+		// validateFullNameOptional(req),
+		validateEmailOptional(req),
+		validatePhoneNumberOptional(req),
+		validateWebsiteOptional(req),
+		validateAvatarOptional(req)
+	]);
 	return returnValidationResult(req);
 };