Conversion du type caractere

**tina-ze-star** · 23/05/2008, 19h31

Voila, j'ai écris un petit programme qui devrait me permettre de convertir une chaine de caractères en un type réel,
je dois convertir cette chaine la par exemple:
"0.2/5+0.56/10.1"
donc je dois en sortir le 0.2 le 5 le 0.56 et le 10.1
donc cette chaine est de forme " réel/ (réel ou entier) + réel / (réel ou entier)....."

Quand je compile le programme il n y a aucun probleme, mais lors de l'execution, que se soit sur visual c++ ou bien c++builder, le meme probleme se pose:
un message d'erreur
celui de builder me dit: "Project1.exe a rencontré un problème et doit fermer. Nous vous prions de nous excuser pour le désagrément encouru."
et a peu pres la meme chose en anglais avec visual.

voila mon programme:

une fonction qui me permet de savoir si c'est de type entier ou réel:

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
 
 
#pragma hdrstop
#include <iostream.h>
#include <stdlib.h>
#include <string.h>
#include <math.h>
 
//Fonction qui renvoit le type de la chaine de caractère
 
int type_chaine (char *t)
{  bool trouve=false  ;
    char *nptr;
    nptr=t;
 
    while (*nptr!='\0' && !trouve )
	{
       if (*nptr=='/')
           trouve =true ;
       nptr++;
	}
	if (trouve)
		return (2);
	else
	{    nptr=t;
         while (*nptr!='\0' && !trouve)
		 {
		   if (*nptr=='.')
			   trouve =true ;
           nptr++;
		 }
 
	    if (trouve)	    return (1);
 
	    else 	     return (0);
	}
};

Ma fameuse fonction de conversion:

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
59
60
61
62
63
64
65
66
67
68
 
 
// Fonction de conversion
void convert_POSS (char *t)
{
     int type;
     float resultat[40];
     float puiss;
     //int sign=1;
     int i;
     char *nptr=t;
     //if (*nptr =='-') sign = -1;
     //if ((*nptr=='-') || ( *nptr=='+')) nptr++;
 
	while (*nptr!='\0')
	{
                i=0;
                while (*nptr!='.')
		   {
	             resultat[i]=resultat[i]*10+*nptr -48;
                     nptr++;
		   }
        	nptr++;
        	puiss=10;
		while (*nptr!='/')
		{
	             resultat[i]=resultat[i]+(*nptr-48)/puiss;
	             puiss*=10;
	             nptr++;
		}
                // resultat[i]=resultat[i]*sign;
		nptr++;
                i++;
 
                type=type_chaine(t);
                if (type==1)
                {
	        	while (*nptr!='.')
		        {
	                   resultat[i]=resultat[i]*10+*nptr -48;
                           nptr++;
		        }
                	nptr++;
                	puiss=10;
 
	        	while (*nptr!='+')
	        	{
                            resultat[i]=resultat[i]+(*nptr-48)/puiss;
	                    puiss*=10;
	                    nptr++;
	        	}
                }
                else
                {
                     while (*nptr!='+')
                    {
                           resultat[i]= resultat[i]* 10 + *nptr - 0x30;
                           nptr++;
                    }
                }
                // resultat[i]=resultat[i]*sign;
		nptr++;
		i++;
 
	}
 
     // return (resultat);
}

et enfin l'appel de cette fonction:

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
 
 
#pragma argsused
int main(int argc, char* argv[])
{
     char a [40];
   char *ptr =a;
 
   int l,i;
   l=1;
   float resultat [40];
   while (l!=0)
   {
  cout << "donner la chaine : ";
  cin >> a;
  convert_POSS(ptr);
  for(i=0;i<40;i++)
  cout << resultat[i] <<endl;
  cout << "voulez vous continuez (0,1) : ";
  cin >> l;
   }
 
        return 0;
}

J'espere que vous allez m'aider a résoudre ce problème!!!!

Merci..

**loufoque** · 23/05/2008, 20h58

Tu accèdes sûrement à la mémoire auquel tu n'as pas le droit.
Tu peux par exemple utiliser valgrind (qui ne fonctionne que sous linux) pour déboguer tes erreurs mémoire.

**yakamoneye** · 23/05/2008, 21h21

regarde au niveau de tes pointeurs, cela ressemble a un problème d'allocation mémoire

**tina-ze-star** · 23/05/2008, 21h53

Je vien de faire des petits changement ds mon programme mais j'ai tjr le mem probleme

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
59
60
61
62
63
64
65
66
67
/ Fonction de conversion
void convert_POSS (char *t)
{
     int type;
     float *resultat;
     float puiss;
     //int sign=1;
     int i;
     char *nptr=t;
     //if (*nptr =='-') sign = -1;
     //if ((*nptr=='-') || ( *nptr=='+')) nptr++;

     resultat = new float [40];
	while (*nptr!='\0')
	{
                i=0;
                while (*nptr!='.')
		   {
	             resultat[i]=resultat[i]*10+*nptr -48;
                     nptr++;
		   }
        	nptr++;
        	puiss=10;
		while (*nptr!='/')
		{
	             resultat[i]=resultat[i]+(*nptr-48)/puiss;
	             puiss*=10;
	             nptr++;
		}
                // resultat[i]=resultat[i]*sign;
		nptr++;
                i++;

                type=type_chaine(t);
                if (type==1)
                {
	        	while (*nptr!='.')
		        {
	                   resultat[i]=resultat[i]*10+*nptr -48;
                           nptr++;
		        }
                	nptr++;
                	puiss=10;

	        	while (*nptr!='+')
	        	{
                            resultat[i]=resultat[i]+(*nptr-48)/puiss;
	                    puiss*=10;
	                    nptr++;
	        	}
                }
                else
                {
                     while (*nptr!='+')
                    {
                           resultat[i]= resultat[i]* 10 + *nptr - 0x30;
                           nptr++;
                    }
                }
                // resultat[i]=resultat[i]*sign;
		nptr++;
		i++;

	}

     // return (resultat);
}

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
#pragma argsused
int main(int argc, char* argv[])
{
   char a [40];
   char *ptr =a;
   float *resultat;
   int l,i;
   l=1;
   while (l!=0)
   {
  cout << "donner la chaine : ";
  cin >> a;
  convert_POSS(ptr);
  for(i=0;i<40;i++)
  cout << resultat[i] <<endl;
  cout << "voulez vous continuez (0,1) : ";
  cin >> l;
   }

        return 0;
}

J'espere vraiment que vous pourrez m'aider, je suis complétement perdue

**yakamoneye** · 23/05/2008, 22h40

je regarde et je te dis quoi...
enfin si je trouve .

**yakamoneye** · 24/05/2008, 00h10

A ta place je passerai tout en statique au niveau des char* un truc du genre

Code :

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

1
2
3
4
5
6
7
8
 
 char chaine[256];
 int indice=0;
 while ( chaine[indice] != '\0' )
 {
   ...
   indice++;
 }

J ai testé ton code sous linux et il me renvoie un incident de segmentation ...

et bien souvent ca vient des pointeurs ...

**koala01** · 24/05/2008, 00h52

Salut,

Et si, quitte à écrire du code C++, vous essayiez d'utiliser le beau, le vrai C++ avec toutes les possibilités qu'il offre

Pourquoi, et ce ne sont que des exemples pratiques, ne pas préférer la classe string, disponible dans l'espace de noms std par inclusion du fichier d'en-tete <string> aux chaines "C style", et envisager le recours aux classes *stringstream et dérivées pour les conversions

