Swift 簡単なアラームアプリ コードの書き方の質問

Question

Swift初心者です。簡単なアラームアプリを作りたいと思っています。

まずクラスに、

を作成しました。

現在ViewControllerにインスタンスを生成し、コードを書いています。

現在やりたいことは、
①MainStoryBoardに設置したUISwitch Buttonをオンにすると、クラスに定義した「vibration」のBool型にon/offと、コンソール上に出力させる

②同様にMainStoryBoard上のUIDatePickerで日にち・時間を選択すると、クラスに定義した「time」のDate型に、yyyy/MM/dd HH:mmのフォーマットで、コンソール上に出力させる

です。
今のところここまで進みました。

エラーが出てしまいRun出来ない状態で、途方に暮れております。
また、②のDatePickerで時間を表示させるやり方が分からず、苦戦しております。

稚拙な説明で分かりづらいかと思いますが、どうかご教示頂けると幸いです。
よろしくお願い致します。

---

追記：
コメントありがとうございます。

こちらがクラスを定義したコードで、

import Foundation
//Alarm型の定義
class Alarm {
    var title: String?
    var time: Date?
    var duration: Int?
    var musicURL: String?
    var vibration: Bool?
    }
 
func initialAlarm() -> Alarm {
    return Alarm()
}

こちらが現在コードを書いているViewControllerです。

//
//  AlarmCreateViewController.swift
//  share_alarm
//
//  Created by -- on 2016-12-11.
//  Copyright © 2016 --. All rights reserved.
//

import Foundation
import UIKit

class AlarmCreateViewController: UIViewController {
    var alarm: Alarm = initialAlarm()
    
    //Durationを入力 
    //When the value of Int is written within UITextField named sender, the values are inserted into the variants "duration" in the class "Alarm".
    @IBAction func input(_ sender: UITextField) {
        if let text = sender.text {
            if let duration = Int(text){
                alarm.duration = duration
                    }
            }
            dump(alarm)
    }

    //Titleを入力
    //When the value of String is written within UITextField named sender, the values are inserted into the variants "title" in the class "Alarm".
    @IBAction func titleText(_ sender: UITextField) {
        if let text2 = sender.text {
            if let title = String(text2){
                alarm.title = title
            }
            dump(alarm)
        }
    }
    
    //vibrationをオン・オフにする
    //When the UISwitch is on, the state of vibration "on" is inserted into the variants "vibration" in the class "Alarm".
    @IBAction func onOffSwitch(_ sender: UISwitch) {
        let buttonState: Bool = sender.isOn
        if sender.isOn {
            let vibrationState = Bool(true)
            alarm.vibration = vibrationState
            dump(alarm)
        }else{
            let vibrationState = Bool(false)
            alarm.vibration = vibrationState
            dump(alarm)
        }
    }

    

    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view, typically from a nib.
    }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
}
}

エラーは、

        let buttonState: Bool = sender.isOn

で出ております。
行き詰まっており、どうしたらよいのか分からない状態で、、お手数ですが、どうかよろしくお願い致します。

ーーーーーー
追記２ 新たなエラー

   //vibrationをオン・オフにする
    //When the UISwitch is on, the state of vibration "on" is inserted into the variants "vibration" in the class "Alarm".
    @IBAction func onOffSwitch(_ sender: UISwitch) {
        var buttonState: Bool = sender.isOn
        if buttonState == true{
            var vibrationState = Bool(on)
            alarm.vibration = vibrationState
            dump(alarm)
        }else{
            let vibrationState = Bool(off)
            alarm.vibration = vibrationState
            dump(alarm)
        }
    }

エラーは、

var vibrationState = Bool(on)
と
var vibrationState = Bool(off)

に出ております。

Comments

[_Kentarou]

全体的な作りが分からないので対象のコードを全て載せてもらってもいいですか？(画像でなくて)どの場所でエラーが起きているのかも分からないです。

Answer 1

まずlet buttonState: Bool = sender.isOnはエラーでは無くワーニング（警告）です。
buttonState変数を定義したのに使用されていないよという警告です。

エラーが出てしまいRun出来ない状態で

と書いてありますがどの部分でエラーが出ているのでしょうか？上記のワーニングはRunはできると思いますので。

②に関しては以下のサイトを参考にするとできると思います。
UIDatePickerの表示

Comments

[Amyz_S]

