improving hashmap conditions

Author: Sharmalm

lightning/src/routing/router.rs

@@ -1243,28 +1243,26 @@ where L::Target: Logger {
 					// around again with a higher amount.
 					if !contributes_sufficient_value || exceeds_max_path_length ||
 						exceeds_cltv_delta_limit || payment_failed_on_this_channel {
+						 
+						let logger_test = crate::util::test_utils::TestLogger::new();
 						let mut is_first_hop = true;
-						// if let Some(target) = first_hop_targets.get(&NodeId::from_pubkey(&our_node_pubkey)) {
-						for (key ,value) in &first_hop_targets {
-							if (key == &NodeId::from_pubkey(&our_node_pubkey)) { 
-								for channel_details in value {
-									if let Some(short_channel_id_target) = channel_details.short_channel_id{
-										if short_channel_id_target == short_channel_id { // short_channel_id is hop id of candidate.
-											if is_first_hop && !contributes_sufficient_value {
-        											log_trace!(logger, "First Hop {short_channel_id} is excluded due to insufficient value");
-    											} else if is_first_hop && exceeds_max_path_length {
-    												log_trace!(logger, "First Hop {short_channel_id} is excluded due to candidate hop excluded max path length");	
-    											} else if is_first_hop && exceeds_cltv_delta_limit {
-    												log_trace!(logger, "First Hop {short_channel_id} is excluded beacause it exceed the maximum total cltv expiry limit");		
-    											} else if is_first_hop && payment_failed_on_this_channel {
-    												log_trace!(logger, "First Hop {short_channel_id} is excluded beacause it was failed previously");
-    											}
-    											is_first_hop = false;			
-    										}
-									}	
-								}
-							}
-						}
+						for (key , _channel) in &first_hop_targets {
+							if key == &$dest_node_id {
+								if is_first_hop && !contributes_sufficient_value {
+        								log_trace!(logger, "First Hop {} is excluded due to insufficient value", short_channel_id);
+        									
+    								} else if is_first_hop && exceeds_max_path_length {
+    									log_trace!(logger, "First Hop {} is excluded due to candidate hop excluded max path length", short_channel_id);
+    									
+    								} else if is_first_hop && exceeds_cltv_delta_limit {
+    									log_trace!(logger, "First Hop {} is excluded beacause it exceed the maximum total cltv expiry limit" , short_channel_id);
+    										
+    								} else if is_first_hop && payment_failed_on_this_channel {
+    									log_trace!(logger, "First Hop {} is excluded beacause it was failed previously" , short_channel_id);
+    								}
+    								is_first_hop = false;			
+    							} // commit
+    						}
 						// Path isn't useful, ignore it and move on.
 					} else if may_overpay_to_meet_path_minimum_msat {
 						hit_minimum_limit = true;