Update camera.cpp

Author: iJeannne

src/world/camera.cpp

@@ -26,24 +26,19 @@ void cg::world::camera::set_position(float3 in_position)
 void cg::world::camera::set_theta(float in_theta)
 {
 	// TODO Lab: 1.04 Implement `cg::world::camera` class
-	theta = in_theta;
+	theta = in_theta * static_cast<float>(M_PI) / 180.f;
 }
 
 void cg::world::camera::set_phi(float in_phi)
 {
 	// TODO Lab: 1.04 Implement `cg::world::camera` class
-	// Ограничим phi, чтобы избежать выворота вверх/вниз
-	const float limit = static_cast<float>(M_PI_2 - 1e-4);
-	if (in_phi > limit) in_phi = limit;
-	if (in_phi < -limit) in_phi = -limit;
-	phi = in_phi;
+	phi = in_phi * static_cast<float>(M_PI) / 180.f;
 }
 
 void cg::world::camera::set_angle_of_view(float in_aov)
 {
 	// TODO Lab: 1.04 Implement `cg::world::camera` class
-	// Ожидается, что сюда приходит значение в градусах из settings — переведём в радианы
-	angle_of_view = in_aov * static_cast<float>(M_PI / 180.0);
+	angle_of_view = in_aov * static_cast<float>(M_PI) / 180.f;
 }
 
 void cg::world::camera::set_height(float in_height)
@@ -75,30 +70,19 @@ void cg::world::camera::set_z_far(float in_z_far)
 const float4x4 cg::world::camera::get_view_matrix() const
 {
 	// TODO Lab: 1.04 Implement `cg::world::camera` class
-	// Вычислим базис камеры из углов
-	const float cth = cosf(theta), sth = sinf(theta);
-	const float cph = cosf(phi),   sph = sinf(phi);
-
-	const float3 dir{
-		cth * cph,
-		sph,
-		sth * cph
-	};
-	const float3 up_world{ 0.f, 1.f, 0.f };
-
-	// right = normalize(cross(up, dir))
+	// Классический look-at: оси камеры и сдвиг
+	const float3 up_world{0.f, 1.f, 0.f};
+	const float3 dir = get_direction();
 	const float3 right = normalize(cross(up_world, dir));
-	// up = cross(dir, right)
 	const float3 up = cross(dir, right);
 
-	// Look-at матрица (row-major): R и t = -R * pos
-	float4x4 V{
+	// Праворукая система; третья строка соответствует -dir
+	return float4x4{
 		{ right.x,        right.y,        right.z,        -dot(right, position) },
 		{ up.x,           up.y,           up.z,           -dot(up, position)    },
 		{ -dir.x,         -dir.y,         -dir.z,          dot(dir, position)   },
 		{ 0.f,            0.f,            0.f,             1.f                  }
 	};
-	return V;
 }
 
 #ifdef DX12
@@ -124,20 +108,14 @@ const DirectX::XMMATRIX camera::get_dxm_mvp_matrix() const
 const float4x4 cg::world::camera::get_projection_matrix() const
 {
 	// TODO Lab: 1.04 Implement `cg::world::camera` class
-	// Перспективная матрица RH с диапазоном глубины [-1,1] в clip (стандартная учебная форма)
-	const float f = 1.0f / tanf(angle_of_view * 0.5f);
-	const float A = f / aspect_ratio;
-	const float B = f;
-	const float C = -(z_far + z_near) / (z_far - z_near);
-	const float D = -(2.f * z_far * z_near) / (z_far - z_near);
-
-	float4x4 P{
-		{ A,   0.f, 0.f,  0.f },
-		{ 0.f, B,   0.f,  0.f },
-		{ 0.f, 0.f, C,    D   },
-		{ 0.f, 0.f, -1.f, 0.f }
+	// Праворукая перспектива с NDC z в [-1,1]
+	const float f = 1.f / tanf(angle_of_view * 0.5f);
+	return float4x4{
+		{ f / aspect_ratio, 0.f, 0.f,  0.f },
+		{ 0.f,               f,   0.f,  0.f },
+		{ 0.f,               0.f,  z_far / (z_near - z_far), -1.f },
+		{ 0.f,               0.f, (z_far * z_near) / (z_near - z_far), 0.f }
 	};
-	return P;
 }
 
 const float3 cg::world::camera::get_position() const
@@ -149,27 +127,24 @@ const float3 cg::world::camera::get_position() const
 const float3 cg::world::camera::get_direction() const
 {
 	// TODO Lab: 1.04 Implement `cg::world::camera` class
-	const float cth = cosf(theta), sth = sinf(theta);
-	const float cph = cosf(phi),   sph = sinf(phi);
-	const float3 dir{ cth * cph, sph, sth * cph };
-	return normalize(dir);
-	
+	// Направление из yaw=theta и pitch=phi; при theta=0, phi=0 — взгляд вдоль -Z
+	return float3{
+		sinf(theta) * cosf(phi),
+		sinf(phi),
+		-cosf(theta) * cosf(phi)
+	};
 }
 
 const float3 cg::world::camera::get_right() const
 {
 	// TODO Lab: 1.04 Implement `cg::world::camera` class
-	const float3 up_world{ 0.f, 1.f, 0.f };
-	const float3 dir = get_direction();
-	return normalize(cross(up_world, dir));
+	return normalize(cross(float3{0.f, 1.f, 0.f}, get_direction()));
 }
 
 const float3 cg::world::camera::get_up() const
 {
 	// TODO Lab: 1.04 Implement `cg::world::camera` class
-	const float3 dir = get_direction();
-	const float3 right = get_right();
-	return cross(dir, right);
+	return cross(get_direction(), get_right());
 }
 const float camera::get_theta() const
 {