Discussion :

[ephiros]

Bonsoir à tous !

Je suis débutant en C, et pour m'entrainer j'ai fait un petit programme en mode console, j'aimerais avoir votre avis, vos critiques dessus afin que je puisse m'améliorer, si il y a des variables mal déclarés, des fonctions inutile, une syntaxe qui pourrait être arrangé, bref, vous voyez le principe.

C'est un petit programme permettant d'afficher le code ASCII correspondant à une lettre entré par l'utilisateur, ou l'inverse, la lettre correspondant au code ASCII rentré par l'utilisateur. Une autre fonction permet d'afficher directement la table ASCII complète et par la suite l'enregistrer dans un fichier si l'utilisateur le souhaite.

J'ai mis le code source en pièce jointe.

voilà j'attends vos remarques/critiques

[mith06]

Salut,

Sur la forme:

1: COMMENTE ton code!!!
2: Ton indentation et bizarre.... Par contre tu utilise des espace au lieur des tab ,ça c'est bien.

Sur le fond:

Code :

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

void enregistrer(char *ascii)

->

Code :

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

void enregistrer(const char *ascii)

tu déclare des constante à 33, 100, 126 et pourquoi pas 42?

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
int tableAscii()
{
    const char start    = '~';
    const char stop     = '!';
    char ascii[stop - start] = {};
    char y,i    = 0;
    int choix   = 0;
 
    printf("~~ Table ASCII ~~\n\n\n\n");
    printf("Lettre ------- Dec\n");
 
    for(y = start; y <= stop; y++)
    {   
        ascii[i] = y;
        printf("%c ------------ %d\n", ascii[i], ascii[i]);
        i++;
    }
 
    printf("\n\n1- Oui\n2- Non\n\n");
    printf("Voulez-vous enregistrer ? : ");
    scanf("%d", &choix);
 
    if(choix == 1)
        enregistrer(ascii);
    else
        printf("A bientot !\n");
}

@+

[droggo]

Pei,

Et j'ajoute : pour des fichiers de cette taille minime, il est préférable de mettre le code directement dans le message plutôt que des liens.