Discussion :

[JeanNoel53]

bonjour
j'ai une liste que je veux initialiser :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

(CAnalysedWord*)(pSeg->pLinguisticSegment->pListWord) = new CAnalysedWord();

et le compilateur me jette :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1>..\..\..\..\Linguistic\Analyser\src\segmentation.cpp(188): error C2106: '='*: l'opérande gauche doit être une l-value

qui m'orientera sera

[ternel]

le C de CAnalizedWord, ce n'est pas pour const, par hasard?

[JeanNoel53]

Je vais vérifier
j'ai un autre résultat
si je ne caste pas le code est:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

pSeg->pLinguisticSegment->pListWord = new CAnalysedWord();

et alors le compilateur dit:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
1>..\..\..\..\Linguistic\Analyser\src\segmentation.cpp(188): error C2440: '='*: impossible de convertir de 'SpecificationLoader::CAnalysedWord *' en 'std::list<_Ty> *'
1>          with
1>          [
1>              _Ty=SpecificationLoader::CAnalysedWord *
1>          ]
1>          Les types pointés n'ont aucun rapport entre eux*; conversion nécessitant reinterpret_cast, cast de style C ou cast de style fonction

vérification faite j'ai mis const au début de la déclaration:
ça ne change rien

[bacelar]

le C de CAnalizedWord, ce n'est pas pour const, par hasard?

@leternel, ne fait pas trop d'assertion sur la "qualité" du code de @JeanNoel53.
Son code, c'est fait à l'arrache sans aucune convention ni rigueur, malgré les plus de 2 ans de mises en garde de ma part dans de multiples forums.
Donc malheureusement sont C, c'est pour "class", bien venu un 1983 avec du "C with class".

Donc @JeanNoel53, pour faire simple, le compilateur t'engueule parce que tu cherches à remplacer une liste de "toto" par un toto.
Désolé, mais là, c'est pas un problème de syntaxe, c'est n'importe quoi.
Qu'est-ce que tu cherches vraiment à faire ? A part faire le code plus sale possible et espérant que le compilateur ni voie rien ?


P.S.: Et tu fais plus de cast à la C dans du code C++, SVP !!!

[JeanNoel53]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

pSeg->pLinguisticSegment->pListWord = new std::list<CAnalysedWord*>();

[ternel]

En effet.

Demeter et les constructeurs, c'est pour décorer?
RAII, c'est pour décorer?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

pSeg->pLinguisticSegment->pListWord = new std::list<CAnalysedWord*>();

Ceci ne devrait même pas être écrit.

le pSeg devrait gérer lui-même la mémoire de son segment linguistique.
Il devrait fournir des services directs, tels que ajouter(CAnalysedWord const&) ou ajouter(CAnalysedWord const*).

le segment linguistique est un détail technique du segment tout court. ou une information parallele (auquel cas, il n'est pas membre).

la liste de mot est un détail technique du segment linguistique, et ne devrait pas être un pointeur.
Je ne vois pas d'utilité à un pointeur alloué de std::list, surtout de pointeurs.