Proposal: AsMaybeAsync #528

Activity

1. vkhorikov commented on Jan 27, 2024

   

   vkhorikov

   on Jan 27, 2024

   Owner

   More actions

   Yes, looks good. Feel free to submit a PR please.
2. RoelantM commented on Sep 18, 2024

   

   RoelantM

   on Sep 18, 2024

   Last edited by vkhorikov

   More actions

   perhaps include a implicit operator as well? i was working on a simular solution/problem.
   Where I am calling a function that returns Result<T, TE>. But my method signature should return a Maybe. So in the case that result.isFailure it needs to convert. Now I have:

    public static async Task<Maybe<T>> ToMaybeAsync<T, TE>(this Task<Result<T, TE>> task) where T : class
    {
        var result = await task.ConfigureAwait(false);
        return result.IsSuccess ? Maybe<T>.From(result.Value) : Maybe<T>.None;
    }

   But i guess the implicit converter would even be better.

    public static class MaybeExtensions
   {
       // Implicit conversion operator from Result<T, TE> to Maybe<T>
       public static implicit operator Maybe<T>(Result<T, TE> result)
       {
           return result.IsSuccess ? Maybe<T>.From(result.Value) : Maybe<T>.None;
       }
   }

   I think i cannot add this directly in my code, can I?
3. vkhorikov commented on Sep 28, 2024

   

   vkhorikov

   on Sep 28, 2024

   Owner

   More actions

   The lack of the implicit conversion is intentional. Check out this article: https://khorikov.org/posts/2021-11-08-converting-result-to-maybe/