Discussion :

[FinalSpirit]

Bonjour,

J'essaye de faire un script pour l'école. Je suis en première année d'info et j'ai pas vu beaucoup de chose en C++ (je suis aux pointeurs pour le moment) et je bug sur un script tout on :

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
34
35
36
37
38
39
40
41
42
 
#include <iostream>
#include <string.h>
using namespace std;
 
const int tailleCodeRevue=4;
const int tailleMaxTitre=100;
 
typedef
  struct
    {
      char codeRevue[tailleCodeRevue];
      int annee,mois;
      char titre[tailleMaxTitre];
    } Article;
 
void lireArticle (Article &art)   // R
{
  string s;
  cout << "Entrez le titre de l'article :\n";
  getline(cin,s);
  art.titre=s;
  cout << "Entreze le code de la revue (3 lettres) :\n";
  cin >> s;
  art.codeRevue=s;
  cout << "Entrez l'année de l'article :\n";
  cin >> art.annee;
  cout << "Entrez le mois de l'article :\n";
  cin >> art.mois;
}
 
void afficherArticle (Article art) // D
{
  cout << "*\t" << art.codeRevue << "\t" << art.mois << "/" << art.annee << "\t" << art.titre << endl;
} 
 
int main ()
{
  Article art;
  lireArticle(art);
  afficherArticle(art);
}

Donc le problème se situe à la ligne :

Code :

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

art.titre=s;

Où il me marque une erreur ainsi qu'à la ligne :

Code :

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

art.codeRevue=s;

Il me dit :
21 ~\articles.cc incompatible types in assignment of `std::string' to `char[100]'
et pour la seconde :
21 ~\articles.cc incompatible types in assignment of `std::string' to `char[4]'

Je ne comprends pas d'où vient l'erreur surtout que c'est le prof qui nous a dit de faire ça

Merci d'avance.

[Luc Hermitte]

La conversion tableau statique (et autres pointeurs sur char) de char vers std::string est implicite via un des constructeurs de std::string ou un des opérateurs d'affectation.
Dans l'autre sens, retour à la case C avec les strncpy. Cf la FAQ.

Le code mélange allègrement C et C++ ... il vaudrait mieux faire un choix. Je ne vois pas l'intérêt de se limiter à une taille fixe dans la structure -- à part à se brider suivant trois axes. Mais bon...

[Laurent Gomila]

http://c.developpez.com/faq/cpp/?pag...TRINGS_to_char

[FinalSpirit]

merci j'ai lu mais je comprends pas et comme je comprends pas le code je peux pas l'utiliser y a pas un truc plus simple ? pour pas que mon prof se demande quoi...?

[Luc Hermitte]

y a pas un truc plus simple ?

Si. Ne pas utiliser de tableaux statiques ou autres char*. Si tu veux mélanger, il n'y a pas plus simple. Et si le code de ton prof était vraiment tel que tu le disais, alors son code était faux, ne pouvant en aucun cas compiler.

[FinalSpirit]

C'est celui ci sans problème... je vais devoir lui dire deux mots
JE peut pas faire directement un :
cin >> art.CodeRevue;

Au moins ce que saisit la personne est directement mis dans la variable....

[Luc Hermitte]

Tu pourrais mais ... cela serait la porte ouverte aux buffer overflows.

[FinalSpirit]

et je viens de tester ça marche pour le code car c'est 3 lettres mais pour le titre ça passe pas car c'est une phrase et rentré une phrase dans une variable de type caractère c'est dur

[FinalSpirit]

Une info complémentaire, c'est spécifié sur le sujet que 'on utilise le codage des chaines de C pas du C++

[BugFactory]

Dans ce cas, il ne faut pas utiliser string mais des char*.

[FinalSpirit]

J'ai trouvé

Bon en effet c'est pas simple de trouver quand on connait tout du C mais bon à mon niveau pour palier le problème je fais :

Code :

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

1
2
3
4
5
6
7
 
  string s;
  cout << "Entrez le titre de l'article :\n";
  getline(cin,s);
  for (int i=0; i<tailleMaxTitre; i++)
    art.titre[i]=s[i];
  art.titre[s.length()]='\0';

Et bon ça marche
Voilà, ben merci quand même, grâce à vos explication j'ai bien compris l'erreur et j'ai pu la résoudre à mon niveau

[Luc Hermitte]

strncpy !! ou encore std::string::copy (si mes souvenirs sont bons).

[FinalSpirit]

On a pas le droit d'utilisé ce qu'on a pas encore vu... strncpy ça me dit vraiment rien donc... :sic:

[Luc Hermitte]

Vu comme ça, c'est sûr.
Au fait, et si la taille de ta chaine est inférieure à tailleMaxTitre ?

[Aurelien.Regat-Barrel]

Tu utilises string, donc vous l'avez vu. Pourquoi utiliser des tableaux de char alors ?

Code :

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

1
2
3
4
5
6
7
8
9
10
const int tailleCodeRevue=4; 
const int tailleMaxTitre=100; 
 
typedef 
  struct 
    { 
      char codeRevue[tailleCodeRevue]; 
      int annee,mois; 
      char titre[tailleMaxTitre]; 
    } Article;

devient

Code :

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

1
2
3
4
5
6
7
8
 
typedef 
  struct 
    { 
      string codeRevue; 
      int annee,mois; 
      string titre; 
    } Article;

note que "struct Article {};" peut remplacer "typedef struct {} Article;"

[FinalSpirit]

C'est le prof qui impose cela... je sais pas pourquoi et je sais que c'est con de sa part mais bon...