Skip to content

Commit 5014bfa

Browse files
committed
duplicate code improvement
1 parent 31880b9 commit 5014bfa

File tree

3 files changed

+25
-42
lines changed

3 files changed

+25
-42
lines changed

framework/src/main/java/org/tron/core/services/http/GetBrokerageServlet.java

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ public class GetBrokerageServlet extends RateLimiterServlet {
2424
protected void doGet(HttpServletRequest request, HttpServletResponse response) {
2525
try {
2626
int value = 0;
27-
byte[] address = getAddress(request);
27+
byte[] address = Util.getAddress(request);
2828
long cycle = manager.getDynamicPropertiesStore().getCurrentCycleNumber();
2929
if (address != null) {
3030
value = manager.getDelegationStore().getBrokerage(cycle, address);
@@ -39,24 +39,4 @@ protected void doPost(HttpServletRequest request, HttpServletResponse response)
3939
doGet(request, response);
4040
}
4141

42-
private byte[] getAddress(HttpServletRequest request) throws Exception {
43-
byte[] address = null;
44-
String addressParam = "address";
45-
String addressStr = request.getParameter(addressParam);
46-
if (StringUtils.isBlank(addressStr)) {
47-
String input = request.getReader().lines()
48-
.collect(Collectors.joining(System.lineSeparator()));
49-
Util.checkBodySize(input);
50-
JSONObject jsonObject = JSONObject.parseObject(input);
51-
addressStr = jsonObject.getString(addressParam);
52-
}
53-
if (StringUtils.isNotBlank(addressStr)) {
54-
if (StringUtils.startsWith(addressStr, Constant.ADD_PRE_FIX_STRING_MAINNET)) {
55-
address = Hex.decode(addressStr);
56-
} else {
57-
address = Wallet.decodeFromBase58Check(addressStr);
58-
}
59-
}
60-
return address;
61-
}
6242
}

framework/src/main/java/org/tron/core/services/http/GetRewardServlet.java

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ public class GetRewardServlet extends RateLimiterServlet {
2525
protected void doGet(HttpServletRequest request, HttpServletResponse response) {
2626
try {
2727
long value = 0;
28-
byte[] address = getAddress(request);
28+
byte[] address = Util.getAddress(request);
2929
if (address != null) {
3030
value = manager.getDelegationService().queryReward(address);
3131
}
@@ -44,24 +44,4 @@ protected void doPost(HttpServletRequest request, HttpServletResponse response)
4444
doGet(request, response);
4545
}
4646

47-
private byte[] getAddress(HttpServletRequest request) throws Exception {
48-
byte[] address = null;
49-
String addressParam = "address";
50-
String addressStr = request.getParameter(addressParam);
51-
if (StringUtils.isBlank(addressStr)) {
52-
String input = request.getReader().lines()
53-
.collect(Collectors.joining(System.lineSeparator()));
54-
Util.checkBodySize(input);
55-
JSONObject jsonObject = JSONObject.parseObject(input);
56-
addressStr = jsonObject.getString(addressParam);
57-
}
58-
if (StringUtils.isNotBlank(addressStr)) {
59-
if (StringUtils.startsWith(addressStr, Constant.ADD_PRE_FIX_STRING_MAINNET)) {
60-
address = Hex.decode(addressStr);
61-
} else {
62-
address = Wallet.decodeFromBase58Check(addressStr);
63-
}
64-
}
65-
return address;
66-
}
6747
}

framework/src/main/java/org/tron/core/services/http/Util.java

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
import java.nio.charset.Charset;
1515
import java.security.InvalidParameterException;
1616
import java.util.List;
17+
import java.util.stream.Collectors;
1718
import javax.servlet.http.HttpServletRequest;
1819
import javax.servlet.http.HttpServletResponse;
1920
import lombok.extern.slf4j.Slf4j;
@@ -31,6 +32,7 @@
3132
import org.tron.common.utils.DecodeUtil;
3233
import org.tron.common.utils.Hash;
3334
import org.tron.common.utils.Sha256Hash;
35+
import org.tron.core.Constant;
3436
import org.tron.core.Wallet;
3537
import org.tron.core.actuator.TransactionFactory;
3638
import org.tron.core.capsule.BlockCapsule;
@@ -433,4 +435,25 @@ public static void printAccount(Account reply, HttpServletResponse response, Boo
433435
}
434436
}
435437

438+
public static byte[] getAddress(HttpServletRequest request) throws Exception {
439+
byte[] address = null;
440+
String addressParam = "address";
441+
String addressStr = request.getParameter(addressParam);
442+
if (org.apache.commons.lang3.StringUtils.isBlank(addressStr)) {
443+
String input = request.getReader().lines()
444+
.collect(Collectors.joining(System.lineSeparator()));
445+
Util.checkBodySize(input);
446+
JSONObject jsonObject = JSONObject.parseObject(input);
447+
addressStr = jsonObject.getString(addressParam);
448+
}
449+
if (org.apache.commons.lang3.StringUtils.isNotBlank(addressStr)) {
450+
if (org.apache.commons.lang3.StringUtils.startsWith(addressStr, Constant.ADD_PRE_FIX_STRING_MAINNET)) {
451+
address = Hex.decode(addressStr);
452+
} else {
453+
address = Wallet.decodeFromBase58Check(addressStr);
454+
}
455+
}
456+
return address;
457+
}
458+
436459
}

0 commit comments

Comments
 (0)