Discussion :

[jlassira]

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

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

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 ; 
 
}

[r0d]

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�.

[jlassira]

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

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

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

[r0d]

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

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
        ]

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

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 )

[jlassira]

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

[farscape]

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 .

[jlassira]

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 !

[r0d]

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

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

[jlassira]

Non !!

j'ai tout chang� en CString :

la signatuer de la fonction

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
 
CString CCard::getFBC()
{
	return FBC;
}

le CArray aussi :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
 
typedef CArray <CString,CString> FBCList ;

et voila la methode :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

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� !

[r0d]

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 :

S�lectionner tout - Visualiser dans une fen�tre � part

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 :

S�lectionner tout - Visualiser dans une fen�tre � part

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 :

S�lectionner tout - Visualiser dans une fen�tre � part

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.

[jlassira]

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 :

S�lectionner tout - Visualiser dans une fen�tre � part

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

[farscape]

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.

[jlassira]

donc pas d'espoir pour recuperer ce resultat ?

[r0d]

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())... )

[farscape]

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

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

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;
	}
};

[Philippe299]

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 :

S�lectionner tout - Visualiser dans une fen�tre � part

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);