1

Qiita の「Swiftで UITextField から UIDatePicker を呼び出して Toolbarに完了ボタンなどを用意する」をswift3で使おうとして次の2点がエラーになってしまいます。改善方法をご教示いただけますか。
1. 34行目

// UIToolBarの設定
toolBar = UIToolbar(frame: ***CGRectMake***(0, self.view.frame.size.height/6, self.view.frame.size.width, 40.0))

CGRectMakeCGRectにすると

Argument labels do not match any....

とエラーになります。これはどのように修正するのでしょうか。

2.//「今日」を押すと今日の日付をセットする」の84行目を Fix it Replaceで修正すると

***let comps: NSDateComponents = calender.components(NSCalendar.Unit.YearCalendarUnit|NSCalendar.Unit.MonthCalendarUnit|NSCalendar.Unit.DayCalendarUnit|NSCalendar.Unit.HourCalendarUnit|NSCalendar.Unit.MinuteCalendarUnit|NSCalendar.Unit.SecondCalendarUnit|NSCalendar.Unit.WeekdayCalendarUnit, fromDate: date)***

No '|' candidates produce the expected contextual result type 'NSCalendarUnit'

とエラーになります。どのように修正するのでしょうか。

http://qiita.com/matsuhisa_h/items/4bb9803828efb89e0305

1 件の回答 1

2

ご質問を書かれる場合は、出来るだけ「状況が再現できる必要十分な情報」を(テキストとして)掲載されるようにしてください。リンクを掲載するのは補助的な情報としては有用ですが、将来リンク切れなどがあると、あなたの質問が(将来の読者にとって)意味をなさなくなってしまいます。

で、あなたのご質問項目を読むと「リンク先のコードをそのままコピペしてXcodeからオートコレクトのサジェスチョンが表示されるところだけ、その修整を当てた状態」になっているようです。

ちなみにそれだけではリンク先のコードを正しく動くようにはできません。あなたが他の修正をうまく当てたことを祈ってますが、上に書いたように必要な情報が無いのでその部分は何とも言えません。


(1) CGRectMake

※結論が早く見たいなら、「結論として」の部分まですっ飛ばしてください。自分の開発スキルを磨きたいなら、リンク先を含め全部に目を通していただきたいところですが。

Swift 3では、多くのインスタンス生成用の関数が削除されました。その殆どは「コンストラクタで代用できるから」です。表題の関数はCGRect生成用の関数ですから、CGRectのコンストラクタの中に代用できる機能があるはず、と言うことになります。

CGRect

Creating Rectangle Values

init(origin: CGPoint, size: CGSize)

Creates a rectangle with the specified origin and size.

init(x: Double, y: Double, width: Double, height: Double)

Creates a rectangle with coordinates and dimensions specified as
floating-point values.

init(x: Int, y: Int, width: Int, height: Int)

Creates a rectangle with coordinates and dimensions specified as
integer values.

init(x: CGFloat, y: CGFloat, width: CGFloat, height: CGFloat)

Creates a rectangle with coordinates and dimensions specified as
CGFloat values.

英語の説明の部分はよくわからなくても、CGRectMakeCGFloat型のパラメータを4つ取る関数だということが分かっていれば、最後のinit(x: CGFloat, y: CGFloat, width: CGFloat, height: CGFloat)が使えるということがわかると思います。

結論としてこんな風に書き換えることになります。

    toolBar = UIToolbar(frame: CGRect(x: 0, y: self.view.frame.size.height/6, width: self.view.frame.size.width, height: 40.0))

(2) NSCalendar.Unit

※結局NSCalendar.Unitは要らないので早く結論が見たければ「さらに修正」の辺りまで飛ばしてください。

NSCalendar.Unit

以前は単なる整数値だったNSCalendarUnit型はOptionSetNSCalendar.Unit型として移入されるようになりました。OptionSetに適合する型では以前の|に相当する演算としてSet型に近い記法(リテラルだけならArray型とも同じ)を使うのですが、ここではその修正をするだけではなく、以下の修正も一緒に行うことにさせてもらいます。

