refactor(group): 采用 slices.Concat 代替 group.mergeOption

Author: caixw

examples/ctx/ctx.go

@@ -51,9 +51,7 @@ func methodNotAllowedBuilder(p types.Node) Handler {
 	})
 }
 
-func notFound(ctx *CTX) {
-	ctx.W.WriteHeader(http.StatusNotFound)
-}
+func notFound(ctx *CTX) { ctx.W.WriteHeader(http.StatusNotFound) }
 
 func NewRouters(o ...mux.Option) *Routers {
 	return group.NewOf[Handler](call, HandlerFunc(notFound), methodNotAllowedBuilder, optionsHandlerBuilder, o...)

group/group.go

@@ -74,30 +74,12 @@ func (g *GroupOf[T]) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 //
 // 新路由会继承 [NewOf] 中指定的参数，其中的 o 可以覆盖由 [NewOf] 中指定的相关参数；
 func (g *GroupOf[T]) New(name string, matcher Matcher, o ...mux.Option) *mux.RouterOf[T] {
-	o = g.mergeOption(o...)
+	o = slices.Concat(g.options, o)
 	r := mux.NewRouterOf(name, g.call, g.originNotFound, g.methodNotAllowedBuilder, g.optionsBuilder, o...)
 	g.Add(matcher, r)
 	return r
 }
 
-// 将 g.options 与 o 合并，保证 g.options 在前且不会被破坏
-func (g *GroupOf[T]) mergeOption(o ...mux.Option) []mux.Option {
-	l1 := len(g.options)
-	if l1 == 0 {
-		return o
-	}
-
-	l2 := len(o)
-	if l2 == 0 {
-		return g.options
-	}
-
-	ret := make([]mux.Option, l1+l2)
-	size := copy(ret, g.options)
-	copy(ret[size:], o)
-	return ret
-}
-
 // Add 添加路由
 //
 // matcher 用于判断进入 r 的条件，如果为空，则表示不作判断。

group/group_test.go

@@ -12,7 +12,6 @@ import (
 	"github.com/issue9/assert/v4/rest"
 
 	"github.com/issue9/mux/v8"
-	"github.com/issue9/mux/v8/internal/options"
 	"github.com/issue9/mux/v8/internal/tree"
 	"github.com/issue9/mux/v8/types"
 )
@@ -39,22 +38,6 @@ func newRouter(a *assert.Assertion, name string, o ...mux.Option) *mux.RouterOf[
 	return r
 }
 
-func TestGroupOf_mergeOption(t *testing.T) {
-	a := assert.New(t, false)
-	g := newGroup(a)
-	oo, err := options.Build(g.mergeOption(mux.Lock(true))...)
-	a.NotError(err).True(oo.Lock).Empty(oo.URLDomain)
-
-	g = newGroup(a, mux.Lock(true))
-	oo, err = options.Build(g.mergeOption()...)
-	a.NotError(err).True(oo.Lock).Empty(oo.URLDomain)
-	oo, err = options.Build(g.mergeOption(mux.Lock(false))...)
-	a.NotError(err).False(oo.Lock).Empty(oo.URLDomain)
-
-	oo, err = options.Build(g.mergeOption(mux.Lock(false), mux.URLDomain("https://example.com"))...)
-	a.NotError(err).False(oo.Lock).Equal(oo.URLDomain, "https://example.com")
-}
-
 func TestGroupOf_Use(t *testing.T) {
 	a := assert.New(t, false)
 	g := newGroup(a)

internal/tree/tree.go

@@ -58,12 +58,11 @@ func New[T any](
 	i *syntax.Interceptors,
 	notFound T,
 	trace bool,
-	methodNotAllowedBuilder,
-	optionsBuilder types.BuildNodeHandleOf[T],
+	methodNotAllowedBuilder, optionsBuilder types.BuildNodeHandleOf[T],
 ) *Tree[T] {
 	s, err := i.NewSegment("")
 	if err != nil {
-		panic("发生了不该发生的错误，应该是 syntax.NewSegment 逻辑发生变化" + err.Error())
+		panic("发生了不该发生的错误，应该是 syntax.NewSegment 逻辑发生变化：" + err.Error())
 	}
 
 	t := &Tree[T]{

router.go

@@ -65,8 +65,16 @@ type (
 //
 // name 路由名称，可以为空；
 // methodNotAllowedBuilder 和 optionsBuilder 可以自定义 405 和 OPTIONS 请求的处理方式；
+// o 用于指定一些可选的参数；
+//
 // T 表示用户用于处理路由项的方法。
-func NewRouterOf[T any](name string, call CallOf[T], notFound T, methodNotAllowedBuilder, optionsBuilder types.BuildNodeHandleOf[T], o ...Option) *RouterOf[T] {
+func NewRouterOf[T any](
+	name string,
+	call CallOf[T],
+	notFound T,
+	methodNotAllowedBuilder, optionsBuilder types.BuildNodeHandleOf[T],
+	o ...Option,
+) *RouterOf[T] {
 	opt, err := options.Build(o...)
 	if err != nil {
 		panic(err)