feat(net): optimize transaction execution logic

Author: xxo1shine

framework/src/main/java/org/tron/core/net/messagehandler/TransactionsMsgHandler.java

@@ -116,8 +116,8 @@ private void handleSmartContract() {
   }
 
   private void handleTransaction(PeerConnection peer, TransactionMessage trx) {
-    if (peer.isDisconnect()) {
-      logger.warn("Drop trx {} from {}, peer is disconnect", trx.getMessageId(),
+    if (peer.isBadPeer()) {
+      logger.warn("Drop trx {} from {}, peer is bad peer", trx.getMessageId(),
           peer.getInetAddress());
       return;
     }
@@ -133,6 +133,7 @@ private void handleTransaction(PeerConnection peer, TransactionMessage trx) {
       logger.warn("Trx {} from peer {} process failed. type: {}, reason: {}",
           trx.getMessageId(), peer.getInetAddress(), e.getType(), e.getMessage());
       if (e.getType().equals(TypeEnum.BAD_TRX)) {
+        peer.setBadPeer(true);
         peer.disconnect(ReasonCode.BAD_TX);
       }
     } catch (Exception e) {

framework/src/main/java/org/tron/core/net/peer/PeerConnection.java

@@ -71,6 +71,10 @@ public class PeerConnection {
   @Getter
   private volatile boolean fetchAble;
 
+  @Setter
+  @Getter
+  private volatile boolean isBadPeer;
+
   @Getter
   @Setter
   private ByteString address;

framework/src/test/java/org/tron/core/net/peer/PeerConnectionTest.java

@@ -0,0 +1,19 @@
+package org.tron.core.net.peer;
+
+import org.junit.Assert;
+import org.junit.Test;
+
+public class PeerConnectionTest {
+
+  @Test
+  public void testVariableDefaultValue() {
+    PeerConnection peerConnection = new PeerConnection();
+    Assert.assertTrue(!peerConnection.isBadPeer());
+    Assert.assertTrue(!peerConnection.isFetchAble());
+    Assert.assertTrue(peerConnection.isIdle());
+    Assert.assertTrue(!peerConnection.isRelayPeer());
+    Assert.assertTrue(peerConnection.isNeedSyncFromPeer());
+    Assert.assertTrue(peerConnection.isNeedSyncFromUs());
+    Assert.assertTrue(!peerConnection.isSyncFinish());
+  }
+}