レビューしやすいコード構造にするには？

メソッド（関数）の長さを「1画面に収まる程度（数十行以内）」に保ち、if文のネスト（入れ子）を浅くし、変数名に省略語（aや tmp 等）を使わず「誰が読んでも意図が伝わる」ように書くことです。

他人が書いた複雑で巨大なコードは、バグが潜んでいてもチームメンバーが解読できず（コードレビューが機能せず）、そのまま本番環境にヤバいバグが混入してしまうからです。

if文の条件が満たされない場合はすぐに return する「早期リターン（ガード節）」を徹底してネストを浅くします。また、「1つのメソッドは1つの仕事しかしない」という原則を守り、複雑な計算は意味のある名前を付けた別のメソッドに切り出します。