Swiftの書き方について教えていただきたい

Swift4でAPIにリクエストしてレスポンスを使って何か処理をするようなことをしたいのですが、イマイチよくわからず下記のようなコードになってしまいます。
とても冗長な感じがするのですがうまいことやるサンプルを教えていただきたいです

func request(callback:((_ result: Dictionary<String,Any>?)->Void)?) {
    let response: Dictionary<String,Any> = [
        "code":200,
        "result": [
	    "id": 1,
	    "name": "user name"
        ]
    ]
    callback!(response)
}

self.request { (response) in
  guard	let result = response else {
    return
  }
  guard	let code = result["code"] as? Int else {
    return
  }
  if code == 200 {
    // result["result"] で何らかの処理
  }
}

fuzzball

2018/08/02 00:43

どの辺に冗長さを感じているのでしょうか？guardをまとめるとか？request()の中は実際は非同期なんですよね？

行動規範の内容に同意します

回答1件

ベストアンサー

僕ならキャストが面倒くさいので

swift
1struct Response {
2
3    struct AAAResult {
4    
5        let id: Int
6        let name: String
7    }
8    
9    let code: Int
10    let result: AAAResult
11    
12    init(id: Int, name: String, code: Int) {
13    
14        self.code = code
15        self.result = AAAResult(id: id, name: name)
16    }
17}

を作って使いますね。（テキトーです）

swift
1func request(callback:((_ result: Dictionary<String,Any>?)->Void)?) {

これは

swift
1func request(callback: ((Dictionary<String, Any>?) -> Void)?) {

のように書くのが普通です。

swift
1    callback!(response)

にしてますけど、これはあり得ませんね。
引数((Dictionary<String,Any>?)->Void)?を非Optionalにするか

swift
1   callback?(response)

にすべきです。

引数のデフォルト値を {_ in }にしてしまう方法もあります。

swift
1func request(callback: (Dictionary<String, Any>?) -> Void = {_ in }) {

あとは、エラーが欲しくなると思うのでcallbackの引数をOptionalではなくいわゆるResult型にするかな？

swift
1enum Result<T> {
2   case value<T>
3   case error<Error>
4}

あるいは

swift
1enum Result<T, E: Error> {
2    case value<T>
3    case error<E>
4}

をモナドにした奴です。

「Swift Result」で検索すればいくつか見つかると思います。

2種類あるのは派閥です。どっちも使えるものなのでどちらを選ぶかは使用者の自由です。

エラーを使わないなら単に冗長さを増すだけなので僕はResultは導入はしません。

投稿2018/08/02 02:39

MasakiHori

総合スコア3384

yoppy0066

2018/08/02 04:49

丁寧にご回答頂きましてありがとうございます。 > callback!(response) がありえない理由引数ではnil許可としてるのに、指定なしで呼び出すとクラッシュするからという意味になるでしょうか？ > func request(callback: (Dictionary<String, Any>?) -> Void = {_ in }) { すみません、こちらの意味がよくわかりませんでした。。教えて頂けますと幸いです