クラス固有の型をモデルからコントローラに戻す必要がありますか？

Question

私はSwiftとiOSの開発を学ぶためのトレーニング追跡アプリケーションを作成しています。私は大学から15年以上も経っていないので、このMVCのことは初めてです。ガイダンスは歓迎されますが、具体的には、モデルとコントローラの間の通信方法に関するベストプラクティスに関する質問があります。

マイモデル（下のクラス名「エクササイズ」）には、独自のタイプ（「ウェイト」、「レプス」など）があり、それらのタイプをコントローラに戻してビューを作成するのは間違いです。私の直感は正しい？コントローラーがクラス固有のデータ構造の代わりに値を要求するように、これを1つ上のレベルで抽象化する必要がありますか？またはこれはMVCにとって全く正常ですか？

私が今やっていることを実証するためのコードですが、間違っています。問題の間違った部分はconfigureView（）にあります。

モデル：

class Exercise { 

    var name = "placeholder name" 
    var formNotes = "placeholder notes" 
    private var repLog: [repLogEntry] = [] 
    private var currentWeights = Weights(warmup25: 0, warmup50: 0, heavy: 0) 

    init() { 
     // some test data 
     name = "Squat" 
     formNotes = "Grasp the bar 1 inch outside knurling\nPull bar apart while squeezing shoulders back" 
     self.recordExercise("16-04-20", weight: 135, reps: Reps(firstSet: 10, secondSet: 8)) 
     self.recordExercise("16-04-22", weight: 135, reps: Reps(firstSet: 11, secondSet: 9)) 
     self.setWeights(135) 


    } 

    // data to be recorded 
    private struct repLogEntry { 
     var date: String 
     var weight: Int 
     var reps: Reps 

    } 

    // reps always come in pairs 
    struct Reps { 
     var firstSet: Int 
     var secondSet: Int 
    } 

    // weights of the exercise 
    struct Weights { 
     var warmup25: Int 
     var warmup50: Int 
     var heavy: Int 
    } 

    func recordExercise(date: String, weight: Int, reps: Reps) { 
     let newRepLogEntry = repLogEntry(date: date, weight: weight, reps: reps) 
     repLog.append(newRepLogEntry) 
    } 

    func setWeights(newWeight: Int) { 
     currentWeights.heavy = newWeight 
     currentWeights.warmup25 = Int(Double(newWeight) * 0.25) 
     currentWeights.warmup50 = Int(Double(newWeight) * 0.50) 
    } 

    func getCurrentWeights() -> Weights { 
     return currentWeights 
    } 

    func getLastWorkoutReps() -> Reps { 
     return repLog.last!.reps 
    } 

    func getLastWorkoutDate() -> String { 
     return repLog.last!.date 
    } 

    func getLastWorkoutWeight() -> Int { 
     return repLog.last!.weight 
    } 
} 

とコントローラ：MVCのために

class DetailViewController: UIViewController { 

    @IBOutlet weak var detailDescriptionLabel: UILabel! 

    @IBOutlet weak var label: UILabel? 

    let Squat = Exercise() 

    var detailItem: AnyObject? { 
     didSet { 
      // Update the view. 
      self.configureView() 
     } 
    } 

    func configureView() { 
     // Update the user interface for the detail item. 
     if let detail = self.detailItem { 
      if let label = self.detailDescriptionLabel { 
       label.text = detail.valueForKey("timeStamp")!.description 
      } 
     } 

     let currentWeights = Squat.getCurrentWeights() 
     let exerciseName = Squat.name 
     let formNotes = Squat.formNotes 
     let warmup25Text = String(currentWeights.warmup25) 
     let warmup50Text = String(currentWeights.warmup50) 
     let heavyText = String(currentWeights.heavy) 
     let lastWorkoutReps = Squat.getLastWorkoutReps() 
     let lastWorkoutDate = Squat.getLastWorkoutDate() 
     let lastWorkoutWeight = Squat.getLastWorkoutWeight() 

     label?.text = "\(exerciseName)\nWarmup (25%): \(warmup25Text)\nWarmup (50%): \(warmup50Text)\nHeavy (100%): \(heavyText)\n\(lastWorkoutDate) Reps @\(lastWorkoutWeight): \(lastWorkoutReps.firstSet) and \(lastWorkoutReps.secondSet)\n\nForm Notes:\n\(formNotes)" 

    } 

    override func viewDidLoad() { 
     super.viewDidLoad() 
     // Do any additional setup after loading the view, typically from a nib. 
     self.configureView() 
    } 

    override func didReceiveMemoryWarning() { 
     super.didReceiveMemoryWarning() 
     // Dispose of any resources that can be recreated. 
    } 
} 

Answer 1

あなたがやっていることは、 '罰金' です。あなたはModel、View、Controllerの区別があり、ControllerはModelとViewの間を仲介しています。

MVCでは、コントローラは通常、ビュー上のデータのすべてのレイアウトとビューのすべてのユーザー対話を管理しているため、大きくて乱雑になります。モデルをビューに渡してビューを構成することもありますが、コントローラがシンプルになりますが、MVCが良くありません。

MVVMの拡張機能であるMVVM - Model View ViewModelの使用を検討できます。基本的には、ModelデータをControllerからどのように表示するかのロジックを取り、それをViewModelに配置します。ViewModelはViewに提供され、ViewModel自体を構成します。これはコントローラを簡素化するだけでなく、モデルからのビューを絶縁します。

したがって、ViewModelにはtext = "\(exerciseName...コードが含まれていますが、それだけです。 Modelオブジェクトをどのようにして表示するかを知るだけです。それはどこに表示されるのか分かりません。 ViewはこのViewModelで構成されているので、必要なデータを取得できますが、どこから来たのかは分かりません。コントローラは、ViewModelを作成してViewに渡す中間的な人間です。