diff options
author | Anton Hrytsevich <ahrytsevich> | 2015-12-25 16:06:19 +0200 |
---|---|---|
committer | Anton Hrytsevich <ahrytsevich> | 2015-12-25 16:06:19 +0200 |
commit | 8304f61a7b4a6bd4257d0105ce9ef166abef0d2a (patch) | |
tree | b3c41ef4cb762a9a5cd9152b6e3cc9e689f54e55 /src/components/connection_handler | |
parent | 968ac8303084222fcf3be8c42f03f0db843c59d8 (diff) | |
download | sdl_core-8304f61a7b4a6bd4257d0105ce9ef166abef0d2a.tar.gz |
Make if statment more wide after review
Make more wide if statment. It include map search.
Diffstat (limited to 'src/components/connection_handler')
-rw-r--r-- | src/components/connection_handler/src/connection_handler_impl.cc | 34 |
1 files changed, 18 insertions, 16 deletions
diff --git a/src/components/connection_handler/src/connection_handler_impl.cc b/src/components/connection_handler/src/connection_handler_impl.cc index f1976c0c40..45215be7c9 100644 --- a/src/components/connection_handler/src/connection_handler_impl.cc +++ b/src/components/connection_handler/src/connection_handler_impl.cc @@ -796,23 +796,25 @@ void ConnectionHandlerImpl::CloseSession(ConnectionHandle connection_handle, protocol_handler_->SendEndSession(connection_handle, session_id); } - SessionMap::const_iterator session_map_itr = session_map.find(session_id); - if (session_map_itr != session_map.end() && connection_handler_observer_) { - const uint32_t session_key = KeyFromPair(connection_id, session_id); - const Session &session = session_map_itr->second; - const ServiceList &service_list = session.service_list; - - ServiceList::const_iterator service_list_itr = service_list.begin(); - for (;service_list_itr != service_list.end(); ++service_list_itr) { - const protocol_handler::ServiceType service_type = - service_list_itr->service_type; - connection_handler_observer_->OnServiceEndedCallback(session_key, - service_type, - close_reason); + if ( connection_handler_observer_ ) { + SessionMap::const_iterator session_map_itr = session_map.find(session_id); + if (session_map_itr != session_map.end()) { + const uint32_t session_key = KeyFromPair(connection_id, session_id); + const Session &session = session_map_itr->second; + const ServiceList &service_list = session.service_list; + + ServiceList::const_iterator service_list_itr = service_list.begin(); + for (;service_list_itr != service_list.end(); ++service_list_itr) { + const protocol_handler::ServiceType service_type = + service_list_itr->service_type; + connection_handler_observer_->OnServiceEndedCallback(session_key, + service_type, + close_reason); + } + } else { + LOG4CXX_ERROR(logger_, "Session with id: " << session_id << " not found"); + return; } - } else { - LOG4CXX_ERROR(logger_, "Session with id: " << session_id << " not found"); - return; } LOG4CXX_DEBUG(logger_, "Session with id: " << session_id << " has been closed successfully"); } |