213. House Robber II

[hroc135]

https://leetcode.com/problems/house-robber-ii/description/

[oda]

返り値をペアにして、第2引数をエラーにするということでしょうか。
Go は Exception がないんですよね。

Go 言語の思想は、私は Google 社内 C++ に影響を受けていると思っています。
Exception はコントロールフローが煩雑になるので好まれません。巨大なコードベースが読めなくなります。
マイクロサービスがたくさんある環境では、FATAL でプロセスが死んでくれると、何が起きているかを外から見ることができてむしろ便利です。

空なことが分かっているのに min, max を呼ぶこと自体が、配列外アクセスに相当するという感覚でしょうか。

[hroc135]

ありがとうございます。一応背景がわかったつもりになりました。大規模コードベースをあまり触ったことのない自分にとってはまだピンとこない部分もありますが、ドキュメントにもあるように「ひどい目」にあった時に思い出せるようにします

あと、よく考えたらslices.Maxでerrorを返すのは変ですね

maxElem, err := slices.Max(s)
if err != nil {
    log.Fatal("empty slice")
}

とするくらいだったら呼ぶ前に空かどうか確認しますね

[oda]

テストケース、もう少しバリエーションがある気はします。
[10, 1, 1, 10, 1]
2連続で入らない場合とか。

あとはいいと思います。