[C] problème de réallocation de la taille d'une chaine [résolu]

problème de réallocation de la taille d'une chaine [résolu] [C] - C - Programmation

Marsh Posté le 26-12-2005 à 17:22:16    

Salut,
Je cherche à coder un petit éditeur de texte , et j'ai un problème sur la gestion d'une de mes boucles. Je cherche à mettre un espace avant une parenthèse si la chaine n'en comporte pas. Mais ma boucle tourne indéfiniment et je n'arrive pas à voir pourquoi? Je suppose que je fais une erreur en réallouant la taille de ma chaine, mais je n'en suis pas sur. Est-ce que quelqu'un peut m'aider?
 
 

Code :
  1. int parentheses (char *chaine)
  2.     {
  3.             int compteur1,compteur2,taille;
  4.              
  5.             compteur1 = 0;
  6.             taille = strlen(chaine);
  7.            
  8.             while (compteur1 < taille)
  9.                   {
  10.                              if ((chaine[compteur1] == '(') || (chaine[compteur1] == ')'))
  11.                                 {
  12.                                           //On efface tous les espaces avant les parenthèses
  13.                                           if (chaine[compteur1 - 1] == ' ')
  14.                                              {                                                   
  15.                                                       while (chaine[compteur1 - 2] == ' ')
  16.                                                             {
  17.                                                                        for (compteur2 = compteur1 - 2; compteur2 < taille - 1; compteur2++)
  18.                                                                            {
  19.                                                                                       chaine[compteur2] = chaine[compteur2 + 1];
  20.                                                                            }
  21.                                                                        compteur1 = compteur1 - 1;                                                             
  22.                                                             }
  23.                                              }
  24.                                           //boucle qui tourne en rond
  25.                                           //On ajoute un espace si le caractère précédent n'est pas une parenthèse (
  26.                                           if ((chaine[compteur1] == '(') && (chaine[compteur1 - 1] != '('))
  27.                                              {
  28.                                                       taille = taille + 1;                                                     
  29.                                                       chaine = (char*) realloc(chaine,(sizeof(char) * taille));
  30.                                                       compteur2 = taille - 1;
  31.                                                       while (compteur2 >= (compteur1 - 1))
  32.                                                           {
  33.                                                                      chaine[compteur2] = chaine[compteur2 - 1];                                                                   
  34.                                                                      compteur2 = compteur2 - 1;
  35.                                                           }
  36.                                                       chaine[compteur1 - 1] = ' ';                                                     
  37.                                              }
  38.                                           //On ajoute un espace après la parenthèse )                                                       
  39.                                           if ((chaine[compteur1] == ')') && (chaine[compteur1 + 1] != ' '))
  40.                                              {                                                   
  41.                                                       taille = taille + 1;
  42.                                                       chaine = (char*) realloc(chaine,(sizeof(char) * taille));                                                     
  43.                                                       for (compteur2 = taille - 1; compteur2 > compteur1 + 1; compteur2--)
  44.                                                           {
  45.                                                                      chaine[compteur2] = chaine[compteur2 - 1];
  46.                                                           }
  47.                                                       chaine[compteur1 + 1] = ' ';                                                                 
  48.                                              }
  49.                                 }
  50.                              compteur1 = compteur1 + 1;
  51.                      }           
  52.             printf("%s\n",chaine);       
  53.             return 0;
  54.     }
  57. int main (void)
  58.    {
  59.          int pause,i;
  60.                  
  61.          char *chaine;
  62.        
  63.          chaine = (char*) malloc(sizeof(char) * 200);
  64.          strcpy (chaine,"ca marche ,ou pas? Ptet  ,(ou ptet pas).Mais bon on s'en tape.  non?" );
  65.        
  66.          parentheses(chaine);
  67.      
  68.        
  69.          scanf("%d",&pause);
  70.          return 0;
  71.    }

Message cité 2 fois
Message édité par bossgama le 27-12-2005 à 00:19:17
Reply

Marsh Posté le 26-12-2005 à 17:22:16   

Reply

Marsh Posté le 26-12-2005 à 17:42:50    

C'est mal indenté, c'est pas comme ca qu'on travaille, je vous l'ai deja dit

Reply

Marsh Posté le 26-12-2005 à 17:53:28    

mal indenté?

Reply

Marsh Posté le 26-12-2005 à 18:00:54    

mal indenté : ça veut dire que qu'il n'y a pas assez d'indentation, pas assez de parenthèses, de guillemets, de virgules, etc.. enfin je crois .
je suis DIIC et c'est ce qu'on m'a enseigné

