Why this for loop does not stop

Question

I have been looking at this code for 20min and I can't understand why this loop is infinite, it should print 30 elements.

Some text, because post is mostly code.

The question demonstrates why compiler warnings should not be ignored — Alex Shesterov, Dec 14 '14 at 21:04
'I have been looking at this code for 20min'. Really, how impressive. Please learn how to debug before you write another line of code. SO is already flooded with posts from people who can code OK, but cannot develop software because they don't have the full skillset rquired to progress beyond 'Hello World'. — Martin James, Dec 14 '14 at 23:43

score 1 · Answer 1 · answered Dec 14 '14 at 21:03

This is a common mistake when programming in C(++), early on, people mistake the equality operator == with the assignment operator =.

What you're doing here with the line if (i = 1){ is setting i to 1 on each iteration of the loop, so i never gets any larger.
To fix it, simply change that line to if (i == 1) {

score 0 · Accepted Answer · answered Dec 14 '14 at 21:03

0

I'm guessing it's because if (i = 1) assigns 1toi. You meant to compareif (i == 1)

answered Dec 14 '14 at 21:03

jpw

44,361
6
66
86

score 0 · Answer 3 · answered Dec 14 '14 at 21:04

0

the if statement is wrong. it should be

if(i == 1)

currently on each loop you are assigning i back to 1. Hence the reason why the loop is infinite

answered Dec 14 '14 at 21:04

JayDev

1,340
2
13
21

Why this for loop does not stop

3 Answers3