Originally Posted by
pheininger
I am going to assume that you are now explicitly initializing review[MAX_REVIEWERS][MAX_OBJECTS] (formerly known as arr[30][30]) to 0.0. [Use names for these things appropriate to your language and the common language of those who will be maintaining the code.]
So change statements in the inner while loop. Get your random reviewer (formerly known as i) and see if that reviewer has already reviewed that object (formerly known as j). If they have, skip the rest of the inner loop. So the if statement is moved and goes from "if (arr[i][j] > 0.0) {" to "if review[reviewer][object] == 0.0) {". So now we know it has not been reviewed. You may want to use a do while loop to make sure that rand() does not return a 0 in the 1/RAND_MAX time that that will happen.
By the way, you may want to make the value 0.0 have the name NOT_REVIEWED and use that for initialization and this check.
.
