Discussion :

[virtuadrack]

Bonjour

je possede le fichier "oneliners" (en piece jointe) qui contient des jokes. Je voudrais que mon pogramme en affiche une au hasard, a chaque fois que l'utilisateur tape Y.

Voici mon code:

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
43
44
45
46
47
48
49
50
51
52
53
54
 
# include <iostream>
# include <fstream>
# include <iomanip>
# include <cstdlib>
using namespace std;
# define   INFILE   "oneliners" /* fichier a ouvrir */
 
main ( )
{
  ifstream f;  /* pointeur */
  static long offset [ 100 ]; /* array ou stocker les jokes */
  int c = 0; /* compteur */
  char iline [ 81 ]; 
  char yesno; /* variable pour l'input (Y/N) */
 
  srand (time (NULL)); /* initialisation du generateur */
  int joke; A
 
  f.open ( INFILE );
  if ( ! f.is_open ( ) ) {
    cerr << "Unable to open " << INFILE << "!\n\n";
    exit ( 1 );
    }
 
  /*
   * Boucle pour lecture du fichier 
   */
  while ( f.getline ( iline, 81) ) {
    if ( iline [ 0 ] == '\0') {      /* a chaque espace chaque joke */
      offset [ c ] = f.tellg ( );   /* utiliser ftell pour y assigner une position */
      c ++; /* incremenation du compteur */
      }
    }
 
  for ( c = 0; offset [ c ] > 0; c ++ ) {
    f.clear ( );
    f.seekg ( offset [ c ] ); /* chercher la position ? */
    f.getline ( iline, 81 );   /* lire la ligne suivante */
    cout << setw ( 3 ) << c + 1 << ". " << iline << endl;
  }
 
  cout << "\n You have "<< c << " jokes. \n";
  cout << "\n Do you want hear a joke ? (Y/N) > ";
  cin >> yesno;
 
 while ( yesno == 'Y' || yesno == 'y' ) {
  joke = rand()  % 16 + 1;
  cout << "\n A random joke would be number " << joke << "\n";
  cout << offset[c] << endl;
  cout << "\n\n Do you want hear a joke ? (Y/N) > ";
  cin >> yesno;
 }
}

Je suis bloque sur plusieurs points.

Est ce que, d'abord, l'utilisation du seekg() et tellg() s'accorde avec le principe de mon programme ?

Je ne crois pas bien proceder avec la generation de 1 jusqu'a 16... ( 16 etant le nombre de joke dans le fichier).

Mais je n'arrive pas a afficher une joke au hasard

Dernier point difficile, une joke ne contient pas qu'une ligne elle peut contenir deux. Comment regler ce probleme aussi ?

Merci de votre aide.

[yan]

Bonjour,
as tu regardé un peut la FAQ??

Sinon, t'as methode pour se repositionné dans un fichier semble bonne. TU pourrai aussi lire ton fichier et stocker les blague dans un vector<std::string>.
Quelque conseil:
1- remplace static long offset [ 100 ]; par std::vector<long> offset(100);
2- remplace char iline [ 81 ]; par std::string iline et lit le fichier comme dans la FAQ
http://cpp.developpez.com/faq/cpp/?p...ture_par_ligne
Ici ca semble faux...

Code :

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

1
2
3
  joke = rand()  % 16 + 1;
  cout << "\n A random joke would be number " << joke << "\n";
  cout << offset[c] << endl;

1- joke = rand() % 16 + 1; te donnera un ciffre entre 1 et 16... hors les indice d'un tableau vont de 0 a N-1.
http://c.developpez.com/faq/?page=no..._random_bornes
2- cout << offset[c] << endl; d'ou vien ce c ??? tu calcul un indice aleatoire et tu utilise un variable qui doit etre fausse....

Voila.
Corrige un peu tout cela est cela devrait marcher

[virtuadrack]

merci pour la reponse

mais j'ai des erreus de compilation quand je remplace char iline [ 81 ] par std::string iline et static long offset [ 100 ] par std::vector<long> offset(100);

Et comment stocker les blagues dans vector<std::string> ? Pourrais-tu etre un peu plus explicite ?

D'autres reponses et points de vues sont les bienvenus.

[yan]

merci pour la reponse

mais j'ai des erreus de compilation quand je remplace char iline [ 81 ] par std::string iline et static long offset [ 100 ] par std::vector<long> offset(100);

quelles erreurs as tu?
as tu ajouter

Code :

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

1
2
#include <string>
#include <vector>

Et comment stocker les blagues dans vector<std::string> ? Pourrais-tu etre un peu plus explicite ?

tu fait une lecture ligne as ligne,
http://cpp.developpez.com/faq/cpp/?p...ture_par_ligne
et quand tu as fini une blague, tu l'ajoute as ton std::vector<std::string> avec un push_back

[virtuadrack]

Je n'ai plus les erreurs de compilations...

en fait, je suis quelqu'un qui a travaille 2 semestres avec C...

et c'est vers la fin qu'on nous demande de switcher au C++

et c'est pas chose facile de se familiariser avec la syntaxe, la transition est difficile... surtout quand on a que 4 jours avant la fin

c'est a cause que j'utilisais gcc pour compiler au lieu g++

anyway, il n'y aurait pas une doc qui pourrait m'expliquer comment ecrire la fonction push_back... car c'est la premiere fois que j'ai affaire au std:: et vetor

Comment il y aura une interaction entre l'array offset et la fonction push_back ?

Merci de ton aide.

[yan]

Je n'ai plus les erreurs de compilations...

en fait, je suis quelqu'un qui a travaille 2 semestres avec C...

et c'est vers la fin qu'on nous demande de switcher au C++

et c'est pas chose facile de se familiariser avec la syntaxe, la transition est difficile... surtout quand on a que 4 jours avant la fin

c'est a cause que j'utilisais gcc pour compiler au lieu g++

anyway, il n'y aurait pas une doc qui pourrait m'expliquer comment ecrire la fonction push_back... car c'est la premiere fois que j'ai affaire au std:: et vetor

Comment il y aura une interaction entre l'array offset et la fonction push_back ?

Merci de ton aide.

C'est vrai que ce n'est pas évident de savoir ce qui est plus adapté au C++. Surtout qu'une partie du C est implémenté en C++.

Pour gcc, c'est bizzare, normalement il aurait du appeler g++. Si ton fichier as une bonne extension. comme : .cc .cpp .cxx ....

Pour le vector, un peu coup de google est voila
http://www.cplusplus.com/reference/stl/vector/
En gros, push_back va augmenter la mémoire du tableau et ajouter un élément.

Si t'as d'autre question, n'hésite pas a mettre un peu de code comme exemple. C'est plus facile pour te répondre