refactor: use slices.Contains to simplify code

ianlv · ianlv · commit ef4b0005a9d2 · 2025-07-02T18:37:34.000+08:00
Signed-off-by: ianlv &lt;sunlvyun@outlook.com&gt;
diff --git a/logger/hook.go b/logger/hook.go
@@ -13,6 +13,7 @@ import (
 	"io"
 	"net/http"
 	"net/smtp"
+	"slices"
 	"time"
 )
 
@@ -93,10 +94,8 @@ func initHookByConfig(conf *viper.Viper) (hook zap.Option, err error) {
 	}
 
 	return zap.Hooks(func(entry zapcore.Entry) error {
-		for _, level := range levels {
-			if level == entry.Level {
-				return internalHook.call(entry)
-			}
+		if slices.Contains(levels, entry.Level) {
+			return internalHook.call(entry)
 		}
 		return nil
 	}), nil

Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,7 @@ import (`
`13`	`13`	`"io"`
`14`	`14`	`"net/http"`
`15`	`15`	`"net/smtp"`
	`16`	`+ "slices"`
`16`	`17`	`"time"`
`17`	`18`	`)`
`18`	`19`
`@@ -93,10 +94,8 @@ func initHookByConfig(conf *viper.Viper) (hook zap.Option, err error) {`
`93`	`94`	`}`
`94`	`95`
`95`	`96`	`return zap.Hooks(func(entry zapcore.Entry) error {`
`96`		`- for _, level := range levels {`
`97`		`- if level == entry.Level {`
`98`		`- return internalHook.call(entry)`
`99`		`- }`
	`97`	`+ if slices.Contains(levels, entry.Level) {`
	`98`	`+ return internalHook.call(entry)`
`100`	`99`	`}`
`101`	`100`	`return nil`
`102`	`101`	`}), nil`