refactor: remove SetFields function and associated tests for cleaner code

Signed-off-by: tbxark <[email protected]>

Author: TBXark

contrib/json-gen-proto/main.go

@@ -2,11 +2,7 @@ package mapper
 
 import (
 	"encoding/json"
-	"fmt"
 	"golang.org/x/exp/constraints"
-	"golang.org/x/text/cases"
-	"golang.org/x/text/language"
-	"reflect"
 )
 
 func Map[S any, T any](source []S, mapper func(S) T) []T {
@@ -30,53 +26,6 @@ func MapStruct[S any, T any](source *S) *T {
 	return &target
 }
 
-func SetFields[S any, T any](source S, target T, ignoreZero bool) (err error) {
-	defer func() {
-		if r := recover(); r != nil {
-			err = fmt.Errorf("panic: %v", r)
-		}
-	}()
-	sourceValue := reflect.ValueOf(source)
-	targetValue := reflect.ValueOf(target)
-	if targetValue.Kind() != reflect.Ptr {
-		err = fmt.Errorf("target must be a pointer")
-		return
-	}
-	for sourceValue.Kind() == reflect.Ptr {
-		sourceValue = sourceValue.Elem()
-	}
-	targetValue = targetValue.Elem()
-	sourceType := sourceValue.Type()
-	for i := 0; i < sourceValue.NumField(); i++ {
-		field := sourceType.Field(i)
-		fieldValue := sourceValue.Field(i)
-		setterName := "Set" + cases.Title(language.Und, cases.NoLower).String(field.Name)
-		//setterName := "Set" + cases.Title(field.Name).String()
-		method := targetValue.Addr().MethodByName(setterName)
-		if method.IsValid() {
-			if method.Type().NumIn() != 1 {
-				err = fmt.Errorf("method %s must have one parameter", setterName)
-				return
-			}
-			methodParamType := method.Type().In(0)
-			if method.Type().In(0) != fieldValue.Type() {
-				if fieldValue.Type().ConvertibleTo(methodParamType) {
-					fieldValue = fieldValue.Convert(methodParamType)
-				} else {
-					err = fmt.Errorf("method %s parameter type must be %s, but got %s", setterName, methodParamType, fieldValue.Type())
-					return
-				}
-			}
-			if ignoreZero && fieldValue.IsZero() {
-				continue
-			}
-			args := []reflect.Value{fieldValue}
-			method.Call(args)
-		}
-	}
-	return
-}
-
 func Page[P constraints.Integer](total, pageSize, defaultSize P) P {
 	if total == 0 {
 		return 0

database/mapper/render.go

@@ -1,7 +1,6 @@
 package mapper
 
 import (
-	"reflect"
 	"testing"
 )
 
@@ -21,74 +20,6 @@ func (t *Target) SetName(name string)       { t.name = &name }
 func (t *Target) SetAge(age int64)          { t.age = int(age) }
 func (t *Target) SetIsActive(isActive bool) { t.isActive = isActive }
 
-func TestSetFields(t *testing.T) {
-	names := []string{"Alice", "", "Charlie"}
-	tests := []struct {
-		name       string
-		source     Source
-		ignoreZero bool
-		want       Target
-	}{
-		{
-			name: "Basic Test",
-			source: Source{
-				Name:     names[0],
-				Age:      25,
-				IsActive: true,
-			},
-			ignoreZero: false,
-			want: Target{
-				name:     &names[0],
-				age:      25,
-				isActive: true,
-			},
-		},
-		{
-			name: "Ignore zero value test",
-			source: Source{
-				Name:     names[1],
-				Age:      0,
-				IsActive: false,
-			},
-			ignoreZero: true,
-			want: Target{
-				name:     nil,
-				age:      0,
-				isActive: false,
-			},
-		},
-		{
-			name: "Partial field test",
-			source: Source{
-				Name:     names[2],
-				Age:      0,
-				IsActive: true,
-			},
-			ignoreZero: true,
-			want: Target{
-				name:     &names[2],
-				age:      0,
-				isActive: true,
-			},
-		},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			target := &Target{}
-			err := SetFields(tt.source, target, tt.ignoreZero)
-			if err != nil {
-				t.Errorf("SetFields() error = %v", err)
-				return
-			}
-
-			if !reflect.DeepEqual(*target, tt.want) {
-				t.Errorf("SetFields() = %v, want %v", *target, tt.want)
-			}
-		})
-	}
-}
-
 func TestMapStruct(t *testing.T) {
 	type structA struct {
 		Name string `json:"name"`