Ubuntu-fr

Nuliel

deviner mot en C: conseils pour progresser

Bonjour,
J'ai fait un début de jeu doù il faut deviner un mot (chaussette ici) en C, le voici

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

#define EXIT_SUCCESS 0
#define EXIT_FAILURE 1
#define REMAINING_TRIES 10

int main()
{
    char word[] = "chaussette";
    char word2[strlen(word)+1];
    bool not_found = true;
    char letter;
    int remaining_tries = REMAINING_TRIES;
    int c;

    for (int i = 0; i < strlen(word); i++)
    {
        word2[i] = '*';
    }
    word2[strlen(word)] = '\0';
 
    printf("Find the word: \n");
    printf("%s \n", word2);
    do
    {
        scanf("%c", &letter);
        while((c = getchar()) != '\n' && c != EOF);
        if (strchr(word, letter) != NULL)
        {
            for (int i = 0; i < strlen(word); i++)
            {
                if (word[i] == letter)
                {
                    word2[i] = letter;
                }
            }
            printf("%s \n", word2);
            if (strcmp(word, word2) == 0)
            {
                printf("You found the word!\n");
                not_found = false;
            }
        }
        else
        {
            if (remaining_tries == 1)
            {
                printf("Game Over\n");
                return EXIT_FAILURE;
            }
            remaining_tries--;
            printf("Letter is not in the word, %d tries remaining\n", remaining_tries);
        }
        
    } while (not_found);
    
    return EXIT_SUCCESS;
}

Quels conseils pourriez vous donner pour améliorer la lisibilité de ce programme, et permettre une saisie sécurisée aussi?
Si vous pouviez ne pas me donner directement le code corrigé mais plutôt m'aiguiller vers les fonctions plus adaptées (en particulier pour la saisie sécurisée), ce serait top!
(en particulier enlever le while((c = getchar()) != '\n' && c != EOF); que je trouve affreux mais qui a le mérite de fonctionner en attendant une solution propre)
Merci d'avance,
Nuliel

---

[ poster un retour de commande ] [ poster une photo ]

grigouille

Re : deviner mot en C: conseils pour progresser

Il faut mettre srlen(word) en cache dans une variable

---

Debian (xfce) 12
HP LaserJet M1132 MFP

Nuliel

Re : deviner mot en C: conseils pour progresser

Merci grigouille, du coup

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

#define EXIT_SUCCESS 0
#define EXIT_FAILURE 1
#define REMAINING_TRIES 10

int main()
{
    char word[] = "chaussette";
    int length_word = strlen(word);
    char word2[length_word+1];
    bool not_found = true;
    char letter;
    int remaining_tries = REMAINING_TRIES;
    int c;

    for (int i = 0; i < length_word; i++)
    {
        word2[i] = '*';
    }
    word2[length_word] = '\0';
 
    printf("Find the word: \n");
    printf("%s \n", word2);
    do
    {
        scanf("%c", &letter);
        while((c = getchar()) != '\n' && c != EOF);
        if (strchr(word, letter) != NULL)
        {
            for (int i = 0; i < length_word; i++)
            {
                if (word[i] == letter)
                {
                    word2[i] = letter;
                }
            }
            printf("%s \n", word2);
            if (strcmp(word, word2) == 0)
            {
                printf("You found the word!\n");
                not_found = false;
            }
        }
        else
        {
            if (remaining_tries == 1)
            {
                printf("Game Over\n");
                return EXIT_FAILURE;
            }
            remaining_tries--;
            printf("Letter is not in the word, %d tries remaining\n", remaining_tries);
        }
        
    } while (not_found);
    
    return EXIT_SUCCESS;
}

C'est vrai que c'est pas utile de calculer 15 fois la longueur du texte fixé.

---

[ poster un retour de commande ] [ poster une photo ]

NicoApi73

Re : deviner mot en C: conseils pour progresser

Salut Nuliel,

Je te conseille de faire plusieurs blocs plutôt qu'une seule fonction main, avec un minimum dans main pour initialiser.

Tu peux également décomposer en MVC, aussi simple soient ils

Nuliel

Re : deviner mot en C: conseils pour progresser

Merci NicoApi73. Comme ça prenait pas tant de lignes, j'ai tout fait dans le main, mais je vais séparer tout ça, ce sera plus propre et plus modulaire.

Au passage, je vais faire beaucoup beaucoup de C cette année, en réalité je prends de l'avance pour pouvoir produire du code lisible rapidement.

Dernière modification par Nuliel (Le 14/09/2020, à 19:15)

---

[ poster un retour de commande ] [ poster une photo ]

NicoApi73

Re : deviner mot en C: conseils pour progresser

D'autant qu'à un moment donné tu vas travailler sur des sujets spécifiques. Par exemple un générateur de mot aléatoire (utilisant un dictionnaire par exemple). Ca ne doit pas impacter la présentation de l'information. A un autre moment tu souhaiteras peut être présenter l'information dans une interface fenêtrée, ça ne doit pas impacter la gestion (et génération) des mots...

Les lignes que tu sauves maintenant te feront perdre beaucoup de temps par la suite

Nuliel

Re : deviner mot en C: conseils pour progresser

Exactement, il faut que je perde maintenant les mauvaises habitudes, et que je prenne de bonnes habitudes de codage.

---

[ poster un retour de commande ] [ poster une photo ]