以下のように無限に2バイトを返し続けるio.Reader
を実装して、そこから奇数バイト(例では5バイト)をio.CopyN
でコピーしようとするとpanicを起こしました。偶数バイトをコピーする場合には起きません。なぜでしょうか?
package main
import (
"io"
"os"
)
type stream int
func (s *stream) Read(p []byte) (n int, err error) {
p[0] = byte('a')
p[1] = byte('b')
return 2, nil
}
func main() {
io.CopyN(os.Stdout, new(stream), 5)
// panic: runtime error: index out of range
}
環境:go version go1.10.3 windows/386
p
の長さが最後の段階で 1 となって、p[1]
にアクセスして panic が発生しています。ですので、こんな感じに書き直す必要があります。p
に対してp[1]=
などと、初歩的なミスでした。頂いたコードはこのように書いてもよいかと考えましたが、そもそも2バイト全てを返せなかった場合にはどうするか(捨てるのか、次のreadに持ち越すのか、など)という仕様の検討が不足していたことにも気付きました。それと、コピーするバイト数を32KB以上にすると奇数バイトでもpanicにならず、その点も実は不思議だったのですが、LimitedReader.Read()のこの判定に入らなくなることが理由だったことも分かりました。