Difficultés-pour-faire-marcher-mon-programme

Bonjour, j'ai essayé de corriger le mieux possibles mon code mais il ne marche toujours pas:

<main.c>/

#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
#include "Fonction.h"


int main()
{
printf("Bienvenue dans le pendu.\n\n");
int nombreDeChancesRestantes = 10;
char motSecret[] = "Marron";
int tailleMotSecret = 0;
tailleMotSecret = compteurMot(motSecret);
char* currentWord = NULL;
currentWord = malloc((tailleMotSecret+1) * sizeof(char));
if(currentWord == NULL)
{
exit(0);
}
dissimulerMot(motSecret, tailleMotSecret, currentWord);
char maLettre;
do
{
printf("Il vous reste %d coups a jouer. \n", nombreDeChancesRestantes);
printf("Quel est le mot secret ? %s \n", currentWord);
printf("Proposez une lettre: ");
maLettre = lireCaractere();
if(strchr(motSecret, maLettre) == NULL)
{
nombreDeChancesRestantes--;
}
else
{
ajouterMaLettre(motSecret, tailleMotSecret, maLettre, currentWord);
}
printf("\n\n");
}
while(strcmp(motSecret, currentWord) != 0 && nombreDeChancesRestantes != 0);

if(nombreDeChancesRestantes == 0)
{
printf("Vous avez perdu !!!\n");
}
else
{
printf("Bravo, vous avez gagne !!!\n");
}
printf("Le mot secret etait donc: %s \n\n", motSecret);
free(currentWord);


return 0;
}

/ <main.c>


<Fonction.h>/

char lireCaractere();
int compteurMot(char motACompter[]);
void dissimulerMot(char motADissimuler[], int tailleMotADissimuler, char motDissimule);
void ajouterMaLettre(char motADissimuler[], int tailleMotDissimuler, char lettreInterdite, char motDissimule);

/ <Fonction.h>


<Fonction.c>/

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

char lireCaractere()
{
char caractere = 0;

caractere = getchar();
caractere = toupper(caractere);
while (getchar() != '\n') ;

return caractere;
}

int compteurMot(char motACompter[])
{
int i = 0, tailleMotSecret = 0;
char lettre;
do
{
lettre = motACompter[i];
i++;
}
while(lettre != '\0');
tailleMotSecret = i;
return tailleMotSecret;
}

void dissimulerMot(char motADissimuler[], int tailleMotADissimuler, char motDissimule)
{
int i;
motDissimule = motADissimuler;
for(i = 0; i < tailleMotADissimuler; i++)
{
motDissimule[i] = '*';
}
}

void ajouterMaLettre(char motADissimuler[], int tailleMotDissimuler, char lettreInterdite, char motDissimule)
{
int i;
motDissimule = motADissimuler;
for(i = 0; i < tailleMotDissimuler; i++)
{
if(motDissimule[i] != lettreInterdite)
{
motDissimule[i] = '*';
}
}
}

/ <Fonction.c>

