Empty chaining

[meian]

gcf/reverse.go

Lines 18 to 20 in de7836c

	if itb == nil {
	return empty[T]()
	}

この書き方をしている場合、引数の itb にnilを渡すと emptyIterable を返すので処理が軽いが、その次に他の関数を適用したときに empty ではなくなるので空の時にも処理が軽くならない

itb := gcf.Filter[int](nil, func(v int) bool { return v > 10})  // この時点では itb は empty
itb = gcf.Filter(itb, func(v int) bool { return v%2 == 0})      // この時点では itb は filterIterable だが要素は1件も取得できない

そのため以下のように修正することで emptyIterable をチェーンできるようにする

func SomeFunc[T any](itb Iterable[T]) Iterable[T] {
    if isEmpty(itb) {
        return orEmpty(itb)
    }
    ....
}

• isEmpty
  • itb がnilか emptyIterable であればtrue
• orEmtpy
  • itb がnilなら emptyIterable、nilでなければ emptyIterable を作って返す

こうすることで empty に対する処理に対して常に empty を返せるので、空の処理の場合に負荷がかからなくなる

itb := gcf.Filter[int](nil, func(v int) bool { return v > 10})  // この時点では itb は empty
itb = gcf.Filter(itb, func(v int) bool { return v%2 == 0})      // empty に対する処理なので empty になる

[meian]

負荷がかからないことを確認するためのテストを用意するかな