Reply

Marsh Posté le 26-12-2005 à 18:02:11    

tous ta fait, mais ca veut haut-ci dire que les noms de variables sont mal choisis

Message cité 1 fois
Reply

Marsh Posté le 26-12-2005 à 18:11:52    

ce n'est pas mon fichier final, mais un fichier temporaire pour faire marcher ma fonction. Quand aux noms de variables... j'ai mis des noms qui me paraissaient logiques, mais je veux pas passer plus de temps la dessus que sur la réalisation du programme proprement dit.
 
Sinon vous ne voyez pas ce qui peut planter dans ma boucle?

Reply

Marsh Posté le 26-12-2005 à 20:25:30    

Ungario a écrit :

tous ta fait, mais ca veut haut-ci dire que les noms de variables sont mal choisis


Ca fout la trouille !
 


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 26-12-2005 à 20:29:46    

je comprends vraiment pas pourquoi ca marche pas. Il doit y avoir un truc tout bete, mais quoi???
 :(

Reply

Marsh Posté le 26-12-2005 à 20:31:43    

bossgama a écrit :

Est-ce que quelqu'un peut m'aider?


Déjà, ça aiderais si tu postais du code compilable...


Compiling: main.c
main.c:2: warning: no previous prototype for 'parentheses'
main.c: In function `parentheses':
main.c:6: error: implicit declaration of function `strlen'
main.c:6: warning: nested extern declaration of `strlen'
<internal>:0: warning: redundant redeclaration of 'strlen'
main.c:29: error: implicit declaration of function `realloc'
main.c:29: warning: nested extern declaration of `realloc'
main.c:29: warning: cast does not match function type
main.c:42: warning: nested extern declaration of `realloc'
main.c:29: warning: redundant redeclaration of 'realloc'
main.c:29: warning: previous implicit declaration of 'realloc' was here
main.c:42: warning: cast does not match function type
main.c:52: error: implicit declaration of function `printf'
main.c:52: warning: nested extern declaration of `printf'
<internal>:0: warning: redundant redeclaration of 'printf'
main.c: In function `main_':
main.c:63: error: implicit declaration of function `malloc'
main.c:63: warning: nested extern declaration of `malloc'
<internal>:0: warning: redundant redeclaration of 'malloc'
main.c:64: error: implicit declaration of function `strcpy'
main.c:64: warning: nested extern declaration of `strcpy'
<internal>:0: warning: redundant redeclaration of 'strcpy'
main.c:69: error: implicit declaration of function `scanf'
main.c:69: warning: nested extern declaration of `scanf'
<internal>:0: warning: redundant redeclaration of 'scanf'
main.c:59: warning: unused variable `i'


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 26-12-2005 à 20:40:13    

Je le reposte, j'avais oublié les includes
my bad emmanuel delahaye :bounce:  
 
Le code ci apres est compilable, mais il tourne en rond de manière infini et je ne comprends pas pourquoi.
 

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  4. //Définition des procédures.
  6. int parentheses (char *chaine)
  7.     {
  8.             int compteur1,compteur2,taille;
  9.              
  10.             compteur1 = 0;
  11.             taille = strlen(chaine);
  12.            
  13.             while (compteur1 < taille)
  14.                   {
  15.                              if ((chaine[compteur1] == '(') || (chaine[compteur1] == ')'))
  16.                                 {
  17.                                           //On efface tous les espaces avant les parenthèses
  18.                                           if (chaine[compteur1 - 1] == ' ')
  19.                                              {                                                   
  20.                                                       while (chaine[compteur1 - 2] == ' ')
  21.                                                             {
  22.                                                                        for (compteur2 = compteur1 - 2; compteur2 < taille - 1; compteur2++)
  23.                                                                            {
  24.                                                                                       chaine[compteur2] = chaine[compteur2 + 1];
  25.                                                                            }
  26.                                                                        compteur1 = compteur1 - 1;                                                             
  27.                                                             }
  28.                                              }
  30.                                            //boucle qui pose problème
  31.                                           //On ajoute un espace si le caractère précédent n'est pas une parenthèse (
  32.                                           if ((chaine[compteur1] == '(') && (chaine[compteur1 - 1] != '('))
  33.                                              {
  34.                                                       taille = taille + 1;                                                     
  35.                                                       chaine = (char*) realloc(chaine,(sizeof(char) * taille));
  36.                                                      
  37.                                                       compteur2 = taille - 1;
  38.                                                      
  39.                                                       for (compteur2 = taille - 1; compteur2 > compteur1 - 1; compteur2--)
  40.                                                           {
  41.                                                                      chaine[compteur2] = chaine[compteur2 - 1];
  42.                                                           }
  43.                                                          
  44.                                                       chaine[compteur1 - 1] = ' ';
  45.                                                       printf("%s\n",chaine);
  46.                                              }
  47.                                           //On ajoute un espace après la parenthèse )                                                       
  48.                                           if ((chaine[compteur1] == ')') && (chaine[compteur1 + 1] != ' '))
  49.                                              {                                                   
  50.                                                       taille = taille + 1;
  51.                                                       chaine = (char*) realloc(chaine,(sizeof(char) * taille));                                                     
  52.                                                       for (compteur2 = taille - 1; compteur2 > compteur1 + 1; compteur2--)
  53.                                                           {
  54.                                                                      chaine[compteur2] = chaine[compteur2 - 1];
  55.                                                           }
  56.                                                       chaine[compteur1 + 1] = ' ';                                                                 
  57.                                              }
  58.                                 }
  59.                              compteur1 = compteur1 + 1;
  60.                      }           
  61.             printf("%s\n",chaine);       
  62.             return 0;
  63.     }
  64.    
  65. int main (void)
  66.    {
  67.          int compteur;
  68.          int pause;
  69.                  
  70.          char *chaine;
  71.        
  72.          chaine = (char*) malloc(sizeof(char) * 200);
  73.        
  74.        
  75.          strcpy (chaine,"ca marche ,ou pas? Ptet  ,(ou ptet pas).Mais bon on s'en tape.  non?" );
  76.        
  77.          parentheses(chaine);
  78.                  
  79.          scanf("%d",&pause);
  80.          return 0;
  81.    }


Message édité par bossgama le 26-12-2005 à 20:40:37
Reply

Marsh Posté le 26-12-2005 à 20:40:13   

Reply

Marsh Posté le 26-12-2005 à 21:39:31    

bossgama a écrit :

Salut,
Je cherche à coder un petit éditeur de texte , et j'ai un problème sur la gestion d'une de mes boucles. Je cherche à mettre un espace avant une parenthèse si la chaine n'en comporte pas. Mais ma boucle tourne indéfiniment et je n'arrive pas à voir pourquoi? Je suppose que je fais une erreur en réallouant la taille de ma chaine, mais je n'en suis pas sur. Est-ce que quelqu'un peut m'aider?


Les indices x-1, x-2 etc. ca fait peur ! Ce genre de problème se résout plutôt à coup de machine à état... Sinon, je conseille de décoreller les problèmes de gestion de mémoire du problème du traitement de la chaine, en créant un objet 'intelligent' de type chaine, qui saurait s'agrandir ou se rétrecir quand il faut...
 
T*str_create()
str_delete(T*)
str_add_queue(T*, int car);
str_ins_head(T*, int car);
str_ins(T*, int car, size_t pos);
str_cat(T*, char const *s);
str_insert(T*, char const *s, size_t pos);
etc.
 
 
http://mapage.noos.fr/emdel/clib.htm
Module FSTR
 
http://mapage.noos.fr/emdel/tad.htm


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 26-12-2005 à 22:12:11    

je regarde ca tout de suite, mais ca a l'air plus compliqué :pt1cable: .
Mici Emmanuel Delahaye

Reply

Marsh Posté le 26-12-2005 à 23:08:23    

Ah oui tiens, j'ai cru que j'avais cliqué sur "poster" tout à l'heure, mais en fait non.
 
Bon alors rapidement : oublie le code pour le moment, et réexamine ton algorithme. Il cloche méchant.
 
Indice : ton algorithme pourrait se nommer "le linceul de Pénélope". [:tinostar]


Message édité par Elmoricq le 26-12-2005 à 23:20:15
Reply

Marsh Posté le 27-12-2005 à 00:19:01    

exact elmoricq. Mais j'ai galéré pour trouver mon erreur :bounce:  
Je savais que ca serait un truc tout bête mais là... vu que je decalais la parenthèse à chaque fois je rebouclais dessus alors forcement ca marchait pas  :pt1cable: .
 
Mais c'est réglé maintenant.
 
Il est sympa ton site, Emmanuel Delahaye. Je vais me plonger dedans à partir de maintenant :).

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed