ouvrire un fichier et modifier

Version imprimable

salut voila mon code pour générer des nombre aléatoirement dans un fichier

Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 void main() { int t[100],*p; FILE* fp; // pointer file if((fp=fopen(C:\\MyFile.txt,"wb"))==NULL) exit(1); for(p=t;p<t+100;p++) *p=(rand()%(50-10+1)+10 //ce forme de (b-a+1)+a [a,b] <->[0,b-a] fwrite(t,sizeofint,100,fp); fclose(fp); }

je veut ds chose plus obtimisé

09/04/2007, 20h58
Bayard

1)

Citation:

je veut ds chose plus obtimisé

Là, il y a en moyenne une faute d'orthographe tout les 2 mots. Le générateur de nombre aléatoire n'est pas celui que l'on croit...

2) Il faut mettre des tabulations. Je ne lis pas de code non tabulé.

3) Je suggère de mettre des accolades après le "if" ainsi qu'après la boucle "for".

4) Il faut utiliser des constantes ("100" en dur partout est regrettable). exit(1) est également à modifier.

5)

Code:

*p=(rand()%(50-10+1)+10

Ca compile sans ; à la fin ?

6) J'ai du mal à trouver une optimisation (en taille j'entends).

Sans correction aucune:

Code:

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
#include <stdio.h>
#include <stdlib.h>
 
 
#define TAILLE 100
 
 
int main (void)
{
   int      t[TAILLE];
   int   *  p  = NULL;
   FILE  *  fp = NULL;
 
 
   if ((fp = fopen ("C:\\MyFile.txt", "wb")) != NULL)
   {
      for (p = t; p < t + TAILLE; p++)
      {
         *p = (rand () % (50 - 10 + 1)) + 10;
      }
 
      fwrite (t, sizeof int, TAILLE, fp);
      fclose (fp);
   }
 
 
   return EXIT_SUCCESS;
}

Tu vois, c'est pas si dur de faire un code propre et de ce fait lisible et agréable pour les autres !

09/04/2007, 22h34
Bayard

Est-il certain que le fwrite est à l'intérieur de la boucle ?
09/04/2007, 22h43
Trap D

Le fwrite n'est pas dans la boucle, et pour avoir du "vrai" aléatoire, il faudrait initialiser le générateur par srand(time(NULL)) par exemple.
Franck H >> J'aurais mis

Code:

int main(void)

mais il est vrai que tu as écrit, "sans correction aucune" ;)
Autre remarque, l'extension ".txt" du fichier est pour le moins abusive, je voudrais bien voir ce que ça donne dans notepad :lol:
09/04/2007, 22h56
Ucom-C++

NN la fonction write n'est pas dans la boucle la boucle s'arrête dans

Code:

*p=rand()%(100-10+1)+10;
10/04/2007, 07h54
Franck.H

Citation:

Envoyé par Bayard

Est-il certain que le fwrite est à l'intérieur de la boucle ?

Oui exacte, c'est ce qui arrive avec des code non mis en forme :lol:

Citation:

Envoyé par Trap D

Franck H >> J'aurais mis

Code:

int main(void)

mais il est vrai que tu as écrit, "sans correction aucune" ;)

Oui c'est vrai :lol: