Which one is more kotliner: A “Builder” object fun...
# composes
Siyamed
10/15/2019, 6:42 AMWhich one is more kotliner: A “Builder” object functions should (1️⃣return this ) or (2️⃣let me use with/apply etc and don’t return this)
2️⃣ 16
1️⃣ 2
e
efemoney
10/15/2019, 9:24 AMCorrect me if i’m wrong but you mean:
1️⃣
Copy code
Builder()
.function()
.anotherFunction()
.build() Copy code
Builder().apply {
function()
anotherFunction()
}l
louiscad
10/15/2019, 10:03 AMNo builder and default argument values is more Kotliny
➕ 7
👆 1
👍 1
☝️ 4
m
Matej Drobnič
10/15/2019, 11:01 AMEven if you return this, you can stull use with/apply etc., so I do not see why you would not return this if you are using builder. There is no downside.
But otherwise I agree with @louiscad
➕ 1
s
Siyamed
10/15/2019, 2:57 PMThanks. Efeturi that's correct.
Siyamed
10/15/2019, 2:59 PMLouis, Regarding the default argument, this is more like StringBuilder, it actually is AnnotatedStringBuilder in order to assist construction of a styled string
Siyamed
10/15/2019, 3:00 PMStringBuilder seems to be an alias to java one, so I wasn't sure what it would look like if was implemented in kotlin
Siyamed
10/15/2019, 3:01 PMThank you all for the responses.
l
louiscad
10/15/2019, 4:27 PMDon't you already have
buildSpannedString { … }SpannedStringBuilders
Siyamed
10/15/2019, 5:29 PMAnnotatedString.Builder will be a Compose component and will be the equivalent of/similar SpannableStringBuilder. Planning to add buildAnnotatedString {...}.
l
louiscad
10/15/2019, 8:07 PMOh yeah, that's why this thread is in the #compose channel. Will it be Android only, or multiplatform ready?
s
Siyamed
10/15/2019, 9:18 PMAnnotatedString is a regular data class
l
louiscad
10/15/2019, 9:25 PMHow would that work with localization? A big when expression? Retrieve from resources on Android? Something else?
louiscad
10/15/2019, 9:26 PMAlso, is there a reason this doesn't implement
Appendables
Siyamed
10/15/2019, 11:06 PMRegarding i18n, that is a wider concept and not defined yet.
Siyamed
10/15/2019, 11:07 PMRegarding appendable, no charsequence support for now. Charsequence bites us on android.
Siyamed
10/15/2019, 11:09 PMWait :) I understand that you think AnnotatedString is directly related to annotation class and tag on Android?
Siyamed
10/15/2019, 11:14 PMIf so, that is not something we intended to :/
l
louiscad
10/15/2019, 11:54 PMI though it was related to the annotation spans you can put in string resources or
SpannedStringlouiscad
10/15/2019, 11:54 PMCharsequence bites us on androidHow?