Discussion :

[3dx2y]

Bonsoir,

En travaillant sur ce problème qui doit comporte des fonctions de saisie d'une chaine, puis de calcul de l'occ du char c puis d'inverser la chaine mais le programme n"execute pas la fonction inverser et je sais pas pourquoi ou comment le resoudre. de l'aide svp!

Voici le code :

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
#include <stdio.h>
#include <stdlib.h>
 
 
/*probleme*/
 
/*saisir*/
char* saisir() {
    int n;
    char* ch;
    printf("saisir la taille de ch:\n");
    scanf("%d", &n);
    printf("saisir votre chaine\n");
    ch = (char*) malloc(n * sizeof(char));
    _flushall();
    gets(ch);
    return (ch);
}
 
/*occ*/
 
int occurance(char* ch) {
    int i;
    int occ = 0;
    for (i = 0; i < strlen(ch); i++) {
        if (ch[i] == 'c')
            occ = occ + 1;
 
    }
    return (occ);
}
 
/*fonction inverse*/
 
char* inverse(char* ch) {
    return (strrev(ch));
}
 
/*programme principal*/
int main() {
    char* ch;
    int occ;
 
    printf("le nb d'occ de char 'c' dans ch =%d\n", occurance(saisir()));
 
    printf("la chaine inversée =%s\n", inverse(ch));
 
    return 0;
}

[Sve@r]

Bonjour

mais le programme n"execute pas la fonction inverser et je sais pas pourquoi ou comment le resoudre.

Il compile déjà??? Parce qu'il fait appel à strrev() qui n'est définie nulle part. Et faire une boucle de 0 à strlen(ch) est la pire des solutions. A chaque itération il recalcule la taille de la chaine. De plus return n'est pas une fonction.

voici le code:

http://club.developpez.com/regles/#LIII-E

[edgarjacobs]

Hello,

1)

2) tu oublies d'allouer de l'espace pour le \0 de fin de chaine

3) on oublie gets(), qui est complètement deprecated, on utilise fgets() à la place

4) je ne connais pas _flushall() , mais en général une fonction de flush ne flush que les fichiers en output

[dalfab]

Bonjour,

Ligne ???, tu définis un pointeur ch et n'y mets rien. Ligne ???, tu passes ce pointeur à la fonction inverse(). Comment pourrait-elle retourner un résultat sensé?

Tu alloues une zone pour y placer une chaine, cette zone n'est jamais libérée.

[3dx2y]

[QUOTE=Sve@r;11894379]
bonjour, oui le programme compile et execute correctement les autres fonctions sauf celle de l'inverse j'ai ajouté la commande

Code c :

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

<string.h>

(faute d'inattention) sinon j'ai changé le code ainsi mais toujourspas d'execution:

Code c :

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

1
2
3
4
5
6
7
 
 
char* inverse(char* ch)
{
 ch=strrev(ch) ;
 return ch;
}

[3dx2y]

Hello,

bonjour,

2) tu oublies d'allouer de l'espace pour le \0 de fin de chaine

je suis encore débutante mais je crois que j'ai alloué de l'espace pour toute la chaine en utilisant la syntaxe "malloc" qui par defaut alloue de l'espace pour toute la chaine en prenant compte de /0
3) on oublie gets(), qui est complètement deprecated, on utilise fgets() à la place
on travaille toujours avec gets en classe c'est pourquoi j'l'ai utilisé en plus j'ai juste un problème au niveau de la fonction inverse qui ne s'execute pas et je sais pas comment corriger mon code
4) je ne connais pas _flushall() , mais en général une fonction de flush ne flush que les fichiers en output

on utilise la commande _flushall() lors d'un saisie de char ou de chaine on l'utilise pour libérer temporairement de l'espace mémoire si on l'utilise pas le code ne s'execute pas.

[Sve@r]

j'ai ajouté la commande <string.h>

Mouais, super. Vu que ça ce n'est pas une commande...

sinon j'ai changé le code ainsi mais toujourspas d'execution:

Et tu l'as lu le message de dalfalb ()? Parce qu'il est quand-même assez explicite. Regarde ton main. A quel moment tu remplis "ch" que tu tentes ensuite d'inverser???

on utilise la commande _flushall() lors d'un saisie de char ou de chaine on l'utilise pour libérer temporairement de l'espace mémoire si on l'utilise pas le code ne s'execute pas.

Oh là là cette salade qu'il y a dans ta tête. Donc si pas de _flushall() pas de programme???

[3dx2y]

Bonjour,

Ligne ???, tu définis un pointeur ch et n'y mets rien. Ligne ???, tu passes ce pointeur à la fonction inverse(). Comment pourrait-elle retourner un résultat sensé?

Tu alloues une zone pour y placer une chaine, cette zone n'est jamais libérée.

voulez vous dire que je dois changer cette instruction ainsi?:

Code :

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

printf("la chaine inversée =%s\n",inverse(*ch));

car meme ainsi le code ne s'execute pas j'sais vraiment pas quoi modifier dans le code pour qu'il s'execute.

[Sve@r]

voulez vous dire que je dois changer cette instruction ainsi?:

Code :

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

printf("la chaine inversée =%s\n",inverse(*ch));

