設定情報ファイルを読み書きするプログラムの設計

Question

Swift(Xcode 9.01)で各種環境設定をファイル保存する Mac アプリケーションを作っています。
ウィンドウの位置や最前面に表示するか、配色などのコンフィグを保存しておいて、次回起動時に同じ状態で起動できるようにするのが目的です。

Swift プログラミング自体不慣れでクラスのもたせ方がよくわからないため、どのように設計したらいいのか見当がついていません。

ひとまず Config クラスを作り、メソッド save() で設定ファイルの保存、load() で設定ファイル(XML)の読み込みができるようにしました。読み書きする情報はすべて Config のプロパティとして保持しています。(Config.windowX, Config.topmost など)

使うにはインスタンス化しなければなりませんがまずどこにそれを書いてよいのかわかりません。とりあえず AppDelegate.swift のプロパティに持たせ、 override init() 時にインスタンス化することにしました。

class AppDelegate: NSObject, NSApplicationDelegate {
    var config:Config?

    override init(){
        super.init()
        self.config = Config()
        self.config.load()
    }
}

NSViewController などからこのデータにアクセスしたいのですが AppDelegate クラスにアクセスする手段を知りません。根本から間違っているのかもしれません。

設定情報を管理するクラスを作る場合どこでインスタンス化し、どのようにアクセスすればいいのでしょうか？

Answer 1

決まりがあるわけではありませんが、僕は次の２つのどちらかを使っています。最近はもっぱら下です。

extension NSApplication {
   var appDelegate: AppDelegate { return delegate as! AppDelegate }
}

extension AppDelegate {
    static var shared: AppDelegate { return NSApplication.shared.delegate as! AppDelegate }
}

---

設定のロードにはapplicationWillFinishLaunching(_:)あるいはapplicationDidFinishLaunching(_:)を使っ方がいいかもしれません。

（追記）
Main.xibなど起動時に読み込まれるnibにWindowなどがある場合、そちらのwindowDidLoad()などが先に呼ばれます。
この場合はawakeFromNib()が利用できます。
ただし、awakeFromNib()は複数回呼ばれる可能性がありますので予防措置が必要です。
この場合はconfigがnilの時だけloadするようにすればいいと思います。
（追記ここまで）

（さらに追記）
Swiftらしい代案を思いつきましたので追記しておきます。

class AppDelegate: NSObject, NSApplicationDelegate {
    lazy var config: Config = {
        let config = Config()
        config.load()
        return config
    }()

   ....
}

---

設定の保存にはUserDefaultsを調べてみるといいかもしれません。
ただ、UserDefaultsには保存できる型というのが決まっていてそれ以外を保存しようとすると若干面倒くさい処理が必要になります。
最近はUserDefaultsをSwiftで簡単に扱うライブラリもいろいろ出ているのでそれらを使えばあまり深く考える必要なく使えます。

Comments

[workr]

コメントありがとうございます。おそらくこちらの実装に問題があるのだと思いますが、applicationWillFinishLaunching()でコンフィグの読み込みを行った際、ビルドは成功するもののViewControllerのviewDidAppear() でコンフィグのプロパティを読み取ろうとした際に EXC_BAD_INSTRUCTION エラーが発生してしまいます。発生時に appDelegate は取得できているようですが appDelegate.config が nil になっていました。applicationWillFinishLaunching()より先にviewDidAppear() が実行されているようでしたが何か解決策はあるのでしょうか？

[MasakiHori]

追記しました

[MasakiHori]

追記し忘れましたが、UIなどへのアクセスが生じないのであればinitでロードしても問題ないです。