``` override fun onBackPressed drawer run if isDrawerOpen op kotlinlang #announcements

Channels

100daysofkotlin

100daysofkotlin-2021

algolialibraries

android-architecture

android-databinding

androidgithubprojects

androidx-xprocessing

arrow-contributors

beepiz-libraries

cincinnati-user-group

codemash-precompiler

codingconventions

competitive-programming

competitivecoding

compose-android

compose-desktop

compose-ui-showcase

connect-audit-events

cscenter-course-2016

domain-driven-design

effective-kotlin

effectivekotlin

embedded-kotlin

event21-community-content

fb-internal-demo

fluid-libraries

framework-elide

getting-started

github-workflows-kt

graviton-browser

image-processing

intellij-plugins

intellij-tricks

introduce-yourself

java-to-kotlin-refactoring

jhipster-kotlin

jvm-ir-backend-feedback

k2-early-adopters

kotest-contributors

kotlin-by-example

kotlin-data-storage

kotlin-foundation

kotlin-in-action

kotlin-multiplatform-contest

kotlin-pakistan

kotlinforbeginners

kotlingforbeginners

kotlinprogrammers

kotlintest-devs

kotlinultimatechallenge

kotlinx-datetime

lambdaworld_cadiz

language-evolution

language-proposals

leedskotlinusergroup

library-development

machinelearningbawas

myndocs-oauth2-server

navigation-architecture-component

ohio-kotlin-users

pattern-matching

pocketgitclient

practical-functional-programming

prozis-android-backup

python-contributors

realworldkotlin

refactoring-to-kotlin

refreshversions

russian-kotlinasfirst

skia-wasm-interop-temp

sofl-user-group

spring-security

squarelibraries

stayhungrystayfoolish

strife-discord-lib

swagger-gradle-codegen

swiss-user-group

tricity-kotlin-user-group

Title

m

Marcin Wisniowski

09/08/2018, 6:10 PM

override fun onBackPressed() {
        drawer?.run {
            if (!isDrawerOpen) {
                openDrawer()
                this
            } else null
        } ?: super.onBackPressed()
    }

n

Nikky

09/08/2018, 6:14 PM

maybe

drawer?.takeIf { !isDrawerOpen }?.apply { openDrawer() }

to avoid if else null and you can also ret rid of the run block then

m

Marcin Wisniowski

09/08/2018, 6:27 PM

Looks good, but

isDrawerOpen

is a method of the

drawer

, so it won't work inside the

takeIf

(and I can't do

drawer.isDrawerOpen

inside since it can be null).

c

chalup

09/08/2018, 6:46 PM

Doesn't takeif provide the object as a lambda parameter?

i.e. I think you should be able to write takeif { !it.isdraweropen }

And I'd use takeunless instead of takeif with negation

k

karelpeeters

09/08/2018, 6:53 PM

isDrawerOpen
doesn't seem to be a member function of
drawer

m

Marcin Wisniowski

09/08/2018, 7:10 PM

drawer?.takeUnless { it.isDrawerOpen }?.apply { openDrawer() } ?: super.onBackPressed()

works great! Thank you Jerzy, pozdrawiam. 🙂 Karel:

drawer

is the receiver in

run

which is why

isDrawerOpen

doesn't look like it uses it.

🤦‍♂️ 1

5 Views

#announcements Join Slack