comment initialiser un tableau dans un constructeur ?

salut:

soit la classe suivante :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

class CCardList { public:   CCard CurrentCardList[200]; int TotalCheckedCards,TotalIgnoredCards,TotalCards;   CCardList(int); virtual ~CCardList(void);   void AddCardtoList(CCard); void UpdateCheckedCards(); void UpdateIgnoredCards(); };

---

si je veux instancier ma classe en ayant une autre taille pour le tableau " CurrentCardList" comment proceder exactement sachant que cette noucelle taille sera par exmple lu a partir de la console?

y a t il un moyen pour declarer ce tableau d'uen façon dyamique permettant de le cha,ger d'une classe à une autre par exmple sans se soucier d'une taille fixe !!

merci

oui. L'utilisation d'un vector règle tous tes problèmes:

Code:

---

1 2	//dans le .h (déclaration) vector<CCard, CCard>; //vector est l'équivalent std de CVector pour la MFC

---

Ainsi, tu n'a plus qu'à utiliser vector.insert(), vector.erase(), etc. Tu n'as plus à te préoccuper la gestion de la mémoire ni rien.
(plus plus d'infos, vois la msdn: vector, ou CVector pour la mfc).

Hope it helps.

Citation:

---

Code:

---

vector<CCard, CCard>

---

---

La déclaration d'un std::vector n'est pas la même que celle d'un CVector. Donc :

Code:

---

vector<CCard>

---

donc tu suggere que je remplace :

Code:

---

1 2	CCard CurrentCardList[200];

---

par

Code:

---

1 2	vector<CCard,CCardlist>;

---

c'est ca non?

euh, la syntaxe est fausse. C'est de ma faute, j'ai fais une erreur. En fait, il faut avant tout savoir si tu veux utiliser la mfc?

oui mon application est basée sur MFC heureusement :) ...donc..?

Bon, j'ai encore fait une erreur: ce n'est pas CVector, mais CArray que tu dois utiliser. Voir la msdn: (http://msdn2.microsoft.com/en-us/library/4h2f09ct.aspx)

Tu va déclarer ton tableau ainsi:

Code:

---

1 2	CArray <CCard, CCard> m_CurrentCardList;

---

Pour insérer un élément: CurrentCardList.Add();
Pour en supprimer un: CurrentCardList.RemoveAt();
etc.

1- oui mais ou est ce que je passe l'element que je vais inserer? sachant qu'il s'agit d'inserer une classe dans un tableu de classe?

2- pourquoi le choix de vector ne serait il pas approprié?

Citation:

---

Envoyé par jlassira

1- oui mais ou est ce que je passe l'element que je vais inserer? sachant qu'il s'agit d'inserer une classe dans un tableu de classe?

---

Un exemple:

Code:

---

1 2 3 4

MonElement* A = new MonElement(); // création d'un élément // MonElement est une classe quelconque qui existe déjà CArray<MonElement, MonElement> MonTableau; //création du tableau MonTableau.Add(A); //Ajout de l'element A dans le tableau.

---

Pour plus de détails, voir la msdn (CArray) ;)

Citation:

---

Envoyé par jlassira

2- pourquoi le choix de vector ne serait il pas approprié?

---

