Discussion :

[Premium]

J'ai un probleme avec ce code car ça ne m'affiche pas le bon résultat.
Est ce que quelqu'un peut me dire ce que je dois modifier :
Merci

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
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include<ctype.h>
 
 
int Parenthese(char *ligne)
{
  int i;
  int lg= strlen(ligne)+1;
  int parOuv=0;
  int parFer=0;
  for(i=0;i!=lg;i++)
    { 
      if(ligne[0]=='(') parOuv++;
      if(ligne[0]==')') parFer++;
    }
  if(parOuv==parFer) return 0;
  return 1;
}
 
int main(void)
{
  char *ligne="()";
  printf("Il %s le meme nombre de '(' et de ')'\n",((Parenthese(ligne))==0)?"y a":"n'y a pas");
  return 0;
}

[Médinoc]

parentheseLigne==0 au lieu de ==1 (tu dois inverser ton test, tout simplement)

[sorry60]

Code :

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

1
2
3
4
5
for(i=0;i!=lg;i++)
    {
      if(ligne[0]=='(') parOuv++;
      if(ligne[0]==')') parFer++;
    }

Pourquoi ne testes tu que la premiere lettre ?
Ton for ne sert à rien si tu ne mets pas

Code :

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

1
2
3
4
5
for(i=0;i!=lg;i++)
    {
      if(ligne[i]=='(') parOuv++;
      if(ligne[i]==')') parFer++;
    }

[Invité]

J'aurais plutôt changé ça :

Code :

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

if(parOuv==parFer)

Par :

Code :

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

if(parOuv!=parFer)

Sinon, merci de mettre un titre plus explicite

[Premium]

Code :

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

1
2
3
4
5
for(i=0;i!=lg;i++)
    {
      if(ligne[0]=='(') parOuv++;
      if(ligne[0]==')') parFer++;
    }

Pourquoi ne testes tu que la premiere lettre ?
Ton for ne sert à rien si tu ne mets pas

Code :

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

1
2
3
4
5
for(i=0;i!=lg;i++)
    {
      if(ligne[i]=='(') parOuv++;
      if(ligne[i]==')') parFer++;
    }

merci c'est vrai qu'en testant que la première lettre 80

[Médinoc]

sinon, pour le retour, j'irais même plus loin que remram44:
Je conseille un return parOuv-parFer

Ainsi, la fonction renvoit zéro si c'est égal, donc tu dois tester ==0 dans ton printf

[Premium]

Que faut' il que je modifie pour recuperer le nombre de ( et le nombre de ) :

[Médinoc]

non, tu retournes juste la différence: Si elle est nulle, c'est qu'il y a le même nombre...

[Emmanuel Delahaye]

J'ai un probleme avec ce code car ça ne m'affiche pas le bon résultat.
Est ce que quelqu'un peut me dire ce que je dois modifier :

Ton code commenté et corrigé.

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
51
52
53
54
55
56
57
58
 
#include <stdio.h>
 
int Parenthese(char *ligne)
{
    int i;
 
    /* -ed-
    int lg= strlen(ligne)+1;
    pourquoi ? De toutes facons, tu parcours la chaine, 
    alors autant utiliser la sentinelle standard pour en 
    determiner la fin... 
    */
    int parOuv=0;
    int parFer=0;
    for (i = 0; ligne[i] != 0; i++)
    {
        /* -ed-
         
                if(ligne[0]=='(')
                    parOuv++;
                if(ligne[0]==')')
                    parFer++;
         
        pourquoi [0], j'aurais mis [i], non ? 
        D'autre part, un switch-case est plus efficace (un seul test)
        */
 
        switch (ligne[i])
        {
            case '(':
            parOuv++;
            break;
            case ')':
            parFer++;
            break;
        }
    }
    /* -ed-
        if(parOuv==parFer)
            return 0;
        return 1;
        
    complique... 
    */
    return parOuv != parFer;
}
 
int main(void)
{
    char *ligne="()";
    printf ("Il %s le meme nombre de '(' et de ')'\n"
            ,((Parenthese(ligne))==0)
            ?"y a"
            :"n'y a pas"
           );
    return 0;
}