「リファクタリング Ruby エディション」によると、一時変数は極力排除した方がいいらしい。メソッドチェーンを使えば、結果をいちいち変数に覚えなくても最終結果だけ得られる。 s.split(/\n/).each_slice(2).map(&:join) でも、結果が意図したものと違った…