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 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132
|
#include<stdlib.h>
#include<stdio.h>
/* -ed- pas standard et inutile.
#include<malloc.h>
*/
#include<string.h>
#define TAB 0
#define DBG 1
static char const s[] =
"0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz";
/* -ed-
void main()
main() retourne int. Toujours.
*/
int main (void)
{
/* -ed-
char nomf[20];
char *tab;
int nb, i, j, taille;
FILE *f;
char const s[] =
"0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz";
Il est inutile, voire nuisible, de centraliser les variables. */
char nomf[20];
printf ("donner le nom du fichier \n");
/* -ed-
scanf ("%s", &nomf);
"%s" attend une l'adresse de type char *. Lorsqu'un tableau est
passé en parametre, il est converti en un pointeur sur le premier
element de ce tableau. Le & est donc inutie (et n'est pas du bon type)
*/
scanf ("%s", nomf);
{
FILE *f = fopen (nomf, "w");
/* -ed- une ouverture de fichier peut echoier (NULL) */
if (f != NULL)
{
int taille;
int nb;
/* -ed- ajoute '\n' */
printf ("Donner la taille en caracteres des mots de passe.\n");
scanf ("%d", &taille);
printf ("Donner le nombre de mots de passe à générer. \n");
scanf ("%d", &nb);
/* -ed-
tab = (char *) malloc (sizeof (char) * taille);
le cast est inutile, voire nuisible.
sizeof (char) vaut 1 par définition
La forme sizeof *p * n est preferable.
malloc() peut echouer. Il manque un test (NULL)
Apres reflexion, le tableau de sert a rien. Suppression.
*/
{
#if TAB
char *tab = malloc (sizeof *tab * taille);
if (tab != NULL)
{
#endif
fprintf (f, " ****************** \n");
fprintf (f, " * MOTS DE PASSES * \n");
fprintf (f, " ****************** \n");
printf ("\n");
{
int i;
/* -ed-
for (i = 0; i <= (nb - 1); i++)
ce genre d'expression me donne mal a la tete.
l'expression canonique est
*/
for (i = 0; i < nb; i++)
{
int j;
for (j = 0; j < taille; j++)
{
#if TAB
tab[j] = s[rand () % (sizeof s)];
fprintf (f, "%c", tab[j]);
#else
/* -ed- Bonne methode.
Attention a ne pas deborder du tableau.
sizeof prend le 0 final (hors-sujet).
*/
int c = s[rand () % (sizeof s - 1)];
fprintf (f, "%c", c);
#endif
}
fprintf (f, "\n\n");
}
}
#if TAB
free (tab), tab = NULL;
}
#endif
}
/* -ed- le fichier doit etre ferme proprement. */
fclose (f), f = NULL;
#if DBG
{
char cde[256] = "type ";
strcat (cde, nomf);
system (cde);
}
#endif
}
}
/* -ed- probablement inutile
system ("pause");
*/
/* -ed- parce que main() retourne un int, donner une valeur valide.
0 signifie OK */
return 0;
} |
Partager