Valeur des variables chars et ints non affichés a la fin :/

#include <stdio.h>

int main (){
    int classe = 0;
    char * classe_name = NULL;
    int hp = 0;

    do {
        printf(
            "Quel est votre classe ?\n"
            "1. Cleric\n"
            "2. Mage\n"
            "3. Voleur\n"
            "4. Guerrier\n"
        );
        scanf("%d", &classe);
        switch (classe) {
            case 1:
                classe_name = "cleric";
                hp = 8;
                break;
            case 2:
                classe_name = "mage";
                hp = 4;
                break;
            case 3:
                classe_name = "voleur";
                hp = 6;
                break;
            case 4:
                classe_name = "guerrier";
                hp = 10;
                break;
            default:
                //system("cls");
                printf("Erreur!");
                getchar();
                getchar();
        }
    } while (!classe_name);
    //system("cls"); .
    printf(
        "Votre classe est %s\n"
        "Vos points de vie : %d hp\n",
        classe_name,
        hp
    );
    return 0;
}

• Corrections de typos diverses (nom de variables, accents manquants)
• Ajout du

  #include

  manquant.
• Conventions de nommage : généralement les variables s'écrivent en minuscule :

• printf

  dans les

  case

  : redondant avec le

  printf

  final.
• Pas besoin de faire plein de

  printf

  à la suite, un seul suffit (un appel de fonction n'est pas complètement gratuit, ce sera donc plus efficace dans l'absolu, même si ici ce sera imperceptible)
• Il faut éviter les labels et les

  goto

  , c'est une mauvaise habitude de programmation et dans 99% des cas on peut s'en passer.
• Il faut éviter la fonction

  system()

  : elle rend ton programme dépendant de la plateforme (ici windows). Pour rendre le programme portable à pour d'autres systèmes d'exploitation (MacOS, Linux), il faudrait s'en passer ou utiliser une librairie qui permet de s'affranchir de ces considérations (e.g.,

  ncurses

  ), voir cette discussion.

&classe

scanf

scanf

#include <stdio.h>

void incrementer(int x) {
    x++;
}

int main(){
    int x = 7;
    printf("%d\n", x); // 7
    incrementer(x);
    printf("%d\n", x); // Pas 8, 7 !
    return 0;
}

&

#include <stdio.h>

void incrementer(int * px) {
    (*px)++;
}

int main(){
    int x = 7;
    printf("%d\n", x); // 7
    incrementer(&x);
    printf("%d\n", x); // 8
    return 0;
}

scanf("%d", &hp)

hp

scanf

hp

&hp

*(&hp)

printf("hp = %d\n", hp);

printf

'\0'

char * s = "toto";

• "crée un bloc de 5 octets, contenant les caractères

  't' 'o' 't' 'o' '\0'

  ,
• puis mémorise dans

  s

  l'adresse de ce bloc,
• puis considère que c'est un pointeur de type

  char *

  (ce qui permet d'avoir

  *s

  ou

  s[i]

  bien définis).

char ClasseN

ClasseN = "Guerrier"

'G'

printf(" ... %s ...", ...)

char *

&ClasseN

ClasseN

printf

struct personnage

• de gérer facilement le cas où tu ajoutes une nouvelle "classe" de personnage, ou où tu changes la quantité de hp sans toucher au code
• de gérer facilement le cas où tu veux ajouter un attribut supplémentaire à une "classe" (en plus des hp : dextérité, points de magie, localisation dans monde, etc.)
• de regrouper toutes les données concernant un joueur dans une structure de données pouvant être renvoyée / remplie par une fonction à laquelle tu passes l'enum correspondant à la classe à créer
• de créer un tableau C ou une autre structure de données statique ou dynamique pour stocker des personnages joueurs et non joueurs et parcourir ces données
• de créer un groupe de personnages en combat et de les passer à une fonction gérant le combat
• etc.

"Guerrier"

const char[8]

char *

$ gcc -Wall -Wextra 37606270.c
37606270.c: In function ‘main’:
37606270.c:4:17: warning: initialization of ‘char’ from ‘char *’ makes integer from pointer without a cast [-Wint-conversion]
  char ClasseN = "Guerrier";
                 ^~~~~~~~~~

