How to use Discriminated Unions Today in C#

Sdílet
Vložit
  • čas přidán 12. 05. 2024
  • Use code REST15 for 15% off the new From Zero to Hero - REST APIs in .NET course: bit.ly/restchapsas
    Become a Patreon and get source code access: / nickchapsas
    Hello everybody I'm Nick and in this video I will show you how you can use Discriminated Unions in C# using a Nuget package called OneOf, so you don't have to wait until the feature makes in in C# and be ready to migrate when it does launch.
    Give OneOf a star on GitHub: github.com/mcintyre321/OneOf
    Workshops: bit.ly/nickworkshops
    Don't forget to comment, like and subscribe :)
    Social Media:
    Follow me on GitHub: bit.ly/ChapsasGitHub
    Follow me on Twitter: bit.ly/ChapsasTwitter
    Connect on LinkedIn: bit.ly/ChapsasLinkedIn
    Keep coding merch: keepcoding.shop
    #csharp #dotnet

Komentáře • 245

  • @jongeduard
    @jongeduard Před rokem +25

    Really great video with very clear example! Also better than your earlier video "Don't throw exceptions in C#. Do this instead". Also with a better nuget package, which is definitely worth to try.
    Of course having the actuall tagged unions implemented in C# would be better, resulting in more fine grained possibilities with pattern matching.
    And I think it would also be great to see a better type aliasing solution in C#. because falling back to inheritance and a ton of code generation is still not the nicest solution.
    But it's all still really great for now.

  • @billy65bob
    @billy65bob Před rokem +2

    I was just pondering ways to do multiple but exclusive return types, and this kind of thing is actually perfect.
    On the face of it, it's exactly what I wanted, but even better

  • @JoeIrizarry88
    @JoeIrizarry88 Před 11 měsíci +2

    Love this and completely agree with your opinion on hidden, magical functionality of Filters or other Pipeline middleware for error handling. The language features force you to handle error conditions where previously (many moons ago) the only way to ensure it didn't get missed was exceptions at runtime.

  • @christianmarpert3844
    @christianmarpert3844 Před rokem

    I come from Dart and there are actually a couple of packages tackling the functional approach. Have been using Options and Either, pattern matching seems to be added soon. I see the big advantage in a cleaner control flow, no exceptions flying around to be catches etc. still a beginner in c#, this looks kind of familiar👍 Nice explanation, thks

  • @asedtf
    @asedtf Před rokem +4

    I saw the notification and dropped everything I was doing to watch this immediately!

  • @GregerOlofsson
    @GregerOlofsson Před rokem

    Love it! Will need to try it. Keep up the great vids!

  • @krccmsitp2884
    @krccmsitp2884 Před rokem

    I love the concept and use similar solutions in newer projects whenever I can.

  • @GrantNelson1
    @GrantNelson1 Před rokem

    Very interesting. At work I'm building an API written in Go. To solve the same issue, I created a monad (as close as Go generics allow) which can either be "not set", "error", or "value". I made it so I could use it for internal fields as well and apply constraints for checking maximum, minimum, is empty, is unique, to set defaults if not set, and so on. It works for what the company needed but I wasn't sure it was a great solution. After seeing this, I realized two things; my solution isn't as elegant (mostly because of Go) and it was an acceptable solution, not something crazy I shouldn't have attempted. Thanks

  • @CiaranGallagher1
    @CiaranGallagher1 Před rokem

    Great video, thanks! Looking forward to giving this a try at some point!

  • @debtpeon
    @debtpeon Před rokem

    I would use it. This is great for writing backend APIs for writing business rules where you have to return multiple result types. Thanks for this video.

  • @rick2591
    @rick2591 Před rokem

    This is really cool. I just had a case use for this. Due to the code being flexible I had multiple fail levels, warning levels, etc.

  • @ilyakurmaz
    @ilyakurmaz Před rokem

    This is something I adore in Typescript. Thanks for the video.

  • @JonathanPeel
    @JonathanPeel Před rokem +28

    I have always found using a F# project for models (in a C# solution) to be a very good solution.
    With this I could even do records before C# had records.
    I use F# a lot, and even have some solutions where the API, domain, shared/kernel are all F# and just use C# for UI, EF.Core and the few places I prefer it.
    My biggest hope if C# created Union type is that there is some sort of compatibility with the Unions in F# already.
    It would be nice to pass unions between the language and have them both act natural.

    • @obinnaokafor6252
      @obinnaokafor6252 Před rokem

      F# union is just a compiler tricks.

    • @obinnaokafor6252
      @obinnaokafor6252 Před rokem

      ​@@JonathanPeel A lot involves some changes in the runtime and even modification if the IL

    • @AbrahamJLR
      @AbrahamJLR Před rokem

      Jonathan, Can you share any repositories where interop is present using F# as the solution domain?
      I am very curious about the implementation.

    • @JonathanPeel
      @JonathanPeel Před rokem

      @@AbrahamJLR I don't have anything on my public github, but it should be easy for you to put a silly example together.
      I will try remember something as well.
      I am replying to this first thing in the morning, so I might forget through the day, shout gain if you want me to do something.
      Or if you come up with something send it as well. I am very new to F# and any learning is good.

    • @JonathanPeel
      @JonathanPeel Před rokem

      @@obinnaokafor6252 But then they should put those back into F# and make it a IL/.Net feature.
      I am just thinking it would really be annoying if C# treated an F# Union as fancy abstract and some concrete, and F# ended up doing the same with C# Unions, and the benefit could not really be felt cross language.

  • @Neonalig
    @Neonalig Před rokem +3

    Man, I remember watching Nick the Greek back in the day. One of the best movies of all time and still cracks me up even to this day. Good times!

    • @bdcp
      @bdcp Před rokem

      it's one of my favorites for sure

  • @gakshay9537
    @gakshay9537 Před rokem

    A very useful nuget package and well explained. Thanks alot

  • @ali_randomNumberHere
    @ali_randomNumberHere Před rokem

    i've been using this since you introduced it a long time ago, It's an amazing package

  • @mohamedsarfaraazosman6419

    nice, i enjoy the fast pace, no wasting time. Discriminate union is looking useful

  • @Denominus
    @Denominus Před rokem +3

    Absolutely will use DUs when they are implemented in C# with exhaustive matching, already use them constantly in F#. They are even more powerful for domain modeling. I don't see myself using OneOf though.

  •  Před rokem +4

    Thank you for the video. Without having DU directly in C# it does seem like the next best thing. I would personally use it, however I do think it's a harder 'sell' in a larger team ... but that will always be a problem with 'altnernative' solutions :)

  • @istovall2624
    @istovall2624 Před rokem

    i use a base controller class with control flow in the basic crud endpoints, validate (fluent), map (automapper), send to service (usually works for non crud endpoints too).
    maybe not the best possible way but it's clean and easy to follow.
    this seems less obfuscated, certainly, but at the cost of adding a lot of 'noise' imo.

  • @PanzerFaustFurious
    @PanzerFaustFurious Před rokem +61

    The type system of typescript has spoiled me

    • @nickchapsas
      @nickchapsas  Před rokem +30

      Languages that support DUs ❤️

    • @user-zk5ym9ut1j
      @user-zk5ym9ut1j Před rokem +3

      Typescript has a completely different type system, structural one, not nominal

    • @obinnaokafor6252
      @obinnaokafor6252 Před rokem +4

      Typescript has the advantage of its types being erased at runtime - by the virtue of seating on top of a dynamic runtime, it has that flexibility with structural typing.

    • @obinnaokafor6252
      @obinnaokafor6252 Před rokem

      @Miguel Yes, TypeScript is a better JavaScript.

  • @kishortiwari1348
    @kishortiwari1348 Před rokem +1

    I guess same as typical Result but more of a deconstructed version & allowing you to perform result checks in a lamda expression. Both approaches allows you to return the desired type or result but this approach looks more natural probably with additional magic.

  • @CodeConstellations
    @CodeConstellations Před rokem +15

    I found your channel yesterday. I am simultaneously ecstatic and overwhelmed to dive in to your content, you're brilliant.
    Do you have a recommendation on when to use minimal apis vs controller apis?

    • @MisterNOmercy
      @MisterNOmercy Před rokem +1

      Personally I always use controller API's because I find the complexity more difficult to manage with minimal API's.
      But it's pretty easy to migrate from minimal API's to controllers, so if you are starting out with a project and your team agrees you can always get started with minimal API's and migrate later. Both approaches will be supported for a very long time, but most innovation will happen in the minimal API space I think. Although that innovation right now is catching up to feature that the controller approach already supports.

    • @reikooters
      @reikooters Před rokem

      Check out FastEndpoints. There's a video about it on this channel.

    • @CodeConstellations
      @CodeConstellations Před rokem

      @@MisterNOmercy Thank you for the feedback! I recently built a minimal api and found a solid way of organizing it, so if that's what is likely to get more support in the future I may want no keep working on that I suppose.

  • @eqwerty610
    @eqwerty610 Před rokem

    Damn, really good video, Nick! And awesome library.

  • @zacky7862
    @zacky7862 Před rokem +1

    Wow this is very helpful. I usually returning as Tuple

  • @modernkennnern
    @modernkennnern Před rokem +4

    There are two things I would love if it worked with this.
    1) Actual switch expression, the match method - while functional - is similar to constructors in that they are one big method with semi-arbitrary values based on ordering, while switch expressions are more like object initializes
    2) generic alias types. `Maybe = OneOf`
    Both of these things require compiler-level changes - especially the latter; not 100% sure if the former is possible with some weird wrapper type combined with Roslyn trickery

    • @chris-pee
      @chris-pee Před rokem

      2 - Yeah I also noticed the missing Maybe (or Option) type. Maybe (pun intended) you could derive it: class Maybe : OneOfBase ?

  • @GufNZ
    @GufNZ Před rokem

    I effectively implemented exactly that pattern for something at work, but the use of implicit operators was considered a bit too magic, and went against the company value of "simple is better than clever" too much. Now I have an explicit ResultOrError.Success(T) or ResultOrError.Fail(Errors) sort of thing.
    BTW: R# and Rider can show inline type info stuff, and that includes a little icon wherever an implicit operator is used.
    PS: Check out R# and Rider's extended colours - colouring things by their semantic meaning, so things like const, enum value, parameter, local mutable var, local singly-assigned var can all be different colours; similarly Class vs Static Class vs Struct vs Record etc.

  • @emmanueladebiyi2109
    @emmanueladebiyi2109 Před rokem

    Very lovely library. Would so much love to see this supported natively in C#

  • @1000percent1000
    @1000percent1000 Před rokem +13

    Coming from Rust, I think this is amazing and it is making me want to try out C#. It's more dynamic than Rust is capable of being which I think is perfect considering the immediate performance cost switching to C# from Rust, but it's promising to me that the more integrated runtime and environment is being used to provide features like this.

    • @driedurchin
      @driedurchin Před rokem

      If you like this, then you'll also probably like Kotlin.

    • @dzllz
      @dzllz Před rokem

      Rust is overrated

    • @driedurchin
      @driedurchin Před rokem +2

      @@dzllz I mean probably, at the end of the day a tool is a tool. That being said, excitement is good!

    • @user-zw5nd5uc1q
      @user-zw5nd5uc1q Před rokem

      @@driedurchin type inferring in C# sucks too much making functional programming and its derived variations pain in the ass

    • @J-Kimble
      @J-Kimble Před 7 měsíci +1

      You'll love F#. It's also derived from ML like rust. It's also pretty safe, but without the borrow checker.

  • @Matt23488
    @Matt23488 Před rokem +8

    This is a neat feature. But it looked very familiar. This video made me realize that a discriminated union solves the same problem that enums solve in Rust. Rust enums are very powerful so this is an exciting feature for C#! Yes Rust code is littered with enums and match statements, which leads to very declarative code, I'm very happy to hear we will get something very similar in native C# in the future.

    • @RogueTravel
      @RogueTravel Před rokem +3

      Yes, I prefer Rust matching and it’s optional type. Discriminated unions here start to approach the power of the optional type in Rust, with Some and None. Kotlin and Swift do something similar in the mobile space, with the generic Result type. Languages seem to be moving away from null in general

    • @johnTheUnigoat
      @johnTheUnigoat Před rokem +6

      In the world of rust the most talked about things seem to be speed and the borrow checker, but I'd argue that one of the most interesting things about rust is it's lack of null, and a type system that forces you to defie and check for every possible state. Basically yes, rust enums are amazing

  • @BillyBraga
    @BillyBraga Před rokem +1

    I didn't know about that source generator, nice! It would be even better if it could generate a struct like the OneOf from the package

  • @microtech2448
    @microtech2448 Před rokem

    Nice one. Out of this video, can you please create a video on how to use fluent validations as attribute at the top of the class if possible?
    Thanks for this video.

  • @orterves
    @orterves Před rokem +6

    Source generators seem to be a very underappreciated power tool for avoiding boilerplate

    • @OllyWood688
      @OllyWood688 Před rokem

      Yeah that was my biggest "hey I should look into this" takeaway from this video too :D

    • @nickchapsas
      @nickchapsas  Před rokem +2

      I think more authors need to make it the norm in their libraries

  • @yanpitangui
    @yanpitangui Před rokem +5

    Hello, Nick!
    Are you able to use this with a Validation Pipeline in mediator? In a way so we don't need to repeat in our services every method that needs validation or something.
    Also, can we use something like a result filter so we don't need to always use match for all our controllers actions? Thanks

  • @digibrett
    @digibrett Před rokem

    Literally need this feature I just learned about just now.

  • @maxhamulyak8013
    @maxhamulyak8013 Před rokem +7

    Would be interesting to see a benchmark comparing the exception flow with the union flow performance wise

    • @nickchapsas
      @nickchapsas  Před rokem +8

      I’ve already made that video. Check my "don’t throw exceptions" video

  • @sudamahebert6978
    @sudamahebert6978 Před rokem +1

    I am not a fan of the match or switch method that require to declare behaviour in the same order than the declared generics. It makes for a terse syntax but it , somewhat, couple the implementation with the consumer.
    I can see the apeal though. I will have to test it out to see if it is really a problem or if i'm just grumpy this morning :D
    I did not know of this techniques, thank you!

  • @amai_zing
    @amai_zing Před rokem +2

    I think this is great - I like self-describing code, and I think this goes a long way towards that - I also strongly dislike error-handling for expected conditions, and I feel this makes the code a lot more dry

  • @verzivull
    @verzivull Před rokem

    That awesome1 I wonder will work with pattern matching since it has a implicit convert?

  • @Yowax
    @Yowax Před rokem

    This is very similar to the Result enum in Rust, I'm happy that there's a way to get something like that in C#. Having errors as values should be a must in any modern programming language nowadays.

  • @alexander_nunezf
    @alexander_nunezf Před rokem +12

    It would have been great if we could use an alias for our OneOf long type in the top part of the file, instead of class inheritance there.Everything else is cool. I would like a next video explaining in depth why this approach is better than throwing builtin/custom exceptions.

    • @VoroninPavel
      @VoroninPavel Před rokem +14

      > It would have been great if we could use an alias for our OneOf long type in the top part of the file
      using SomeName= Namespaces.OneOf;
      If you have some fixed set of types for the whole project, then you can make global alias:
      global using SomeName= Namespaces.OneOf;
      > explaining in depth why this approach is better than throwing builtin/custom exceptions.
      Cons of Exceptions:
      Performance - exceptions are expensive. Especially, if some branch of code execution is an expected and frequent case.
      Implicit - It's impossible to know from method's signature what exceptions method throws.
      Because of the previous: No compiler safety - compiler cannot guarantee caller checks all cases.
      Easy to break Liskov's principle.
      When an exception is thrown, components up the call stack may end up having corrupted state.

    • @wojciechwilimowski985
      @wojciechwilimowski985 Před rokem +2

      One possible justification is that it's certainly better if you have fallback actions for error scenarios. Task.WhenAll is a good case for this as well, because you don't have to remember to implement fishing the exceptions out from the individual tasks.

  • @ermanafacan
    @ermanafacan Před rokem

    my next project is definetely using this

  • @michbushi
    @michbushi Před rokem

    THis is deadly! thank you, man!

  • @CrippleX89
    @CrippleX89 Před rokem +4

    I'm a really big fan of discriminated unions ever since I learnt about them while developing in Swift. In my humble opinion, Swift has a much cleaner solution though: enums! You can add different parameters to each individual case and extract the arguments when switching on the case. Absolutely brilliant!

    • @the_wilferine
      @the_wilferine Před rokem +1

      Completely agree. This is one of my biggest missed features from Swift

    • @CrippleX89
      @CrippleX89 Před rokem

      @@the_wilferine absolutely! I keep telling this to my colleagues and they look at my like “wtf are you complaining about” but once you’ve used it, it just makes so much sense.

  • @jakoblindblad191
    @jakoblindblad191 Před rokem

    Great stuff. Thank you

  • @oldwhitowl
    @oldwhitowl Před rokem

    Great video Nick, as always! The OneOf library doesn't seem to play very nicely with return types of IEnumerable though. Is this an issue with my design, should i wrap IEnumerables in a container model? Seems a bit of fudge to get it to work with this library. I currently use CSharpfunctionalExtensions but like the idea of being able to return more than 2 result types.

  • @killymxi
    @killymxi Před rokem

    What the video is missing is the comparison with the current best approach without external dependencies.
    A common interface for possible results can often be used.

  • @dankogulsoy
    @dankogulsoy Před rokem

    Nice work

  • @x0rld159
    @x0rld159 Před rokem

    That's a really nice feature, I'm currently working on framework 4.8 and there is no nullable so that could be a nice alternative

  • @ramanam123
    @ramanam123 Před rokem

    This also make sure developers don't miss handling any return value and this is enforced compile time which is neat
    And code becomes explicit as well because there is no magic that Dev has to discover as to where does this exception is handled

  • @CodySkidmorenh
    @CodySkidmorenh Před rokem

    To anyone watching this video. Nick's REST API course is well worth taking.

  • @Mi44Jean
    @Mi44Jean Před rokem +3

    I love the concept, it feels like F#. I would love to organize my code like that. However not sure how I feel about using that while it's not built-in.

  • @jameshancock
    @jameshancock Před rokem

    I love it. I don’t love the refactoring when it’s part of the language that’s coming but I suppose you get that no matter what.

    • @nickchapsas
      @nickchapsas  Před rokem +2

      You'll have to wait at least 2-3 years for that so don't worry too much about it

  • @_Aarius_
    @_Aarius_ Před rokem +1

    Unions in TS and value enums in Rust are such great features that theyre the only reason I don't use c# for basically everything. If c# had them natively, Itd be basically perfect

  • @kostasgkoutis8534
    @kostasgkoutis8534 Před rokem +1

    I guess I am the only one who would prefer to write DUs by hand. Just create a struct with private constructor, turn all the value constructors of the case classes to public factory methods, and you are 80% done.. Now you have your own type with no dependency on an external package.

  • @juliendebache4965
    @juliendebache4965 Před rokem +3

    You can use an empty abstract base record type instead. Works great.

    • @nickchapsas
      @nickchapsas  Před rokem

      You aren't forced to handle every implementation and you don't know where this class is inherited into

    • @juliendebache4965
      @juliendebache4965 Před rokem

      @@nickchapsas That's true. I think I prefer it because I can use the pattern matching / 'inline' deconstruction and it looks more like F#. But I understand how it's more error prone.

  • @marikselazemaj3428
    @marikselazemaj3428 Před rokem

    A much needed feature

  • @peter.dolkens
    @peter.dolkens Před rokem +2

    I feel targeted 🤣
    It's like you recorded this just for me after our discussion on null vs Empty collections the other day - I like it.
    👍

    • @peter.dolkens
      @peter.dolkens Před rokem +1

      I'd love to see result.Match replaced with a contract mapping that decides the result code based on either an underlying interface, or static mapping to some of the OneOf types.
      Also interested in the Swagger/OpenAPI support for this - definitely will have a play around with this package - thanks for sharing!

  • @MarcoVervoort
    @MarcoVervoort Před rokem

    Awesome!

  • @Arbenowskee
    @Arbenowskee Před rokem +1

    This looks like the UnionAttribute of c# Language-ext library and then you can use pattern matching with switch expression.

  • @williamsbotchway2471
    @williamsbotchway2471 Před rokem

    This makes c# more functional

  • @andreasmewald2439
    @andreasmewald2439 Před rokem +1

    Sounds a little bit like `Validation` of LanguageExt, which either returns a valid object or a list of errors.

  • @deepakdagar_change_is_in

    Use a struct Carrying statuscode and object?
    Do a switch on the statuscode in controller

  • @PhilHaselden
    @PhilHaselden Před rokem

    I wonder if you have looked at the ErrorOr package from Amichai Mantinband Nick? I wonder how it compares.

  • @nexaroth
    @nexaroth Před rokem

    Very good video, I didn't know much about discriminated unions. What I usually do is I create a generic Result object in my service layer and another generic Result object in my API layer (with a mapper). What do you think about my approach, when you campare it with Middleware validation and Discriminated unions ?

    • @nickchapsas
      @nickchapsas  Před rokem +1

      DUs and the result object in general is a cross cutting concern so you don't need multilpe result objects.

  • @mihaiga
    @mihaiga Před rokem

    I have tried to use this package some time ago but it did not handle serialization and deserialization. I ended up implementing a similar class that can do serialization. My context is Azure Durable Functions where serialization is heavily used when returning from Activity Functions. I hope that the official implementation will be able to handle serialization correctly.

  • @TheSuperAllumette
    @TheSuperAllumette Před rokem

    Ah ah, I thought the validation middleware was pretty cool, but I see your point :)

  • @antonmartyniuk
    @antonmartyniuk Před rokem

    I really like the idea of discriminated unions, the service looks more cleaner without exceptions being thrown, etc. But what I don't like is a Match method with a set of lambdas in the controller, this looks somelike ugly. I would prefer the following JS-like syntax more: var (movie, notFound, validationError) = await service.UpdateAsync(); and then you check if you have a movie - return Ok(), if not found - return NotFound(), otherwise BadRequest(). Again it's not ideal but for me it looks better than a set of lambdas. Btw, Nick have you checked the performance when using OneOf comparing to versions that can return multiple values in one class but without throwing exceptions ? How much overhead do the lambdas have?

  • @grant_vine
    @grant_vine Před rokem

    What is the overall “cost” of the switch statement and in your opinion when do things go a bit bat shit crazy with switch statements of 10+ possible results lol, it seems to “shortcut” right to the right place but is this an expensive operation or actually highly efficient switching?

  • @chaschannel8253
    @chaschannel8253 Před rokem +1

    Nice! Interested to see what OneOf looks like in Swagger?

    • @nickchapsas
      @nickchapsas  Před rokem

      I wouldn’t know since I use produces attributes. I don’t leave my swagger docs left to what the framework manages to infer automatically

  • @vladyslavhrehul2185
    @vladyslavhrehul2185 Před rokem

    @Nick Chapsas feature is really great, but what you forgot to mention is that this library don`t work with interfaces.
    I`ve also cheked ValueOf lib.
    Any solutions to do the same with interfaces?
    for the Ienumerable I can call .ToList().
    But is it possible to do it without ToList?

  • @KasperDahlTangen
    @KasperDahlTangen Před rokem +2

    What about using alias instead of creating a new class with inheritance and source generator?

  • @diadetediotedio6918
    @diadetediotedio6918 Před rokem

    I really want someday C# will copy the "inline" of Kotlin, it would be huge and absolutely amazing for things like OneOf and dunet, because it will reduce greatly the overhead of calling these lambdas instead just placing the code right there

  • @lextr3110
    @lextr3110 Před rokem

    Waiting for c# to add it to the language with syntactic sugar.. I just hope it's gonna be just the right amount of sugar and the right type. Wonder how much time it will take to add this simple concept

  • @LukeVilent
    @LukeVilent Před 11 měsíci

    Since switching to Python, I am getting unable to program whithout discriminated unions. I've been advocating that languages basically converge now, and here we have yet another example of convergence between C# and Python.

  • @mzuhry4263
    @mzuhry4263 Před rokem

    Hi Nick,
    I am interested to buy your complete course bundle.
    may I know if there is any discount code for that?

    • @nickchapsas
      @nickchapsas  Před rokem

      The bundle is already discounted by 20%. There isn’t a further discount

  • @T___Brown
    @T___Brown Před rokem

    I like this a lot but i would rather see this implemented on the client side. That way i can have a distinct expected switch for each api call. Oneof movie,notfound,validation,exception so i can do similar to a promise and have reusable handlers. Essentially a quick use switch because switch doesnt understand multiple return types.

  • @javi_malaga
    @javi_malaga Před rokem

    How this works with autorest? How your swagger documentation could work with this to return one or another?

  • @Rein______
    @Rein______ Před rokem +3

    I am torn on this one. On the one hand i like the descriptiveness of the method signatures. But on the other hand i worry about readability. This adds error checking and mapping code everywhere while with exceptions you just dont see it. Also how composable is a codebase when all methods can return unions, how do you deal with errors comming from a few layers away? I kinda like exceptions, they dont get in the way

    • @WillEhrendreich
      @WillEhrendreich Před rokem

      That is exactly the problem that DU's solve, you DON'T see the places it's likely to break. He gave a really good example of it I think, where from the outside of that function it was impossible to tell with certainty that it actually was going to behave in 3 ways, when it really looks like it can only behave in 2.
      Also he pointed out that that returned bool was not really very definitive either, it was an indication of the truth or falsehood of... Hopefully the whole thing being successful, but simply to succeed or fail is lacking context entirely.
      There is only usually one way for something to succeed, and very typically many ways to fail, and if you don't have a good idea of why you might have failed, then your program simply cannot be expected to behave in a way that isn't accounted for.
      You're completely right in that there is more visually to digest, but in a CSharp situation it's not really much of a choice for there not to be, unless you simply don't care about the programming api to be as clear as possible about it's underlying complexity.
      And that's the core of the problem, what's being described here is an issue of inherent complexity, there is no avoiding the fact that by the nature of the request being made, it has 3 behaviors at minimum. To change that it would have to be broken up somehow, and it might not be feasible to break it up in some situations.
      If you care quite a bit about readable code that still completely expresses the domain behavior, (and you care to entertain recommendations that you didn't ask for, lol) I highly recommend looking at using FSharp. The solution he's using an external library for in CSharp is part of the basic structure of the language in FSharp. It's a first class citizen there, and it makes so much of a difference to have the language built around the feature you're using. I recommend the fantastic beginner course by Matthew Crews on the FastFSharp channel. Also www.fsharpforfunandprofit.com is killer.

  • @ja_mcito
    @ja_mcito Před rokem +1

    great

  • @hodor2704
    @hodor2704 Před rokem

    it's not directly related to the topic here but I've got a question to anyone who'd be interested: if you make a web service call in a method, would you handle the exception within the method for 500 or network failure and populate a failed result which is defined in return type, or would you not handle and expect it to be handled somewhere else?

    • @mikewiebel
      @mikewiebel Před rokem

      My exception handling middleware catches the exception, logs it, and returns a 500 error.

  • @IMarvinTPA
    @IMarvinTPA Před rokem

    I think the phrase I was waiting for is "Mutually Exclusive".

  • @DummyFace123
    @DummyFace123 Před rokem

    Can you inherit oneof to support many http responses and only need to use one generic parameter? So like HttpOf?

  • @GiovanniCostagliola
    @GiovanniCostagliola Před rokem

    IMHO OneOf is excellent to represent alternative types but it is less beneficial to model a processing pipeline (with failures). In these cases a Result monad is more expressive and its composability is a major value. Something like Result should be more effective.

  • @mariocamspam72
    @mariocamspam72 Před rokem

    PolySharp kicking into 12th gear soon

  • @jamesbennett5421
    @jamesbennett5421 Před rokem

    OneOf seems useful. When you get to Match, I wonder if it will be too obscure in the future when new guy tries to maintain the code.

    • @nickchapsas
      @nickchapsas  Před rokem

      Quite the opposite. You have clear return paths with clear responsibilities. If anything it makes it easier

  • @dhollm
    @dhollm Před rokem

    For many situations I've found the simple approach of returning a tuple indicating (a) success/failure and (b) relevant object upon success ... works well. Sort of a poor man's version of this video. Since you only look at (b) if (a) is true, doesn't matter much what the default value of (b) is in case of failure.

  • @mihaiprocopi5207
    @mihaiprocopi5207 Před rokem

    Does this library supports default match case?

  • @metaltyphoon
    @metaltyphoon Před rokem

    I really hope that once the C# team add DUs, the BCL should have new namespaces alas System.Collections.Generic.Discriminated, to make use of DUs instead of exceptions.

  • @adrian_franczak
    @adrian_franczak Před rokem

    Maybe they will Add monads in future for chaining this into pipes

  • @urbanelemental3308
    @urbanelemental3308 Před rokem

    I'm weighing the pros and cons of using this versus a monad pattern. Any thoughts?

    • @user-tk2jy8xr8b
      @user-tk2jy8xr8b Před rokem

      They are orthogonal, you can have a monad over a DU

  • @jzascinski1
    @jzascinski1 Před rokem

    did anyone watch this with subtitles generated by CZcams? It starts with "hello everybody I'm naked in this video" 🤣

  • @vencislavvidov
    @vencislavvidov Před rokem

    Can we use something similar as Result match for Validation Result?

    • @nickchapsas
      @nickchapsas  Před rokem

      Sure but when you have 3 or 4 possible return types then Result is not enough

    • @vencislavvidov
      @vencislavvidov Před rokem

      @@nickchapsas I mean success or error, i do not like that if:)

  • @MMMM-vc5oi
    @MMMM-vc5oi Před rokem

    Can't you define a result class with status enumeration field for example, then return it and inside the controller map them to response?!

    • @nickchapsas
      @nickchapsas  Před rokem

      Sure you can but that doesn’t make it a good approach. Having one type per result and having the user to explicitly handle, alongside all the relevant type information is a better approach

    • @MMMM-vc5oi
      @MMMM-vc5oi Před rokem

      Yes, the generic approach is more suitable to avoid defining classes for each, thanks Nick

  • @danielmaki2792
    @danielmaki2792 Před rokem

    Does the OneOf package actually also improve the performance of the API in this case?

    • @nickchapsas
      @nickchapsas  Před rokem +1

      If you come from thowing exceptions and catching them for validation, then yes, significantly.

  • @TECHN01200
    @TECHN01200 Před 3 měsíci

    If and when c-style or discriminated unions come to C# (I don't care which), I am going to be so happy never needing to represent data with an abstract type ever again. Programming against an interface is a scourge against cache locality and memory fragmentation.

  • @ayyyyyyylmao
    @ayyyyyyylmao Před rokem +1

    Fugo

  • @jakoblindblad191
    @jakoblindblad191 Před rokem

    Is it possibe to use this with async?

  • @rbi91
    @rbi91 Před rokem +1

    Nice implemetation(use case), but i still prefer the version with nullable objects as return type, with the validation part in advance(in a separate function). Returning multiple objects violates SOLID principles (#DoOneThing)

    • @rtl6832
      @rtl6832 Před rokem

      The problem with the SRP principle is "single responsibility" doesn't have a clear cut definition. What constitutes a single responsibility is subjective to a certain extent. So to you it may appear as a violation, but to others, not so much.
      I actually don't agree with this approach either, but for different reasons.

  • @lordicemaniac
    @lordicemaniac Před rokem

    well isn't this just monad "maybe"? anyway, it is nice solution, what i'm not sure about is "Match". Is it positional? so if i later change positions of those oneof types, this will do different things than i would like and compiler may not even say something, i would probably rather use switch statement based on types returned just to be sure

    • @user-tk2jy8xr8b
      @user-tk2jy8xr8b Před rokem

      `Maybe` is a DU of the unit and the type. This OneOf is not really properly tagged, but rather indexed, so you can't have a good switch over it and you're left with the type eliminator (that's what the Switch method is essentially)