請問你們中的任何人都可以告訴如何重構以下功能嗎？我想要一個功能，例如。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}
1 回答
UYOU
TA貢獻1878條經驗 獲得超4個贊
雖然調用幾乎相同，內容也幾乎相同，但它們的類型不同。 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)
一個返回 a *v1.PodList，另一個返回 a *v1.ServiceList。這兩個不同的對象都有Items，但一個是 a []*Service，另一個是 a []*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
}
而且您無需進行大量類型檢查即可獲得所需的簡潔性。
- 1 回答
- 0 關注
- 128 瀏覽
添加回答
舉報
0/150
提交
取消