Ainsi, nous ne le répéterons jamais assez, <iostream.h> est obsolète, et il faut lui préférer <iostream> (même si cela implique de penser que tout se trouve dans l'espace de noms std)

De la même manière, string.h et stdlib.h sont des en-tête issus du C dont l'usage est obsolète en C++.

Et il y aurait tant à dire quant à l'inclusion du fichier math.h, à commencer par le fait que, s'agissant de simple conversions, il est purement et simplement inutile, étant donné qu'il déclare plutôt les fonctions nécessaires à la trigonométrie et à la gestion de nombres complexes ...

**yakamoneye** · 24/05/2008, 04h01

voici les bases "rapides" de la classe string :
string
depuis son principal constructeur jusqu'aux methodes les plus utilisées ...
cette classe evite bien des problèmes d'algorithmique ou de gestion de la mémoire... mais c'est la solution de facilité !!!

**coyotte507** · 24/05/2008, 16h56

Salut,

dans tous les cas similaires à ça:

Code :

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

while (*nptr!='.')

si tu veux ne pas avoir de segfault, fait:

Code :

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

while (*nptr!='.' && *nptr != '\0')

La même chose avec le '+', le '/', ...

Sinon, décommente cette ligne:

Code :

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

   // return (resultat);

Dans ton 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
18
19
20
21
22
23
int main(int argc, char* argv[])
{
   char a [40];
   float *resultat;
   int l,i;
   l=1;
   while (l!=0)
   {
       cout << "donner la chaine : ";
       cin >> a;
       resultat = convert_POSS(a);
       for(i=0;i<40;i++)
            cout << resultat[i] <<endl;
       cout << "voulez vous continuez (0,1) : ";
       while (!(cin >> l))
       {
          cin.clear();
       }
       delete [] resultat;
   }
 
   return 0;
}

Je crois que ça améliore les choses par rapport à la segfault, mais je ne sais pas si les résultats sont ceux que tu escomptes.

Enfin, faire ça avec les stringstreams est beaucoup plus simple:

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
#include <sstream>
#include <iostream>
#include <string>
 
using namespace std;
 
float * convert_POSS(const string &item)
{
     float *resultat = (float*)calloc(40,sizeof(float));
     char c;
     istringstream donnees;
     int i = 0;
 
     donnees.str(item);
     while (donnees >> resultat[i++])
     {
            donnees >> c; // peut-être un meilleur moyen de faire ça!
     }
 
     return resultat;
}
 
int main ()
{
     string a;
     int continuer = true;
     float *resultat;
 
     while (continuer)
     {
          cout << "Entrez la chaine: ";
          cin >> a;
 
          resultat = convert_POSS(a);
          for (int i = 0; i < 40; i++)
          {
                  cout << resultat[i] << endl;
          }
          cout << "Voulez vous continuer? (0/1): ";
          while (!(cin >> continuer))
          {
                  cin.clear();
          }
          free(resultat);
      }
 
      return 0;
}

Y as-tu le droit, ou pas?

**tina-ze-star** · 24/05/2008, 20h04

coyotte, t un petit genie, a ton age... je m'incline devant toi.
Le programme que tu ma proposé marche très bien, je n sais même pas comment!! waw. la je vais le décortiquer, et essayer de le comprendre et comprendre toutes les instructions que tu a utilisé, puis le peaufiner et faire des petits changements,
car ça c'été un petit test, je n'utiliseré pas un simple tableau pour ranger les valeur mais plutot une matrice à 2 colonnes où seront placé les deux premieres valeurs qui se trouvent avant le +(deus à deux).

Et si je trouve quelques petits probleme je vais faire appel a toi (j'espere que tu n'aura pas marre de moi

).

En tout cas Merci c tres gentil.

E merci aux deux autres, c tres gentil, de vous casser la tete pour moi.

Je te tiens au courant..

**coyotte507** · 24/05/2008, 23h31

Ok,

n'hésite pas si tu as des questions, mon code n'est pas très commenté

Conversion du type caractere

C++

Discussions similaires

Partager

Partager