下記コードのようにresultにcase .none が必要になる設計は好ましくないでしょうか？

func fetchData(completion: @escaping ((Result<String, Error>)?) -> Void) {...} fetchData { result in switch result { case .some(.success(let data)): print("Success: \(data)") case .some(.failure(let error)): print("Error: \(error)") case .none: print("Result is nil") } }