Swiftで似たようなViewControllerを複数実装する場合のベストプラクティスについて

Question

###前提・実現したいこと
はじめまして。SwiftでのiOSアプリ開発を独学でやっている初心者です。
現在Smart Newsのようなニュースアプリを作成しています。開発中のニュースアプリでは、NewsFeedというトレンドの記事を一覧表示するViewControllerと、ユーザーが選択したカテゴリの最新記事を一覧表示するViewControllerの2種類があります。(Smart Newsでいう"トップ"というトレンド記事と"テクノロジー"などのカテゴリごとのニュースの2種類） ２つのViewControllerでは表示するものは同じですが、WebAPIのエンドポイントが異なっていたり若干の違いがあります。最初は1つのViewControllerにてif文で処理を分岐していたのですが、それぞれ独自の処理が増えてきてViewControllerが肥大化したためViewControllerを分けたいと思っています。

###聞きたいこと
2つのViewControllerに分ける際に、表示するViewは全く同じなのでBaseViewControllerで共通処理を書いて、NewsFeedとカテゴリごとのViewControllerへ継承させようと思ったのですが、そのうちBaseViewControllerが肥大化したりして挙動を把握するのが大変になったり、なんのオブジェクトを保持しているのか把握するのが大変になったりしそうで、結局根本的な解決にならない気がしています。Swiftはプロトコル指向なのでプロトコルを使ったもっと良い実装方法があるのではないかと思うのですが、知見を共有していただけないでしょうか？
プロトコルを使った方法に限らず、うちは普段こういう風に実装してるなどでも構いません。

もう一つ、NewsFeedやカテゴリの記事一覧ではUITableViewを使っているのですが、BaseViewControllerなどの親クラスを作らない場合は、UITableViewDelegateなどのプロトコルにViewController毎に準拠させて処理を書くことになりますか？

※具体的に聞きたいことがあれば補足致します。
ざっくりとした質問で申し訳ありませんがどうぞよろしくお願い致します。

Answer 1

DI（Dependency Injection）を活用しましょう。
以下のサイト等が参考になります。

• [Swift] DIって何？ 実践編
• 節子、それViewControllerやない...、FatViewControllerや...。
• iOSDCでテストしづらいコードをテストしやすくするための方法について話しました

共通の処理はスーパークラスで実装するよりも専用のクラスを作成して注入するのも良いでしょう。

画面がほぼ同じであれば、画面の差異について注入してあげればよいと考えます。

class TrendViewController: UIViewController {
    init(router: Router,
        reloader: Reloader,
        repository: Repository

    ) {
        newsCollectionView = NewsCollectionView(
                reloader: Reloader,
                viewConfig: TrendViewConfig())
    }
}

class CategoryViewController: UIViewController {
    init(keyword: String,
        router: Router,
        reloader: Reloader,
        repository: Repository

    ) {
        newsCollectionView = NewsCollectionView(
                reloader: Reloader,
                viewConfig: CategoryViewConfig())
    }
}

class NewsCollectionView: UICollectionView {
    init(reloader: Reloader
        viewConfig: CollectionViewConfig)
}

protocol CollectionViewConfig {
    func showRegisterButton() -> Bool
    func showFloatingMenu() -> Bool
    func titleColor() -> UIColor
}

（変数の宣言等、色々と省いていますが）
上記の場合、各ViewControllerでNewsCollectionViewを持っていますが
viewConfigの引数でそれぞれ異なるクラスを渡しています。
CollectionViewConfigプロトコルでは画面の差異の部分だけの情報を扱います。
トレンドモードであればお気に入り登録のボタンは表示するけど、ユーザカテゴリモードであれば表示しないなど。
おそらく、現状はモード毎に処理を分けるif文で構成されていると思われます。
その方法ですとモードが増えるたびに処理が複雑になっていきます。
NewsCollectionViewには振る舞いのみを記述し、モードは意識させないことでカスタマイズがしやすくなると考えます。

UITableViewDelegateやUITableViewDatasourceが同じ処理だとして親クラスがなければ各ViewControllerに実装するのかという質問については、各ViewControllerに実装せずに
NewsFeedCollectionViewDatasourceAndDelegateのような、
UICollectionViewDelegateとUICollectionViewDatasourceを実装したクラスを
独自のクラスを作ってしまっても良いかと考えます。

Comments

[ishikawa_pro]

参考サイトやサンプルのコードまで提示していただいてありがとうございました。
DIについてはそもそも知らなかったですし、MVPなどのアーキテクチャに変える事などは頭になかったので、かなり視野が広がりました！
今回は、Swinjectなどを導入することも検討しながらDIを使ってみることにしてみます。