Skip to content

Commit c0f1242

Browse files
author
WeiziPlus
committed
1.0.8.2 修复findbugs提示的问题
1 parent d6b8659 commit c0f1242

File tree

3 files changed

+10
-10
lines changed

3 files changed

+10
-10
lines changed

springboot/demo-common/src/main/java/com/weiziplus/common/base/BaseService.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -725,7 +725,7 @@ protected <T> T baseFindOneDataByClassAndBaseWhereList(Class<T> clazz, List<Base
725725
if (null == clazz || null == baseWhereList || 0 >= baseWhereList.size()) {
726726
return null;
727727
}
728-
if (1 == baseWhereList.size() && BaseWhere.Where.EQUAL.equals(baseWhereList.get(0).getWhere())) {
728+
if (1 == baseWhereList.size() && BaseWhere.Where.EQUAL.getValue().equals(baseWhereList.get(0).getWhere())) {
729729
log.debug("如果baseWhere数组只有一个的话推荐使用baseFindOneDataByClassAndColumnAndValue");
730730
}
731731
String tableName = getTableName(clazz);
@@ -823,7 +823,7 @@ protected <T> List<T> baseFindListByClassAndBaseWhereList(Class<T> clazz, List<B
823823
if (null == clazz || null == baseWhereList || 0 >= baseWhereList.size()) {
824824
return null;
825825
}
826-
if (1 == baseWhereList.size() && BaseWhere.Where.EQUAL.equals(baseWhereList.get(0).getWhere())) {
826+
if (1 == baseWhereList.size() && BaseWhere.Where.EQUAL.getValue().equals(baseWhereList.get(0).getWhere())) {
827827
log.debug("如果baseWhere数组只有一个的话推荐使用baseFindOneDataByClassAndColumnAndValue");
828828
}
829829
String tableName = getTableName(clazz);

springboot/demo-pc/src/main/java/com/weiziplus/pc/common/interceptor/AuthorizationInterceptor.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -300,19 +300,19 @@ private boolean handleSign(HttpServletRequest request, HttpServletResponse respo
300300
return true;
301301
}
302302
TreeMap<String, String[]> stringTreeMap = new TreeMap<>(request.getParameterMap());
303-
Set<String> keySet = stringTreeMap.keySet();
304303
String signParam = "__sign";
305-
if (!keySet.contains(signParam)) {
304+
if (!stringTreeMap.containsKey(signParam)) {
306305
handleResponse(response, ResultUtils.error("签名错误"));
307306
return false;
308307
}
309308
StringBuilder stringBuilder = new StringBuilder();
310-
for (String key : keySet) {
309+
for (Map.Entry<String, String[]> stringEntry : stringTreeMap.entrySet()) {
310+
String key = stringEntry.getKey();
311311
//跳过__sign参数
312312
if (signParam.equals(key)) {
313313
continue;
314314
}
315-
stringBuilder.append(key).append("=").append(stringTreeMap.get(key)[0]).append("&");
315+
stringBuilder.append(key).append("=").append(stringEntry.getValue()[0]).append("&");
316316
}
317317
String s = Md5Utils.encodeNotSalt(stringBuilder.substring(0, stringBuilder.length() - 1));
318318
if (stringTreeMap.get(signParam)[0].equals(s)) {

springboot/demo-web/src/main/java/com/weiziplus/web/common/interceptor/AuthorizationInterceptor.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -260,19 +260,19 @@ private boolean handleSign(HttpServletRequest request, HttpServletResponse respo
260260
return true;
261261
}
262262
TreeMap<String, String[]> stringTreeMap = new TreeMap<>(request.getParameterMap());
263-
Set<String> keySet = stringTreeMap.keySet();
264263
String signParam = "__sign";
265-
if (!keySet.contains(signParam)) {
264+
if (!stringTreeMap.containsKey(signParam)) {
266265
handleResponse(response, ResultUtils.error("签名错误"));
267266
return false;
268267
}
269268
StringBuilder stringBuilder = new StringBuilder();
270-
for (String key : keySet) {
269+
for (Map.Entry<String, String[]> stringEntry : stringTreeMap.entrySet()) {
270+
String key = stringEntry.getKey();
271271
//跳过__sign参数
272272
if (signParam.equals(key)) {
273273
continue;
274274
}
275-
stringBuilder.append(key).append("=").append(stringTreeMap.get(key)[0]).append("&");
275+
stringBuilder.append(key).append("=").append(stringEntry.getValue()[0]).append("&");
276276
}
277277
String s = Md5Utils.encodeNotSalt(stringBuilder.substring(0, stringBuilder.length() - 1));
278278
if (stringTreeMap.get(signParam)[0].equals(s)) {

0 commit comments

Comments
 (0)