search

yumemi-inc / ios-training

株式会社ゆめみ iOS研修
Apache License 2.0
306 stars 41 forks source link

YumemiWeather を iOS 13 以降にしてはどうか。 #74

Open yumemi-kumagai opened 6 months ago

yumemi-kumagai commented 6 months ago

YumemiWeather の API の一部に iOS 13 以降で有効と明記されたものがいくつかあります。

たとえば、非同期で天気情報を拾ってくるメソッドがそれに該当します。

@available(iOS 13, macOS 10.15, *)
    public static func asyncFetchWeather(_ jsonString: String) async throws -> String {

これが使えなくても同期版があるため問題はないのですけれど、課題の環境条件に「基本的に最新の安定版を利用すること」とあるのと、バグ解消の課題で使う Example プロジェクトが iOS 17.2 を求めているのを踏まえると、YumemiWeather で一部の機能を iOS 13 以降で解禁するよりも、YumemiWeather 自体を iOS 13 以降にするのが自然かもしれません。

yumemi-kumagai commented 6 months ago

非同期版の使用を求める課題が存在しないこと、そして同期版さえあれば非同期処理を実装できること（課題でもある）を踏まえると、むしろ既存の非同期版 API を廃止する手もありそうです。

iOS 13 以降に制限されているものは非同期に関するものなので、それによって当 Issue を解消できます。