Kotlin Multiplatform Broadcast

Kotlin Multiplatform Broadcast

Что бы вы изменили в это коде и почему? fun PersonDto?.toPerson(): Person? = if (this == null) { null } else { Person( name = name, surname = surname, age = age ) }

Не делайте single-expression функции, если она занимает у вас больше 1 строки

Пример кода из поста выше читается очень сложно из-за многострочного if/else, который используется как выражения для функции чтобы не писать тело. Правило просто - в функции больше одной строки, то точно объявляете тело {}

Исключения бывают, о них ниже

Код и поста станет выглядеть так

fun PersonDto?.toPerson(): Person? {
     if (this == null) {
         return null
     } else {
         return Person(
              name = name,
              surname = surname,
              age = age
         )
     }
}

www.tgoop.com/kotlin_broadcast/1642

4.9K viewsedited  Jan 29 at 15:13

Не делайте single-expression функции, если она занимает у вас больше 1 строки

Пример кода из поста выше читается очень сложно из-за многострочного if/else, который используется как выражения для функции чтобы не писать тело. Правило просто - в функции больше одной строки, то точно объявляете тело {}

Исключения бывают, о них ниже

Код и поста станет выглядеть так

fun PersonDto?.toPerson(): Person? {
     if (this == null) {
         return null
     } else {
         return Person(
              name = name,
              surname = surname,
              age = age
         )
     }
}

BY Kotlin Multiplatform Broadcast