CArray est l'équivalent MFC du vector de la std. C'est exactement pareil (sauf l'implémentation), sauf que la std est portable. Mais bon, si tu utilises déjà la mfc...

Code:

---

1 2 3 4 5 6 7 8 9 10

include "CCard.h" .class CCardList { public:   //CCard CurrentCardList[200] ; CArray < CCard , CCard> CurrentCardList ; .... }

---

le compilateur me genere une erreur au niveau de "CArray < CCard , CCard> CurrentCardList ;"

et voiçi l'erreur:

Code:

---

1 2 3

CCardList.h(9): error C2143: erreur de syntaxe : absence de ';' avant '<' CCardList.h(9): error C2501: 'CCardList::CArray' : Spécificateurs de type ou de classe de stockage manquants

---

je ne comprends pas !! pourtant c la bonne syntaxe!!

tu as bien inclu le <afxtempl.h>?

oui c'est bon merci ca compile :)...je vais decouvrir ce type maintenant :)

salut,

malgré que j'ai résolu le probleme ..une petite anomaliedemeure un peu inquietante pour moi :)

au fait j'ai essaye d'utiliser CArray pour construire un tableau de string ...comme suit:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

#include "..\include\ccardlist.h" #include "CCard.h" #include <afxtempl.h> typedef CArray <string,string> FBCList;   ... FBCList CCardList::getRemainingCards(CCardList m_CurrentCardList) { FBCList UntreatedCards ;   for ( int i = 0 ; m_CurrentCardList.CurrentCardList.GetSize() ; i++) { CCard a ; if (a.getchecked() == false) UntreatedCards.Add(a.getFBC());   }   return UntreatedCards.Copy() ;   }

---

le compilateur me genere l'erreur suivante au niveau de la ligne return :

Code:

---

1 2 3 4 5 6 7

CCardList.cpp(74): error C2660: 'CArray<TYPE,ARG_TYPE>::Copy' : la fonction ne prend pas 0 arguments with [ TYPE=std::string, ARG_TYPE=std::string ]

---

comment je peux y remedier pour retourner enfin mon tableau de string? ou est l'anomalie? de quel constructeur manquant s'agit il ?

merci

quelle est la signature de CCard ::getFBC() ?

ha aussi, essaie de remplacer des string par des CString. Tant qu'à utiliser la mfc...

[edit]argh, je n'avais pas vu: la méthode Copy prends un argument. Voir la msdn: http://msdn.microsoft.com/library/de...ons_carray.asp[/edit]

Code:

---

1 2 3 4 5

string CCard::getFBC() { return FBC; }

---

c'est la bonen signature :( ..elle retourne string exactement le type d'un elemen,t à ajouter ...je ne comprends pas!

voir mon post précédent

alors c'est ou la faute selon vous? pourquoi je ne peux pas retourncer ce CArray ?

y aura pas de problemes si je passe quand meme un string dans la variable CString? pas de conversion necessaire?

j'ai essaye !! mais en vain !! je ne comprends pas une simple operation comme celle la echoue!!! pourquoi je ne peux pas retourner le resultat comme ca :(!

tu utilises mal la méthode Copy

Citation:

---

Envoyé par ton compilo

'CArray<TYPE,ARG_TYPE>::Copy' : la fonction ne prend pas 0 arguments

---

Voir la msdn: http://msdn.microsoft.com/library/de...ons_carray.asp

crois moi je l'ai enlevé et ca compile toujours pas avec la meme erreur !!:(

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

#include "..\include\ccardlist.h" #include "CCard.h" #include <afxtempl.h> typedef CArray <string,string> FBCList;   ... FBCList CCardList::getRemainingCards(CCardList m_CurrentCardList) { FBCList UntreatedCards ;   for ( int i = 0 ; m_CurrentCardList.CurrentCardList.GetSize() ; i++) { CCard a ; if (a.getchecked() == false) UntreatedCards.Add(a.getFBC());   }   return UntreatedCards ;   }

---

Il est impossible que tu obtiennes la même erreur si tu n'utilises plus la méthode Copy().
1/ Fais un 'clean' pour 'rebuild'
2/ Je ne pense pas que l'erreur vienne de là, mais je te conseille fortement de remplacer les 'string' (std lib) par des CString(mfc). Question d'homogénéité.

crois moi c'est l'erreur que j'aie :)

Code:

---

1 2 3 4 5 6 7 8

CCardList.cpp(75): error C2558: class 'CArray<TYPE,ARG_TYPE>' : pas de constructeur de copie disponible ou le constructeur de copie est déclaré 'explicit' with [ TYPE=std::string, ARG_TYPE=std::string ]

---

je vais essayer de changer en CString et je et ferai savoir.....juste une minute :)

Citation:

---

Envoyé par jlassira

Code:

---

1 2 3 4 5 6 7

CCardList.cpp(74): error C2660: 'CArray<TYPE,ARG_TYPE>::Copy' : la fonction ne prend pas 0 arguments with [ TYPE=std::string, ARG_TYPE=std::string ]

---

---

Citation:

---

Envoyé par jlassira

Code:

---

1 2 3 4 5 6

CCardList.cpp(75): error C2558: class 'CArray<TYPE,ARG_TYPE>' : pas de constructeur de copie disponible ou le constructeur de copie est déclaré 'explicit' with [ TYPE=std::string, ARG_TYPE=std::string ]

---

---

C'est la même erreur?

L'erreur que tu obtiens maintenant (C2558) est intéressante et fréquente lorsqu'on utilise des templates. Elle proviens du fait que que la méthode Add() utilise un constructeur de recopie. En gros, elle prend ton objet en paramètre, et elle essaie d'en faire une copie pour l'insérer. Pour cela, elle appelle le constructeur de recopie de l'objet que tu lui passe (cf http://c.developpez.com/faq/cpp/?pag...tructeur_copie )

donc je ne peux pas retourner le resultat souhaitée c'est ca !!

si tu veux utiliser les string ,il est plus simple de faire un std::vector<string>
avec les CString tu pourras faire std::<CString> ou CArray<CString,CString>.
la stl et les MFC n'utilise pas la meme philosophie.
exemple la stl privilegie les fonctions de conversions (c_str()) plutot que les operateurs de conversions .
:D

Oui je l'ai compris ca ....mais pourquoi quand j'essaie de retourner ce resultat sosu forme de CArray ca me genere cette erreur!! c'est un peu bizarre non!! ...r0d m'a explique un peu le probleme mais je trouve pas vraiement de quoi il s'agit ni comment y remedier :(!

Si, c'est possible de faire comme tu veux faire.

As-tu remplacé tes 'string' par des 'CString'?

Non :( !!

j'ai tout changé en CString :

la signatuer de la fonction

Code:

---

1 2 3 4 5

CString CCard::getFBC() { return FBC; }

---

le CArray aussi :

Code:

---

1 2	typedef CArray <CString,CString> FBCList ;

---

et voila la methode :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

FBCList CCardList::getRemainingCards(CCardList m_CurrentCardList) //void CCardList::getRemainingCards(CCardList m_CurrentCardList) { FBCList UntreatedCards ;   for ( int i = 0 ; m_CurrentCardList.CurrentCardList.GetSize() ; i++) { CCard a ; if (a.getchecked() == false) UntreatedCards.Add(a.getFBC()); // c'est bien un CString   }   return UntreatedCards;   }

---

mais l'erreur persiste :( :
[code]
CCardList.cpp(75): error C2558: class 'CArray<TYPE,ARG_TYPE>'*: pas de constructeur de copie disponible ou le constructeur de copie est déclaré 'explicit'
with
[
TYPE=CString,
ARG_TYPE=CString
]

(/code]

voilà :( !

Bon, pas de panique ;)
Déjà, je viens d'implémenter ce que tu tente de faire chez moi, et ça fonctionne très bien. Voilà ce que j'ai fais:

fichier CCard.h:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

class CCard { public: CCard(); ~CCard();   void SetString(CString str); CString GetMyString();   private: CString m_myString; };

---

fichier CCard.cpp:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

CCard::CCard(){}   CCard::~CCard(){}   void CCard::SetString(CString str) { m_myString = str; }   CString CCard::GetMyString() { return m_myString; }

---

mainframe.cpp:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

... #include "afxtempl.h" ... typedef CArray <CString, CString> FBCList ; ... void UneFonction() { FBCList UntreatedCards ; CCard a; a.SetString("toto"); UntreatedCards.Add(a.GetString()); }

---

Jette un coup d'oeil là-dessus, essaye de comprendre comment ça fonctionne, et peut-être que tu trouvera pourquoi ton code ne compile pas.

Sinon, envoie plus de détails, et n'hésite pas à faire des 'clean' et des 'rebuild'.

Courage, tu vas y arriver :)

Hope it helps.

merci pour ton effort :) ca me fait vraiement plaisir :)

