cedric
12/09/2015, 3:53 PMcedric
12/09/2015, 3:53 PMjw
12/09/2015, 3:56 PMrafal
12/09/2015, 3:56 PMmadisp
12/09/2015, 5:22 PMdharrigan
12/09/2015, 5:37 PMdharrigan
12/09/2015, 5:37 PMfragmentManager.findFragmentByTag("eventsFragment")?.apply {
this.isVisible
}
rafal
12/09/2015, 5:55 PMdharrigan
12/09/2015, 5:58 PMcedric
12/09/2015, 6:02 PMisVisible
is a getter, isn’t it a no op to do that?cedric
12/09/2015, 6:02 PMapply
contains side effect stuff, e.g. settersdharrigan
12/09/2015, 6:02 PMcedric
12/09/2015, 6:03 PMdharrigan
12/09/2015, 6:03 PMcedric
12/09/2015, 6:03 PMfindWindow().apply {
width = 100
height = 100
}
cedric
12/09/2015, 6:04 PM?.isVisible
on the returned instancedharrigan
12/09/2015, 6:05 PMdharrigan
12/09/2015, 6:05 PMrafal
12/09/2015, 6:05 PMrafal
12/09/2015, 6:06 PMdharrigan
12/09/2015, 6:06 PMrafal
12/09/2015, 6:07 PMlet {}
cedric
12/09/2015, 6:07 PMif (fragment?.isVisible!!)
dharrigan
12/09/2015, 6:08 PMdharrigan
12/09/2015, 6:08 PMdharrigan
12/09/2015, 6:08 PMcedric
12/09/2015, 6:09 PMlet
is fine for 1) scoping and 2) when you need an if
without an else
rafal
12/09/2015, 6:09 PMif (fragment?.isVisible!!)
this will probably throw an NPE when fragment is nullcedric
12/09/2015, 6:09 PMcedric
12/09/2015, 6:09 PM