Erreur de constructeur C++. Impossible d'initialiser le tableau de chaînes

Question

Pourquoi ne peut-on pas initialiser mon tableau de chaînes dans mon constructeur? Je reçois l'erreur suivante: Erreur interne du compilateur: Erreur de segmentation | Sur ces deux lignes du constructeur: suits = {"Clubs", "Diamonds", "Hearts", "Spades"}; dénominations = {"Ace", "2", "3", "4", "5", "6", "7", "8", "9", "10", "Jack", "Reine" ","Roi"};

class Card 
    { 
     public: 
     Card(int n); 
     Card(string d, string s); 
     int getNumber(); 
     string getDenomination(); 
     string getSuit(); 
     void setNumber(int n); 
     void setDenomination(string d); 
     void setSuit(string s); 
     void printMe(); 
     void compareMe(Card c); 

     private: 
     int number; 
     string denomiation; 
     string suit; 
     string suits [4]; 
     string denominations [13]; 
    }; 


    Card::Card(int n) 
    { 
     suits = {"Clubs", "Diamonds", "Hearts", "Spades"}; 
     denominations = {"Ace","2","3","4","5","6","7","8","9","10","Jack","Queen","King"}; 

     int denIndex, suitIndex; 

     denIndex = 51 % 13; 
     suitIndex = 51/13; 

     number = n; 
     denomiation = denominations[denIndex]; 
     suit = suits[suitIndex]; 

    } 

Answer 1

utilisation std::vector ou boost::array, un exemple simplifié de votre code est ci-dessous

#include <boost/assign/list_of.hpp> 

#include <boost/array.hpp> 

#include <iostream> 
#include <string> 

class Card 
{ 
public: 
    explicit Card(unsigned n) : 
     _number(n), 
     _suits(boost::assign::list_of("Clubs")("Diamonds")("Hearts")("Spades")), 
     _denominations(boost::assign::list_of("Ace")("2")("3")("4")("5")("6")("7")("8")("9")("10")("Jack")("Queen")("King")) 
    { 

    } 

private: 
    unsigned _number; 
    boost::array<std::string, 4> _suits; 
    boost::array<std::string, 13> _denominations; 
}; 

int 
main() 
{ 
    Card foo(5); 

    return 0; 
} 

Answer 2

Vous ne pouvez utiliser l'initialisation du tableau littéral à la construction, pas l'affectation.

int good[3] = { 1, 2, 3 }; // OK 
int bad[3]; 
bad = { 1, 2, 3 }; // not so good 

Vous devrez copier le tableau manuellement (ou utiliser un récipient comme vector ou boost::array.

Answer 3

A partir du code affiché, je suppose que cela devrait être membre de données statiques les menbres suits et denominations, que est qu'ils auront toujours les mêmes données pour chaque instance Card

Je vous suggère de les rendre statiques et les initialiser à la portée du fichier dans votre fichier cpp (en dehors de votre déclaration de classe).

Déclaration:

class Card 
{ 
    public: 
    Card(int n); 
    Card(string d, string s); 
    int getNumber(); 
    string getDenomination(); 
    string getSuit(); 
    void setNumber(int n); 
    void setDenomination(string d); 
    void setSuit(string s); 
    void printMe(); 
    void compareMe(Card c); 

    private: 
    int number; 
    string denomiation; 
    string suit; 
    static const string suits [4];     // static const 
    static const string denominations [13];  // static const 
}; 

Définition:

const string Card::suits[4] = {"Clubs", "Diamonds", "Hearts", "Spades"}; 
const string Card::denominations[13] = {"Ace","2","3","4","5","6","7","8","9","10","Jack","Queen","King"}; 

EDIT: Comme Robert S. Barnes a déclaré, ce qui les rend static const est encore mieux.

Espérons que ça aide.

Answer 4

Si vous obtenez un ICE (erreur de compilation interne), alors vous devriez vraiment signaler le code à celui qui a écrit votre compilateur. Les compilateurs sont censés imprimer les avertissements et les erreurs, mais pas les erreurs de segmentation.