From 574f1b5265a344e3703f8a74e00267a570f3ffe6 Mon Sep 17 00:00:00 2001 From: zxysilent Date: Tue, 14 Jul 2026 17:42:25 +0800 Subject: [PATCH] refactor: replace traditional for loops with range loops for improved readability --- bind.go | 2 +- json_test.go | 6 +++--- perf_bench_test.go | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/bind.go b/bind.go index b562f3ca4..b0331a88a 100644 --- a/bind.go +++ b/bind.go @@ -187,7 +187,7 @@ func bindMetaFor(typ reflect.Type) *bindStructMeta { } n := typ.NumField() meta := &bindStructMeta{fields: make([]bindFieldMeta, n)} - for i := 0; i < n; i++ { + for i := range n { f := typ.Field(i) meta.fields[i] = bindFieldMeta{ index: i, diff --git a/json_test.go b/json_test.go index c369e41da..952f37d23 100644 --- a/json_test.go +++ b/json_test.go @@ -136,7 +136,7 @@ func TestDefaultJSONCodec_Decode_RejectsTrailingData(t *testing.T) { // followed by a short one must each decode to exactly their own input. func TestDefaultJSONCodec_Decode_PooledBufferReuse(t *testing.T) { e := New() - for i := 0; i < 50; i++ { + for i := range 50 { longName := strings.Repeat("x", 1000+i) var long user err := deserializeJSON(e, fmt.Sprintf(`{"id":%d,"name":%q}`, i, longName), &long) @@ -159,7 +159,7 @@ func TestDefaultJSONCodec_Decode_PooledBufferConcurrent(t *testing.T) { var wg sync.WaitGroup errs := make([]error, n) got := make([]user, n) - for i := 0; i < n; i++ { + for i := range n { wg.Add(1) go func(i int) { defer wg.Done() @@ -168,7 +168,7 @@ func TestDefaultJSONCodec_Decode_PooledBufferConcurrent(t *testing.T) { }(i) } wg.Wait() - for i := 0; i < n; i++ { + for i := range n { assert.NoError(t, errs[i]) assert.Equal(t, user{ID: i, Name: strings.Repeat("n", i+1)}, got[i]) } diff --git a/perf_bench_test.go b/perf_bench_test.go index 8d6fecafd..245ac0757 100644 --- a/perf_bench_test.go +++ b/perf_bench_test.go @@ -51,7 +51,7 @@ func BenchmarkServeHTTP_Param(b *testing.B) { // Exercises the global middleware chain (finding #1). Five pass-through middlewares. func BenchmarkServeHTTP_Middleware(b *testing.B) { e := New() - for i := 0; i < 5; i++ { + for range 5 { e.Use(func(next HandlerFunc) HandlerFunc { return func(c *Context) error { return next(c) } })