Selon mon compilateur, il y aurait 3 erreurs qui se situent aux lignes suivantes :
motDissimule[i] = '*';
if(motDissimule[i] != lettreInterdite)
motDissimule[i] = '*';
Les 3 erreurs sont les mêmes:
error: subscripted value is neither array nor pointer
Il faut garder en tête que mon but est de réaliser un jeu de pendu. Je dois garder le mot secret dissimuler. Lorsque l'utilisateur entre une lettre qui se trouve dans le mot secret une ou plusieurs fois je dois l'afficher (toutes les lettres si la lettre se trouve plusieurs fois dans le mot secret. J'ai cherché et cherché encore et c'est le seul moyen que j'ai trouvé. Même s'il y en a surement d'autres je ne veut pas que vous me le disiez. Je vous demande, s'il vous plait, de m'aider à faire de mon code un code valable sans erreurs ou instructions interdites.
Merci pour votre aide.

Tu t'es trompé dans les balises...
Il faut le mettre entre deux balises "code c". Exemple :
<"code c">/*ici tu mets ton code*/<"/code"> (sans les guillemets).

exit(0);
Pas terrible dans le main(). Mets plutôt return EXIT_FAILURE;

Tu as oublié de changer : char lireCaractere() pour le mettre en int. getchar() renvoie un int et non un char...

void dissimulerMot(char motADissimuler[], int tailleMotADissimuler, char motDissimule)
Attention, motDissimule n'est pas un char mais une adresse sur une zone. Il faut donc faire : char *motDissimule.

motDissimule = motADissimuler;
Que veux-tu faire avec ça ?

Après, il reste essentiellement des erreurs d'algorithme. Mais, je te conseille déjà de corriger et que tu repostes correctement (avec les bonnes balises) ton code pour qu'on y voie plus clair.

motDissimule = motADissimuler;
Que veux-tu faire avec ça ?

motDissimule est en fait currentWord et motADissimuler est motSecret. Je dois remplacer à l'affichage le mot secret qui est "marron" ici par "******". C'est pour cela que j'ai créer une chaine de caractere currentWord. Cependant, motSecret doit rester "marron" et non "******" donc j'utilise currentWord qui est égale à motSecret donc "marron" et je remplace toutes ses lettres par * grâce à la fonction dissimulerMot qui se trouve avant la boucle do-while. Ensuite vient la boucle, l'utilisateur propose une lettre. Si cette lettre se trouve bien dans le motSecret alors je l'ajoute avec la fonction ajouterMaLettre à currentWord tout en laissant les autres lettres dissimulées par des *. Si la lettre proposée ne correspond pas, je n'utilise pas la fonction et je diminue le nombre de chances restantes à l'utilisateur. La boucle doit s'arrêter si current word = motSecret donc lorque l'utilisateur aura trouver toutes les lettres ou lorsque son nombres de chances sera égale à zero. Cependant, mon programme ne veut pas fonctionner. J'ai suivi vos consignes mais la fonction lireCaractere() renvoie la variable caractere qui de type char alors pourquoi la mettre de type int. Je l'ai fait comme même comme vous l'avez dit et j'ai aussi remplacé exit(0) par EXIT_FAILURE. Et j'ai remplacé char motDissimule par char* motDissimule dans la fonction dissimulerMot. Voila mon code :

main.c :

#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
#include "Fonction.h"


int main()
{
    printf("Bienvenue dans le pendu.\n\n");
    int nombreDeChancesRestantes = 10;
    char motSecret[] = "Marron";
    int tailleMotSecret = 0;
    tailleMotSecret = compteurMot(motSecret);
    char* currentWord = NULL;
    currentWord = malloc((tailleMotSecret+1) * sizeof(char));
    if(currentWord == NULL)
    {
        EXIT_FAILURE;
    }
    dissimulerMot(motSecret, tailleMotSecret, currentWord);
    char maLettre;
    do
    {
        printf("Il vous reste %d coups a jouer. \n", nombreDeChancesRestantes);
        printf("Quel est le mot secret ?   %s \n", currentWord);
        printf("Proposez une lettre:   ");
        maLettre = lireCaractere();
        if(strchr(motSecret, maLettre) == NULL)
        {
            nombreDeChancesRestantes--;
        }
        else
        {
            ajouterMaLettre(motSecret, tailleMotSecret, maLettre, currentWord);
        }
        printf("\n\n");
    }
    while(strcmp(motSecret, currentWord) != 0 && nombreDeChancesRestantes != 0);

    if(nombreDeChancesRestantes == 0)
    {
        printf("Vous avez perdu !!!\n");
    }
    else
    {
        printf("Bravo, vous avez gagne !!!\n");
    }
    printf("Le mot secret etait donc: %s \n\n", motSecret);
    free(currentWord);


    return 0;
}

Fonction.h :

char lireCaractere();
int compteurMot(char motACompter[]);
void dissimulerMot(char motADissimuler[], int tailleMotADissimuler, char motDissimule);
void ajouterMaLettre(char motADissimuler[], int tailleMotDissimuler, char lettreInterdite, char motDissimule);

Fonction.c

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

int lireCaractere()
{
    char caractere = 0;

    caractere = getchar();
    caractere = toupper(caractere);
    while (getchar() != '\n') ;

    return caractere;
}

int compteurMot(char motACompter[])
{
    int i = 0, tailleMotSecret = 0;
    char lettre;
    do
    {
        lettre = motACompter[i];
        i++;
    }
    while(lettre != '\0');
    tailleMotSecret = i;
    return tailleMotSecret;
}

void dissimulerMot(char motADissimuler[], int tailleMotADissimuler, char* motDissimule)
{
    int i;
    motDissimule = motADissimuler;
    for(i = 0; i < tailleMotADissimuler; i++)
    {
        motDissimule[i] = '*';
    }
}

void ajouterMaLettre(char motADissimuler[], int tailleMotDissimuler, char lettreInterdite, char motDissimule)
{
    int i;
    motDissimule = motADissimuler;
    for(i = 0; i < tailleMotDissimuler; i++)
    {
        if(motDissimule[i] != lettreInterdite)
        {
            motDissimule[i] = '*';
        }
    }
}

caractere doit être de type int...
caractere va remplacer maLettre après utilisation de la fonction. maLettre est de type char cela serait mieux de mettre caractere de type char et comme la fonction lireCaractere retourne caratere elle doit être elle aussi de type char je pense. Sinon on met également maLettre de type int mais vu qu'il va venir se placer dans currentWord qui est de type char... Mais bon, j'ai quand meme mis caractere et la fonction de type int parce que vous m'avez dit que c'est à cause du getchar() or je ne connais pas trop cette fonction c'était la première fois que je l'utilisait.
Sinon on y est presque... Le programme se lance correctement mais il ne fonctionne pas correctement. Voici mon code re...re...remastorisé :

main.c :

#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
#include "Fonction.h"


int main()
{
    printf("Bienvenue dans le pendu.\n\n");
    int nombreDeChancesRestantes = 10;
    char motSecret[] = "Marron";
    int tailleMotSecret = 0;
    tailleMotSecret = strlen(motSecret);
    char* currentWord = NULL;
    currentWord = malloc((tailleMotSecret + 1) * sizeof(char));
    if(currentWord == NULL)
    {
        EXIT_FAILURE;
    }
    dissimulerMot(currentWord, tailleMotSecret);
    char maLettre;
    do
    {
        printf("Il vous reste %d coups a jouer. \n", nombreDeChancesRestantes);
        printf("Quel est le mot secret ?   %s \n", currentWord);
        printf("Proposez une lettre:   ");
        maLettre = lireCaractere();
        if(strchr(motSecret, maLettre) == NULL)
        {
            nombreDeChancesRestantes--;
        }
        else
        {
            ajouterMaLettre(motSecret, tailleMotSecret, maLettre, currentWord);
        }
        printf("\n\n");
    }
    while(strcmp(motSecret, currentWord) != 0 && nombreDeChancesRestantes != 0);

    if(nombreDeChancesRestantes == 0)
    {
        printf("Vous avez perdu !!!\n");
    }
    else
    {
        printf("Bravo, vous avez gagne !!!\n");
    }
    printf("Le mot secret etait donc: %s \n\n", motSecret);
    free(currentWord);


    return 0;
}

Fonction.h :

void dissimulerMot(char* motADissimuler, int taille);
char lireCaractere();
void ajouterMaLettre(char* motSecret, int taille, char lettreAAjouter, char* currentWord);

Fonction.c :

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

void dissimulerMot(char* motADissimuler, int taille)
{
    int i;
    for(i = 0; i < taille; i++)
    {
        motADissimuler[i] = '*';
    }
}

int lireCaractere()
{
    int caractere = 0;

    caractere = getchar();
    caractere = toupper(caractere);
    while (getchar() != '\n') ;

    return caractere;
}

void ajouterMaLettre(char* motSecret, int taille, char lettreAAjouter, char* currentWord)
{
    int i;
    for(i = 0; i < taille; i++)
    {
        if(motSecret[i] != lettreAAjouter)
        {
            currentWord[i] = '*';
        }
        else
        {
            currentWord[i] = lettreAAjouter;
        }
    }
}

Voila les problèmes dans le programme :
1) Il y a à chaque fois un caractère ou un chiffre différent qui vient s'ajouter à la fin de currentWord.
2) Lorsque la lettre entrée est incorrecte pas de problème le programme agit comme je lui demande mais lorsque celle-ci se trouve bien dans le motSecret (par exemple si je tape la lettre a qui se trouve bien dans le mot marron, le programme agit comme si cela n'était pas le cas et cela pour toutes les autres lettres qui se trouve bien dans le mot SAUF pour la lettre M qu'il ajoute correctement à currentWord lorsque je la propose.

Comment est-ce que je pourrais vous montrer mon programme console pour que vous puissiez voir les problèmes du programme de vous même parce que là j'ai aucune idée pour les résoudre.

J'ai oublié de mettre la fonction lireCaractere de type int dans Fonction.h quand je vous est envoyé le message mais là c'est corrigé désolé...
Mais la situation est restée la même...
En attente d'une suggestion de votre part...
Merci pour votre aide.

Le deuxième problème de mon programme venait du fait que lorsque j'ai défini le mot secret j'avais écrit :

char motSecret[] = "Marron"

Mais j'ai modifié et j'ai écrit:

char motSecret[] = "MARRON"

Je pense que vous avez compris d'où venait le problème.
Maintenant les autres lettres sont traités correctement sauf qu'il y a maintenant un autre problème:
Lorsque je tape par exemple la lettre m current word devient M***** mais ensuite je tappe par exemple r et là currentWord devient **RR**
La lettre précédente redevient un * !!!!

Désolé je ne l'est pas posté mais j'avais compris pourquoi la lettre précédente redevenait un * .
Cependant il reste le problème suivant :
1) Il y a à chaque fois un caractère ou un chiffre différent qui vient s'ajouter à la fin de currentWord.
Vous me dîtes :
Tu n'as pas rajouté le '\0' comme j'avais dit ;-) dans ta fonction dissimulerMot().
Je ne comprend pas. Je ne me rappelle pas que vous m'ayez dis ça. Je ne voit pas où mettre le '/0' .

