Bonjour,j'ai crée une classe Voilere. Celle ci contient un tableau de pointeurs sur des objets de type Oiseau. Je dois créer une méthode libere(int rang) qui libere une case du tableau (un Oiseau) et qui décale vers la gauche tous les élément qui étaient à droite de ce rang. voilà ce que j'ai écrit: void Voliere :: libere(int rang) { delete tab[rang]; int i; for(i=rang+1;i

void Voliere :: libere(int rang)
{
           delete tab[rang];
           for(int i = rang+1; i<nbOiseau; i++) tab[i-1]=tab[i];
          nbOiseau--;
}

C++ (@++)

Euh... Désolé j'ai pas compris ce que je devais changer... Peux tu me réexpliquer? Merci

On ne supprime qu'une seule classe Oiseau.
On enlève donc
nbOiseau=nbOiseau-1
de la boucle.

C++ (@++)

Bas tu mets "nbOiseau=nbOiseau-1;" dans ta boucle for, c'est débile. SAKingdomt'a donné la version corrigée.

Oui ça j'ai compris en fait. Le probleme c'est que lorsque supprime le premier oiseau de la liste, je me retrouve bien avec deux oiseaux (au lieu de trois auparavant). Mais le décalage que je fais ne doit pas être bon puisque au lieu d'avoir mes deux oiseaux restant je me retrouve avec un seul oiseau en double. Ex: j'ai Oiseau O1, O2, O3. je supprime 01 (qui disparait bien) mais je me retrouve avec O2 et O2. (merci luthor j'avais compris!!!!)

J'ai compris et trouvé merci encore pour l'aide.

Même avec mon code ?
Dans ce cas, ça signifit que nbOiseau ne contient pas le bon nombre de classes.

C++ (@++)

Oups.
Quel était le problème ?

C++ (@++)