Discussion :

[Orbeaman]

Bonjour,

Je débute le C++. En reprenant un exemple du livre de Pearson avec quelques modifications (nom des labels essentiellement, j'obtiens le message d'erreur suivant avec Visual Studio Community 2017:

Error	C3867	'std::vector<student,std::allocator<_Ty>>::capacity': non-standard syntax; use '&' to create a pointer to member

Malheureusement, je n'ai pas encore les aptitudes à débugger sous Visual Studio. Pouvez-s'il vous plaît m'aider ? (fichier complet en pièce jointe)

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
 
//display vector properties
template<class T> void display_vector(const std::vector<T>& v);
 
int main()
{
...
}
 
template<class T> void display_vector(const std::vector<T>& v)
{
	std::cout << "max_size() = " << v.max_size() << "\tsize() = " << v.size() << "\tcapacity() = " << v.capacity << "\t" << (v.empty()? "empty": "not empty") << std::endl;
 
	for (unsigned int i = 0; i < v.size(); i++)
		std::cout << v[i] << std::endl;
 
	std::cout << std::endl;
}

Merci par avance !

[jo_link_noir]

Le compilateur dit qu'il faut utiliser & pour récupérer la valeur de la fonction membre capacity. Or tu veux l'appeler et il te manque les parenthèses pour le faire: v.capacity().

2 choses concernant la boucle:

- v.size() retourne un std::size_t, pas un unsigned int.
- il est préférable d'utiliser les boucles sur intervalle: for (T const& x : v) std::cout << x << std::endl;

[Orbeaman]

Merci pour ce retour, j'avais effectivement oublié les () après la méthode capacity, j'ai aussi changé l'itération dans la bouble en ++i et 2 ou 3 autres p'tites choses dans le code sans trop d'importance.

Merci pour l'optimisation des boucles, mais à vrai dire je débute