pointeur qui se dé ininitialise

Version imprimable

Bonjour à tous,

J'ai un code qui fonctionne mais avec un pointeur qui se déinitialise?

Code:

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
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
/*  ttt-stripped <t.txt   */

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

#define CINQPAGES 10240
#define LIGNES 100
#define Date: __DATE__

char strconv[10240];
char *toConsole ( char *p ) ;


char *toConsole ( char *p )
{
   CharToOemBuffA( p, strconv, 10240 );
   return strconv;
}

int main( void )
{
   char *pointeur = ( ( void * )0 );
   char *strst = ( ( void * )0 );
   char *reset = ( ( void * )0 );
   const char *cmp = "End_of_it.";
   int k = 1;
   int i = 0;
   int j = 0;
   char inligne[100] = {0};
   char intexte[3024] = {0};
   char substring[250] = {0};

   reset = &intexte[0];

   while ( ( ( void * )0 ) != fgets( inligne, sizeof inligne, ( &_iob[0] ) ) ) {
      strcat( intexte, inligne );
   }

   strcat( intexte, "\n" );

   strcat( intexte, cmp );
   strcat( intexte, "\n" );
   free( inligne );

   while ( 0 == strstr( substring , cmp ) ) {

      j = 0;

      substring[j++] = intexte[i];
      i++;

      while ( 1 != isupper( ( unsigned char )intexte[i] ) ) {
         substring[j++] = intexte[i];
         i++;
      }

      j--;

      while ( 0 == isgraph( ( unsigned char )substring[j] ) ) {
         j--;
      }
      j++;

      substring[j] = '\0';
      printf( "after %d--->%s<----\n\n", k++, substring );
      strst = &intexte[i];

      while ( ( ( void * )0 ) != ( pointeur = strstr( strst, substring ) ) ) {
         *pointeur++ = ' ';
         do {
            *pointeur++ = ' ';

         } while ( 1 != isupper( ( unsigned char )*pointeur ) );
         *( pointeur - 1 ) = ' ';
      }
   printf( "%s\n", toConsole( reset ) );  /*  OK  */
   }

   printf( "\n----------fin  -------------\n" );

   printf( "%s\n", toConsole( reset ) );  /*  ligne vide   */

   free( intexte );
   return 0;
}

Fichier.txt sur stdin:

Code:

1
2
3
4
5
6
7
 
---------------------   <    t.txt  -------------------------------
 
C1ar, il sera ok hah bomb adil.	    Every shared library has a special.
C1ar, il sera ok hah bomb adil.
E3very shared library has a special.
---------------------   fin   t.txt  -------------------------------

:calim2:

04/06/2009, 16h42
matafan

Tu pourrais faire un effort pour au moins expliquer ce que tu attends de ton programme, et en quoi ce que tu obtiens diffère de ce que tu attends...
04/06/2009, 17h08
J4e8a16n

Il enlève les phrases qui se répètent. (doublons)

Mon problème c'est que le pointeur reset (dans la boucle) imprime le texte à la console mais n'imprime rien toute de suite après la dernière boucle à la fin.

>----------fin -------------
est imprimé.
'reset' n'imprime rien.

Pourquoi perdrait-t-il ses infos?
04/06/2009, 19h34
J4e8a16n
Question complémentaire:

Est-ce que je pourrais écrire:
Code:

1 2 int intexte[3024] = {0}; int substring[250] = {0};
pour écrire:

Code:

while ( 1 != isupper(*pointeur ) );

au lieu de:
Code:

1 2 char intexte[3024] = {0}; char substring[250] = {0};
pour éviter d'avoir à écrire

Code:

while ( 1 != isupper( ( unsigned char )*pointeur ) );

Bon ben, marci ben ...
05/06/2009, 10h39
diogene
* A propos de ton code :
- ( ( void * )0 ) se remplace plus correctement par NULL :

Code:

char *pointeur = NULL ;....

- reset = &intexte[0]; s'écrit plus simplement :

Code:

reset = intexte;

- ( &_iob[0] ) se remplace de façon plus standard par stdin

- Enfin à partir de

Code:

free( inligne );

