Top 5 Mistakes That Make Your Kotlin Code a Mess

  Рет қаралды 23,454

Philipp Lackner

Philipp Lackner

Күн бұрын

Пікірлер
@vaibhav47sharma
@vaibhav47sharma 2 жыл бұрын
Another readability issue is not using trailing commas in method signatures and their invocations if they span multiple lines. The problem with this is that if we are to add a new parameter in the future the reviewer will see 2 lines of diff instead of 1 making the review harder. Furthermore the git blame will say that we are the one who added the previous parameter. As kotlin supports trailing commas I highly recommend thinking about using them.
@makkhan3153
@makkhan3153 2 жыл бұрын
It really give you pleasure, when you have written the code in this manner.
@laOLIVAesVERDE
@laOLIVAesVERDE 2 жыл бұрын
I would create sealed interface to express authentication result. - Success - Error with message By the way, I enjoyed your CI/CD course👍 It was so great!
@AntonioGonzalezsanchez
@AntonioGonzalezsanchez 2 жыл бұрын
Holy stuff my man, this video is GOLD. Where were you when I was learning Android some 4 years ago?
@myrabohikechi2168
@myrabohikechi2168 2 жыл бұрын
Thanks so much Phil, most time I just use some stuff without really knowing the negative side of it, maybe because I see senior devs use it.
@agungwatanabe318
@agungwatanabe318 2 жыл бұрын
I always use named parameter except for short functions that are self describing. Example: Modifier.height(56.dp) instead of Modifier.height(height = 56.dp)
@pablovirus
@pablovirus Жыл бұрын
Thanks for the tips! These are the things you only get from experience or from someone mentoring you
@chase-2-2
@chase-2-2 2 жыл бұрын
Talking about named parameters: Intellij supports adding "parameter names" to java functions. It will then add comments that give you the name of the java parameter and while this does not give you any of the kotlin features, it's still very helpful
@anasfarhad2845
@anasfarhad2845 2 жыл бұрын
I am soo happy to discover that I already take care of these code readability mistakes but I got to learn some new tricks as well :)
@anasfarhad2845
@anasfarhad2845 2 жыл бұрын
@Philipp_Lackner1 Omg really? I can't believe it
@anasfarhad2845
@anasfarhad2845 2 жыл бұрын
How Can I claim my prize? Im soo excited
@tch.777
@tch.777 2 жыл бұрын
I just enjoy to hear every word of you in every single video... Thank you!! 🙏
@domonk7450
@domonk7450 2 жыл бұрын
I really like videos like this that go over general improvements
@denisoluka
@denisoluka 2 жыл бұрын
Thank you for the tip. I used inline lamda functions a couple of times. I finally realize my mistake. Thanks again
@piotr8731
@piotr8731 2 жыл бұрын
If you already took the time to extract if conditions to variables you might as well create third one "val shouldHighlightMessage = isLocalUserAdmin || areUserFriends", and call your sendMessage() function once, instead of two times. This will speed up reading process even more as developer would not need to go trough 2 "sendMessage" logics
@dennisgithuku9340
@dennisgithuku9340 2 жыл бұрын
Great content Phil.
@ekstrapolatoraproksymujacy412
@ekstrapolatoraproksymujacy412 2 жыл бұрын
Maybe you should say something about whether performance is compromised by adding unnecesary variable definitions or other instructions that do nothing? is compiler getting rid of those automatically etc?
@PhilippLackner
@PhilippLackner 2 жыл бұрын
You won't ever notice a performance difference by introducing a variable 😅 and yes the compiler optimizes a lot
@ekstrapolatoraproksymujacy412
@ekstrapolatoraproksymujacy412 2 жыл бұрын
​@@PhilippLackner Unless this function is called a billion times? ;) I don't know much about kotlin jvm etc, but I know a thing or two about "lowerish" level programming for very low power microcontrollers when you have to think 10 times if you really need this float or it can be done with just an int, I'm aware that CPU in average android device is at least million times faster than low power microcontroller and such thing in almost all cases will not be even detectable, but I think it should be said why that is, because otherwise we will end up with new generation of programmers that don't actually know what they are doing and that starts to be a plague in my biased opinion. I don't expect that high level android programmer will be fluent in ARM assembly, but it is important to know what you don't know.
@dankal444
@dankal444 2 жыл бұрын
@@ekstrapolatoraproksymujacy412 readability is most important, at the end you check performance of whole application (or your part of it) and only if you find a bottleneck you can sacrafice a bit of readability for faster code. Happens very rarely
@emmanuelmtera5936
@emmanuelmtera5936 2 жыл бұрын
I once faced this problem but as time goes i learn and now create better code.
@SriHarshaChilakapati
@SriHarshaChilakapati 2 жыл бұрын
I'd also avoid passing boolean parameters to functions as much as possible. Most of the times, it makes a lot more sense to create an enum and pass that instead.
@PhilippLackner
@PhilippLackner 2 жыл бұрын
Why would you pass an enum if you have something that has a clear true and false value 🥴
@tashilapathum3709
@tashilapathum3709 2 жыл бұрын
@@PhilippLackner I think it's better if we had to extend the functionality later. For example, to add another user type BOT, other than FRIEND or ADMIN
@mv2e19
@mv2e19 2 жыл бұрын
Kotlin supports named parameters, so boolean parameters aren't that bad. For example, "Hello, World!".contains("hello", ignoreCase = true) is quite readable.
@PhilippLackner
@PhilippLackner 2 жыл бұрын
@@mv2e19 Enums make sense if you have a boolean where the negated value is unclear. For example isLeft. It's not clear what it means if isLeft is false, could be isRight, isTop, isBottom. But if the negated value is clear like for isLoading, isDropDownShowing etc. please use booleans
@mv2e19
@mv2e19 2 жыл бұрын
@@PhilippLackner oh absolutely, I agree. I just wanted to add that booleans have their place. Not everything should be an enum
@OlegGolubev_yolo
@OlegGolubev_yolo 2 жыл бұрын
i wish more people will follow this practice, sometimes joining a new projects is a very difficult process :|
@ug333
@ug333 2 жыл бұрын
Love the video, and totally agree with your notes. Thanks! One small difference for me: I would pull the logic for local user admin and is friend to functions. I tend towards boundary testing, and that's much easier when those are broken into functions. There are other reasons as well, but I would imagine you would have already made it a function in those cases Again, thanks!
@danieldawson8018
@danieldawson8018 2 жыл бұрын
What formatter do you use? Seems a lot better than my setup.
@pinoy_marites_beh
@pinoy_marites_beh 2 жыл бұрын
A good analogy for the null argument are use-cases like a calculator or a some monetary use-case, numbers should start with null instead of 0 or 0.0, but for a counter/counting use-case 0 should be a default value
@annunzarizzle
@annunzarizzle 2 жыл бұрын
why is his x-code dark theme so chill, when i download the one on android studio the colors are very bright
@aquilesorei
@aquilesorei 2 жыл бұрын
Thanks for your advices 🙏🏾
@bombinaround
@bombinaround 2 жыл бұрын
If highlighted messages are a first class feature, would it not make more sense to have two send methods in the interface - send normal message and send highlighted message. Then you can lose the Boolean altogether
@aptemkov
@aptemkov 2 жыл бұрын
Hey, Phillip, can you say, what is your Theme in Android studio and what is your font?
@frank.koenig
@frank.koenig 2 жыл бұрын
Hello Philipp. I am curious. Could you show some big projects you were involved in?
@adambickford8720
@adambickford8720 2 жыл бұрын
I almost *never* use local variables like this; i'd much rather extract them into functions. You still end up with a nice 'named/semantic ' chunk of code but it's: 1. more readable/abstract; don't bother me with the implementation of deriving friends or admin, i only care about the result 2. it clearly limits scope of that result, reducing cognitive load 3. its reusable/extensible/composable/etc
@sabuta100
@sabuta100 2 жыл бұрын
In a more general suggestion (despite being Kotlin or not), I would add; try to avoid very large files, not doing so makes the code reading hard/confusing/boring 😅, probably keeping the number of lines between 1k and 1.5k is still a good option (do so ONLY when needed, I mean, if your file is shorter than that, it's totally fine and great). Also large files might kill your Android Studio 🥲.
@TheLatvianboy
@TheLatvianboy 2 жыл бұрын
1k is already 3x what you would want from a file.
@TheTuxtrons
@TheTuxtrons 2 жыл бұрын
With 500 my AS is already agonizing lol
@JimPekarek
@JimPekarek 2 жыл бұрын
On the other extreme, don't go full enterprise Java and spread logic across 75 different files with 10 layers of inheritance so you have to go on a treasure hunt to figure out what every line of code actually does. There's usually a happy medium.
@shahzamanrai2617
@shahzamanrai2617 2 жыл бұрын
Which Font are you using?
@emmanuelpregnolato5026
@emmanuelpregnolato5026 2 жыл бұрын
Thank you Phil, I did enjoyed this capsule.
@gori_maheswari8994
@gori_maheswari8994 2 жыл бұрын
Thank you so much❤
@FreedivingTrainer
@FreedivingTrainer 2 жыл бұрын
Hi! Thanks for your videos! Why in your example projects you don't use preview of the whole screen during developing?
@victorlapin2080
@victorlapin2080 2 жыл бұрын
Philipp, what approach would you recommend for localized strings in error messages?
@hossamqandel5638
@hossamqandel5638 2 жыл бұрын
Could you tell me how can we mix between UiEvent and UiText sealed classes to show a Snackbar with translated message? @Philipp Lackner
@rhen4610
@rhen4610 2 жыл бұрын
this is pretty helpful on technical interview
@robchr
@robchr 2 жыл бұрын
Variable names are important but I don't think these examples are more readable with the added variables. I still will need to parse logic to see what logic is actually occurring.
@mustafaammar551
@mustafaammar551 2 жыл бұрын
very cool video thank you bro wish you all the best👍👍👍👍
@Torte-Roblox
@Torte-Roblox 2 жыл бұрын
Good video! I just want to add, when using intellij IDE you can use the .val - shortcut to create a new val of an expression. Simply put ".val" behind the expression et voila.
@ПавелЗубко-ц8ч
@ПавелЗубко-ц8ч 2 жыл бұрын
Thanks man!!!! Cool!!!
@DaleKingProfile
@DaleKingProfile 2 жыл бұрын
When you extracted the expression to a variable you should have done that using the built in receptions instead of doing it by hand so you cannot make a mistake.
@vladdiachuk562
@vladdiachuk562 Жыл бұрын
We need same video for Compose!
@pavelschannel-alittleoutof3532
@pavelschannel-alittleoutof3532 2 жыл бұрын
This is just standard coding practice. Can be for any language....
@scottbiggs8894
@scottbiggs8894 7 ай бұрын
"Don't be afraid to introduce more variables to make code more readable." I wish I could tattoo that on the heads of some of my old co-workers. I completely disagree with removing the comment. Keep it. If there's a bug, it's nice to know what the programmer was TRYING to do. This way we can tell if the algorithm is faulty or the implementation of it. Comments explain what you're trying to do; the code shows how it is accomplished. I've wasted hundreds of hours of my life debugging people's code; knowing the difference between the programmers' intent vs how it was implemented is invaluable.
@Daaaaaaavid
@Daaaaaaavid 2 жыл бұрын
Hey, I wanted to ask what monitor do you use for Android Studio? I have a 27" 4k only, and it's not enough to work with 4 columns in AS, so I'm wondering which one to buy to use as a central monitor or as second one to the current one.
@pratikdhage3491
@pratikdhage3491 2 жыл бұрын
Thanx Philip bro 🤟
@argahutama
@argahutama 2 жыл бұрын
I spotted 1 more mistake: You don't rearrange your code (CMD + Option + L)
@syedovaiss
@syedovaiss 2 жыл бұрын
Woah! Didn't doing these mistakes ♥️
@yasserakbbach7342
@yasserakbbach7342 2 жыл бұрын
That would be a useful Playlist 👌
@HARSHSHARMA-ic6qo
@HARSHSHARMA-ic6qo 2 жыл бұрын
Great video. Waiting for the result off the giveaway. I hope will be replacing my 7 year old laptop.
@technicholy1299
@technicholy1299 2 жыл бұрын
Disagree on the empty string for error message. As long as it is consistent across the app, then the .isEmpty() function does all you need, safely.
@progmf
@progmf Жыл бұрын
The „areUsersFriends“ is not a good refactoring, you should rather move the list of „localUsers“ from the first part of the expression to a variable and keep the rest wirhin the condition. Not a fan of moving conditions to variables just to name them either, it bloats the code; just like the named parameters do. „checkIfAuthenticated“ is also bad naming, „checkAuthentication“ is suitable.
@theonline5703
@theonline5703 2 жыл бұрын
Not gonna a lie i didn't know you but i just started watching your videos after you posted the giveaway! ✊
@PhilippLackner
@PhilippLackner 2 жыл бұрын
Thanks and welcome to the club!
@tashi7160
@tashi7160 2 жыл бұрын
read uncle bob clean code, it will change/improve your coding style drastically.
@peterwestlin8052
@peterwestlin8052 2 жыл бұрын
Named parameters.
@sacarymoviesamu
@sacarymoviesamu 2 жыл бұрын
i like this, as i am a noob developper, lol
@KotlinBek
@KotlinBek 2 жыл бұрын
thanks
@programaths
@programaths 2 жыл бұрын
for the null, better add a boolean alongside or an enum when relevant. The rational behind that is that one variable should not encode two things. In the case of errorMessage being null, it encodes two things: whether there was an error and if so, the error message. With a boolean, you can do: if(auth.isError){ display(auth.errorMessage) } instead of: if(auth.errorMessage!=null){ display(auth.errorMessage) } Also, "==true" or "==false" should not be used. As for comments explaining "what", they also indicate that some code pertain to a function. I used that when doing QC ^^ And with Kotlin, you don't lose much with inline functions. So, it's worth to even put a one liner in a function.
@PhilippLackner
@PhilippLackner 2 жыл бұрын
With that argument nullables shouldn't be used at all, since they ALWAYS encode the existence of a value and the value if it exists. Can't agree with that 😅
@programaths
@programaths 2 жыл бұрын
@@PhilippLackner Yes, null is the one million mistake!!!
@thangaduraiselvaraj9318
@thangaduraiselvaraj9318 2 жыл бұрын
First Cmt🥳
@JagdishOnYT
@JagdishOnYT 2 жыл бұрын
Not actually 🤣
@hossamqandel5638
@hossamqandel5638 2 жыл бұрын
by the way Philipp .. Could you build a new project for example chat app?❤
@PhilippLackner
@PhilippLackner 2 жыл бұрын
Already have that on my channel
@swedishpsychopath8795
@swedishpsychopath8795 2 ай бұрын
kotlin is messy as hell. Better use java and OOP
@PhilippLackner
@PhilippLackner 2 ай бұрын
😂😂😂
@mkc0321
@mkc0321 2 жыл бұрын
amazing
@JagdishOnYT
@JagdishOnYT 2 жыл бұрын
I am waiting for Giveaway announcement. 😂
@pradyumnx
@pradyumnx 2 жыл бұрын
❤️❤️❤️❤️
@UmerFarooq-vk9be
@UmerFarooq-vk9be 2 жыл бұрын
No matter what you do... Kotlin syntax is a total mess by itself 🤦
@PhilippLackner
@PhilippLackner 2 жыл бұрын
?
@peterbrown4516
@peterbrown4516 2 жыл бұрын
Excellent
@cristianowinter4382
@cristianowinter4382 2 жыл бұрын
Hey Phil, congrats on all the work you have done. Your job is fantastic, and you have been helping many developers with your content!! Maybe, another improvement, what about moving it.isLocalUser && it.isAdmin to a method inside the User class as one of its members, something like users.values.any {it.isLocalUserAdming()}? I noticed that people are no longer adding behaviours or using the state of an Object.
new technologies that will change the world
8:56
Techno Point
Рет қаралды 4
1% vs 100% #beatbox #tiktok
01:10
BeatboxJCOP
Рет қаралды 67 МЛН
人是不能做到吗?#火影忍者 #家人  #佐助
00:20
火影忍者一家
Рет қаралды 20 МЛН
Earn Six Figures Learning a High-Income Skill in 90 Days
7:02
90 Days Better
Рет қаралды 613
5 Fatal Coroutine Mistakes Nobody Tells You About
18:15
Philipp Lackner
Рет қаралды 89 М.
Make Your Code Clean With the SOLID Principles
18:24
Philipp Lackner
Рет қаралды 102 М.
I've Used Jetpack Compose For 4 Years - Here's the Good and the Bad
16:12
Advanced Kotlin: Mastering Delegation in Kotlin
18:54
kt whisperer
Рет қаралды 9 М.
Let, Also, Apply, Run, With - Kotlin Scope Functions
11:44
Philipp Lackner
Рет қаралды 99 М.
THIS Is How You Measure the Performance of Your Android App
21:30
Philipp Lackner
Рет қаралды 45 М.
Should You Use Compose State or StateFlow in Your ViewModels?
13:59
Philipp Lackner
Рет қаралды 84 М.