AN
Map использует unsafeTransform вместо transform, если я правильно понимаю.
Ну это не так важно, он почти наверняка лучше оптимизирован чем то, что вы напишите сами.
Size: a a a
2020 September 08
A
Ну там просто менее монструозный вывод типов получается. Вообще общая рекомендация - не изобретать велосипед без острой необходимости.
Тут как бы не совсем велосипед, т.к. transform, я так понимаю, для публичного пользования. Хотя map, конечно, лучше подходит.
PE
Не подскажите, знатоки, чем приоритетнее?
PE
Это мне лид написал, в код ревью
VP
Приоритетнее то, что написано в кодстайле проекта/команды. А если его нет, то на вкус и цвет, если там одно выражение, влезающее в одну строку.
PE
Ну его по сути и нет, относительно такого, ладно, буду обсуждать
t
То, что приоритетнее в вашей компании. Синтаксис с
= имеет неочевидный сайд-эффект: функция начинает возвращать значение, если результат тела — не Unit. Это в теории может сломать ссылки на методы, не совпадающие по сигнатуреPE
не, я разумеется пишу когда надо его)
просто видимо код стайл такой в команде
просто видимо код стайл такой в команде
IO
как это переписать, чтобы идея не ругалась на блокирующий вызов?
VP
Ну его по сути и нет, относительно такого, ладно, буду обсуждать
Тогда можно идти далеко с такими замечаниями 🙂
PE
Тогда можно идти далеко с такими замечаниями 🙂
в плане?
VP
в плане?
Нет никакого смысла придираться к кодстайлу, если он не формализован
PE
Нет никакого смысла придираться к кодстайлу, если он не формализован
Ну да, верно
VP
Alt-Enter -> Disable inspection
IO
Alt-Enter -> Disable inspection
так себе
VP
так себе
Да эта кривая инспекция ругается на методы, которые не могут быть блокирующими, ей дорога именно туда
IO
Да эта кривая инспекция ругается на методы, которые не могут быть блокирующими, ей дорога именно туда
а тот метод реально блокирующий, я чекнул
IO
проблема в том, что withContext(Dispatchers.IO) { try { x() }} не вырубает эту инспекцию
VP
а тот метод реально блокирующий, я чекнул
readTree из строки?
IO
readTree из строки?
дыа