mode préféré de remplissage d'un vecteur C++ de structs

Question

Alternative 1, réutiliser une variable temporaire:

Sticker sticker; 
sticker.x = x + foreground.x; 
sticker.y = foreground.y; 
sticker.width = foreground.width; 
sticker.height = foreground.height; 
board.push_back(sticker); 

sticker.x = x + outline.x; 
sticker.y = outline.y; 
sticker.width = outline.width; 
sticker.height = outline.height; 
board.push_back(sticker); 

Alternative 2, la portée de la variable temporaire:

{ 
Sticker sticker; 
sticker.x = x + foreground.x; 
sticker.y = foreground.y; 
sticker.width = foreground.width; 
sticker.height = foreground.height; 
board.push_back(sticker); 
} 

{ 
Sticker sticker; 
sticker.x = x + outline.x; 
sticker.y = outline.y; 
sticker.width = outline.width; 
sticker.height = outline.height; 
board.push_back(sticker); 
} 

Alternative 3, écrire directement à la mémoire vectorielle:

{ 
board.push_back(Sticker()); 
Sticker &sticker = board.back(); 
sticker.x = x + foreground.x; 
sticker.y = foreground.y; 
sticker.width = foreground.width; 
sticker.height = foreground.height; 
} 

{ 
board.push_back(Sticker()); 
Sticker &sticker = board.back(); 
sticker.x = x + outline.x; 
sticker.y = outline.y; 
sticker.width = outline.width; 
sticker.height = outline.height; 
} 

Quelle approche préférez-vous?

Edit: Pour des raisons de cette discussion, on suppose que les missions doivent être effectuées par un à l'extérieur d'un constructeur

Answer 1

option Mon - donner un autocollant constructeur qui prend les paramètres. puis:

board.push_back(Sticker(outline.x, foo.bar, etc.)); 

Edit: code pour illustrer les noms des paramètres constructeur:

#include <iostream> 
using namespace std; 

struct S { 
    int a, b; 
    S(int a, int b) : a(a), b(b) { 
    } 
}; 

int main() {  
    S s(1, 2); 
    cout << s.a << " " << s.b << endl; 
} 

Answer 2

1. Alternative Pourquoi créer un champ juste pour une variable? Il y a généralement une portée englobante à proximité (au minimum, vous devriez garder vos fonctions/procédures de petite taille afin de pouvoir le définir).

Pourquoi? Vous pouvez créer un nom de variable plus court, par ex. st dans ce cas. Puisque l'affectation sera à proximité, il ne devrait pas y avoir de perte de clarté. En fait, il semblera plus simple et plus propre.

En outre, si le vecteur doit être déréférencé/accessible à partir de plusieurs autres niveaux d'indirection, il simplifiera également le code.

Answer 3

board.resize (nombre d'autocollants);

itérer Puis, à travers tout le vecteur et les paramètres définis.

Answer 4

Que diriez-vous WinForms style:

// Class members 

Sticker sticker1; 
Sticker sticker2; 
Board board; 

// Initialization 

void InitBoard() 
{ 
    sticker1.x = x + foreground.x; 
    sticker1.y = foreground.y; 
    sticker1.width = foreground.width; 
    sticker1.height = foreground.height; 

    sticker2.x = x + outline.x; 
    sticker2.y = outline.y; 
    sticker2.width = outline.width; 
    sticker2.height = outline.height; 

    // Add to board 
    board.push_back(sticker1); 
    board.push_back(sticker2); 
}