refactor patch function

internal/statsapi/feed.go

@@ -36,40 +36,51 @@ type about struct {
 	CaptivatingIndex json.Number
 }
 
-func patch(obj any, path string, value any) error {
+func setValue(obj any, key string, value any) error {
-	if string(path[0]) == "/" {
+	var err error
-		path = path[1:]
+	switch v := obj.(type) {
+	case *FeedResponse:
+		log.Println("Formerly %v", (*v)[key])
+		(*v)[key] = value
+	case map[string]any:
+		log.Println("Formerly %v", v[key])
+		v[key] = value
+	case []any:
+		idx, err := strconv.Atoi(key)
+		if err != nil {
+			break
+		}
+
+		if len(v) <= idx {
+			arr := make([]any, idx+1)
+			for i, val := range v {
+				arr[i] = val
+			}
+			v = arr
+		}
+		v[idx] = value
+	default:
+		err = errors.New("couldn't determine type")
 	}
 
+	return err
+}
+
+func expand(v []any, idx int) {
+	if len(v) <= idx {
+		arr := make([]any, idx+1)
+		for i, val := range v {
+			arr[i] = val
+		}
+		v = arr
+	}
+}
+
+func patch(obj any, path string, value any) error {
 	var err error
 	first, rest, found := strings.Cut(path, "/")
 	if !found {
-		switch v := obj.(type) {
+		setValue(obj, first, value)
-		case *FeedResponse:
-			log.Println("Formerly %v", (*v)[first])
-			(*v)[first] = value
-		case map[string]any:
-			log.Println("Formerly %v", v[first])
-			v[first] = value
-		case []any:
-			idx, err := strconv.Atoi(first)
-			if err != nil {
-				break
-			}
-
-			if len(v) <= idx {
-				arr := make([]any, idx+1)
-				for i, val := range v {
-					arr[i] = val
-				}
-				v = arr
-			}
-			v[idx] = value
-		default:
-			err = errors.New("couldn't determine type")
-		}
-
-		return err
 	}
 
 	switch v := obj.(type) {
@@ -83,14 +94,7 @@ func patch(obj any, path string, value any) error {
 			break
 		}
 
-		if len(v) <= idx {
+		expand(v, idx)
-			arr := make([]any, idx+1)
-			for i, val := range v {
-				arr[i] = val
-			}
-			v = arr
-		}
-
 		err = patch(v[idx], rest, value)
 	default:
 		err = errors.New("couldn't determine type")