Discussion :

[Sylargh]

Bonjour, en executant mon code ci dessous, une erreur m'apparait :
Expression: Vector suscrit out of range ( Le but etant de supprimer les nombres paires dans le second tableau, en utilisant vector et erase )
Je ne comprend pas cette erreur, et solicite de l'aide merci

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
24
25
26
27
28
29
30
31
32
33
#include <iostream>
#include <conio.h>
#include <vector>
 
using namespace std;
 
void main()
{
	const int nMax = 20;
	vector<int> tab(20);
	for (int a = 0; a < nMax; a++)
	{
		tab[a] = rand() % 101;
	}
	cout << "Voici le tableau de 20 entiers donne aleatoirement." << endl;
	for (int a = 0; a < nMax; a++)
	{
		cout<<" | "<< tab[a];
	}
	cout << "\n\nVoici le tableau sans les nombres paires :" << endl;
	for (int a = 0; a < nMax; a++)
	{
		if (tab[a]%2==0)
		{
			tab.erase(tab.begin());
		}
		else
		{
			cout << " | " << tab[a];
		}
	}
	_getch();
}

[Bousk]

Salut,

tu parcours ton tableau sur 20 cases, et en chemin tu supprimes des cases.

[Sylargh]

Je comprend, cependant quand je le diminue s'est automatiquement la derniere valeur qui est supprimée,
pouvez vous m'indiquez comment faire?

[bacelar]

Lisez la documentation.
http://www.cplusplus.com/reference/vector/vector/erase/

Ligne 25, vous êtes sûr d'avoir supprimé le bon élément ???

Après être passé une fois en ligne 25, la taille du tableau est de 19.
Pensez-vous qu'après cela les bornes de la boucle en ligne 21 sont pertinentes ?

[stendhal666]

Le mieux c'est de prendre le réflexe d'utiliser la bibliothèque standard, en particulier les algorithmes:

Code :

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

1
2
3
4
5
6
7
8
9
#include <algorithm>
#include <vector>
 
int main() {
 
  std::vector<int> v = { 0, 1, 2, 3, 4, 5 };
  v.erase( std::remove_if(v.begin(), v.end(), [](int x) { return x % 2 != 0; }) , v.end() );
 
}

std::remove_if(b, e, p) repousse tous les éléments qui satisfont à p au bout du conteneur et renvoie un itérateur qui pointe après le dernier élément pour lequel p est faux.

[Sylargh]

Merci pour vos reponses,

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
24
25
26
27
28
29
#include <iostream>
#include <conio.h>
#include <vector>
#include <algorithm>
 
using namespace std;
 
void main()
{
	int nMax = 20;
	vector<int> tab(nMax);
	for (int a = 0; a < nMax; a++)
	{
		tab[a] = rand() % 101;
	}
	cout << "Voici le tableau de 20 entiers donne aleatoirement." << endl;
	for (int a = 0; a < nMax; a++)
	{
		cout<<" | "<< tab[a];
	}
	cout << "\n\nVoici le tableau sans les nombres paires :" << endl;
	for (int a = 0; a < nMax; a++)
	{
		tab.erase(remove_if(tab.begin(), tab.end(), [](int x){ return x % 2 == 0; }), tab.end());
		nMax = nMax - 1;
			cout << " | " << tab[a];
	}
	_getch();
}

Avec ce code, le but de l'exercice est réalisé, cependant j'ai un dernier probleme, ici ma derniere valeur aleatoire était 83, il devrait donc apparaitre
à la fin, mais il n'y ai pas. Sinon pour les autres valeurs impairs ton astuce marche Merci