Je l'avais mis dans mon tout premier post : https://forums.commentcamarche.net/forum/affich-30653791-difficultes-pour-faire-marcher-mon-programme#1

Une fois que tu as fait :
for(i = 0; i < taille; i++)
{
motADissimuler[i] = '*';
}
dans ta fonction dissimulerMot(), tu as juste à faire : motADissimuler[taille]='\0';
Justement, c'est pour ça qu'il fallait faire taille+1 dans le malloc(). Sinon, il n'y avait pas la place pour mettre le '\0'.

Je me suis amusé à améliorer mon programme mais maintenant le problème est revenu...
Voici mon ce que j'ai ajouté par rapport à mon code précédent en plus du motADissimuler[taille]='\0'; dans la fonction dissimulerMot() :

int main()
{
    printf("Bienvenue dans le pendu.\n\n");
    int nombreDeMots = 0;
    int numLigne = 0;
    int indice = 0;
    int nombreDeChancesRestantes = 10;
    char motSecret[TAILLE_MAX] = {0};
    char caractereActuel;
    FILE* fichier = NULL;
    fichier = fopen("Lybrary.txt", "r");
    if(fichier != NULL)
    {
        do
        {
            caractereActuel = fgetc(fichier);
            if(caractereActuel == '\n')
            {
                nombreDeMots++;
            }
        }
        while(caractereActuel != EOF);
        rewind(fichier);
        srand(time(NULL));
        indice = rand() % nombreDeMots;
        rewind(fichier);
        while(numLigne < indice)
        {
            if(fgetc(fichier) == '\n')
            {
                numLigne++; 
            }
        }
        fgets(motSecret, TAILLE_MAX, fichier);
        fclose(fichier);
    }
    else
    {
        printf("Impossible d'ouvrir le fichier Lybrary.txt");
    }

Je te conseille de faire une fonction pour compter le nombre de mots.
Poste ton programme en entier. Ca sera plus simple :-).