For Loop Won't eEecute

Expand Post »

Hi im zee and i'm new to javascript..
The javascript function below..examines if a string value contains only numbers..
My problem is..I can't figure out why it doesn't continue with the 2nd loop and iterate j..please help.!

JavaScript / DHTML / AJAX Syntax (Toggle Plain Text)
<script language="javascript">
<!--
	value = "23e";
	isNumber(value);
 
	function isNumber(value){
		document.write('I\'m inside the function isNumber' + '</br>');
		document.write('The value = ' + value + '</br>' );
 
		document.write('The length of value = ' + value.length);
		var isAllValid=true;						document.write(isAllValid);
		var validChars="0123456789";		document.write('</br>' + 'validChars = ' + validChars + '</br>');
		var current = "";
 
		for(i=0; i<value.length; i++){
			current = value.charAt(i);  
			document.write('current = ' + current + '</br>');
 
				for(j=0; j<validChars.length; j++){	
					document.write('j = ' + j + '</br>');
					document.write('current = ' + current + '  ' + 'validChars = ' + validChars.charAt(j) + '</br>');
					if(current == validChars.charAt(j)){
							document.write('break!');
							break;
					}
					if(j==validChar.length){
							document.write('oops! 1-9 evaluated...no match');
							isAllValid=false;
							break;
					}
				}
 
		}
		return isAllValid;
	}
 
-->
</script>
<script language="javascript">
<!--
	value = "23e";
	isNumber(value);
	
	function isNumber(value){
		document.write('I\'m inside the function isNumber' + '</br>');
		document.write('The value = ' + value + '</br>' );
		
		document.write('The length of value = ' + value.length);
		var isAllValid=true;						document.write(isAllValid);
		var validChars="0123456789";		document.write('</br>' + 'validChars = ' + validChars + '</br>');
		var current = "";
		
		for(i=0; i<value.length; i++){
			current = value.charAt(i);  
			document.write('current = ' + current + '</br>');
			
				for(j=0; j<validChars.length; j++){	
					document.write('j = ' + j + '</br>');
					document.write('current = ' + current + '  ' + 'validChars = ' + validChars.charAt(j) + '</br>');
					if(current == validChars.charAt(j)){
							document.write('break!');
							break;
					}
					if(j==validChar.length){
							document.write('oops! 1-9 evaluated...no match');
							isAllValid=false;
							break;
					}
				}
				
		}
		return isAllValid;
	}
	
-->
</script>

Last edited by poison_bubbles; Aug 30th, 2010 at 10:07 pm. Reason: wrong spelling

Similar Threads

Loop Problem in Python
Problem with EOF and for loop in C
Tried to get PHP inside a Javascript loop to run over and over in PHP
How can I make a loop that changes? in Python
Python loop help in Python

poison_bubbles

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

1 posts
since Aug 2010

Permalink

Sep 1st, 2010

Re: For Loop Won't eEecute

Could you not just use the match method?

JavaScript / DHTML / AJAX Syntax (Toggle Plain Text)
var str="A string";
if(str.match(/^[0-9]+$/)){
   alert("Only Numbers");
} else{
   alert("Not only numbers");
}
// Would alert "Not only numbers"
var str="A string";
if(str.match(/^[0-9]+$/)){
   alert("Only Numbers");
} else{
   alert("Not only numbers");
}
// Would alert "Not only numbers"

avario

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

13 posts
since Apr 2009

Permalink

Sep 1st, 2010

Re: For Loop Won't eEecute

Click to Expand / Collapse Quote originally posted by poison_bubbles ...

Hi im zee and i'm new to javascript..
The javascript function below..examines if a string value contains only numbers..
My problem is..I can't figure out why it doesn't continue with the 2nd loop and iterate j..please help.!
JavaScript / DHTML / AJAX Syntax (Toggle Plain Text)
<script language="javascript">

</script>
<script language="javascript">  </script>

There are 2 problems in this code.
1). line=26, "if(j==validChar.length)" you have variable name as validChars and int this line you used wrongly.

2). line=26, value of j and value of validChars.length will never match.

to optimize this code u can use parse() method.

james chaco

Reputation Points: 10

Solved Threads: 1

Newbie Poster

Offline

24 posts
since Sep 2008

This thread is more than three months old

No one has posted to this discussion for at least three months. Please let old threads die and do not reply to them unless you feel you have something new and valuable to contribute that absolutely must be added to make the discussion complete. Otherwise, please start a new thread in this forum instead.

Previous Thread in JavaScript / DHTML / AJAX Forum Timeline: Javascript to php?

Next Thread in JavaScript / DHTML / AJAX Forum Timeline: Scrolling of images..

DaniWeb Message

Cancel Changes

User Name
Password