Il ne t'a rien dit de changer, il t'a juste posé une question : à quel moment tu remplis "ch" avant de vouloir la renverser ?
Normalement en répondant à sa question (que j'ai reprise dans mon message précédent) ça devrait t'amener à changer ce qu'il faut comme il le faut.

j'sais vraiment pas quoi modifier dans le code pour qu'il s'execute.

Programmer ce n'est pas mettre du code au pif en espérant qu'il tombera en marche.

[3dx2y]

et voici encore la fenetre d'execution:


le problème est juste que le programme n'execute pas la fonction "inverse":

Programmer ce n'est pas mettre du code au pif en espérant qu'il tombera en marche.
[/QUOTE] ouai, j'sais très bien je ne programme pas au hasard,je veux juste comprendre quel est le problème dans mon code et trouver des solutions

[Sve@r]

je veux juste comprendre quel est le problème dans mon code

Pour la troisième fois, réponds à la question: à quel moment, dans le main, as-tu rempli "ch" ???

[3dx2y]

Il t'a déjà été expliqué: dans ton main, tu inverses "ch" mais nulle part tu n'as rempli "ch". Donc tu inverses un tableau vide.

mais j'ai déja rempli la chaine dans cette ligne c'est juste imbriqué dans la commande de l'occurance:

Code :

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

printf("le nb d'occ de char 'c' dans ch =%d\n",occurance(saisir()));

[3dx2y]

Pour la troisième fois, réponds à la question: à quel moment, dans le main, as-tu rempli "ch" ???

je vous ai déjà répondu. en plus le code ne fonctionnera pas dès le début si j'ai pas saisie la chaine

[Sve@r]

mais j'ai déja rempli la chaine dans cette ligne c'est juste imbriqué dans la commande de l'occurance:

Code :

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

printf("le nb d'occ de char 'c' dans ch =%d\n",occurance(saisir()));

Nan, cette instruction ne remplit pas "ch". Si tu avais ne serait-ce que tenté de l'afficher par exemple tu l'aurais vu immédiatement

en plus le code ne fonctionnera pas dès le début si j'ai pas saisie la chaine

[3dx2y]

Nan, cette instruction ne remplit pas "ch". Si tu avais ne serait-ce que tenté de l'afficher par exemple tu l'aurais vu immédiatement

ainsi non?

Code :

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

1
2
for (i=0;i<strlen(ch);i++)
   printf("%s",ch[i]);

sinon je ne comprend pas pourquoi je dois remplir la chaine alors que j'ai demandé à l'utilisateur de la saisir

[Sve@r]

ainsi non?

Code :

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

1
2
for (i=0;i<strlen(ch);i++)
   printf("%s",ch[i]);

Tu as probablement appris en cours comment afficher une chaine. C'est ainsi qu'on te l'a appris???

Sinon afficher caractère par caractère (alors qu'on peut l'afficher directement) ça marche mais à 2 ou 3 détails

1. on utilise "%c" pour afficher un caractère, pas "%s"
2. on ne met pas strlen() dans une boucle, car la fonction est appelée à chaque tour de boucle. Une chaine se termine par '\0' donc on peut tester ch[i] != '\0'

sinon je ne comprend pas pourquoi je dois remplir la chaine alors que j'ai demandé à l'utilisateur de la saisir

Ne confonds pas les messages que tu affiches à l'écran et les actions de ton code. Ton programme affiche des trucs oui, mais si derrière tu n'écris pas les instructions pour que ce qui a été saisi par l'utilisateur aille réellement dans "ch"... ben ça n'y va pas tout seul.

[3dx2y]

[QUOTE=Sve@r;11894717]Tu as probablement appris en cours comment afficher une chaine. C'est ainsi qu'on te l'a appris???

Sinon afficher caractère par caractère (alors qu'on peut l'afficher directement) ça marche mais à 2 ou 3 détails

1. on utilise "%c" pour afficher un caractère, pas "%s"
2. on ne met pas strlen() dans une boucle, car la fonction est appelée à chaque tour de boucle. Une chaine se termine par '\0' donc on peut tester ch[i] != '\0'

bonjour,
voilà j'ai corrigé ainsi mais le problème persiste

Code :

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

1
2
for (i=0;ch[i]!='\0';i++)
 printf("ch=%c",ch[i]);

[Sve@r]

voilà j'ai corrigé ainsi mais le problème persiste

Code :

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

1
2
for (i=0;ch[i]!='\0';i++)
 printf("ch=%c",ch[i]);

Ca ce n'est pas une correction, c'est juste pour te montrer que ch n'est pas rempli.

La correction, maintenant, c'est de remplir ch. Ca te parait pas logique à toi de remplir un truc avant de vouloir l'inverser ?????????????

[3dx2y]

Ca ce n'est pas une correction, c'est juste pour te montrer que ch n'est pas rempli.

La correction, maintenant, c'est de remplir ch. Ca te parait pas logique à toi de remplir un truc avant de vouloir l'inverser ?????????????

oui compris, j'l'ai rempli en utilisant le code ci dessus puis j'i'ai inversé mais toujours pas d'excution de la partie inverse.

[foetus]

Il faut refaire le main

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
int main() {
    char* ch;
 
    ch = saisir();
 
    if (ch != NULL) {
        printf("le nb d'occ de char 'c' dans ch =%d\n", occurance(ch));
 
        printf("la chaine inversée =%s\n", inverse(ch));
 
        free(ch);
    } else {
        printf("main - error : allocation ch\n");
    }
 
    return 0;
}

Édit : Youpi j'ai des -1