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

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

ただいまの
回答率

87.49%

ViewControllerをグローバルで定義したい

解決済

回答 1

投稿

  • 評価
  • クリップ 1
  • VIEW 681

score 17

質問

現在、ビューワーアプリを製作中です。
メインのViewControllerクラスで定義した変数を色々なクラスで参照したいため、ViewControllerをグローバルで定義したいと思っています。
グローバルで定義しても問題ありませんでしょうか。
サンプルアプリを見ていると大抵AppDelegateクラス内で定義されています。
グローバルで定義して試してみたところ動作的には問題なさそうですが、今後致命的な問題が出ないか心配です。
また、その他に良い方法がありましたら教えていただきたいです。

初歩的な質問で申し訳ありませんが、ご教示のほどお願い申し上げます。

該当のソースコード

///********グローバルで定義
var g_Vc : ViewController!     // 共用ビューコントローラー


///******** ViewController
class ViewController: UIViewController{

    var appDelegate:AppDelegate = UIApplication.shared.delegate as! AppDelegate

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


///******** AppDelegate
@UIApplicationMain
class AppDelegate: UIResponder, UIApplicationDelegate{

    var window: UIWindow?
    var NaviController: NavigationController?

    // アプリを起動する際に呼び出されるメソッド
    func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool {

        // ウィンドウの設定
        g_Vc = ViewController()
        NaviController = NavigationController(rootViewController: g_App.m_Vc)
        self.window = UIWindow(frame: UIScreen.main.bounds)
        self.window?.rootViewController = NaviController
        self.window?.backgroundColor = UIColor.white
        self.window?.makeKeyAndVisible()

        return true
    }
}

補足情報

Xcode:Version 10.2
Swift5
iOS12.2

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • msa_winnie

    2019/09/11 16:58

    t_obara様

    丁寧に教えていただきありがとうございます。
    理解不足で申し訳ありません。

    以下のようにAppDelegate内に参照したい変数を定義して、
    他の色々なクラスで参照したいときは、UIApplicationからdelegateを取得してアクセスするということであっていますでしょうか。

    **************************************
    class AppDelegate: UIResponder, UIApplicationDelegate{
    〜〜
    var m_sansyo : Int = 0 // 参照したい変数
    〜〜
    }

    **************************************
    class 色々なクラス{
    〜〜
    var appDelegate:AppDelegate = UIApplication.shared.delegate as! AppDelegate

    // 参照したい変数を取得
    func getsansyo() -> Int {
    return appDelegate. m_sansyo
    }
    〜〜
    }

    キャンセル

  • t_obara

    2019/09/11 17:05

    やってみましょう。デバッガで参照するとよくわかるかと思います。

    キャンセル

  • msa_winnie

    2019/09/11 17:27

    t_obara様

    やってみました。上記の方法で無事に参照することができました。
    AppDelegateを利用することで、グローバル変数にしなくても済むのですね。
    AppDelegateについてはよくわからない部分が多かったのでこれから勉強します。
    ありがとうございました。

    キャンセル

回答 1

checkベストアンサー

0

グローバルで定義しても問題ありませんでしょうか。

動作上問題が無いかという意味であれば、動作チェックして問題がなければ大丈夫でしょうね。
一般的にはグローバル変数はメンテナンスの観点からなるべく減らすべきと考えられています。

また、その他に良い方法がありましたら教えていただきたいです。 

そのためには、ViewControllerの何の変数をどこのクラスで使いたいのか、そもそも別のクラスでその変数を参照する必要があるのか、といったもろもろを含めて判断しないといけませんので、今の情報だけで○○がオススメという事は言えません。
(かと言って全部情報を出されても私は回答できかねますが)

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/09/11 16:35

    takabosoft様

    教えていただきありがとうございます。
    >一般的にはグローバル変数はメンテナンスの観点からなるべく減らすべきと考えられています。
    そうですよね。安易にグローバル変数にするのは避けたいです。

    >今の情報だけで○○がオススメという事は言えません。
    メインのViewControllerの子供のViewControllerで変数を参照したりしています。
    たしかに参照しなくても済む可能性があるので、再度コードを見直してみます。

    キャンセル

  • 2019/09/12 11:53 編集

    通常は、データは別に持っていて、
    各viewの表示・利用の際に読み出すのが一般的だと思います。
    AViewに
    let animalName: String = "ライオン"
    let animalImage: UIImage = UIImage(named: "lion.png")
    などある場合に、

    他のview(BView)でわざわざ
    let name: String = AView.animalName
    let image: UIIMage = AView.animalImage
    などとはせず、

    取得時or初回起動時(AViewなどで)userdefaults(他のデータ永続化方法でも可)などに保存すれば、
    BViewでuserdefaults.string(forKey: "animalName")で読み出せます。
    CViewでも同様で、管理しやすいと思います。

    viewcontroller自体をグローバルにするのは、
    不要なもの(viewのlifecycleなど)まで保持されてしまうことが懸念されます。

    どの永続化方法がいいのかは、時と場合によります。
    簡便的にはuserdefaultsですが、なんでもかんでも保存すれば良いという訳でもなく、議論を呼ぶところではありますが、、、

    キャンセル

  • 2019/09/13 13:03

    hameji様

    教えていただきありがとうございます。
    userdefaultsなどを利用する方法も参考にさせていただきます。

    キャンセル

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

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

関連した質問

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