fmt.Println("Enter position to delete::")
fmt.Scanln(&pos)
new_arr := make([]int, (len(arr) - 1))
k := 0
for i := 0; i < (len(arr) - 1); {
if i != pos {
new_arr[i] = arr[k]
k++
i++
} else {
k++
}
}
for i := 0; i < (len(arr) - 1); i++ {
fmt.Println(new_arr[i])
}
我正在使用这个命令从切片中删除一个元素，但它不起作用，请建议。
当前回答
T. Claverie目前投票最多的答案是正确的，但我发现如果只在需要时执行交换，即对片的最后一个元素以外的所有元素执行交换，算法会更清晰。这可以通过一个简单的if守卫来实现。
秩序不重要/没有进行边界检查
func remove(s []int, i int) []int {
// bring element to remove at the end if its not there yet
if i != len(s)-1 {
s[i] = s[len(s)-1]
}
// drop the last element
return s[:len(s)-1]
}
其他回答
for index, item := range movies{
if item.ID == "123"{
movies = append(movies[:index], movies[index+1:]...)
break
}
}
你需要稍微修改一下你的代码，
new_arr := make([]int, (len(arr) - 1))
for i := 0; i < len(arr); i++ {
if i != pos {
new_arr = append(new_arr, arr[i])
}
}
为了更有效的循环，你可以使用这个
for i, item := range arr {
...
}
最后，您可以通过使用本机切片功能来实现这一点
new_arr = append(arr[:2], arr[3:])
最后一个解决方案是删除索引2中的元素，并将新切片放在new_arr中。
我采用下面的方法切片删除项目。这有助于提高其他人的可读性。也是不可变的。
func remove(items []string, item string) []string {
newitems := []string{}
for _, i := range items {
if i != item {
newitems = append(newitems, i)
}
}
return newitems
}
因为Slice是由数组支持的因为你不可能从数组中删除一个元素而不重新洗牌内存，我不想写这么难看的代码;下面是一个伪代码，用于保存已删除项的索引;基本上，我想要一个有序的切片，即使在删除后位置也很重要
type ListSlice struct {
sortedArray []int
deletedIndex map[int]bool
}
func lenSlice(m ListSlice)int{
return len(m.sortedArray)
}
func deleteSliceElem(index int,m ListSlice){
m.deletedIndex[index]=true
}
func getSliceElem(m ListSlice,i int)(int,bool){
_,deleted :=m.deletedIndex[i]
return m.sortedArray[i],deleted
}
for i := 0; i < lenSlice(sortedArray); i++ {
k,deleted := getSliceElem(sortedArray,i)
if deleted {continue}
....
deleteSliceElem(i,sortedArray)
}
m := ListSlice{sortedArray: []int{5, 4, 3},deletedIndex: make(map[int]bool) }
...
没有必要检查每一个元素，除非你关心的内容，你可以利用切片追加。试试吧
pos := 0
arr := []int{1, 2, 3, 4, 5, 6, 7, 9}
fmt.Println("input your position")
fmt.Scanln(&pos)
/* you need to check if negative input as well */
if (pos < len(arr)){
arr = append(arr[:pos], arr[pos+1:]...)
} else {
fmt.Println("position invalid")
}
