guard の書き方を統一した

[kcd/KCD.git] / KCD / MissionStatus.swift

diff --git a/KCD/MissionStatus.swift b/KCD/MissionStatus.swift
index b8ad937..2f358dd 100644 (file)
--- a/KCD/MissionStatus.swift
+++ b/KCD/MissionStatus.swift
@@ -9,99 +9,136 @@
 
 import Cocoa
 
-fileprivate enum State: Int {
+private enum State: Int {
+    
     case none = 0
     case hasMission = 1
     case finish = 2
     case earlyReturn = 3
 }
 
-class MissionStatus: NSObject {
+final class MissionStatus: NSObject {
+    
     private let number: Int
     private let controller: NSArrayController
     private var didNotify = false
     private var realTime: TimeInterval = 0.0 {
+        
         didSet { time = realTime as NSNumber }
     }
-    dynamic var name: String?
-    dynamic var time: NSNumber?
-    dynamic var state: NSNumber?
-    dynamic var missionId: NSNumber? {
+    
+    @objc dynamic var name: String?
+    @objc dynamic var time: NSNumber?
+    @objc dynamic var state: NSNumber?
+    @objc dynamic var missionId: NSNumber? {
+        
         didSet { updateState() }
     }
+    @objc dynamic var milliseconds: NSNumber?
+    @objc dynamic var fleetName: String?
     
     init?(number: Int) {
-        guard (2...4).contains(number) else { return nil }
+        
+        guard case 2...4 = number else { return nil }
+        
         self.number = number
         controller = NSArrayController()
+        
         super.init()
-        controller.managedObjectContext = ServerDataStore.default.managedObjectContext
+        
+        controller.managedObjectContext = ServerDataStore.default.context
         controller.entityName = Deck.entityName
         controller.fetchPredicate = NSPredicate(format: "id = %ld", number)
         controller.automaticallyRearrangesObjects = true
         controller.fetch(nil)
         
-        bind("state", to: controller, withKeyPath: "selection.mission_0")
-        bind("missionId", to: controller, withKeyPath: "selection.mission_1")
+        bind(NSBindingName(#keyPath(state)), to: controller, withKeyPath: "selection.mission_0")
+        bind(NSBindingName(#keyPath(missionId)), to: controller, withKeyPath: "selection.mission_1")
+        bind(NSBindingName(#keyPath(milliseconds)), to: controller, withKeyPath: "selection.mission_2")
+        bind(NSBindingName(#keyPath(fleetName)), to: controller, withKeyPath: "selection.name")
+    }
+    
+    deinit {
+        
+        unbind(NSBindingName(#keyPath(state)))
+        unbind(NSBindingName(#keyPath(missionId)))
+        unbind(NSBindingName(#keyPath(milliseconds)))
+        unbind(NSBindingName(#keyPath(fleetName)))
     }
     
     private func updateState() {
+        
         guard let state = state as? Int,
-            let stat = State(rawValue: state)
-            else { return print("unknown State") }
-        if stat == .none || stat == .finish  {
+            let stat = State(rawValue: state) else {
+                
+                print("unknown State")
+                return
+        }
+        
+        if stat == .none || stat == .finish {
+            
             if stat == .none { didNotify = false }
+            
             name = nil
             time = nil
+            
             return
         }
         
-        guard let missionId = self.missionId as? Int
-            else { return }
-        guard let mission = ServerDataStore.default.masterMission(by: missionId)
-            else {
-                name = "Unknown"
-                DispatchQueue(label: "MissionStatus")
-                    .asyncAfter(deadline: .now() + 0.33) {
-                        self.updateState()
-                }
-                return
+        guard let missionId = self.missionId as? Int else { return }
+        
+        guard let mission = ServerDataStore.default.masterMission(by: missionId) else {
+            
+            name = "Unknown"
+            DispatchQueue(label: "MissionStatus")
+                .asyncAfter(deadline: .now() + 0.33) {
+                    self.updateState()
+            }
+            return
         }
+        
         name = mission.name
     }
     
     func update() {
+        
         if name == nil {
+            
             time = nil
+            
             return
         }
-        guard let t = controller.value(forKeyPath: "selection.mission_2") as? Int
-            else {
-                name = nil
-                time = nil
-                return
+        
+        guard let milliSeconds = milliseconds as? Int else {
+            
+            name = nil
+            time = nil
+            return
         }
-        let compTime = TimeInterval(Int(t / 1_000))
-        let now = Date()
-        let diff = compTime - now.timeIntervalSince1970
         
-        if diff < 0 { realTime = 0 }
-        else { realTime = diff }
+        let compTime = TimeInterval(Int(milliSeconds / 1_000))
+        let diff = compTime - Date().timeIntervalSince1970
+        
+        realTime = diff < 0 ? 0 : diff
         
         if didNotify { return }
         if diff >= 1 * 60 { return }
         
-        guard let fleetName = controller.value(forKeyPath: "selection.name") as? String
-            else { return }
+        guard let fleetName = fleetName else { return }
+        
         let notification = NSUserNotification()
         let format = NSLocalizedString("%@ Will Return From Mission.", comment: "%@ Will Return From Mission.")
         notification.title = String(format: format, fleetName)
         let txtFormat = NSLocalizedString("%@ Will Return From %@.", comment: "%@ Will Return From %@.")
         notification.informativeText = String(format: txtFormat, fleetName, name!)
-        if UserDefaults.standard.playFinishMissionSound {
+        
+        if UserDefaults.standard[.playFinishMissionSound] {
+            
             notification.soundName = NSUserNotificationDefaultSoundName
         }
+        
         NSUserNotificationCenter.default.deliver(notification)
+        
         didNotify = true
     }
 }