选择哪种类型是更好的 - 对于重构 golang 函数来说？

本篇文章向大家介绍《选择哪种类型是更好的 - 对于重构 golang 函数来说？》，主要包括，具有一定的参考价值，需要的朋友可以参考一下。

请问你们谁能告诉我如何重构以下函数吗？

我想要一个功能，例如。 getresource(name string, resourcetype ????) []v1.??????，但不知道它的返回类型是什么。

func getPods(name string) []v1.Pod {
    pods, err := clientset.CoreV1().Pods(namespace).List(context.TODO(), getListOption(name))

    if err != nil {
        panic(err.Error())
    }
    return pods.Items
}

func getServices(name string) []v1.Service {
    services, err := clientset.CoreV1().Services(namespace).List(context.TODO(), getListOption(name))

    if err != nil {
        panic(err.Error())
    }
    return services.Items
}

正确答案

虽然调用几乎相同，内容也几乎相同，但它们的类型不同。 CoreV1().Pods(namespace).List 是：

list(ctx context.context, opts metav1.listoptions) (*v1.podlist, error)

但是 CoreV1().Services(namespace).List 是

list(ctx context.context, opts metav1.listoptions) (*v1.servicelist, error)

一个返回 *v1.podlist，另一个返回 *v1.servicelist。这两个不同的对象都有 items，但一个是 []*Service，另一个是 []*Pod。

我个人会考虑不包装。我会调用底层函数，在两种情况下使用相同的 corev1 客户端。 我当然不会尝试将它们组合成一个函数。

如果这里有什么需要组合的话，那就是命名空间和名称：

type byname struct {
   name string
   namespace string
   v1 corev1.corev1interface
}

func (b *byname)services(ctx context.context) []v1.service , error {
   return b.v1.services(namespace).list(ctx,  getlistoption(b.name))
}

func (b *byname)pods(ctx context.context) []v1.pod, error {
  return b.v1.pods(b.namespace).list(ctx, getlistoption(b.name))
}

现在您不必传递名称和命名空间：

b := &ByName{
  Name: name,
  Namespace: namespace,
  v1: clientset.CoreV1(),
}

if svcs, err :=  b.Services(); err != nil {
   return nil, err
} else if pods, err := b.Pods(); err != nil {
  return nil, err
} else {
  ... continue processing
}

并且您无需进行大量类型检查即可获得所需的简洁性。

本篇关于《选择哪种类型是更好的 - 对于重构 golang 函数来说？》的介绍就到此结束啦，但是学无止境，想要了解学习更多关于Golang的相关知识，请关注golang学习网公众号！