Hi,

A probably fairly simple question, but I seem to have usef the c++ 'continue' statement incorrect.

What I do is the following (pseudo code)

for(materials=0;materials<nrMaterials++materials)

{

for(meshes=0;mesh<nrMeshes;++meshes)

{

if(materials[material].index[meshes].uses = false) continue;

// otherwise do thing with this mesh

}

}

What I try to achieve is that 'continue' stops the iteration for meshes[m] and goes to the next iteration in this for loop, meshes[m+1].

From debugging it looks like it's not doing that.

Any ideas/ suggestions on what I might be doing wrong?

You don't have to use the continue statement for this anyway if you rewrite the code as follows:

for(materials=0;materials<nrMaterials++materials)
{
    for(meshes=0;mesh<nrMeshes;++meshes)
    {
        if(materials[material].index[meshes].uses )
        {
            // otherwise do thing with this mesh
        }
   }
}

Thanks, I'll check the real code to see if the '=' is OK in the real code.

Maybe it's also a good idea to do what nightcreature mentioned (also more readable).

++ for NightCreature's post: don't test a boolean against true or false, just test the boolean. The boolean itself *is* true or false. If your bool variables are named sensibly the code is more readable without the comparison. Use the 'not' operator to test for false, i.e. if (!something.isDead()) ...

Got it, it was not the continue statements causing all the unneccesary state changes, but me running the same function for blended mesh instances going through each effect, material and mesh.

The real code was/ is:

			for(mat=0;mat<mRenderQueue.mNrMaterials;++mat)
			{
				if(!mRenderQueue.MatUsesEffect(fx, mat)) continue;
				if(!pD3dscene->PreSelectMaterial(mat, fx)) return false;							// 2x SetFloatArray, 1x SetTexture									   							
				for(m=0;m<mRenderQueue.mMaterialData[mat].nrMeshes;++m)		
				{
					mesh = mRenderQueue.mMaterialData[mat].meshIds[m];
					if(!mRenderQueue.MeshUsesEffect(fx, mesh)) continue;
					if(!pD3dscene->mMeshes[mesh].SetBuffers(mD3ddev)) return false;					// 1x SetStreamSource, 1x SetIndices

I might change the 'continue' style by making it more readble

if(......)

{

do stuff

}

instead of if(!...) continue;

Now off the fix my indices for going through blended mesh instances.