zdogma / aikatsulyric

アイカツ歌詞と関連動画を閲覧できる iOS アプリ

AikatsuLyric

Description

アイカツの歌詞を確認できる iOS アプリ。
完全に個人用のため、配布はなし。

曲一覧	曲詳細

Install

pod install

Contribution

1. Fork it ( https://github.com/zdogma/AikatsuLyric/fork )
2. Create your feature branch (git checkout -b my-new-feature)
3. Commit your changes (git commit -am 'Add some feature')
4. Push to the branch (git push origin my-new-feature)
5. Create new Pull Request

Author

[zdogma](https://github.com/zdogma/)

aikatsulyric's Issues

残っている force_cast の警告を解消したい

#6

Song のデータの型(Entity)を定義して持ち回したい

#1 (comment)

持ち回すデータの型(Entity)を定義する

ViewControllerのdataをSong型として定義してあげるとスッキリします。あとJSONマッピングをObjectMapperなどを使ってやるとJSONとEntity間の変換が見通しやすくなりますよ！

import ObjectMapper

struct Song: ImmutableMappable {
    let title: String
    let text: String
    let thumbnail_url: String
    let series: String
    let scene: String
    let singer: String
    let embed_movie_src: String

    init(map: Map) throws {
        title = try map.value("title")
        text = try map.value("text")
        thumbnail_url = try map.value("thumbnail_url")
        series = try map.value("series")
        scene = try map.value("scene")
        singer = try map.value("singer")
        embed_movie_src = try map.value("embed_movie_src")
    }
}
そんでSong型オブジェクトをSongListTableViewControllerとViewControllerで持ち回すようにするといいです。ローカルのJSONファイルから[Song]に変換するメソッドはたとえば次のようになります。

func loadSongs() throws -> [Song] {
    if
        let path = Bundle.main.path(forResource: "aikatsu_songs", ofType: "json"),
        let data = FileManager.default.contents(atPath: path) {
        let json = try JSONSerialization.jsonObject(with: data, options: JSONSerialization.ReadingOptions.allowFragments) as! [[String: Any]]
        return try Mapper<Song>().mapArray(JSONArray: json)
    } else {
        // なにかがおかしいよエラー
        throw CustomError.somethingWrong
    }
}

// どっか別のファイルに定義してもよい
enum CustomError: Error {
  case somethingWrong
}

SwiftLint を導入したい

#1 (comment)

お作法的なところ
SwiftLintに従って書くと大体コミュニティーのベストプラクティスに則れます。
https://github.com/realm/SwiftLint
Cocoapodsで導入してXcodeのBuildPhaseでチェックするようにするといいです。

エラーハンドリングを見直したい

#6

例外の作法

なんつっても例外を握りつぶさないってとこですね。
こちらも参照してみてください。
https://ez-net.jp/article/B2/cwtBwURB/pp3ZiIOQz2Cc/
Result型でのError表現などにも触れられています。

Result型使いたいときは
https://github.com/antitypical/Result
これが良いです。

SwiftLint に則って Caution 対応したい

Caution の数が多そうであれば種類別に PR を分けて対応する。
#1 (comment)

特に
force_cast
force_try
force_unwrapping
をなるべく避けるようにするとよりSwiftらしくなると思います。

例えばこの辺のforce unwrappingを治すとしたら、
var dataを暗黙的アンラップ型で定義して、dataに値が渡ってなかったら非正常系というのを型で表現し、try ~ catchで例外を補足して適当にエラー表示する
Optional Chaining、Optional Bindingを使って、dataにデータが渡されてるとき(正常系)と、渡されてないとき(非正常系)を表示分けする
if let thumbnailUlr = data?.thumbnailUrl, !thumbNailUrl.isEmpty {
// データがあるとき
} else {
// データがないとき
}
のどちらかといった感じです。