お手本の動画で、このゲームに関するさまざまな値をクリックイベントの中で初期化しているのですが、それらの処理を関数にまとめて切り出しました。

お手本と同じように動作しているようにみえるのですが、なにか不都合なことがあるでしょうか？

関数の名前は「init()」です。