遍历切片并通过“ append()”对其进行修改时出现“切片边界超出范围”错误

时间:2018-11-17 18:46:34

标签: go slice

我编写了一个函数,可以用一个字符串([] string)中的一个单词替换重复的单词序列。我使用“范围”来循环切片,并使用“ append()”来修改切片。下面是代码:

func RemoveContinuosStrings(input []string) []string {  
        top:=0

        for i,_ := range input {
                if input[i] != input[top] {
                        if top != i-1 {
                                input = append(input[:top+1], input[i:]...)
                        }   
                        top = i 
                }   
        }   
        return input[:top+1]
}

func main() {

        scanner := bufio.NewScanner(os.Stdin)
        fmt.Println("enter text")
        input := make([]string, 0, 0)

        for scanner.Scan() {
                input = append(input, scanner.Text())
        }   

        fmt.Println("INPUT", input)
        input = RemoveContinuosStrings(input)
        fmt.Println("OUTPUT", input)
}

执行上述代码时,我看到以下错误:

===================================================  
    vignesh@vignesh-ThinkPad-E470 ~/go-book/chapter4 $ ./4_6_eliminate_duplicate_string  
    enter text
    qwe
    asd
    asd
    zxc
    asd
    asd
    INPUT [qwe asd asd zxc asd asd]
    panic: runtime error: index out of range

    goroutine 1 [running]:
    main.RemoveContinuosStrings(0xc420088000, 0x6, 0x8, 0x20, 0x0, 0x0)
        /home/vignesh/go-book/chapter4/4_6_eliminate_duplicate_string.go:11 +0x2a0
    main.main()
        /home/vignesh/go-book/chapter4/4_6_eliminate_duplicate_string.go:33 +0x33a
    vignesh@vignesh-ThinkPad-E470 ~/go-book/chapter4 $ 
=======================================================

因此,通过“范围”将其附加到要迭代的切片上是正确的还是不允许的。我在网上看到了几个示例,它们确实附加到切片上,但是在不使用“范围”的情况下进行了迭代(例如:对于i = 0; i

1 个答案:

答案 0 :(得分:0)

在循环中更改数组大小时,请记住使用简单的for而不是for range

func RemoveContinuosStrings(input []string) []string {
    top:=0

    for i:=0;i<len(input);i++ {
        if input[i] != input[top] {
            if top != i-1 {
                input = append(input[:top+1], input[i:]...)
                i=top+1
                top=top+1
            }else{
                top = i
            }
        }
    }
    return input[:top+1]
}
相关问题