Discussion :

[Ghalloun]

Bonjour à tous,
j'ai un problème avec le type vector, que je voudrais utiliser dans un petit algorithme qui donne une liste de nombres premiers.
Voici les deux lignes qui posent problème au compilateur :

Code :

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

1
2
3
vector<int> Listepremiers( int );
et
std::cout << "..." <<Listepremiers(1) <<".\n";

Merci d'avance pour votre aide désintéressée.
Ghislain

[Klaim]

Sans message d'erreur (et/ou sans un contexte plus précis) on ne peut pas t'aider.

Mais j'imagine au pif que tu as juste oublier d'inclure les headers <iostream> et <vector>?

[Bakura]

A tout hasard,

Code :

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

1
2
vector<int> Listepremiers;
std::cout << "..." <<Listepremiers[1] <<".\n";

[Davidbrcz]

PO >> Avec le code de Bakura, si t'ajoute pas de donné, ca va faire boom

[Bakura]

Je sais je sais ^^. Mais il va être quand même assez intelligent pour rajouter des choses entre les deux, sinon ça n'a pas de sens .

[loufoque]

std::vector<int> n'a pas d'operator<< défini.

[3DArchi]

std::vector<int> n'a pas d'operator<< défini.

Et alors?
(pour rappel :ostream& operator<<()

[loufoque]

Et alors c'est pour ça qu'il a une erreur.

ListePremiers est une fonction qui prend un entier et qui renvoit un std::vector<int>.

std::cout << ListePremiers(1); tente donc d'invoquer operator<<(std::ostream&, std::vector<int>) qui n'existe pas.

[Spout]

Et alors c'est pour ça qu'il a une erreur.

ListePremiers est une fonction qui prend un entier et qui renvoit un std::vector<int>.

std::cout << ListePremiers(1); tente donc d'invoquer operator<<(std::ostream&, std::vector<int>) qui n'existe pas.

+1, si l'on s'en tient au code fourni par Ghalloun, et pas à celui fourni par Bakura.

[Ghalloun]

J'ajoute une précision. Je me suis aperçu que je voulais allouer dynamiquement mon vecteur Listepremiers.

Il y a principalement deux endroits où ça dérape. Je vous donne les morceaux de code et les messages d'erreur:

Code :

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

std::cout << Listepremiers[1] << ".\n" ;

Avec comme message d'erreur :
pointer to a function used in arithmetic

Et

Code :

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

1
2
3
vector<long> *Liste(long nombre);
...
Liste = malloc (taille * sizeof(long) );

Avec comme messages d'erreur qui s'applique à la deuxième ligne :
assignment of le type de mon vecteur
cannot convert void* in le type de mon vecteur

Je pense qu'il y a peut-être une confusion entre pointeurs et adresses ou un problème de type de mon vecteur, mais je laisse la parole aux spécialistes.

[loufoque]

Code :

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

vector<long> *Liste(long nombre);

déclare une fonction "Liste" qui prend un long et retourne un pointeur vers un vecteur de long.
Tu ne peux certainement*pas* utiliser "Liste" comme lvalue.

[Ghalloun]

déclare une fonction "Liste" qui prend un long et retourne un pointeur vers un vecteur de long.
Tu ne peux certainement*pas* utiliser "Liste" comme lvalue.

Dis-moi Loufoque, pourquoi est-ce que je dois créer une fonction Liste qui renvoie au vecteur, au lieu d'utiliser simplement le vecteur pour stocker mes nombres premiers?

Et pourquoi ma Liste ne peut-elle pas être une lvalue?

Ghalloun

[loufoque]

Tu n'es pas du tout obligé de faire ça, c'est toi qui écris ça...
Si tu expliques clairement ce que tu veux faire, je te donnerai la syntaxe.

[3DArchi]

Ce serait plutôt quelque chose comme ça:

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
int main()
{
   std::vector<long> *L_pMonVecteur;
   long L_lNombre;
   L_lNombre = 4;
// Allocation de ton vecteur:
   L_pMonVecteur = new std::vector<long>(L_lNombre);

// Utilisation de ton vecteur:
   (*L_pMonVecteur)[0]=2;
   (*L_pMonVecteur)[1]=3;
   (*L_pMonVecteur)[2]=5;
   (*L_pMonVecteur)[3]=7;
   std::cout<<"4 premier nombres premiers: "<<std::endl;
   std::copy(
      L_pMonVecteur->begin(),
      L_pMonVecteur->end(),
      std::ostream_iterator<int>(std::cout, " ")
   );
   std::cout<<std::endl;

// Libération de ton vecteur
   delete L_pMonVecteur;

   std::cout<<"OK"<<std::endl;

   return 0;
}

P.S. (loufoque): effectivement, je n'avais pas compris ta remarque car je ne voyais pas le problème avec

Code :

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

1
2
vector<int> Listepremiers;
std::cout << "..." <<Listepremiers[1] <<".\n";

[JolyLoic]

J'ajoute une précision. Je me suis aperçu que je voulais allouer dynamiquement mon vecteur Listepremiers.

Je ne vois vraiment pas à quoi ça pourrait bien servir...

[Ghalloun]

Je te réponds, JolyLoic, au sujet de l'allocation dynamique.
Je pense qu'elle est nécessaire parce que je voudrais créer une fonction qui prend en entrée un entier et qui renvoie en sortie un tableau d'entiers de taille variable, la liste des nombres premiers inférieurs à l'entier d'entrée. Pour cette liste, j'ai choisi le contenant vector.

Dans le code

Code :

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

Liste = malloc (taille * sizeof(long) );

j'ai retiré "Liste =" et il n'y a plus de message d'erreur. Mais je me demande maintenant comment le code sait dans quoi allouer ce que je lui demande d'allouer.

Mais j'ai toujours trois messages d'erreur pour la ligne d'affectation de Liste[i] du code suivant :

Code :

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

1
2
3
4
5
i=0;
for (k=2; k<=n; k++)
  if (Entier[k]==0)
    {*Liste[i]=k;
    i=i+1;}

Les voici :
pointer to a function used in arithmetic
assignment of read only location
cannot convert 'long int' to 'std::vector<long int, std::allocator<long int>>'
in assignment

[JolyLoic]

Un vector est un tableau de taille variable. Il n'y a aucunement besoin de l'allouer dynamiquement pour ça. Après, en interne, il fait certainement lui de l'allocation dynamique, mais ça ne concerne pas l'utilisateur.

Ex d'une fonction qui retourne un tableau contenant tous les entiers entre 1 et n :

Code :

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

1
2
3
4
5
6
7
8
9
vector<int> f(int n)
{
  vector<int> result;
  for (int i=1 ; i<n ;++i)
  {
     result.push_back(i);
  } 
  return result;
}