What's wrong with this simple code

kedarp · 03-12-2013, 01:25 AM

Code:

#!/usr/bin/perl
=cut
Program to sum the squares of first 10 odd nos.
=cut

$count=0;
$i=0;
$sum=0;
while(1){
	
	$i++;
	if($i/2!=0){
		print "$i\n";
		$sum+=$i**2;
		$count++;
	}
	if($i/2==0){
		print "Ok\n";
	}
	if($count==10){
		last;
	}
}
print "The sum of first 10 odd nos. = $sum";

The o/p I get from this code is -

Code:

1
2
3
4
5
6
7
8
9
10
The sum of first 10 odd nos. = 385

I am not able to figure it out. Why is the if statement
not working.

evo2 · 03-12-2013, 01:42 AM

Hi,

I don't code in perl, but would be very surprised if its "/" operator is modulus division. Perhaps you want "%"?

Evo2.

kedarp · 03-12-2013, 02:48 AM

Quote:

Originally Posted by evo2

Hi,

I don't code in perl, but would be very surprised if its "/" operator is modulus division. Perhaps you want "%"?

Evo2.

Oops. Actually these scripting language can really make you forget programming.
I just thought this was a syntax error.

grail · 03-12-2013, 06:03 AM

I know many disagree with me, but I do think it is lazy programming to write an infinite loop to be broken out of when you could simply
test your count variable in the while clause, or better yet, use a for loop as you know then end point.

kedarp · 03-12-2013, 10:13 AM

Quote:

Originally Posted by grail

I know many disagree with me, but I do think it is lazy programming to write an infinite loop to be broken out of when you could simply
test your count variable in the while clause, or better yet, use a for loop as you know then end point.

That was my first try to the program.

kedarp · 03-12-2013, 10:15 AM

Quote:

Originally Posted by grail

I know many disagree with me, but I do think it is lazy programming to write an infinite loop to be broken out of when you could simply
test your count variable in the while clause, or better yet, use a for loop as you know then end point.

That was my first try to the program. But thanks for the suggestion.

kedarp · 03-12-2013, 10:16 AM

Quote:

Originally Posted by grail

I know many disagree with me, but I do think it is lazy programming to write an infinite loop to be broken out of when you could simply
test your count variable in the while clause, or better yet, use a for loop as you know then end point.

That was my first try to the program. But thanks for the suggestion.