Question

我在浏览github时发现了这个功能：

func Accumulate(s []string, f func(st string) string) (result []string) {
    for _, v := range s {
        result = append(result, []string{f(v)}...)
    }
    return result
}

这种方法可以简化为以下内容，还是我遗漏了一些内容：

func Accumulate(s []string, f func(st string) string) (result []string) {
    for _, v := range s {
        result = append(result, f(v))
    }
    return result
}

Answer 1

您编写的版本是正确且理智的。我写了一个基准：

package p

import (
    "testing"
    "strings"
)

var s = []string{"hello", "world", "this", "new", "world"}

func BenchmarkAcc1(b *testing.B) {
    for n := 0; n < b.N; n++ {
        Accumulate(s, strings.ToUpper)
    }
}

func BenchmarkAcc2(b *testing.B) {
    for n := 0; n < b.N; n++ {
        Accumulate2(s, strings.ToUpper)
    }
}

以下是我得到的一些结果：

% go test -benchtime=10s -bench=.
testing: warning: no tests to run
PASS
BenchmarkAcc1   10000000          1510 ns/op
BenchmarkAcc2   10000000          1492 ns/op
ok      _/home/satran/test  33.064s

正如您所看到的，也没有任何性能提升，事实上您的版本效果更好。

以下是来源：

package p

func Accumulate(s []string, f func(st string) string) (result []string) {
    for _, v := range s {
        result = append(result, []string{f(v)}...)
    }
    return result
}

func Accumulate2(s []string, f func(st string) string) (result []string) {
    for _, v := range s {
        result = append(result, f(v))
    }
    return result
}

variadic pack and unpack in go lang：＆＃39; [] string {f（v）} ...＆＃39;

1 个答案: