C++ - for循環中的值未正確設置

Question

我想設置類三角形的三個對象上的第二個和第三個角的值。如果明確完成的，這看起來是這樣的：

triangle_mesh[0].set_vector_point2(vector_anchors[1]); 
triangle_mesh[1].set_vector_point3(vector_anchors[1]); 
triangle_mesh[1].set_vector_point2(vector_anchors[2]); 
triangle_mesh[2].set_vector_point3(vector_anchors[2]); 
triangle_mesh[2].set_vector_point2(vector_anchors[3]); 
triangle_mesh[0].set_vector_point3(vector_anchors[3]); 

這工作！打印這些三角形，我得到（NB第一個彎道已經設置 - 這是不是一個問題）：

triangle0 is (0, 0, -1) (1, 0, 0) (-0.5, -0.866025, 0) 
triangle1 is (0, 0, -1) (-0.5, 0.866025, 0) (1, 0, 0) 
triangle2 is (0, 0, -1) (-0.5, -0.866025, 0) (-0.5, 0.866025, 0) 

首先，雖然，這是醜陋的，其次，我有超過它必須推廣到案件三個三角形設置。我的代碼是：

for (int longitude = 0; longitude < num_longitudes; longitude++){ 
    SurfaceVector current_anchor = vector_anchors[1 + longitude]; 
    triangle_mesh[longitude].set_vector_point2(current_anchor); 
    triangle_mesh[(longitude + 1) % num_longitudes].set_vector_point3(current_anchor); 
} 

* n.b。 num_longitudes是3 *

我檢查一切我能想到的，但現在當我打印出我的三角形，我得到：

triangle0 is (0, 0, -1) (-0.5, -0.866025, 0) (-0.5, -0.866025, 0) 
triangle1 is (0, 0, -1) (-0.5, -0.866025, 0) (-0.5, -0.866025, 0) 
triangle2 is (0, 0, -1) (-0.5, -0.866025, 0) (-0.5, -0.866025, 0) 

沒有人有任何想法可能是想錯了？

編輯

上的三角形vector_point變量的指針，並設置這樣的：

void set_vector_point1(SurfaceVector vector_point) { vector_point1 = &vector_point; } 

Answer 1

裏面就你的問題：

void set_vector_point1(SurfaceVector vector_point) { vector_point1 = &vector_point; } 

您指向一次臨時（函數調用完成後，vector_point停止存在）。更改它，以便正確複製SurfaceVector。

Answer 2

我會改變：

void set_vector_point1(SurfaceVector vector_point) { vector_point1 = &vector_point; } 

到

void set_vector_point1(SurfaceVector& vector_point) { vector_point1 = &vector_point; } 

或類似的東西。

在當前版本中，vector_point將是您傳遞的任何內容的副本，並且在您存儲指向不再存在的對象的指針後將不再存在。

在第二個中，vector_point是對該函數外部較長壽命的對象的引用。存儲一個指針就沒有問題，因爲當你使用指針時，對象仍然存在。

訣竅是確保對象的壽命比所有指向它的指針長。

此外：

感謝@Nim在下面的評論：

另外在for循環，其中行是：

SurfaceVector current_anchor = vector_anchors[1 + longitude]; 

這應該有可能是一個參考太..目前它也是一個副本。這樣你就可以編輯數組中的實際對象，而不是隨副本一起玩並扔掉它們。所以我會改變這一行爲：

SurfaceVector& current_anchor = vector_anchors[1 + longitude];