au fait mon probleme ce ,'est pas dans l'ajour d'un element CString mais plutot quand ej veux retourner l'objet FBCList comme resultat!

si tu chaneg la signature de ta methode de façon a ce qu'elle retourne un FBCList et non plus un void et que tu ajoute un:

Code:

---

1 2 3 4 5 6 7 8 9 10

FBCListUneFonction() { FBCList UntreatedCards ; CCard a; a.SetString("toto"); UntreatedCards.Add(a.GetString()); return UntreatedCards }

---

tu verras qeu ca se coince :)

tu ne peux pas ecrire ça .
le CArray ne dispose pas de constructeur de copie.
soit tu l'implementes dans une classe derivée de CArray soit tu retournes une reference .mais a ce moment la l'objet ne doit pas etre temporaire.
:D

donc pas d'espoir pour recuperer ce resultat ?

Si, comme te dit farscape, il y a 2 solutions pour le faire (mais c'est peut-être un peu compliqué, je ne connais pas ton niveau en c++):

1/ tu créés une nouvelle classe dérivée de CArray, et tu lui implémente un constructeur de recopie.

2/ tu utilise une référence sur ton CArray:
au lieu de FBCList UneFonction(), tu fais:
DWORD UneFonctin(FBCList &maListe){...}
rem: le DWORD retourné est juste pour signaler qu'i est important de gérer les erreurs dans ce genre de fonctions.
Dans ce cas, tu pourra utiliser maListe (maListe.Add, maListe.removeAt, etc.) mais il faut faire attention à l'allouer avant l'appel de UneFonction et de ne pas la désallouer n'importe quand (c'est à dire: faire attention ou et comment elle est déclarée/allouée, et ne pas oublier de la désallouer (removeAll())... )

la gestion du contructeur de copie et de l'operateur d'affectation peut etre réalisée facilement :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

template <class TYPE, class ARG_TYPE> class CArrayEx : public CArray<TYPE, ARG_TYPE> {   public: CArrayEx (){} CArrayEx (const CArrayEx &x) { *this=x; }     public: CArrayEx &operator = (const CArrayEx &x) { if(this != &x) { Copy(x); } return *this; } };

---

:D

Si CArray ce comporte de la meme manière que CObArray la méthode CArray::insert prend un pointeur sur l'objet et non l'objet en tant que telle. Vérifie ça si oui tu devras utiliser un pointeur sur ton objet comme suit:

Code:

---

1 2 3 4 5 6 7 8

MonObjet *objet = new MonObjet(); CArray liste; liste.insert(objet); // ou encore: MonObjet objet; CArray liste; liste.insert(&objet);

---