JetPack Compose - Best practices for method length

user_4623473 0
Viewed 318

I am trying to integrate detekt with JetPack Compose project and as a rule

 LongMethod:
    active: true
    threshold: 60

and as compose recommend passing each parameter in separate line the method length exceeds the 100 lines of code, So I am wondering what is the best threshold to use for composable functions or if there is a way to exclude composables from this rule?

there are some detekt customization for compose here but nothing about this rule

detekt android-jetpack-compose kotlin android
1 Answers

I would suggest to encapsulate your params in a data class. This is something similar what's is done is some Material Compose components like: Scaffold (which uses ScaffoldState, or LazyColumn/LazyListState, etc.

This recommendation is described here.

Instead of using:

@Composable
fun VerticalScroller(
    scrollPosition: Int,
    scrollRange: Int,
    onScrollPositionChange: (Int) -> Unit,
    onScrollRangeChange: (Int) -> Unit
) {

you should use:

@Stable
interface VerticalScrollerState {
    var scrollPosition: Int
    var scrollRange: Int
}

@Composable
fun VerticalScroller(
    verticalScrollerState: VerticalScrollerState
) {
user_10943330
Related