Discussion :

[bl4cksky]

Bonjour,

J'ai un souci de c++ concernant une fonction dont le but est de rassembler des pointeurs dans un seul tableau. Voici la fonction qui pose problème :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
vector<PE*> ATC::comptePE()
{
    vector<PE*> tableau;
    for (int i=0;i<m_PEemp.size();i++)
    {
        for (int j=0;j<m_PEemp[i].getsize();j++)
        {
            tableau.push_back(m_PEemp[i].getPE(j));
        }
    }
    return tableau;
}

Voici l'idée : un objet nommé ATC possède un vecteur d'objets PEemp (emplacements de PE) qui chacun contiennent un vecteur de pointeursr d'objet PE. Cette fonction est créé afin de rassembler des pointeurs de PE en un seul tableau qui servira a comparer les PE. Piti schéma explicatif fait sous paint en pièce jointe.

Cette fonction parcoure les PEemp avec la première boucle sur i puis les PE de chaque PEemp avec la seconde boucle sur j. Je pense que le problème vient de la fonction suivante, située dans la classe PEemp :

Code :

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

PE* PEemp::getPE(int i) const {return m_PE[i];}

Et je comprends paaaaaaaaas *sob*

merci d'avance pour vos réponses.

[Bousk]

Bonsoir,

avec le message d'erreur ça serait mieux...
heureusement la boule de cristal fonctionne ce soir.

Ta fonction getPE est const et retourne un objet membre non const*, y'a fort à parier que le compilo t'insulte de quelque chose à ce niveau.
Si ta fonction est const, le membre retourné doit être const, sinon la méthode ne doit pas être const.

[bl4cksky]

Bonjour,

Désolé j'ai oublié de mentionner l'erreur. Mais le compilateur ne me renvoi aucune erreur, le programme s’exécute sans souci. Seulement au moment d'utiliser la fonction le programme plante et une fenêtre Windows s'affiche, "PGRM.exe a cessé de fonctionner."

Et en utilisant le débugger il me renvoi une segmentation fault. Et en utilisant des cout un peu partout j'ai déterminé que c'est de la que vient l'erreur. En effet si j'écrit

Code :

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

cout << "bla";

avant les deux boucles le bla apparaît dans la console avant le plantage, et il n'apparaît pas si je l'écrit avant le return.

Encore désole de l'oubli ca n'arrivera plus (j'espère)

[Iradrille]

Et en utilisant le débugger il me renvoi une segmentation fault. Et en utilisant des cout un peu partout j'ai déterminé que c'est de la que vient l'erreur. En effet si j'écrit

Code :

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

cout << "bla";

avant les deux boucles le bla apparaît dans la console avant le plantage, et il n'apparaît pas si je l'écrit avant le return.

Hello,

Regardes la stacktrace au moment du plantage, tu sauras exactement d'où ça vient (sans avoir à mettre des cout partout).
Les deux choses qui peuvent poser problèmes dans ces boucles sont tes fonctions : getsize() et getPE().

Il est cependant possible que le plantage viennent de l'utilisation de ton tableau (qui je suppose est faite juste après que tu l'ai récupéré). Tu ne flush pas la sortie, il est donc possible que tes cout ne s'affichent pas immédiatement.
Un pointeur sur un objet invalide est vite arrivé ..

@Bousk, le tableau/vector est un membre de la classe, (vector de PE*), on peut donc retourner une copie d'un élément du vector dans une fonction const (ici un PE*) ? Pas de problèmes de const-correctness ici je pense.

[bl4cksky]

Salut,

D'abord merci de vos réponses rapides.

@Iradrille
Atta atta atta moi y'en a petit mec qui code depuis deux mois. Qu'est-ceque tu veux dire par "Tu ne flush pas la sortie"?

quant à la fonction getsize(), voici son code :

Code :

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

int PEemp::getsize() const {return m_PE.size(); }

Et le tableau est utilisé dans cette fonction du main() :

Code :

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

1
2
3
4
5
6
7
8
9
10
vector<PE*> comptePE()
{
    vector<PE*> PEatc,PEtot;
    PEatc=ATC->comptePE();
    for (int i=0;i<PEatc.size();i++)
    {
        PEtot.push_back(PEatc[i]);          
    }
    return PEtot;
}

Dans l'idéal il y aurait davantage que l'objet ATC comme stockage de PE, mais pour les tests je n'ai mis que celui-là. Avec ensuite une fonction de comparaison, mais que je n'ai pas encore codé. Pour l'instant le résultat est juste stocké dans le main() :

Code :

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

vector<PE*> ToutesLesPE(comptePE());

[Médinoc]

Où est gérée la durée de vie des objets PE?

[Iradrille]

Atta atta atta moi y'en a petit mec qui code depuis deux mois. Qu'est-ceque tu veux dire par "Tu ne flush pas la sortie"?

En fonction du système (et du compilo probablement), les cout ne sont pas instantanés.

Il sont affichés quand il y a un retour à la ligne.

Code :

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

1
2
3
std::cout << "a ";
Sleep(3000); // attente 3sec, pas portable, juste pour l'exemple
std::cout << "b" << std::endl;

Ici "a" et "b" seront probablement affichés en même temps, après l'attente de 3 secondes.

Est-ce que tu utilises tes pointeurs après les avoir rassemblés dans un vector ? (Pour afficher une propriété de tes objets par exemple)
Je pense, comme Médinoc, que ça vient de là, tu dois avoir un pointeur invalide quelque part.

Montre nous le bout de code qui crée tes objets PE pour voir.