This Is My FAVORITE Error Handling Class

  Переглядів 22,455

Philipp Lackner

Philipp Lackner

Місяць тому

In this video I'll show you my FAVORITE Error Handling Class!
💻 Let me be your mentor and become an industry-ready Android developer in 10 weeks:
pl-coding.com/drop-table-ment...
⭐ Courses with real-life practices
⭐ Save countless hours of time
⭐ 100% money back guarantee for 30 days
⭐ Become a professional Android developer now:
pl-coding.com/premium-courses/
Get my FREE PDF about 20 things you should never do in Jetpack Compose:
pl-coding.com/jetpack-compose...
Regular programming advice on my Instagram page: / _philipplackner_
Join my Discord server:
/ discord
Get the source code for this video on GitHub:
github.com/philipplackner/Cle...

КОМЕНТАРІ: 101
@thebigboi5357
@thebigboi5357 Місяць тому
I agree that using a custom Result with a sealed hierarchy of errors is the way to go. However, I think you should have included the extension functions that really make this class useful. Things like ifSuccess, ifError, getOr, getOrNull, recover, map, and, andAlso, orElse, etc. Michael Bull’s Result class, Arrow’s Either, and Rust’s Result have many more examples for functions that make “railway oriented programming” possible with function chaining.
@DiegoNovati1
@DiegoNovati1 Місяць тому
To avoid all ypur suggested implementations, I use 'arrow-kt' and each use case returns a Either nomad: left contains the failure, right contains the success; the caller uses the 'fold' method to check the left and the right cases. As further details, the 'when' on the left case has to implement an always present case: UnexpectedError. The UnexpectedError is created when the use case detects an unmanaged exception that is automatically converted to the UnexpectedError to avoid the crash of the App without the need of putting every use case inside a try/catch.
@mirza5373
@mirza5373 Місяць тому
I think, this goes into infinite creativity. He only explains the bare bone of how the Result class works. You can just add your own functionality that fits your needs
@user-ge3yl6nu5o
@user-ge3yl6nu5o Місяць тому
can you point to some articles or courses explaining these please?
@thebigboi5357
@thebigboi5357 Місяць тому
@@user-ge3yl6nu5o checkout Rust’s or Michael Bull’s implementation
@pelealexandru
@pelealexandru Місяць тому
Thank you thank you thank you! I saw a very similar video yesterday from Stevdza-San and I pointed out the same concerns you had about localisation and separation of concerns. I was about to write my first Medium article on this but then your video came out with more or less the same solution. Awesome video Philip!
@skarloti
@skarloti Місяць тому
Absolutely! The advantage of using a sealed interface instead of strings is huge.👍
@solokiller5883
@solokiller5883 Місяць тому
I had good luck two days ago and I was looking for a solution to this problem and a way to deal with different types of errors, but the matter was terrible, and it happened that today you are posting a video on this topic and delving into it in depth. Thank you.
@CryptoCodeZone
@CryptoCodeZone Місяць тому
Very nice Improvement, Oh boy, much to learn, much to re-factor!
@ArthurKhazbs
@ArthurKhazbs Місяць тому
The idiomatic Kotlin implementation of the classic Result pattern - a wonderful thing to know! Oh, and also - awesome usage of "when subject capture" at 12:10 - I didn't know we could do "when val"!!
@hoeric5605
@hoeric5605 Місяць тому
Thanks for philipp, i've learn a useful and efficient way to write error class, and do error handling. It would be easier to write out logic codes ✌😁
@JPilsonSumbo
@JPilsonSumbo Місяць тому
Furthermore you could map the status code of network request to your enums so that you avoid doing too many "when" . ... Great video btw
@lukesmoljak8617
@lukesmoljak8617 Місяць тому
If you have a use case class that throws its own domain level errors that also communicates with a repository, how would you encapsulate both DataError as well as the usecase's errors? I can see these types of scenarios happening where the usecase returns errors of type Error supertype only. It would be preferable to know the exact subset of errors it could be without looking at the usecase code. Any ideas?
@baranidharanmurali469
@baranidharanmurali469 Місяць тому
I do this by writing custom Either sealed class left being error and right being the data we don't need the entire arrow library just few of those extensions like fold makes it lot easier. As always good tutorial 👏
@nesletchimaew9209
@nesletchimaew9209 Місяць тому
sometimes you don't need data for a success result. sometimes you need data for an error. Also either is totally unreadable
@baranidharanmurali469
@baranidharanmurali469 Місяць тому
​@@nesletchimaew9209 in those cases u can write other extension functions to just get left or right from either and I use it mostly in network calls which will have both the data and error most of the time and as I mentioned extension functions like fold just has 2 lambdas first being the actual data and second the error I mean I think it is much easier than writing if else or when expression to determine if it is data or error all the time. If u haven't tried the fold or other extensions i request u to try.
@ishchuk_
@ishchuk_ Місяць тому
@@nesletchimaew9209 ok
@JPilsonSumbo
@JPilsonSumbo Місяць тому
You definitely need a better naming 😢
@abv99997
@abv99997 Місяць тому
Great video, thank you a lot!
@mohannadyoussef8683
@mohannadyoussef8683 Місяць тому
Nice explanation, Thank you
@roushanali1918
@roushanali1918 Місяць тому
very logical, great explanation
@AndrewKupreychik
@AndrewKupreychik Місяць тому
Good content I'm looking for!
@leunamvon2565
@leunamvon2565 23 дні тому
Philipp has so good and solid approaches that I implement them also in iOS.
@gekylafas
@gekylafas Місяць тому
Mapping 4xx or 5xx errors from retrofit's (or ktor's) exceptions to your own enums may seem redundant to some, but this is the only way to isolate your presentation layer from the actual networking library used by the data layer. Which in turn makes it easy to switch networking libraries if the need arises, e.g. you start using Kotlin Multiplatform (which retrofit does not support). Great video, as usual!
@kivan26
@kivan26 Місяць тому
You can always extract or abstract away the process of converting http error codes,or onError(Exception) callbacks into your own Error structure, so you don't repeat yourself.
@MrVipulLal
@MrVipulLal Місяць тому
Great video, as usual.
@Ayor88
@Ayor88 Місяць тому
what about the "class Loading(data: T? = null): Ressource(data)" you had in your Ressource class before ? Would you still have a similar mechanism in the Result interface or do you manage it in an another way ?
@wojtekkrystyniak
@wojtekkrystyniak Місяць тому
it's better to use Arrow's Either which is pretty much the same thing but additionally provides plenty of utility functions
@prashantwosti
@prashantwosti Місяць тому
If you could achieve it in a simple way, why use third-party libs? I'm not saying using Arrow lib is bad but it could be unnecessary.
@Mike-er2ih
@Mike-er2ih Місяць тому
​@@prashantwostiThis. I don't like throwing huge libraries into project as a 1st solution
@ericksli
@ericksli Місяць тому
@@prashantwosti You can archieve it by writing custom code, but you have to write unit test cases and extension functions by yourself Also, for a project that have different collaborators, it would be better to stick with some de facto libraries to save the onboarding time
@johngray2875
@johngray2875 Місяць тому
Yup, I just started using arrow-kt about a month ago, and I love it.
@mustafaammar551
@mustafaammar551 Місяць тому
thank you bro
@ubersticks
@ubersticks Місяць тому
How do you recommend dealing with "Loading" state? It seems closely related. Solutions often include all-in-one (Success|Error|Loading) or creating a second class that is used in the Presentation layer that includes Loading.
@danielayodeji3641
@danielayodeji3641 Місяць тому
That should be handle in your presentation layer
@HuteriManza
@HuteriManza 2 дні тому
I would say this is simpified version, in practices, there are more than this, and usually we will have common error handling to handle cases such as unauthorised token, internal server error, generic error message, and we would not want to handle this in every api call and this things require strategy to reduce boilerplate.
@shifthackz
@shifthackz Місяць тому
Interesting approach which looks clean, but is there actually a need to introduce another domain layer for errors? You can just use regular kotlin Result class (which also can return non-nullable data), and throw a different instances of custom Throwable classes for different error cases (or even introduce a custom base sealed interface for domain layer throwables), then just catch throwable in a ViewModel and map a different throwables to the resource strings and show different messages? And in my opinion when some domain logic functions throw exceptions makes code more readable and does not need some custom mapping when using it with mapping/flat-mapping with coroutines or rx.
@PhilippLackner
@PhilippLackner Місяць тому
I don't like having to "remember" that I have to catch exceptions and then needing to go to the data layer to see which ones could be thrown. I guess it's a matter of taste in the end, but a wrapper class tells you which error types it could return and forces you to handle them
@fardalakter4395
@fardalakter4395 Місяць тому
I use this approach in vm and use generic exception handler and map throwable for possible errors from both remote and local to a string res. It just a matter of single error handler vs modular error handler. The trade off for phillip's approach is that it generating a lot of files and code and complex for beginner-intermediate level devs in trade of clean separated code
@mobAppAR
@mobAppAR Місяць тому
Why don't we inject context using hilt?
@McRookworst
@McRookworst Місяць тому
Isn't this just what exceptions are for? Executing use cases in a runCatching block is more concise and standard than all this boilerplate. I understand that some might dislike working with exceptions, but you should at least suggest the possibility.
@asuras2891
@asuras2891 Місяць тому
While I like the approach in general, I couldnt figure out yet how to use this on server side. Like when I have a ktor backend and I use such an error handling, I want to send the result back to the client via rest or websockets so my client can show the result or the error. Since we have a sealed data structure with generics trying to serialize them with kotlinx-serialization-json turned out to be a nightmare. Type erasure kicks in hard here. You can package a lego star wars set in the backend and unwrap a lego harry potter set in the frontend - not sure if lightsabers stand a chance vs magicians!
@Daaaaaaavid
@Daaaaaaavid Місяць тому
I prefer Google's AndroidNow approach using UIStates.
@abdussamad833
@abdussamad833 Місяць тому
Great way to handling errors, But how can we use it for multimodule architectures? could you please help to me?
@petibah27
@petibah27 Місяць тому
Hi, Philipp can you make a projet where you use koin and ktor please
@lhk9805
@lhk9805 Місяць тому
Awesome😊
@isolino.ferreira
@isolino.ferreira Місяць тому
Great video! How about mapping data error into custom domain exceptions? I used to inject context into repositoryImpl, them throwing custom exceptions with the right string. If the erro message comes from backend, I just throw the custom exception with that message.
@PhilippLackner
@PhilippLackner Місяць тому
Below some other comments I explained why I'm not too big of a fan for using exceptions for ALL errors
@robchr
@robchr Місяць тому
I see wrapped errors a lot and it makes processing the successful path require a flatMap. I prefer to throw errors and have the root coroutine catch and handle the error. This leads to much cleaner data, domain layers since they do not need to handle errors and only the coroutine in the ViewModel is concerned with errors.
@PhilippLackner
@PhilippLackner Місяць тому
I don't like this too much since you then always need to look back at the data layer to know which types of errors/exception could be thrown. There's also nothing that forces you to catch these, so it's easy to forget
@thebigboi5357
@thebigboi5357 Місяць тому
I think one of the biggest reasons not to use exceptions for domain-specific stuff like this is exhaustiveness. If you use typed-errors, you can then create an error hierarchy with a sealed class, and you can guarantee at compile-time that every possible domain error is handled
@5erTurbo
@5erTurbo Місяць тому
I agree with @robchr. I personally prefer exceptions, because you know they will cancel code execution, if something goes wrong. And exceptions also provide a useful stacktrace - result classes (mondas) don't. Using a monad is not a silver bullet. You can call a function and forget to actually use Result return value. Monads force you to deal with wrapper object on every step of the way - you can't propagate errors between function, like you can exceptions.
@RonnyBubke
@RonnyBubke Місяць тому
What you really need is only the data and catching exceptions on the presentation layer. For general errors just use an error handler which translates the error into an action or a message. That's it.
@PhilippLackner
@PhilippLackner Місяць тому
Don't like having to remember to catch all types of exceptions and always needing to look which ones could be thrown. A wrapper class tells you all different error types that could be relevant in a specific use case and you can decide which ones to handle
@RonnyBubke
@RonnyBubke Місяць тому
@@PhilippLackner You don't need to remember all exceptions when you have Tests.
@RonnyBubke
@RonnyBubke Місяць тому
@@PhilippLackner You can simple catch all exceptions and then have an global error handler wich can be extend for custom handlings. The advantage is you can handle the value directly and don't need to look if it wasn't a success or not, the success is the default and errorhandling is the exception.
@ArthurKhazbs
@ArthurKhazbs Місяць тому
Thrown exceptions are more difficult to click and read through than returned failure objects, especially when done across architecture layers
@UsmonWasTaken
@UsmonWasTaken Місяць тому
Just use the Arrow library, instead of reinventing the wheel. You can even just copy the Either class from their git repo if you don't want the whole library :)
@gorudonu
@gorudonu Місяць тому
This
@javiere.gonzalez1021
@javiere.gonzalez1021 11 днів тому
I would but Arrow's Left/Right nomenclature sucks and gives off code stink
@technophile_
@technophile_ Місяць тому
I've not watched the full video yet, but it looks like this approach is very similar to how errors are handled in swift
@coldwised
@coldwised Місяць тому
But UiText is not Serializable, how do we save state?
@jimpauloovejera2599
@jimpauloovejera2599 Місяць тому
instead of passing message: String from your old sample you could just pass in enum class there :D and its gonna be the same but less boilerplate
@makix08
@makix08 Місяць тому
Yes I like it !
@malikkissoum730
@malikkissoum730 Місяць тому
How to deal with the case I want to show different pieces of UI depending on what error occurred, not just show a message, I want to show a floating smiley face when there is a network error and a blinking button when there is a password error
@TheMikkelet
@TheMikkelet Місяць тому
This seems overly complicated when you can just throw custom errors (NotFoundException, TimeoutException) and also use the *built in* coroutine function runCatching
@harishpadmanabh6857
@harishpadmanabh6857 Місяць тому
How to handle the scenario when we need to show the message in response as error message and not the string from resources?
@nsshurtz
@nsshurtz Місяць тому
I'd recommend that the Succes result not take an E type (it doesn't care about the error) and have it use Nothing as it's error type for implementing the interface. Same thing goes for the D type on the Error implementation.
@PhilippLackner
@PhilippLackner Місяць тому
That makes it hard to write utility functions for the Result class, since they need to extend it with both generic params
@kacetal
@kacetal Місяць тому
​@@PhilippLacknerTo make writing easier, you can use Kotlin contracts.
@StreetsOfBoston
@StreetsOfBoston Місяць тому
​@@PhilippLackner That should not be an issue, since both type parameters of Result are covariant and "Nothing" is a sub-class of all types. I.e. If you'd put Nothing for E type in Success and Nothing for D type in Error, then `fun doSomeUtilThing(r: Result...)` can be called with with a Success (Result) or an Error (Result).
@davidgaier7324
@davidgaier7324 Місяць тому
Instead of naming the data class error of result interface, we could name it Failure like scala did and don't have collision with error sealed interface
@MD90X
@MD90X Місяць тому
Is there a book or a tutorial you would recommend for learning kotlin multiplatform ?
@Elian-Fabian
@Elian-Fabian Місяць тому
He has a KMM premium course.
@josephofem5448
@josephofem5448 Місяць тому
How can you handle a network request that returns its own custom error message. Say in your example, the provided password is wrong and the error message communicates that.
@pelealexandru
@pelealexandru Місяць тому
As an alternative to UiText, is it a bad practice to inject the application context in the ViewModel and then use that to get string resources?
@germenwong
@germenwong Місяць тому
not good
@sevbanthebuyer
@sevbanthebuyer Місяць тому
yes definitely. because context objects are expensive and your viewmodels lives longer than your screens. you will end up having large memory usage
@pelealexandru
@pelealexandru Місяць тому
hi @@sevbanthebuyer, thanks for your reply. to my knowledge, the application context is a singleton which is already there in the memory anyway, so just injecting a reference to it in the ViewModel wouldn't have any impact on performance what so ever. please correct me if i'm wrong though
@pelealexandru
@pelealexandru Місяць тому
hey@@germenwong! can you elaborate?
@user-lk1wm1ul6w
@user-lk1wm1ul6w Місяць тому
Help me understand this. I see we’re mapping the server error code to defined error type without considering the actual error message returned from the server. What if we want to show the actual error message returned from the server?
@PhilippLackner
@PhilippLackner Місяць тому
Then you need to extend this with a separate string field, but that wouldn't support localization
@coldwised
@coldwised Місяць тому
OOP is killing me 💀
@aabhishek4911
@aabhishek4911 Місяць тому
What will happen when sealed interface Error and PasswordError are in different modules ?
@Elian-Fabian
@Elian-Fabian Місяць тому
I think that's probably a mistake, I don't think it should really be sealed. Also, I don't see what's the purpose of defining such interface, it should work the same if you get rid of it. Finally adding the generic type of data to the error type and the error type to the data does not seem to be necessary for me.
@mihies
@mihies Місяць тому
Any particular reason for using data class for UiText.DynamicString and class for UiText.StringResource?
@ArthurKhazbs
@ArthurKhazbs Місяць тому
My guess is for making them automatically equatable by value, just in case
@mihies
@mihies Місяць тому
​@@ArthurKhazbs Yes, but why only one of the two?
@dleonardo3238
@dleonardo3238 Місяць тому
You and stevdza competing lmao? Same day same video lol
@fardalakter4395
@fardalakter4395 Місяць тому
Lol, happy for that, love both of them
@ubersticks
@ubersticks Місяць тому
at 07:19 it is still confusing to me why the "typealias RootError = Error" is helpful. Can you explain this a bit more?
@abdelrahmankhaled7575
@abdelrahmankhaled7575 Місяць тому
because he define a new class inside the result sealed class which has name of "Error" , so to not conflict with the other "Error" sealed interface class , he used the typealias to make the sealed interface class take a name of "RootError" instead of "Error" only inside this file
@csabasemler6806
@csabasemler6806 Місяць тому
Thanks for the video Philipp! I have a question that has been bothering me for a long time. Why are you talking so fast? In this way, it is difficult to understand the essence of your video. I have come across this many times. (Or am I just slow? :-O)
@sahilsapehia4789
@sahilsapehia4789 Місяць тому
1st one to comment
@saileshlimbu2373
@saileshlimbu2373 Місяць тому
no you are not 😂
MVVM vs. MVI - Understand the Difference Once and for All
18:40
Philipp Lackner
Переглядів 23 тис.
Этого От Него Никто Не Ожидал 😂
00:19
Глеб Рандалайнен
Переглядів 10 млн
Functional Error Handling In Kotlin: Either and Result
55:11
Rock the JVM
Переглядів 4,1 тис.
Best OS for programming? Mac vs Windows vs Linux debate settled
8:41
3 Clean Code Hacks I Bet You Didn't Know (Kotlin & Android)
12:30
Philipp Lackner
Переглядів 24 тис.
The Do's and Don'ts of Error Handling • Joe Armstrong • GOTO 2018
45:31
GOTO Conferences
Переглядів 67 тис.
4 levels of Rust error handling
46:35
timClicks
Переглядів 8 тис.
Sealed Classes for UI State are an ANTI-PATTERN - Here's why!
9:53
Philipp Lackner
Переглядів 16 тис.
I've Tested Android Studio's New Code Companion Gemini
12:29
Philipp Lackner
Переглядів 13 тис.
Get Rid of Exceptions in Your Code With the Result Pattern
13:06
Milan Jovanović
Переглядів 38 тис.
Clean Architecture IS about Vertical Slicing, actually!
15:24
About Clean Code
Переглядів 29 тис.
Apple, как вас уделал Тюменский бренд CaseGuru? Конец удивил #caseguru #кейсгуру #наушники
0:54
CaseGuru / Наушники / Пылесосы / Смарт-часы /
Переглядів 3,2 млн
The ARM chip race is getting wild… Apple M4 unveiled
4:07
Fireship
Переглядів 679 тис.
Apple Event - May 7
38:22
Apple
Переглядів 6 млн
Вы поможете украсть ваш iPhone
0:56
Romancev768
Переглядів 215 тис.