イベントハンドラはHTMLから分離した方が良いのか？

コードだけ見ても、どのイベントがインストールされているか分からないというのはその通りですが、最近のブラウザには素晴らしいインスペクタが付いているので、綺麗にレンダリングされたソースを見ながら、イベントリスナーのチェックをすることもできますから、さほど困ることも無いと思います。

またインスペクタのコンソールも、クラスやファンションや変数のタイプアヘッドもするものもありますから、コンソールも使いこなせるようになると、コードの書き換え無しにクラスやファンクションのデバッグができます。むしろクラスの方が、テスト段階やデバッグではインスタンスの変更や追加が可能ですから、ファンクションのデバッグよりも楽かもしれません。

分かりにくいかどうかは、ツールに対する慣れなので、好みの問題はあると思いますが、イベントハンドラがオブジェクトに直接コーディングされていない方が、コードの書き換え無しにデバッグやテストを続けられるので便利かなと、個人的には思います。

http://game.guide.withabout.net/guide_game/gp333/about/G10246/CU20050315A/test2.htm

上のページをインスペクタで見ながら、テストしてみると、その良さが分かると思いますが、いかがでしょうか。