else-Anweisung wird scheinbar ignoriert

Question

Wenn die innere if-Anweisung nicht als wahr ausgewertet wird, würde ich erwarten, dass der Else-Block ausgeführt wird (appProtocol auf APP_PROTO_RTP gesetzt). Dies geschieht jedoch nicht. Stattdessen scheint die else-Anweisung vollständig ignoriert zu werden. Ich kann mir nicht vorstellen, warum das so ist.

Wie Sie in meiner gdb-Sitzung sehen können, funktioniert die if-Anweisung zum ersten Mal, und appProtocol wird (wie erwartet) auf APP_PROTO_SIP gesetzt. Beim zweiten Mal schlägt das if fehl, aber anstatt in die else zu gehen und appProtocol auf APP_PROTO_RTP zu setzen, wird die Funktion vollständig zurückgesetzt, ohne dass appProtocol gesetzt wird. appProtocol bleibt auf APP_PROTO_INVALID gesetzt (der Wert, mit dem es im PacketRecord ctor initialisiert wird).

Breakpoint 1, PacketRecord::determineAppProtocol (this=0x805c6c8) at PacketRecord.cpp:156 
156  if (ipProtocol == IP_PROTO_UDP) 
(gdb) step 
158   std::istringstream ss(udpData); 
(gdb) 
159   std::string line; 
(gdb) 
160   if (getline(ss, line) && (line.find("SIP/2.0") != std::string::npos)) 
(gdb) 
162    appProtocol = APP_PROTO_SIP; 
(gdb) 
167   } 
(gdb) 
173 } 
(gdb) continue 
Continuing. 

Breakpoint 1, PacketRecord::determineAppProtocol (this=0x8065388) at PacketRecord.cpp:156 
156  if (ipProtocol == IP_PROTO_UDP) 
(gdb) step 
158   std::istringstream ss(udpData); 
(gdb) 
159   std::string line; 
(gdb) 
160   if (getline(ss, line) && (line.find("SIP/2.0") != std::string::npos)) 
(gdb) 
167   } 
(gdb) 
173 } 
(gdb) 

Answer 1

Sie ersetzen sollte, ist

appProtocol == APP_PROTO_RTP; 

von

appProtocol = APP_PROTO_RTP; 

(keine doppelte Gleichheitszeichen)

Die else-Anweisung ausgeführt. Aber Sie geben den Wert für appProtocol nicht darin ein.

Answer 2

Sie ordnen nicht zu, Sie vergleichen. Verwenden Sie =, nicht ==

Answer 3

Sie verwenden eine Gleichheit Aussage hier:

appProtocol == APP_PROTO_RTP; 

eher als ein Stelldichein.

Der richtige Code ist:

appProtocol = APP_PROTO_RTP;