【fix sonar】 JobAlarmer.java文件
This commit is contained in:
+3
-3
@@ -29,8 +29,8 @@ public class JobAlarmer implements ApplicationContextAware, InitializingBean {
|
|||||||
@Override
|
@Override
|
||||||
public void afterPropertiesSet() throws Exception {
|
public void afterPropertiesSet() throws Exception {
|
||||||
Map<String, JobAlarm> serviceBeanMap = applicationContext.getBeansOfType(JobAlarm.class);
|
Map<String, JobAlarm> serviceBeanMap = applicationContext.getBeansOfType(JobAlarm.class);
|
||||||
if (serviceBeanMap != null && serviceBeanMap.size() > 0) {
|
if (!serviceBeanMap.isEmpty()) {
|
||||||
jobAlarmList = new ArrayList<JobAlarm>(serviceBeanMap.values());
|
jobAlarmList = new ArrayList<>(serviceBeanMap.values());
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -44,7 +44,7 @@ public class JobAlarmer implements ApplicationContextAware, InitializingBean {
|
|||||||
public boolean alarm(XxlJobInfo info, XxlJobLog jobLog) {
|
public boolean alarm(XxlJobInfo info, XxlJobLog jobLog) {
|
||||||
|
|
||||||
boolean result = false;
|
boolean result = false;
|
||||||
if (jobAlarmList!=null && jobAlarmList.size()>0) {
|
if (jobAlarmList!=null && !jobAlarmList.isEmpty()) {
|
||||||
result = true; // success means all-success
|
result = true; // success means all-success
|
||||||
for (JobAlarm alarm: jobAlarmList) {
|
for (JobAlarm alarm: jobAlarmList) {
|
||||||
boolean resultItem = false;
|
boolean resultItem = false;
|
||||||
|
|||||||
Reference in New Issue
Block a user