Алгоритм Лианга—Барски. C++

[vladrrom]

Здравствуйте. Нужен взгляд со стороны на код.. Всё вроде сделано по плану, но отсечение отрезка идёт только по нижней части и то не при всех отрезках.

bool Clip(float &x1, float &x2, float &y1, float &y2, float xmin, float ymin, float xmax, float ymax)
{
	float tmin = 0, tmax = 1, i = 1;
	float x11=566, x21, y11, y21;
	
step2:
	if (i > 4)
	{
		x11 = x1 + (x2 - x1)*tmin;
		x21 = x1 + (x2 - x1)*tmax;
		y11 = y1 + (y2 - y1)*tmin;
		y21 = y1 + (y2 - y1)*tmax;
		x1 = x11; y1 = y11; x2 = x21; y2 = y21; return true;
	}
	else
	{
		float Pi, Qi;
		if (i = 1) { Pi = x1 - x2; Qi = x1 - xmin; }
		if (i = 2) { Pi = x2 - x1; Qi = xmax - x1; }
		if (i = 3) { Pi = y1 - y2; Qi = y1 - ymin; }
		if (i = 4) { Pi = y2 - y1; Qi = ymax - y1; }
		if (Pi == 0)
			if (Qi < 0) return false;
			else
			{
				++i; 
				goto step2;
			}
		if (Pi > 0)
		{
			if (tmax > (Qi / Pi)) 
				tmax = Qi / Pi;
		}
		else if (Pi > 0)
		{
			if (tmin < (Qi / Pi)) 
				tmin = Qi / Pi;
		}
		if (tmin > tmax)
			return false;
		else
		{
			++i;  
			goto step2;
		}
	}
}

Задание прикреплено. И полный проект прилагается.

За труд могу заплатить.

[MansMI]

mansmi@rambler.ru пишите

[evg_m]

		if (Pi > 0)
		{
			if (tmax > (Qi / Pi)) 
				tmax = Qi / Pi;
		}
		else if (Pi > 0)   // и КАК это сработает, учитывая первую строку.
		{
			if (tmin < (Qi / Pi)) 
				tmin = Qi / Pi;
		}