jefferyjob / go-easy-utils Goto Github PK

Tracking issue for:

• https://github.com/jefferyjob/go-easy-utils/security/code-scanning/4

Tracking issue for:

• https://github.com/jefferyjob/go-easy-utils/security/code-scanning/2

Tracking issue for:

• https://github.com/jefferyjob/go-easy-utils/security/code-scanning/8

Tracking issue for:

• https://github.com/jefferyjob/go-easy-utils/security/code-scanning/3

There is a bug in how jsonToStruct parses maps and slices.

1. Does not support parsing map type
2. Parse the slice type, if the data type is inconsistent, panic error

type Target struct {
	Followed map[string]bool `json:"followed"`
	Fids     []string        `json:"fids"`
}

func TestJsonToStruct(t *testing.T) {
	data := `
	{
		"followed": {
			"43015653": true,
			"43015666": false
		},
		"fids": ["43015653",43015666]
	}
	`

	var event Target
	err := JsonToStruct(data, &event)
	fmt.Printf("%+v, err:%s \n", event, err)
}

The specific error description is as follows

panic: reflect.Set: value of type float64 is not assignable to type string [recovered]
	panic: reflect.Set: value of type float64 is not assignable to type string

At the beginning of the design, due to the strong typing properties of the language, it is not recommended to define data types vaguely. So the definition of interface{} and any type is not supported.

However, considering the affordability of some businesses, these two types need to be supported, so this requirement will support the definition of interface{} and any type.

func TestJsonToStructAny1(t *testing.T) {
	jsonData := `{
		"name": "make",
		"age": "10"
	}`
	type Target struct {
		Name interface{} `json:"name"`
		Age  uint        `json:"age"`
	}
	var target Target
	err := JsonToStruct(jsonData, &target)
	if err != nil {
		t.Errorf("err %s", err)
		return
	}
}

I think errors like this can be improved to json_ to_ struct. go, it is convenient for developers to make incorrect type assertions

func Chunk[T any](slice []T, size int) [][]T {
var chunks [][]T
for i := 0; i < len(slice); i += size {
end := i + size
if end > len(slice) {
end = len(slice)
}
chunks = append(chunks, slice[i:end])
}
return chunks
}

Tracking issue for:

• https://github.com/jefferyjob/go-easy-utils/security/code-scanning/1

v1.0 为了快速适应本公司的业务，所以没有考虑any和泛型
v2.0 当前正在开发和自测中，将会在不久支持泛型和any，以兼容各个业务需求

有不少同学提出这个问题，这里统一回复一下，请大家期待后续版本的发布。

Tracking issue for:

• https://github.com/jefferyjob/go-easy-utils/security/code-scanning/7

go.mod中go的版本是1.18，已经支持泛型，为何没有应用泛型，存在大量的重复代码和interface{}。

// MergeSlices 将多个slice合并成一个slice
func MergeSlices(slices ...[]interface{}) []interface{} {}

// MergeStrSlices 将多个slice合并成一个slice
func MergeStrSlices(slices ...[]string) []string {}

// MergeIntSlices 将多个slice合并成一个slice
func MergeIntSlices(slices ...[]int) []int {}

可以参考lancet

这种方法完全可以不使用反射，可以这样写func Ceil[T float32 | float64](num T) int {
if num == 0{return 0 }
return int(math.Ceil(float64(num)))
}