VC
лол, я не заметил проблему)
Size: a a a
2021 August 04
I
ну да, с первого взгляда не очевидно ))
M
Это как раз область стат анализаторов. И в случае конкретно одного выражения - никто не пишет { Option(10) }.map, а скорее пишут { Option(10).map }. Поэтому именно в таком виде не вызывает сомнений, что челик хотел ко всему логическому блоку
VC
ну все, вот теперь можно закапывать скобки. с сигнификант-индентацией такой фигни не будет.
ZM
я воспринял пример как идею, а не буквально в таком виде
но все равно тогда надо как-то различать кейсы попроще и посложнее
но все равно тогда надо как-то различать кейсы попроще и посложнее
D
+
хз как тут стат анализатор помог бы
хз как тут стат анализатор помог бы
VC
ну помог бы если бы такой рул был
D
замените if else на fold из mouse и будет счастье
GP
А как бы он работал?
M
Да. Специфический рул для if{}.method else {}.method - такие рулы - хлеб стат анализаторов для с++, когда вроде бы конструкция и разрешенная компилятором и имеет даже смысл, но отличается от обычного кода и потенциально может приводить к малозаметным ошибкам.
D
вроде можно настроить scalafmt чтобы он убирал скобки в if else (ну если там что-то простое в ветках)
D
ну и тогда бы глазами было виднее наверное
D
if (flag) Option() else Option().map()
GP
хм да, такое было б хорошо
P
так может быть сложнее, чем одна строка
D
тогда метод надо декомпозить :)
ZM
имхо, в идеале надо проверять есть ли тест на этот кусок
если нету, то тогда ругаться что потенциально опасное место
если нету, то тогда ругаться что потенциально опасное место
D
Pavel соглы?
P
надо тогда еще стат анализатор, который скажет, что у тебя в if-else блоки больше 1 строки (мб такое есть, я хз)
D
ругать надо ревьювера тогда