1 回答
TA贡献1844条经验 获得超8个赞
您提出的解决方案不正确。问题是当您从切片中删除一个元素时,所有后续元素都将被shift。但是循环不知道您更改了底层切片和循环变量(索引)像往常一样增加,即使在这种情况下它不应该因为那样您跳过一个元素。
如果切片包含 2 个彼此相邻的元素,这两个元素都需要删除,则第二个将不会被检查,也不会被删除。
所以如果你删除一个元素,循环变量必须手动递减!让我们看一个例子:删除以 开头的单词"a":
func conditionMeets(s string) bool {
return strings.HasPrefix(s, "a")
}
解决方案(在Go Playground上尝试使用以下所有其他示例):
a := []string{"abc", "bbc", "aaa", "aoi", "ccc"}
for i := 0; i < len(a); i++ {
if conditionMeets(a[i]) {
a = append(a[:i], a[i+1:]...)
i--
}
}
fmt.Println(a)
输出:
[bbc ccc]
或者更好:使用向下循环,因此您无需手动减少变量,因为在这种情况下,移位的元素位于切片的“已处理”部分。
a := []string{"abc", "bbc", "aaa", "aoi", "ccc"}
for i := len(a) - 1; i >= 0; i-- {
if conditionMeets(a[i]) {
a = append(a[:i], a[i+1:]...)
}
}
fmt.Println(a)
输出是一样的。
替代多次移除
如果您必须删除“许多”元素,这可能会很慢,因为您必须进行大量复制(append()复制)。想象一下:你有一个包含 1000 个元素的切片;仅删除第一个元素需要将 999 个元素复制到前面。还将创建许多新的切片描述符:每次删除元素都会创建2 个新的切片描述符 ( a[:i], a[i+1:])加上 a必须更新( 的结果append())。在这种情况下,将不可移动元素复制到新切片可能更有效。
一个有效的解决方案:
a := []string{"abc", "bbc", "aaa", "aoi", "ccc"}
b := make([]string, len(a))
copied := 0
for _, s := range(a) {
if !conditionMeets(s) {
b[copied] = s
copied++
}
}
b = b[:copied]
fmt.Println(b)
此解决方案分配与源长度相同的切片,因此不会执行新的分配(和复制)。此解决方案也可以使用range循环。如果您想要结果a,请将结果分配给a: a = b[:copied]。
输出是一样的。
就地替代进行多次移除(和一般用途)
我们还可以通过一个循环来“就地”移除,通过维护 2 个索引并在同一切片中分配(向前复制)不可移除元素。
要记住的一件事是,我们应该将已删除元素的位置归零,以删除无法访问的值的引用,以便 GC 可以完成其工作。这也适用于其他解决方案,但仅在此处提及。
示例实现:
a := []string{"abc", "bbc", "aaa", "aoi", "ccc"}
copied := 0
for i := 0; i < len(a); i++ {
if !conditionMeets(a[i]) {
a[copied] = a[i]
copied++
}
}
for i := copied; i < len(a); i++ {
a[i] = "" // Zero places of removed elements (allow gc to do its job)
}
a = a[:copied]
fmt.Println(a)
输出是一样的。尝试Go Playground上的所有示例。
- 1 回答
- 0 关注
- 173 浏览
添加回答
举报