Discussion :

[Ucom-C++]

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

Code :

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

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é

[Bayard]

1)

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 :

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

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

Ca compile sans ; à la fin ?

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

[Franck.H]

Sans correction aucune:

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
#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 !

[Bayard]

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

[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 :

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

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

[Ucom-C++]

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

Code :

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

*p=rand()%(100-10+1)+10;

[Franck.H]

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

Franck H >> J'aurais mis

Code :

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

int main(void)

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

Oui c'est vrai