ご親切にコメント頂き感謝しております。
確認したところワーニングでした。失礼いたしました。
エラーはAppDelegateのところに{//Thread1: signal SIGABRT.と出ていました。

現在コードを見直し書きなおしたところ、違うエラーが出てしまい、「Use of unresolved identifier “on”」「Use of unresolved identifier “off”」と出ております。対象のコードを追記にアップさせて頂きます。非常に恐縮ですが、ご教示頂けると幸いです。よろしくお願い致します。

[_Kentarou]

Boolはtrueかfalseなのでon、offではありませんよ。
やっている事は変数の代入して更に代入してということなので以下でよいのでは？

@IBAction func onOffSwitch(_ sender: UISwitch) {
alarm.vibration = sender.isOn
}

[_Kentarou]

> エラーはAppDelegateのところに{//Thread1: signal SIGABRT.と出ていました。

おそらくコンソールにエラーが出ていると思うのでそちらを載せてください。

[Amyz_S]

ご返信ありがとうございます。
Boolの使い方とコードのヒントもありがとうございます。
@IBAction func onOffSwitch(_ sender: UISwitch) {
alarm.vibration = sender.isOn
}
以上で書き直し、runを試みたところ、コンソールには

2017-01-08 16:42:41.985 share_alarm[22797:731361] *** Terminating app due to uncaught exception 'NSUnknownKeyException', reason: '[<share_alarm.AlarmCreateViewController 0x7f97a5c19930> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key Vibration.'
*** First throw call stack:
(
0 CoreFoundation 0x000000010cde3d4b __exceptionPreprocess + 171
1 libobjc.A.dylib 0x000000010a1e221e objc_exception_throw + 48
2 CoreFoundation 0x000000010cde3c99 -[NSException raise] + 9
3 Foundation 0x0000000109cf09df -[NSObject(NSKeyValueCoding) setValue:forKey:] + 291
4 UIKit 0x000000010a84c293 -[UIViewController setValue:forKey:] + 88
5 UIKit 0x000000010aac079e -[UIRuntimeOutletConnection connect] + 109
6 CoreFoundation 0x000000010cd889e0 -[NSArray makeObjectsPerformSelector:] + 256
7 UIKit 0x000000010aabf122 -[UINib instantiateWithOwner:options:] + 1867
8 UIKit 0x000000010a8529c5 -[UIViewController _loadViewFromNibNamed:bundle:] + 386
9 UIKit 0x000000010a8532e7 -[UIViewController loadView] + 177
10 UIKit 0x000000010a85361c -[UIViewController loadViewIfRequired] + 201
11 UIKit 0x000000010a85a062 -[UIViewController __viewWillAppear:] + 118
12 UIKit 0x000000010a8851d3 -[UINavigationController _startCustomTransition:] + 1290
13 UIKit 0x000000010a895e48 -[UINavigationController _startDeferredTransitionIfNeeded:] + 697
14 UIKit 0x000000010a896fdb -[UINavigationController __viewWillLayoutSubviews] + 58
15 UIKit 0x000000010aa8ddd7 -[UILayoutContainerView layoutSubviews] + 223
16 UIKit 0x000000010a776ab8 -[UIView(CALayerDelegate) layoutSublayersOfLayer:] + 1237
17 QuartzCore 0x000000010fc29bf8 -[CALayer layoutSublayers] + 146
18 QuartzCore 0x000000010fc1d440 _ZN2CA5Layer16layout_if_neededEPNS_11TransactionE + 366
19 QuartzCore 0x000000010fc1d2be _ZN2CA5Layer28layout_and_display_if_neededEPNS_11TransactionE + 24
20 QuartzCore 0x000000010fbab318 _ZN2CA7Context18commit_transactionEPNS_11TransactionE + 280
21 QuartzCore 0x000000010fbd83ff _ZN2CA11Transaction6commitEv + 475
22 UIKit 0x000000010a6aad9b _UIApplicationFlushRunLoopCATransactionIfTooLate + 206
23 UIKit 0x000000010aeb577c __handleEventQueue + 5672
24 CoreFoundation 0x000000010cd88761 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 17
25 CoreFoundation 0x000000010cd6d98c __CFRunLoopDoSources0 + 556
26 CoreFoundation 0x000000010cd6ce76 __CFRunLoopRun + 918
27 CoreFoundation 0x000000010cd6c884 CFRunLoopRunSpecific + 420
28 GraphicsServices 0x000000010ed23a6f GSEventRunModal + 161
29 UIKit 0x000000010a6b1c68 UIApplicationMain + 159
30 share_alarm 0x0000000109bfef1f main + 111
31 libdyld.dylib 0x000000010dd9368d start + 1
32 ??? 0x0000000000000001 0x0 + 1
)
libc++abi.dylib: terminating with uncaught exception of type NSException
(lldb)
と表示されております。

[_Kentarou]

Vibrationという名前のパーツがエラーの様にかいてありますが、コードとStoryboardをひも付けたパーツでそのような名前のパーツを結んだ覚えはありますか？
またその名前をコード側で変更しましたか？

[_Kentarou]

それと以下の様な記述がありますが、var alarm: Alarm = Alarm()　と同じ事なのでメソッドを作る必要がないかと思います。

func initialAlarm() -> Alarm {
return Alarm()
}

[Amyz_S]

ご返信大変感謝しております。また、何度も何度も質問し、申し訳ありません。
確認したところ、ViewControllerのパーツの名前がなぜかVibrationという名前に置き換わっていたため、１からやり直しコードを書いたところ問題無く動きました。動いた時は涙が出ました。。。どうもありがとうございます。

また、メソッドに関してのアドバイスもありがとうございます。
Duration, Title, Vibrationは希望通りにクラスの変数に代入され、コンソールに出力されるようになったので、残りのUIDatePickerのコードも書いていきたいと思います。

[Amyz_S]

教えて頂いたサイトを参考に書いて見ます！！！本当にありがとうございました。