Discussion :

[lucky]

Bonjour,

sous VC++ 2010 (MFC) mode debug 64 bits (win7), j'utilise une bête fonction atof() pour convertir un tableau de char ; lors de la première conversion je récupère bien le double correspondant mais lors des conversions suivantes j'ai 0.0000.. et non la valeur ; ma chaine se termine bien par le caractère nul et j'ai comme chaine un truc du genre "0.7932". (observé avec espion sur la variable...)
Est ce lié à l'environnement 64 bits ou autres ?

Merci

[bacelar]

Votre code svp.

[lucky]

voici le code (en gros) de la fonction appelé plusieurs fois dans l'exécution

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
double fct(...)
{
double d_resistance = 0.0 ;
static char  buffer[60] = {'\0'} ;
static char  buffdest[60] = {'\0'} ;
 
//...........
 
 
		if (....)
		{
			buffer[7] = 0 ;                         // ex : buffer est " 0.7210Ohm" 
			memcpy(buffdest,buffer+1,7);
			// 			i = 1 ;
			if ( (buffer[1]) == '>')
			{
				d_resistance = 0.0 ;
				......
				return d_resistance ;
			} 
			else
			{
 
				.....
// point d'arret >> buffdest est bien = "0.7210" dans notre exemple
                                d_resistance = atof(buffdest);           // premier passage OK, second d_resistance = 0.00000000000
				.....
				m_st_recept_rs232 = strtmp ;
			}
		}
                else
                {
                       ....
                       return 0.0 ;
               }
 
return  d_resistance;
}

Merci

[lucky]

je viens de mettre :

Code :

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

			memset(buffdest,'\0',sizeof(buffdest));

avant le memcpy :
problème identique

[lucky]

Je viens d'essayer à la place :

Code :

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

d_resistance = strtod(buffdest,&stopstr);

problème identique !!!!

[bacelar]

Code fragile, static plus multi-threading => nimprotenawak

Code difficile à lire : indices, pointeurs, opération mémoire sur des char*, magique value '>' ...

Concrètement, faites un code complet qui reproduit le problème.

[lucky]

dernier code essayé (car j'ai un peu tout essayé...)
si vous arrivez à vous y retrouver :

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
 
double CPosteCtrleBougiesGCSView::RS232LectureResistance(void)
	{
		static char  buffer[60] = {'\0'} ;
// 		static char  buffdest[20] = {'\0'} ;
		char *  buffdest ;
		char * stopstr ;
		static CString strtmp ;
		double d_resistance = 0.0 ;
		UINT i = 1 ;
 
 
		if (RS232EnvoieRecept(charx04,buffer))         // reception de la chaine dans buffer
		{
			buffer[7] = 0 ;                                     // à partir du 7eme caractère : à éliminer
			buffdest = buffer ;
// 			memset(buffdest,'\0',sizeof(buffdest));
// 			memcpy(buffdest,buffer+1,7);
 
			if ( (buffer[1]) == '>')                             // car dans certain cas j'ai ">>>" comme chaine
			{
				d_resistance = 0.0 ;
				strtmp = _T("Pas de résistance lue") ;
				m_st_recept_rs232 = strtmp ;
				m_st_recept_rs232.SetBkColor(ROUGEVIF);
				return d_resistance ;
			} 
			else
			{
 
// 				d_resistance = 0.0f ;                                                          //*****************************************************************************
//  				d_resistance = atof(buffdest);                                            // le problème de conversion se trouve ici en testant en debug la variable d_resistance / buffdest (buuffdest = bonne chaine
				d_resistance = strtod(buffdest,&stopstr);                            // essayé avec des variables static ou non static...                      
				m_st_recept_rs232 = strtmp ;                                           //*****************************************************************************
			}
		}
		else
		{
			strtmp = _T("Erreur de lecture résistance") ;
			m_st_recept_rs232 = strtmp ;
			m_st_recept_rs232.SetBkColor(ROUGEVIF);
			return 0.0f ;
		}
 
		RS232EnvoieRecept(charx00,buffdest);
 
// 		strtmp.Format(_T("%s"),buffer+1);
		m_st_recept_rs232 =  strtmp;
 
		return d_resistance ;
	}

Merci