0

Kotlinにある mapNotNull を実装しようとして(Swiftなので名前はmapNotNilの方が適切か?)

Call can throw, but it is not marked with 'try' and the error is not handled

というエラーが出ました。

extension Array {
    func mapNotNull<ElementOfResult>(_ transform: (Element) throws -> ElementOfResult) rethrows -> [ElementOfResult] {
        return self.compactMap {
            $0
        }.map {
            transform($0) // ここでエラー
        }
    }
}

メソッド定義を

func mapNotNull<ElementOfResult>(_ transform: (Element) -> ElementOfResult) -> [ElementOfResult]

と変えた場合、エラーは消えるのですが、これだと mapNotNull に渡したクロージャが例外を起こすコードを書いた場合コンパイルエラー(to non-throwing function typeのようなコンパイルエラー)となるため、本来の map や compactMap と違う挙動となるため、よくないと考えています(本来のmap や comapctMap に近づけたいです)

うまく rethrows させるにはどう書けばよいのでしょうか?

追記:
そもそもElementが配列オブジェクトができたタイミングで型が決まるので、mapNotNull は実現できないきもしてきました...

さらに追記
OptionalType もしくは OptionalProtocol はどういったテクニックなのでしょうか?

に別の質問を足しました

2 件の回答 2

1

エラーを取るだけなら

extension Array {
    func mapNotNull<ElementOfResult>(_ transform: (Element) throws -> ElementOfResult) rethrows -> [ElementOfResult] {
        return try self.compactMap { // tryを付ける
            $0
        }.map {
            try transform($0) // tryをつける
        }
    }
}

としてください。

あなたの欲しいメソッドはおそらくはこうです。

extension Array {
    func mapNotNull<T, ElementOfResult>(_ transform: (T) throws -> ElementOfResult) rethrows -> [ElementOfResult] 
        where Element == Optional<T> {
            return try compactMap { try $0.map(transform) }
    }
}
3
  • ありがとうございます。よいですね!(upvoteしました) とはいえ、下記のように(こんな書き方を普通しないと思いますが)すると、let a = [1, 2, "3", nil].mapNotNull { String($0) } $0 が、 Type of expression is ambiguous without more context となってしまいます。 これがなぜか OptionalType を使った場合 $0Any型であるとSwiftは認識するようです。 Commented 2019年7月25日 8:20
  • Array<Any>型を対象とする場合は上の方法にする必要があります。 2か所に try が必要です。
    – masakih
    Commented 2019年7月26日 6:31
  • さすがにAnyの件はイレギュラーな気もしてきましたので、「承認」もいたしました Commented 2019年7月26日 7:03
0

https://stackoverflow.com/a/44614084/1979953

を参考に よくわかっていないですが... 下記でできました。

protocol OptionalType {
    associatedtype Wrapped
    var optional: Wrapped? { get }
}

extension Optional: OptionalType {
    var optional: Wrapped? { return self }
}

extension Sequence where Iterator.Element: OptionalType {
    func removeNils() -> [Iterator.Element.Wrapped] {
        return self.compactMap { $0.optional }
    }

    func mapNotNil<R>(_ transform: (Iterator.Element.Wrapped) throws -> R) rethrows -> [R] {
        var temp: [R] = []

        for e in self {
            if let ee = e as? Iterator.Element.Wrapped {
                let t = try transform(ee)
                temp.append(t)
            }
        }

        return temp
    }
}

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

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