Дубликат Struct в C++

Question

Как я могу копировать один struct в другой. В настоящее время следующий код компилируется, но продолжает сбой во время выполнения. Есть лучший способ сделать это?

struct Trip 
{ 
    int startX; 
    int startY; 
    int endX; 
    int endY; 
    int suppress 
}; 

struct Feedback 
{ 
    int startX; 
    int startY; 
    int endX; 
    int endY; 
    int suppress; 
}; 

vector<Trip> tripList; 
vector<Trip> TTMx[288]; 
TTMX[0] = &tripList; 
vector<Feedback> Tripfeed[288]; 

    for(time = 0; time < 288; time++){ 
      for (int trp=0; trp < tripList.size(); trp++) { 

       Tripfeed[time][trp].startX = tripList[trp].startX; 
       Tripfeed[time][trp].startY = tripList[trp].startY; 
       Tripfeed[time][trp].endX = tripList[trp].endX; 
       Tripfeed[time][trp].endY = tripList[trp].endY; 
       Tripfeed[time][trp].suppress = tripList[trp].suppress; 
    } 
    } 

Answer 1

Tripfeed объявлен как массив 288 пустых векторов , Они пусты, потому что это поведение конструктора по умолчанию для векторного класса. Он выходит из строя из-за нарушения доступа при попытке доступа к элементам этих векторов, поскольку векторы пусты. Для каждого вектора вы должны изменить его размер до желаемого размера до доступа к его элементам. Вот правильный код:

struct Feedback 
{ 
    int startX; 
    int startY; 
    int endX; 
    int endY; 
    int suppress; 
}; 

vector<Feedback> Tripfeed[288]; 

for(time = 0; time < 288; time++) 
{ 

    Tripfeed[time].resize (tripList.size()); 

    for (int trp=0; trp < tripList.size(); trp++) 
    { 
     Tripfeed[time][trp].startX = tripList[trp].startX; 
     Tripfeed[time][trp].startY = tripList[trp].startY; 
     Tripfeed[time][trp].endX = tripList[trp].endX; 
     Tripfeed[time][trp].endY = tripList[trp].endY; 
     Tripfeed[time][trp].suppress = tripList[trp].suppress; 
    } 
} 

Answer 2

ваша декларация Tripfeed должна быть

vector<vector<Feedback>> Tripfeed(288, vector<Feedback>(tripList.size(), Feedback())); 

Answer 3

Если у вас есть приличный компилятор с диапазоном на основе цикла доступного:

#include <algorithm> 

// ... 

for (std::vector<Feedback>& f : Tripfeed) 
{ 
    f.resize(tripList.size()); // all vectors in Tripfeed are initially empty 
    std::copy(tripList.begin(), tripList.end(), f.begin()); 
} 

Answer 4

использовать resize(), чтобы выделить достаточное количество памяти для вектора. ваш код должен выглядеть примерно так. Тем не менее, я рекомендую использовать 2D-векторы для объявления Tripfeed (std :: vector < std :: vector < Обратная связь>> Tripfeed).

for(int time = 0; time < 288; time++){ 
    Tripfeed[time].resize(tripList.size()); 
    for (int trp=0; trp < tripList.size(); trp++) { 
     Tripfeed[time][trp].startX = tripList[trp].startX; 
     Tripfeed[time][trp].startY = tripList[trp].startY; 
     Tripfeed[time][trp].endX = tripList[trp].endX; 
     Tripfeed[time][trp].endY = tripList[trp].endY; 
     Tripfeed[time][trp].suppress = tripList[trp].suppress; 
    } 
    }