もっとも、そもそも DiceRoll::parse はコンストラクタだと思うので
nil を返すという仕様はどうか？という気がします。
実際の使われ方を見てないけれど、ここは raise_error が自然では
無いかと思います。