0

Долго вчитывался в документацию, но не до конца понял, как оно работает.

Внутренний голос нашептывает, что вот эти две строки должны делать одно и то-же:

private fun makePost(post: Map<String, String>): RequestBody {
    val body = FormBody.Builder()
1:    post.forEach { (k, v) -> body.add(k, v) }
2:    post.forEach { body::add }
    return body.build()
}

Но не делают. первая работает, а вторая нет (body остается пустым, но и ошибок нет)

растолкуйте, пожалуйста, где мой внутренний голос облажался и можно ли(как?) использовать тут function reference?

5
  • Можно поинтересоваться версией Kotlin, что вы используете? 7 мая 2017 в 19:31
  • @EugeneKrivenja последней, которая плагином в андроид студию подтягивается. 1.1.2-3
    – rjhdby
    7 мая 2017 в 19:36
  • Уверены что в build.gradle именно эта версия? Плагин-плагином, а вдруг не подтягивается? :) 7 мая 2017 в 19:45
  • @EugeneKrivenja конечно уверен. ext.kotlin_version = '1.1.2-3', classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"? compile "org.jetbrains.kotlin:kotlin-stdlib-jre7:$kotlin_version"
    – rjhdby
    7 мая 2017 в 19:49
  • Ваш внутренний голос прав, по моему мнению, см. ответ. 7 мая 2017 в 20:16

2 ответа 2

0

Отлично, что с версией все ясно. Вот мое обьяснение:

Метод forEach для вашей Map оперирует элементами Map.Entry<String, String>, а это не то же самое, что два параметра типа String, необходимые методу body::add.
Но Kotlin вместо того, чтобы ругнуться, делает вид что сможет развернуть элемент, как делает в варианте №1, но что-то не получается.

Думаю, что можете зарепортить им баг. Как минимум на то, что нет ни ошибки, ни предупреждения, но код не работает.

Вот ссылка на имплементацию этой фичи.
https://youtrack.jetbrains.com/issue/KT-6947

Это похоже на специфический кейс для Map, который упустили в тестах новой фичи.

5
  • Ответ разработчиков: Your code is incorrect. To pass a method reference to a function, you need to enclose it in parentheses, not in curly braces: bar.forEach(foo1::add)
    – rjhdby
    8 мая 2017 в 8:35
  • Все равно это баг, post.forEach { body::add } не должен компилиться тогда. 8 мая 2017 в 11:50
  • youtrack.jetbrains.com/issue/KT-17053
    – rjhdby
    8 мая 2017 в 12:30
  • По сути это совершенно нормальное поведение языка, вводящее в заблуждение неочевидностью синтаксиса. Фактически создается ничего не делающая лямбда
    – rjhdby
    8 мая 2017 в 12:33
  • Отлично, в 1.1.3 проверим :) 8 мая 2017 в 14:29
0

Ответ разработчиков:

Your code is incorrect. To pass a method reference to a function, you need to enclose it in parentheses, not in curly braces: bar.forEach(foo1::add)

При передаче (method)function reference в функцию необходимо использовать обычные скобки, а не фигурные.

Следующие три строки выполняют одно и то же:

post.forEach ({ (k, v) -> body.add(k, v) })
post.forEach { (k, v) -> body.add(k, v) }
post.forEach (body::add)

Обратите внимание, что метод, на который осуществляется ссылка, должен возвращать Unit, иначе даже не скомпилируется.

Если единственным параметром является лямбда, то обычные скобки можно опустить и оставить только фигурные (строки 1 и 2)

2
  • Интересно. А не известно ли, как интерпретируются фигурные скобки в таком случае?
    – yeputons
    8 мая 2017 в 9:21
  • @yeputons обновил ответ
    – rjhdby
    8 мая 2017 в 10:22

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.