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

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

ただいまの
回答率

90.76%

  • Swift

    6699questions

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

  • Xcode

    3850questions

    Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

  • iOS

    3797questions

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

Swift 同じ機能を2回書いてしまう。

解決済

回答 1

投稿 編集

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

soujomitsuura

score 16

Swift4&xcode9.2で開発しています。

前置きが少し長いですが、サンプルコードは極力少なくします。

前置き

私は既存のUITabbarのデザインが少し気に入らないので自作しています。

後々カスタマイズなどがしやすいようにとカスタムクラスを作ったりしてそのUIと機能
(function)を分けるようにしています。

主にUIの部分をカスタムクラス&xibでつくり、画面遷移などのfunc部分をViewControllerに書いています。

UITabbarと同じものなので、遷移先のページにも同じUIが設置され、同じ機能が設置されます。

イメージ説明

質問

UIをカスタムクラス で分離し、ViewControllerにてインスタンス生成し、表示するところまでは可能でした。
しかし、画面遷移はUIViewController内でしか行えないため、遷移用のfuncに関しては遷移前と遷移後のクラス両方に書かなくてはならないのではと考えています。
全く同じナビゲーションメニューのUIと機能を2回も書かなくてはならないことに不満を感じています。

これをなんとかまとめたいです。
そのための提案や意見をお願いします。

コードは下記に記載します。

ファイルは
FirstViewcontroller.swift
SecondViewcontroller.swift (白紙のため省略)
NavigationMenu.swift

//FirstViewController

import UIKit

class FirstViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()
        // ナビゲーション設定
        navigation()
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }

}

//問題の機能とUIのカスタマイズソース
//この辺をSecondViewControllerにも書きそうだが、書きたくない。
extension FirstViewController{

    func navigation() {

        //navigationデザイン
        let navigationSize     = CGRect(x:0, y:screenHeight - 50, width: screenWidth, height: 50)
        let screenSize         = CGRect(x:0, y:0, width: screenWidth, height: screenHeight)
        let bgColor            = UIColor(red: 255/255,green: 255/255,blue: 255/255,alpha: 0.7)
        let blurEffect         = UIBlurEffect(style: .extraLight)
        let visualEffectView   = UIVisualEffectView(effect: blurEffect)
        let bgView             = UIView(frame: screenSize)
        let commonNavigation   = NavigationMenu.instance()
        bgView.backgroundColor = bgColor
        visualEffectView.frame = navigationSize
        commonNavigation.frame = navigationSize
        self.view.addSubview(visualEffectView)
        self.view.addSubview(bgView)
        self.view.addSubview(commonNavigation)
        commonNavigation.iconSetting()

        //navigationアクション
        let sWHalf        = screenWidth / 2
        let iconAFrame    = CGRect(x:0, y:screenHeight - 50, width: sWHalf, height: 50)
        let iconBFrame    = CGRect(x:sWHalf, y:screenHeight - 50, width: sWHalf, height: 50)
        let iconAAction   = UIButton()
        let iconBAction   = UIButton()
        iconAAction.frame = iconAFrame
        iconBAction.frame = iconBFrame
        iconAAction.addTarget(self,
                              action: #selector(self.iconATapped(sender:)),
                              for: .touchUpInside)
        iconBAction.addTarget(self,
                              action: #selector(self.iconBTapped(sender:)),
                              for: .touchUpInside)
        self.view.addSubview(iconAAction)
        self.view.addSubview(iconBAction)
    }

    @objc func iconATapped(sender : AnyObject) {
        //ここにFirstViewcontrollerへの画面遷移を書く
    }

    @objc func iconBTapped(sender : AnyObject) {
        //ここにSecondViewcontrollerへの画面遷移を書く
    }
}
//Storyboardと共に作られたナビゲーション用のコード

import UIKit

class NavigationMenu: UIView {
    let firstViewController = FirstViewController()

    @IBOutlet weak var navigationMenu: UIView!
    @IBOutlet weak var iconAButton: UILabel!
    @IBOutlet weak var iconBButton: UILabel!
    @IBOutlet weak var iconALabel:  UILabel!
    @IBOutlet weak var iconBLabel:  UILabel!

    var iconA:String! = "?"
    var iconB:String! = "?"
    var textA:String! = "TESTA"
    var textB:String! = "TESTB"

    class func instance() -> NavigationMenu {
        return UINib(nibName: "NavigationMenu", bundle: nil).instantiate(withOwner: self, options: nil)[0] as! NavigationMenu
    }

    func iconSetting(){
        iconAButton.text = iconA
        iconBButton.text = iconB
        iconALabel.text  = textA
        iconBLabel.text  = textB
    }

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

NavigationをおこなうためのRouterを作成して各クラスにDependency Injectionしてはいかがでしょうか。
以下の記事が参考になるかと思います。

また、Routerに限らず共通化できそうな処理や抽出できそうな処理は責務別に他のクラスとして作成すると、ファイルの総数は増えますがメンテナンス性が高くなります。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/17 13:28 編集

    方法が少し異なりましたが、うまく機能を分けました。ありがとうございます。

    キャンセル

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

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

関連した質問

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

  • Swift

    6699questions

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

  • Xcode

    3850questions

    Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

  • iOS

    3797questions

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