C++ vecteurs de classes avec des constructeurs

//Using g++ and ubuntu.
#include <vector>
using namespace std;

Définir une classe:

class foo(){
(...)
foo(int arg1, double arg2);
}

Constructeur:

foo::foo(int arg1, double arg2){ 
(...) //arrays whose length depend upon arg1 and arg2
}

Je voudrais faire quelque chose comme ceci:

vector<foo> bar(10); //error: no matching function for call to 'foo::foo()'
bar[0] = new foo(123, 4.56);
(...)

Une autre méthode (que j'aime moins) est d'utiliser push_back:

vector<foo> bar; //works
bar.push_back(new foo(123, 4.56)); //throws similar error.
//Omitting the "new" compiles but throws a "double free or corruption (fasttop)" on runtime.

Je veux les différents éléments du vecteur d'être construit différemment, donc je ne veux pas utiliser la "séquence Répétitive constructeur".
Ce qui devrait être fait?

OriginalL'auteur Kevin Kostlan | 2010-08-09

  1. 4

    Pourquoi êtes-vous à l'aide de new lorsque aucune dynamique de la mémoire doit être créé? De cours à l'aide de new échoue, il en résulte un foo* quand push_back accepte un foo. (C'est ce que vous avez un vecteur d', après tout.)

    Quel est le problème avec push_back? Si vous souhaitez réserver de la mémoire à l'avant, utilisez reserve(); de fournir un numéro dans le constructeur de vector fait que de nombreuses copies de la deuxième paramètre (qui est implicitement foo(), qui ne fonctionne pas donc vos erreurs), ce qui n'est pas le même que tout simplement réserver de la mémoire.

    Si faire les choses correctement (pas de new) se bloque, la faute est dans votre code et pas de vecteur. Vous n'avez probablement pas écrit une classe qui gère les ressources.* (Rappelez-vous Les Trois Grands, utilisez le copie-et-swap idiome.)

    *je dis cela parce que vous vous dites "//arrays whose length depend upon arg1 and arg2
    ", ce qui je pense signifie que vous avez new[] dans votre classe, quelque part. Sans le Grand les Trois, votre gestion des ressources sera un échec.

    Vous ne devriez pas être la gestion des ressources de toute façon, les classes ont une responsabilité. Cela signifie qu'il doit être un tableau dynamique, ou l'utilisation d'un tableau dynamique, mais pas les deux à gérer et à utiliser un tableau dynamique. Afin de factoriser les ressources dans leur propre classe, et ensuite faire une autre classe (la vôtre) qui les utilise. Un tableau dynamique est un std::vector, de sorte que vous êtes déjà fait avec qui. Tout de temps vous avez besoin d'un tableau dynamique, utilisez un vector; il n'y a jamais une raison de ne pas.

    +1 pour une très bonne réponse. Si je le pouvais, je voudrais vous donner un +1 pour ne pas dire que vous devriez toujours utiliser std::vector, mais que vous devriez toujours utiliser un vecteur.

    OriginalL'auteur GManNickG

  2. 4
    vector<foo> bar(10); //error: no matching function for call to 'foo::foo()'

    C'est un échec parce que le std::vector constructeur que vous appelez est

    explicit vector ( size_type n, const T& value= T(), const Allocator& = Allocator() );

    Comme vous pouvez le voir, il essaye de remplir le vecteur avec les 10 appels au constructeur par défaut de foo qui n'existe pas.

    Aussi, tous vos exemples mettant en vedette new va échouer parce que le vecteur attend un objet de type foo, pas foo *. En outre, l'évolution de vector<foo *> échoue trop, sauf si vous avez manuellement delete chaque membre avant d'effacer le vecteur. Si vous voulez vraiment aller de l'allocation dynamique de la mémoire de l'itinéraire créer un vector< shared_ptr< foo > >. shared_ptr est disponible dans le Bibliothèques Boost ou si votre compilateur comprend TR1 bibliothèques, il va être présent dans le <memory> d'en-tête dans le std::tr1 espace de noms ou si votre compilateur a le C++0x bibliothèques il sera disponible dans le std espace de noms.

    Ce que vous devriez faire est la suivante:

    vector<foo> bar;
bar.reserve(10);
bar.push_back( foo( 1, 2 ) );
...
...
bar.push_back( foo( 10, 20 ) ); //10 times
    Sans doute convient de mentionner que shared_ptr est une partie de l'bibliothèques boost ou C++0x
    Cela seul ne fonctionne pas, mais j'ai besoin d'avoir le big 3.
    Je soupçonne que vous devez utiliser std::vector de sorte que vous n'avez pas. Faire facteur, les ressources provenant de l'utilisation, ne pas faire les deux.

    OriginalL'auteur Praetorian

  3. 0

    std::vector toujours créer des éléments basés sur le constructeur par défaut qui vous n'avez pas de définir dans l'extrait ci-dessus.

    la méthode push_back est confrontée à un double problème parce que vous n'avez pas à gérer le constructeur de copie.

    OriginalL'auteur YeenFei