Viacheslav
Romɑn
Крылатый
лол
Крылатый
https://www.spacetelescope.org/images/heic1502a/zoomable/
Крылатый
Годноты вам
Крылатый
Или порвать арбузер https://svs.gsfc.nasa.gov/vis/a010000/a013200/a013285/TESS_south_hires_azeq_no_labels_16K.jpg
Arsenii
ООПГМ это уже зарегистрированная болезнь?
Romɑn
Arsenii
Я просто чищу тут коммиты одного европейского коллеги, человек нёс корпорат в наши серые массы
Arsenii
Тесты на кастомные контракты внутри объектов, которые поднимают исключение, если null
Arsenii
И всё
Romɑn
Arsenii
Ну тут иногда проскакивают прям красоты
Arsenii
С 15 дженериками
Ayrat
Вот что сегодня родил скала гений
val allCombos =
List(true, false).flatMap(i => List(true, false).map(j => (i,j)))
Тип код “на вырост”, ВДРУГ в булеане третий вариант появится, тогда комбинации САМИ появятся новые
Ayrat
удобно чо. Написать четыре тупля в листе религия не позволила
Ayrat
пойду поплачу
Ilya
Зачем руками? Лучше кодген.
Ivan
а если тип с була изменится на другой?
Ivan
вполне обычная ситуация, когда бул вырастает в энум
Ilya
Поэтому лучше кодген!
Vladislav
Vladislav
или как ее
Denis
Vasily
Карты Карно !!!!!1111
Denis
логгеры в монаду не засовывают ?
Диёр
а если тип с була изменится на другой?
Сегодня только прикручивал апиху где в поле win в качестве значения могли быть только "Win" и "Failure". Почему не бул я до сих пор не понял
Ayrat
таблица истинности?
всё проще, просто все комбинации двух булеанов чтобы потом подать их в тест
Ayrat
ну типа вот использование
for((oldCeilingEligible, newCeilingEligible) <- allCombos)
{
val oldTarget = priceEntry(oldPrice, targetEligible)
val oldCeiling = priceEntry(oldPrice, oldCeilingEligible)
val oldAlgo = MarketplaceAlgoPriceEntity(target = Some(oldTarget), ceiling = Some(oldCeiling))
val newTarget = priceEntry(newPrice, !targetEligible)
val newCeiling = priceEntry(newPrice, newCeilingEligible)
val newAlgo = MarketplaceAlgoPriceEntity(target = Some(newTarget), ceiling = Some(newCeiling))
CrtRulesFilter.isImportantUpdate(newAlgo, oldAlgo) should be (true)
}
где allCombos вон выше определен.
Но как бы факт в том как эти allCombos определены
Ayrat
я больше про то что четыре тупля булов проще записать явно, чем через флатМап с вложенным мапом
Ayrat
это блять целых 4 комбинации
Ayrat
навсегда
Ayrat
вот когда поменяется на енум, тогда и комбинации можно поменять. Пока даже не на горизонте
Vladislav
Roman
За такую хуйню он бы вообще сразу казнил наверное
Ivan
в коде для тестов я бы не искал оплошностей подобного рода и "оверинжиниринга".
Ayrat
я к слову даже nitpick в ревью не написал, просто поплакал внутрь
Ayrat
я не придираюсь к таким мелочам
Roman
нет, не все. Я вот залез в бд слой у нас на днах. Короче, в абсолютно каждой доменной сущности у нас присутствует поле Persistance option, в котором инфа про партишн ключи и прочую ебалу. А опшн пушто с клиента-то прилетает без него. А мы, такие молодцы, сами все это проставляем. Такая вот история
Ayrat
ну пишет так чувак и ладно
Roman
Ayrat
бережешь нервы? Или типа choose your battles?
я ненавижу ревью с сотней коментов. Код должен быстро уходить в продакшн без обмусоливания мелочей и личных код стайлов.
Если не противоречит правилам линтинга мне норм
Лично я (лично я) пишу коменты только на реальные ошибки
Ayrat
но я знаю людей которые приёбываются ко всему на свете если что-то выглядит не так, как они хотят
Roman
ну, подобная хуйня просто опасна. Тут дело не в личном стиле, а в том, что вместо двух секунд на чтение этого кода уходит 2 минуты. И в тестах это еще безобидно, но когда такое начинает везде появляться, это же жопа
Roman
То есть это начинает сильно снижать общую производительность команды
Ayrat
понимаю, верю. Но всех под себя не переделать. Я конечно хотел бы клонировать себя, но нимагу
Ayrat
я оставляю место для личного кодаизливания
Roman
Но вообще я тоже начинаю проникаться похуизмом. Оч приятная штука
Ayrat
хочешь ты перебирать булеаны флатмапом, пожалуйста
Ayrat
я просто поплачу внутрь
Ayrat
ну и сюда напишу
Ivan
(позанудствую немношк) картезианов продукт двух булов - 6 значений. легко можно ошибиться. флатмап ошибки никогда не сделает.
Ayrat
Bonart
Ayrat
в скале булеаны вроде ненулябельны, ща проверю
Ivan
нет, мой косяк.
Ayrat
не, ну ошибиться в переборе ВСЕХ вариантов двух булов конечно можно
Ayrat
ВСЕХ ЧЕТЫРЕХ
Ayrat
но я бы тогда посоветовал пойти в проституки
Bonart
Roman
ну если ты лид, то у тебя и рычагов больше, чтоб поток говна ослабить
Roman
Vladislav
Roman
просим просим
Bonart
просим просим
Когда я смогу понизить долю нецензурной лексики до единиц процентов
Roman
да ебошь, тут незазорно
Bonart
А так я был близок к выходу в космос по украинскому варианту
Roman
что это значит?
Roman
Укр Космос? Космос ДБ?