-Wall

-Wall -Wextra

$ gcc -Wall -Wextra 37606270.c
37606270.c: In function ‘main’:
37606270.c:7:5: warning: implicit declaration of function ‘printf’ [-Wimplicit-function-declaration]
     printf("Quel est votre classe ?\n");
     ^~~~~~
37606270.c:7:5: warning: incompatible implicit declaration of built-in function ‘printf’
37606270.c:7:5: note: include ‘<stdio.h>’ or provide a declaration of ‘printf’
37606270.c:1:1:
+#include <stdio.h>
 int main ()
37606270.c:7:5:
     printf("Quel est votre classe ?\n");
     ^~~~~~
37606270.c:11:30: error: expected expression before ‘.’ token
     printf("4. Guerrier\n"); .
                              ^
37606270.c:16:19: warning: character constant too long for its type
         ClasseN = 'cleric';
                   ^~~~~~~~
37606270.c:16:19: warning: overflow in conversion from ‘int’ to ‘char’ changes value from ‘1701996899’ to ‘99’ [-Woverflow]
37606270.c:21:9: error: ‘ClassN’ undeclared (first use in this function); did you mean ‘ClasseN’?
         ClassN = 'mage';
         ^~~~~~
         ClasseN
37606270.c:21:9: note: each undeclared identifier is reported only once for each function it appears in
37606270.c:21:18: warning: multi-character character constant [-Wmultichar]
         ClassN = 'mage';
                  ^~~~~~
37606270.c:26:19: warning: character constant too long for its type
         ClasseN = 'voleur';
                   ^~~~~~~~
37606270.c:26:19: warning: overflow in conversion from ‘int’ to ‘char’ changes value from ‘1818588530’ to ‘114’ [-Woverflow]
37606270.c:31:18: warning: character constant too long for its type
        ClasseN = 'guerrier';
                  ^~~~~~~~~~
37606270.c:31:18: warning: overflow in conversion from ‘int’ to ‘char’ changes value from ‘1919509874’ to ‘114’ [-Woverflow]
37606270.c:35:9: warning: implicit declaration of function ‘system’ [-Wimplicit-function-declaration]
         system("cls");
         ^~~~~~
37606270.c:37:9: warning: implicit declaration of function ‘getchar’ [-Wimplicit-function-declaration]
         getchar();
         ^~~~~~~
37606270.c:37:9: note: ‘getchar’ is defined in header ‘<stdio.h>’; did you forget to ‘#include <stdio.h>’?
37606270.c:42:20: error: expected expression before ‘.’ token
     system("cls"); .
                    ^
37606270.c:44:34: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘int *’ [-Wformat=]
     printf("Vos points de vie : %d HP\n", &HP);
                                 ~^        ~~~
                                 %ls

#include <stdio.h>
#include <stdlib.h>
#include <string.h>


int main ()
{
    int classe = 0;
    char ClasseN[256];
    int HP = 0;
    B:
    printf("Quel est votre classe ?\n");
    printf("1. Cleric\n");
    printf("2. Mage\n");
    printf("3. Voleur\n");
    printf("4. Guerrier\n");
    scanf("%d", &classe);
    switch (classe) {
        case 1:
        printf("Vous etes un cleric.");
        strcpy(ClasseN, "cleric");
        HP=8;
        break;
        case 2:
        printf("Vous etes un mage.");
        strcpy(ClasseN, "mage");
        HP=4;
        break;
        case 3:
        printf("Vous etes un voleur.");
        strcpy(ClasseN, "voleur");
        HP=6;
        break;
        case 4:
        printf("Vous etes un guerrier.");
        strcpy(ClasseN, "guerrier");
        HP=10;
        break;
        default:
        system("cls");
        printf("Erreur!");
        getchar();
        getchar();
        goto B;

    }
    system("cls");
    printf("Votre classe est %s\n", &ClasseN);
    printf("Vos points de vie : %d HP\n", HP);
    return 0;
}