Discussion :

[FenX.]

Bonjour à tous

Voici un résumé du code qui me pose un petit problème :

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
69
70
 
typedef map<char*, char*> M_OID;
typedef map<char*, M_OID> M_OIDS;
typedef map<char*, M_OIDS> M_PDUS;
typedef map<char*, M_PDUS> M_LOGONS;
typedef map<char*, M_LOGONS> M_STS;
 
M_OID mapOid;
M_OIDS mapOids;
M_PDUS mapPdus;
M_LOGONS mapLogons;
M_STS mapSts;
 
 
void fillMaps()
{
	mapOid["Name"] = ".1.2";	
	mapOids["OID12"] = mapOid;
	mapPdus["pdu1"] = mapOids;
	mapLogons["Normal"] = mapPdus;
	mapSts["ABC123"] = mapLogons;
}
 
 
void readMaps()
{
	M_OIDS::iterator itOid;
	M_PDUS::iterator itPdu;
	M_LOGONS::iterator itLogon;
	M_STS::iterator itSt;
 
	M_OID mapOidFound;
	M_OIDS mapOidsBrowsed;
	M_PDUS mapPdusFound;
	M_LOGONS mapLogonsFound;
 
	itSt = mapSts.find("ABC123");
	if ( itSt != mapSts.end() )
	{
		mapLogonsFound = itSt->second;
 
		itLogon = mapLogonsFound.find("Normal");
		if ( itLogon != mapLogonsFound.end() )
		{
			mapPdusFound = itLogon->second;
 
			// Dans mon code réel, je ne connais pas le nom de la PDU où se trouve l'OID recherché,
			// alors je vais toutes les parcourir
			for ( itPdu = mapPdusFound.begin(); itPdu != mapPdusFound.end(); itPdu++ ) 
			{
				mapOidsBrowsed = itPdu->second;
 
				itOid = mapOidsBrowsed.find("OID12");
				if ( itOid != mapOidsBrowsed.end() )
				{
					mapOidFound = itOid->second;
 
					printf("OID Found: %s", mapOidFound["Name"]); 
				}
			}
		}
	}
}
 
 
void main()
{
	fillMaps();
	readMaps();
}

• En effet, j'ai une erreur sur chacune des lignes de type : <= Edit : Corrigé par Poukill
  

  Code :

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

  mapxFound = itX->second();

• De plus, le find ne semble pas fonctionner... : lors du " if iterator != map.end() " il ne rentre jamais dans le then...

Où se trouve le problème d'après vous ? Car là j'avoue que je sèche complètement...

Merci d'avance pour votre aide !

[poukill]

Code :

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

mapxFound = itX->second;

suffira!

Quelques remarques:

• pourquoi utiliser des char*, et pas des std::string ?
• Le printf peut avantageusement être remplacé par un std::cout

[FenX.]

Tout simplement, suis-je bête

Concernant le fait que j'utilise des char* et printf, c'est à cause du programme que je dois faire évoluer et qui a déjà été ( mal ! ) codé de cette manière...

Et concernant mon problème de find ?

Merci beaucoup pour ta réponse en tout cas

[Laurent Gomila]

find ne pourra jamais fonctionner avec des char*, car la comparaison se fera sur les adresses et non sur le contenu des chaînes.

[poukill]

Remplace tes char* par des std::string !

[poukill]

Euh... Laurent, j'ai trouvé ça dans la doc de la STL:

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
struct ltstr
{
  bool operator()(const char* s1, const char* s2) const
  {
    return strcmp(s1, s2) < 0;
  }
};
 
int main()
{
  map<const char*, int, ltstr> months;
 
  months["january"] = 31;
  months["february"] = 28;
  months["march"] = 31;
  months["april"] = 30;
  months["may"] = 31;
  months["june"] = 30;
  months["july"] = 31;
  months["august"] = 31;
  months["september"] = 30;
  months["october"] = 31;
  months["november"] = 30;
  months["december"] = 31;
 
  cout << "june -> " << months["june"] << endl;
  map<const char*, int, ltstr>::iterator cur  = months.find("june");
  map<const char*, int, ltstr>::iterator prev = cur;
  map<const char*, int, ltstr>::iterator next = cur;    
  ++next;
  --prev;
  cout << "Previous (in alphabetical order) is " << (*prev).first << endl;
  cout << "Next (in alphabetical order) is " << (*next).first << endl;
}

Y'a bien des char * non?
Et l'utilisation de find ne pose pas de problèmes ?