How do I supress this code rather using many if/else loops

Question

1.00/5 (1 vote)

See more:

we are submitting a registration form, where in the user has to input the details for that we are providing certain alerts.Now the question is instead of using many ifelse loops can we supress the code for checking the validations.(i think we can do using for loop) Would be happy if any anyone puts the code

JavaScript

function validateFields() {

	valid = true;
	if (isEmpty(frm.id)) {
		alert("ID cannot be empty");
		valid = false;

	} else if (isEmpty(frm.name)) {
		alert("Name cannot be empty");
		valid = false;

	}

	else if (isEmpty(frm.age)) {
		alert("Age cannot be empty");
		valid = false;
	}
	else if (isEmpty(frm.mobile)) {
		alert("Mobile cannot be empty");
		valid = false;
	}
	else if (isEmpty(frm.email)) {
		alert("Email cannot be empty");
		valid = false;
	
	} 
	else if (isEmpty(frm.city)) {
		alert("city cannot be empty");
		valid = false;
	}

	else if (isNaN(frm.age.value)) {
		alert("Age should be a number");
		valid = false;
	}

	

	else if (isNaN(frm.mobile.value)) {
		alert("Mobile should be a number");
		valid = false;
	}

	
	return valid;

}

function isEmpty(obj) {
	if (obj.value == "")
		return true;
	else
		return false;
}

Posted 24-Aug-15 21:44pm

Member 11933436

Updated 24-Aug-15 22:13pm

Andy Lanng

v3

Add a Solution

Comments

Andy Lanng 25-Aug-15 3:58am

Please don't code dump. It is considered impolite.
It is always best to add a quick sentence to tell us:
1: What you are trying to achieve
2: What the exact problem is with your code AND where in the code
3: *Tell us if you have debugged the code and what the results were

*Not everyone is aware that you can debug javascript in the browser, so this is easily forgiven

Member 11933436 25-Aug-15 4:07am

we are submitting a registration form, where in the user has to input the details for that we are providing certain alerts.Now the question is instead of using many ifelse loops can we supress the code for checking the validations.(i think we can do using for loop) Would be happy if any anyone puts the code

Andy Lanng 25-Aug-15 4:16am

Thanks ^_^
I updated the question for you. You can do this with the "Improve Question" button above these comments.
You should reply to peoples comments using the "reply" button. You have to hover over the comment to find it, though :S That way we get notified that you have responded.
You'll find you get a lot more responses following these steps.

I have posted a solution. Let me know if it suits your needs ^_^

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Andy Lanng · Accepted Answer · 2015-08-24T22:12:00

There is a way that is a bit of a hack. You can use switch(true):