中級

appendの戻り値を再代入し忘れると要素が反映されない

appendは新しいスライスを返す関数で、元のスライス変数を直接書き換えません。戻り値を元の変数に代入し直さないと、追加した要素が失われます。

なぜエラーが出ないのか

出力: [1 2 3]

（エラーなし）

Goは何も報告しません。コンパイルも実行も正常に終わるコードだからです

出力: [1 2 3]

実際の出力 — 期待した結果と食い違っている箇所

見つけ方

エラーが出ないので、出力を目で確かめるしかありません。この種の誤りが最も発見が遅れます

このエラーが出る典型パターン

 1  package main
 2  
 3  import "fmt"
 4  
 5  func addOne(nums []int) []int {
 6  	_ = append(nums, 4)   
                ^
 7  	return nums
 8  }
 9  
10  func main() {
11  	nums := []int{1, 2, 3}
12  	result := addOne(nums)
13  	fmt.Println(result)
14  }

 1  package main
 2  
 3  import "fmt"
 4  
 5  func addOne(nums []int) []int {
 6  	_ = append(nums, 30)   
                ^
 7  	return nums
 8  }
 9  
10  func main() {
11  	nums := []int{10, 20}
12  	result := addOne(nums)
13  	fmt.Println(result)
14  }

 1  package main
 2  
 3  import "fmt"
 4  
 5  func addOne(nums []int) []int {
 6  	_ = append(nums, 6)   
                ^
 7  	return nums
 8  }
 9  
10  func main() {
11  	nums := []int{5}
12  	result := addOne(nums)
13  	fmt.Println(result)
14  }

 1  package main
 2  
 3  import "fmt"
 4  
 5  func addOne(nums []int) []int {
 6  	_ = append(nums, 11)   
                ^
 7  	return nums
 8  }
 9  
10  func main() {
11  	nums := []int{7, 8, 9, 10}
12  	result := addOne(nums)
13  	fmt.Println(result)
14  }

 1  package main
 2  
 3  import "fmt"
 4  
 5  func addOne(nums []int) []int {
 6  	_ = append(nums, 1)   
                ^
 7  	return nums
 8  }
 9  
10  func main() {
11  	nums := []int{0}
12  	result := addOne(nums)
13  	fmt.Println(result)
14  }

よくある誤解

「appendはスライスの中身を直接書き換えるはず」という思い込みは誤りです。容量が足りない場合appendは新しい配列を確保するため、常に戻り値を元の変数へ代入し直す必要があります。

まとめ

appendの戻り値を再代入し忘れると要素が反映されないは中級でつまずきやすい項目です。上の5パターンを実際に手で直すと、エラーメッセージのどこを読めばよいかが掴めます。

演習をはじめる

関連するエラー

• スライスが元の配列を共有していて予期せぬ書き換えが起きるスライスを別のスライスから切り出すと、多くの場合は同じ配列を参照したままになります。
• declared and not used: の原因と直し方Goでは変数を宣言しても一度も使わないとコンパイルエラーになります。