質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.61%

  • Swift

    7052questions

    Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

  • iOS

    3906questions

    iOSとは、Apple製のスマートフォンであるiPhoneやタブレット端末のiPadに搭載しているオペレーションシステム(OS)です。その他にもiPod touch・Apple TVにも搭載されています。

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

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 2
  • VIEW 582

ishikawa_pro

score 1

前提・実現したいこと

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

聞きたいこと  

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

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

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

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

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

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

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を実装したクラスを
独自のクラスを作ってしまっても良いかと考えます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/12/30 16:28

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

  • 受付中

    swiftのクラス内で作成したボタンのイベントを発生させたい

    いつもお世話になっております。 アイフォンアプリ初心者です。 現在swiftでアイフォンアプリを作成しております。 カレンダーで日付をクリックすると、画面が遷移するような形を作り

  • 解決済

    [Swift]ViewControllerの初期化

    ViewControllerに表示してあるデータを初期化する方法はありますか? はじめにViewControllerに表示したデータを初期化して、新しいデータを表示したいと思って

  • 受付中

    中学2年生のswift初心者

    はじめまして。 swiftでwebブラウザアプリを作っています。 ですが、なぜかエラーが出ます。 import UIKit class ViewController: UIVie

  • 受付中

    UIButtonのサブクラスを作成する

    Swiftで電卓アプリを作ろうとしています。以下の流れを考えています。 ・UIButtonクラスを継承したサブクラスを作成 ・UIButtonのサブクラスにボタンのプロパティとボタ

  • 受付中

    swiftで、TableViewに罫線が表示されないため困っています。

    質問があります、よろしくお願い申し上げます。 解決したいこと 下記のコードでは、アプリに罫線が表示されずに困っています。 理想図 よろしくお願い申し上げます。 全コ

  • 解決済

    storyboard?.instantiateViewControllerWithIdentifie...

    参考書のサンプルアプリを作っているのですが、わからないプロパティ(?)があったので質問させていただきます。 下記のコードが全体なのですが if let nextQuestion

  • 解決済

    Text.delegate = self←これについて質問です

    タイトルの通りなんですがselfを単体で使うのはどういうわけでしょうか?selfの後にドットを挟んでそのクラス内のプロパティやメソッドを指定するならわかりますが、selfを単体で使

  • 受付中

    SwiftでPagingMenuControllerを使用してCurrentPageを出力したい

    SwiftでPagingMenuController(ver1.1.2)を使用して「ページをめくったときに」遷移先のページのCurrentPageを出力したいです。調べたところPa

同じタグがついた質問を見る

  • Swift

    7052questions

    Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

  • iOS

    3906questions

    iOSとは、Apple製のスマートフォンであるiPhoneやタブレット端末のiPadに搭載しているオペレーションシステム(OS)です。その他にもiPod touch・Apple TVにも搭載されています。