| Each function definition can only have one loop. |
Do not mix loops and recursion [LOOP-WITH-RECURSION: 1-4 points]
| A function that contains a loop must not also use recursion. |
If code is only performed at the end of the last iteration of a loop, then it should be written after the loop [LOOP-END: 1 point]
Look at the following example.
int loopdemo(int n)
{
for(int i = 0; i < n; i++)
{
doSomething(i);
if(i == n-1)
{
finish(i+1);
}
}
}
Notice that statement finish(i+1) is only done at the end of the
last iteration of the loop. This function should be written
as follows.
int loopdemo(int n)
{
for(int i = 0; i < n; i++)
{
doSomething(i);
}
finish(n);
}
|
Do not change the value of a for-loop control variable in the loop body [FOR-BODY-CHANGE: 1-3 points]
A for-loop
control variable (any variable that is modified in the third
part of a for-loop heading) should only be modified in the for-loop heading,
not in the body of the loop. For example, do not write
for(i = 1; i < n; i++)
{
...
i = n;
...
}
If you want to break out of a loop, use break or return.
|
Do not change a for-loop end value in the loop without justification [FOR-BODY-END: 1-3 points]
Do not change the end-value of a for-loop just as a way to get
out of the loop. For example,
for(i = 1; i < n; i++)
{
...
if(condition)
{
n = 0;
}
...
}
is not a good way to cause the loop to end. If you want to
exit a for-loop, use break or return.
|
Do not simulate two nested loops or two consecutive loops using a single loop. [NESTED-LOOP-SIMULATION: 2-8 points]
It is possible to do a lot in a single loop by managing variables in
complicated ways. Imagine that you want to do two nested for-loops,
as follows.
for(int i = 0; i < n; i++)
{
for(int j = 0; j < m; j++)
{
doSomething(i,j);
}
}
But that uses two loops, so you simulate it with one loop, as follows.
int i = 0;
int j = 0;
while(i < n)
{
doSomething(i,j);
j++;
if(j == m)
{
j = 0;
i++;
}
}
That is not an acceptable way to avoid two loops in one function.
Just write a function to handle the inner loop.
void doAll(int i)
{
for(int j = 0; j < m; j++)
{
doSomething(i,j);
}
}
Now your loop becomes
for(int i = 0; i < n; i++)
{
doAll(i);
}
|