Using Swift with Cocoa and Objective-C (Swift 3.0.1)の中のFoundationからBridged Typesの部分をご覧になってください。

Swift 3では、参照型である幾つかのデータ型に対応する値型のデータ型が追加されています。今回のコードの中から抜粋すると…

NSCalendar       Calendar
NSDateComponents DateComponents
NSDate           Date
NSLocale         Locale

(NSDateFormatterは?、と思われるかもしれませんが、こちらは「クラス名からNSが取れただけ」でまた別物です。)

現在のSwiftはこれらのデータ型に対して、積極的に値型の方を使うような修正はサジェストしてこないことが多いようですが、できるだけ将来に通じるSwift的なコードにしたければ、全部値型の方を使用することをお勧めしておきます。

そういったことを当てはめると、リンク先のdateToStringメソッドはこんな感じになったのですが…

func dateToString(date: Date) -> String {
    //Swiftが型推論を絶対に間違えない場所では、型名は省略
    //OptionSet型やenumの定数は適切な場所では型名部分を省略できる(Calendar.Identifier.gregorian -> .gregorian)
    let calender = Calendar(identifier: .gregorian)
    //OptionSet型では`Set`型風に定数リテラルを使用する、型名部分も省略可
    let comps = calender.dateComponents([.year, .month, .day, .hour, .minute, .second, .weekday], from: date)
    
    //DateFormatterは参照型なので、letが適切
    let date_formatter = DateFormatter()
    //`weekdays`は`String?`(=`Optional<String>`)の配列になってしまうと言う非常に悪い書き方、しかも内容の変更はしないのだから`let`を使うべき
    let weekdays: Array  = [nil, "日", "月", "火", "水", "木", "金", "土"]
    
    date_formatter.locale     = Locale(identifier: "ja")
    date_formatter.dateFormat = "yyyy年MM月dd日(\(weekdays[comps.weekday!])) "
    
    return date_formatter.string(from: date)
}

とここまでコードを書いて、気がついたのですが、上記のコードでDate型の値を文字列に変換すると、曜日を入れたかったのであろう部分によくわからない変な値が埋まってしまいます。

そもそも極めて標準的な曜日名を入れるのにweekdaysなんて自前の配列を用意する必要はありませんし、そうするとあなたが苦労されたNSDateComponentsを使用する必要も無いということになります。

(記事の日付はSwift 2.1時代のものですが、これはその頃のSwiftを使っていたとしても発生するバグなので、「動作確認が十分なされていないコードを掲載している」と言うことになります。)

そこら辺をさらに修正するとこんなコードが出来上がります。

func dateToString(date: Date) -> String {
    //DateFormatterは参照型なので、letが適切
    let date_formatter = DateFormatter()
    //日本語Localeを使用して曜日の1文字表記をしたいならweekdaysなんて配列はいらない
    
    date_formatter.locale     = Locale(identifier: "ja")
    date_formatter.dateFormat = "yyyy年MM月dd日(E) " //<-`E`は曜日出力用のフォーマット文字
    
    return date_formatter.string(from: date)
}

※上記のコードを使用するなら、関連して他の部分を若干修正する必要があります。

ネット上のSwift情報には「いくらSwift2の時代でもこの書き方はないだろう」とか、上記のようにろくに動作確認もされていないと思われるコードが氾濫しています。(これはお勧めできない、どこから、全く動かないという記事がGoogleの上位に来たりします…。)本当にこんなやり方で正しいんだろうかというのを判断できるだけのセンスを磨いていってください。

なお、よく探すと同じ著者自身がSwift 3版の記事を既に公開されているという場合もあります。Swift 3で開発されるなら、初めからSwift 3用の記事を(少々時間をかけてでも)探し出した方が、余計なことに煩わされずに済みますよ。


最初に書いたようにこれだけではリンク先記事のコードを動くようにはできないのですが、この回答に書かれたこと自体に疑問点や誤りなどがありましたらコメントなどで、まだ動くようにできない部分が残りましたら別スレッドで新たな質問として、お知らせください。

1

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。