これは好みかもしれませんが。
for文内でbreakしてその後何も処理せずreturnする場合は、
そもそもbreakせずにここでreturnする方が私は好きです。

「`return`は1つ」というのは構造化プログラミングとしてC言語などでは好まれています。
(最近はどうか知らないけど、昔の情報だと多い)
「出口は1つの方が可読性高いだろう」というのがその理由ですが、
「出口」の解釈が時代や人によって異なります。

また、私はC言語をがっつり書いたことがないので分かりませんが、
C言語ではreturnは1つの方が読みやすいのかもしれません。

「呼び出し側から見て、その関数の結果が意味することは必ず1つ」というのを「出口は1つ」と捉える考え方もあります。
つまり、条件に応じて結果が戻り値を返したり標準出力したりするのは出口が1つとは言えない。というわけです。

http://d.hatena.ne.jp/minekoa/20090207/1234031444