Ditto to what Hodgman said, but just another observation. For each new tree, you're just adding a position to the previous tree's position. So, you'll never find tree i+1 getting placed on the other side of tree i-1 than tree i. Your values for mp_x and mp_z are always positive. The way OpenGL (and graphics in general) treats transformations is as a series of matrix multiplications. When you translate once, you "move" to a spot. Another translation will, unless you tell it otherwise, start "moving" from that spot. Effectively, after enough trees, you'll be placing them at infinity. Or INT_MAX, anyway.
In short, you have a couple options to not have this happen:
for (int n = 0; n < 10; n++){ int TreeType = (rand()%4)+1; int mp_x = (rand()%10)+1; int mp_z = (rand()%10)+1; glTranslatef(mp_x,0.0f,mp_z); DrawTree(TreeType, 4.0); // undo the previous translation glTranslatef(-mp_x, 0.0f, -mp_z);}// orfor (int n = 0; n < 10; n++){ int TreeType = (rand()%4)+1; int mp_x = (rand()%10)+1; int mp_z = (rand()%10)+1; // start with a new copy of the current transformation matrix glPushMatrix( ); // transform this new matrix glTranslatef(mp_x,0.0f,mp_z); DrawTree(TreeType, 4.0); // don't forget to pop that new matrix off of the matrix stack glPopMatrix( );}
Your problems are probably a combination of the srand(10) issue and this. If not, let us know!
Doesn't look "random" that much doesn't it?
