GetLocal() type-safety, naming, const, and formatting cleanups

Co-authored-by: Jon Atack <jon@atack.com>

src/net.cpp

@@ -148,30 +148,27 @@ uint16_t GetListenPort()
 // find 'best' local address for a particular peer
 [[nodiscard]] static bool GetLocal(CService& addr, const CNode& peer)
 {
-    if (!fListen)
+    if (!fListen) return false;
-        return false;
 
     int nBestScore = -1;
     int nBestReachability = -1;
     {
         LOCK(g_maplocalhost_mutex);
-        for (const auto& entry : mapLocalHost)
+        for (const auto& [local_addr, local_service_info] : mapLocalHost) {
-        {
             // For privacy reasons, don't advertise our privacy-network address
             // to other networks and don't advertise our other-network address
             // to privacy networks.
-            const Network our_net{entry.first.GetNetwork()};
+            const Network our_net{local_addr.GetNetwork()};
             const Network peers_net{peer.ConnectedThroughNetwork()};
             if (our_net != peers_net &&
                 (our_net == NET_ONION || our_net == NET_I2P ||
                  peers_net == NET_ONION || peers_net == NET_I2P)) {
                 continue;
             }
-            int nScore = entry.second.nScore;
+            const int nScore{local_service_info.nScore};
-            int nReachability = entry.first.GetReachabilityFrom(peer.addr);
+            const int nReachability{local_addr.GetReachabilityFrom(peer.addr)};
-            if (nReachability > nBestReachability || (nReachability == nBestReachability && nScore > nBestScore))
+            if (nReachability > nBestReachability || (nReachability == nBestReachability && nScore > nBestScore)) {
-            {
+                addr = CService{local_addr, local_service_info.nPort};
-                addr = CService(entry.first, entry.second.nPort);
                 nBestReachability = nReachability;
                 nBestScore = nScore;
             }