From 36415932101a994f1b471b6922939853358c3610 Mon Sep 17 00:00:00 2001 From: LamGC Date: Thu, 2 Sep 2021 18:55:05 +0800 Subject: [PATCH] =?UTF-8?q?refactor:=20=E8=B0=83=E6=95=B4=20logger=20?= =?UTF-8?q?=E8=8E=B7=E5=8F=96=E6=96=B9=E5=BC=8F.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 调整方法名称以避免 Groovy 误将其当做 Getter 来获取脚本日志记录器. --- .../script/groovy/trigger/TimerTrigger.java | 22 +++++++++---------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/src/main/java/net/lamgc/oracle/sentry/script/groovy/trigger/TimerTrigger.java b/src/main/java/net/lamgc/oracle/sentry/script/groovy/trigger/TimerTrigger.java index e78c7e4..f23e266 100644 --- a/src/main/java/net/lamgc/oracle/sentry/script/groovy/trigger/TimerTrigger.java +++ b/src/main/java/net/lamgc/oracle/sentry/script/groovy/trigger/TimerTrigger.java @@ -23,7 +23,7 @@ public class TimerTrigger implements GroovyTrigger { SCHEDULER.setThreadFactory(new ThreadFactoryBuilder() .setNameFormat("Groovy-TimerTrigger-%d") .build()); - SCHEDULER.setErrorHandler(t -> getLog().error("脚本执行时发生异常.", t)); + SCHEDULER.setErrorHandler(t -> triggerLog().error("脚本执行时发生异常.", t)); SCHEDULER.initialize(); } @@ -44,24 +44,24 @@ public class TimerTrigger implements GroovyTrigger { @Override public synchronized void run(Closure runnable) { if (future != null) { - getLog().warn("脚本存在多个 run 代码块, 已忽略."); + triggerLog().warn("脚本存在多个 run 代码块, 已忽略."); return; } if (trigger == null) { - if (!getLog().isDebugEnabled()) { - getLog().warn("脚本尚未设置 Cron 时间表达式, 任务将不会执行(堆栈信息请检查调试级别日志)."); + if (!triggerLog().isDebugEnabled()) { + triggerLog().warn("脚本尚未设置 Cron 时间表达式, 任务将不会执行(堆栈信息请检查调试级别日志)."); } else { - getLog().warn("{} - 脚本尚未设置 Cron 时间表达式, 任务将不会执行(堆栈信息请检查调试级别日志).", this); - getLog().warn("{} - 脚本尚未设置 Cron 时间表达式, 任务将不会执行.\n{}", this, Throwables.getStackTraceAsString(new Exception())); + triggerLog().warn("{} - 脚本尚未设置 Cron 时间表达式, 任务将不会执行(堆栈信息请检查调试级别日志).", this); + triggerLog().warn("{} - 脚本尚未设置 Cron 时间表达式, 任务将不会执行.\n{}", this, Throwables.getStackTraceAsString(new Exception())); } return; } else if (runnable == null) { - if (!getLog().isDebugEnabled()) { - getLog().warn("脚本尚未设置 Cron 时间表达式, 任务将不会执行(堆栈信息请检查调试级别日志)."); + if (!triggerLog().isDebugEnabled()) { + triggerLog().warn("脚本尚未设置 Cron 时间表达式, 任务将不会执行(堆栈信息请检查调试级别日志)."); } else { - getLog().warn("{} - 脚本尚未设置任务动作, 任务将不会执行(堆栈信息请检查调试级别日志).", this); - getLog().warn("{} - 脚本尚未设置任务动作, 任务将不会执行.\n{}", this, Throwables.getStackTraceAsString(new Exception())); + triggerLog().warn("{} - 脚本尚未设置任务动作, 任务将不会执行(堆栈信息请检查调试级别日志).", this); + triggerLog().warn("{} - 脚本尚未设置任务动作, 任务将不会执行.\n{}", this, Throwables.getStackTraceAsString(new Exception())); } return; } @@ -76,7 +76,7 @@ public class TimerTrigger implements GroovyTrigger { } } - private static Logger getLog() { + private static Logger triggerLog() { return LoggerFactory.getLogger(TimerTrigger.class); }