while循环为什么会多循环一次？

代码如下：

//坐标转换
	while( 2 <= id)		//如果大于等于2，调用WordCoordinates
	{	
		cos = idParam[id-2][0];
		sin = idParam[id-2][1];
		px  = idParam[id-2][2];
		py  = idParam[id-2][3];	//读cos sin x y 参数【id-2】

		xz = cos * *x - sin * *y  - px * cos + py * sin;
		yz = (cos * *y + sin * *x + px * sin - py * cos)/(cos * cos - sin * sin);

		printf("\ntest\t%d\t%0.2f %0.2f\n",id,xz,yz);//test

		id = id - 1;

		if( 1 < id)
		{
			WordCoordinates(&xz,&yz,id,idParam);	//递归调用转换函数，直到转换成世界坐标。
		}
		else
		{
			*x = xz;
			*y = yz;
			return;
		}
	}
	return;

在这段代码中，当id运行到id = 1 的时候会进入：
*x = xz;
*y = yz;
return;
但是此时并没有结束循环，居然在id = 1 的时候又进入了while循环，并且id 变成了id = 2，请问各位老师这是什么情况？是我的逻辑有问题吗？

7 个解决方案

#1

递归调用吧？

#2

while (2 <= id) 多余了。已经在后面递归处理了，这里不要再做循环。
试试：

//坐标转换
	//while( 2 <= id)		//如果大于等于2，调用WordCoordinates
	cos = idParam[id-2][0];
	sin = idParam[id-2][1];
	px  = idParam[id-2][2];
	py  = idParam[id-2][3];	//读cos sin x y 参数【id-2】
	xz = cos * *x - sin * *y  - px * cos + py * sin;
	yz = (cos * *y + sin * *x + px * sin - py * cos)/(cos * cos - sin * sin);
	printf("\ntest\t%d\t%0.2f %0.2f\n",id,xz,yz);//test
	--id;
	if( 1 < id)
	{
		WordCoordinates(&xz,&yz,id,idParam);	//递归调用转换函数，直到转换成世界坐标。
	}
	else
	{
		*x = xz;
		*y = yz;
	}
	return;

#3

谢谢！您解决我的问题了。但是我十分疑惑的是：
在else中的break，或者 return;不起作用呢？

#4

引用 2 楼 amoyman 的回复:

while (2 <= id) 多余了。已经在后面递归处理了，这里不要再做循环。
试试：
C/C++ code?1234567891011121314151617181920//坐标转换    //while( 2 <= id)        //如果大于等于2，调用WordCoordinates    cos = idParam[id-2][0];    sin =……

您解决了我的问题了

#5

break是跳出与它最近的循环，
return是从函数返回。

你所说的 "在else中的break，或者 return;不起作用"，
建议你在else后加上花括号{}

《高质量C++编程》里，建议不管语句多少，都加上{}
我一直是这样坚持的，没发生过你说的这种情况。

#6

不要使用
while (条件)
更不要使用
while (组合条件)
要使用
while (1) {
 if (条件1) break;
 //...
 if (条件2) continue;
 //...
 if (条件3) return;
 //...
}
因为前两种写法在语言表达意思的层面上有二义性，只有第三种才忠实反映了程序流的实际情况。
典型如：
下面两段的语义都是当文件未结束时读字符
whlie (!feof(f)) {
 a=fgetc(f);
 //...
 b=fgetc(f);//可能此时已经feof了！
 //...
}
而这样写就没有问题：
whlie (1) {
 a=fgetc(f);
 if (feof(f)) break;
 //...
 b=fgetc(f);
 if (feof(f)) break;
 //...
}
类似的例子还可以举很多。

#7

引用 5 楼 amoyman 的回复:

break是跳出与它最近的循环，
return是从函数返回。

你所说的 "在else中的break，或者 return;不起作用"，
建议你在else后加上花括号{}

《高质量C++编程》里，建议不管语句多少，都加上{}
我一直是这样坚持的，没发生过你说的这种情况。

恩 我也一直是这么做的