swiftな感じにリファクタリング

[yimajo]

このPull Requestは

• optionalの変数はnilになっているのでnil代入する必要はなく
• 引数の初期値がある場合は呼び出し側が省略できるのでoptionalのクロージャにはnilを明示
• optionalのクロージャはbindするよりoptional chaining的にそのまま呼び出してbindで二度同じ変数名を書かないでよいように
• （好みの問題でやってなかったのかもしれませんが）MARKで実装を分けるときはextensionで実装すると分かりやすいかと

もしよければ参考に :)

[koogawa]

@yimajo プルリクエストありがとうございます！

optionalの変数はnilになっているのでnil代入する必要はなく
引数の初期値がある場合は呼び出し側が省略できるのでoptionalのクロージャにはnilを明示

なるほど、仰るとおりですね。

optionalのクロージャはbindするよりoptional chaining的にそのまま呼び出してbindで二度同じ変数名を書かないでよいように

たしかにこちらの方が良いですね。

（好みの問題でやってなかったのかもしれませんが）MARKで実装を分けるときはextensionで実装すると分かりやすいかと

好みではなく、単純に知りませんでした😫
とても勉強になります。

[koogawa]

まだまだ Swift には慣れていないので、非常に勉強になりました！:bow:
マージ致します。