tout peut arriver : on ne peut appliquer un free que sur un tableau alloué dynamiquement par une fonction de la famille de malloc(). Ce n'est pas le cas ici et le programme va planter à un moment ou à un autre ou avoir un comportement incompréhensible.

* A propos de ta question complémentaire :
Le choix des tableaux de char ou de int ne doit dépendre de devoir faire un cast ou non. Il dépend de la logique du programme : quelles grandeurs le tableau est-il censé représenté ?
Tu peux écrire
Code:

1 2 3 while ( 1 != isupper(*pointeur ) ); // ou while ( !isupper(*pointeur ) );
que *pointeur soit un char ou un int. Aucun cast n'est nécessaire dans un cas comme dans l'autre.
Accessoirement, je te signale que isupper a un argument dont le type n'est pas un unsigned char mais est un int

J'ai nettoyé mon code manuellement. Les trucs genre ( ( void * )0 ) appartenaientt au préprocesseur. J'ai enlevé les unsigned.

Je ne comprends pas pourquoi la sortie de boucle (0 == strstr(substring , cmp) déréférencie ou le (reset) modifie autrement de façon à imprimer rien.

J'ai essayé de débugger avec gdb
(gdb) target exec ttt.exe
(gdb) run < t.txt
Je ne sais trop comment faire.
(ici un tutoriel de base pour gdb en ligne de commande serait le bienvenue)

J'ai aussi tenté code block mais là non plus rien de clair. En fait je trouve cela encore plus complexe quoique plus attrayant.
--
En remplaçant (0 == strstr(substring , cmp) par --k > 0 tout baigne.

Code:

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
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
 
/*  ttt <t.txt   > t2.txt */
 
#include <windows.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>
 
#define CINQPAGES 10240 
#define LIGNES 100
char strconv[CINQPAGES];
char  *toConsole (char *p) ;
 
 
char  *toConsole (char *p) 
{
CharToOemBuff(p,strconv,CINQPAGES);
return strconv; 
} 
 
int main( void )
{
   char *pointeur = NULL;
   char *strst = NULL;
   char *reset = NULL;
   const char *cmp = "End_of_it.";
   int i = 0;
   int j = 0;
   int k = 12;
   char inligne[100] = {0};
   char intexte[3024] = {0};
   char substring[250] = {0};
 
   reset = intexte;
 
   while ( NULL != fgets( inligne, sizeof inligne, stdin ) ) {
      strcat( intexte, inligne );
   }
   strcat( intexte, "\n");
   strcat( intexte, cmp);
   strcat( intexte, "\n");
 
   /*   LOOP  */
 
   while ( /*   0 == strstr(substring , cmp)   */    --k > 0    ) {
      j = 0;
 
      /*  créons la phrase à chercher sans '\n' ou espaces blanc terminaux  */
      substring[j++] = intexte[i];
      i++;
 
      while ( 1 != isupper( intexte[i] ) ) {
         substring[j++] = intexte[i];
         i++;
      }
 
      j--;
 
      while (substring[j] == ' ' ||  substring[j] == '\n' || substring[j] == '\t') { /*  0 == faux  */
         j--;
      }
      j++;
 
      substring[j]  = '\0';
      printf("after-->%s--\n\n", substring);
      strst = &intexte[i];
 
       if(strlen(substring) ==  0 )continue;
 
      while ( NULL != ( pointeur = strstr( strst, substring ) ) ) {
 
         do {
             *pointeur++ = ' ';  
 
         } while ( 1 != isupper(*pointeur));
 
         *( pointeur - 1 ) = '\n';
      }
      fflush( stdout );
   }
 
   /*   FIN  LOOP  */
 
 
printf( "\n----------fin  -------------\n" );
 
printf( "%s", toConsole(reset));
 
return EXIT_SUCCESS;
}
 
/*    ttt <t.txt   > t2.txt & t2.txt  */

Il manquait ceci:

Code:

strcat( intexte, "\nT\n");

ici

Code:

1
2
3
4
5
6
7
 
  while ( NULL != fgets( inligne, sizeof inligne, stdin ) ) {
   strcat( intexte, inligne );
   }
   strcat( intexte, "\n");
   strcat( intexte, cmp);
   strcat( intexte, "\nT\n");