Сап двач, к нам в Газпромбанк пришёл новый автотестер который написал это, ваши ответы
Аноним 18/10/23 Срд 19:42:16 #1 №2883991
Сап двач, к нам в Газпромбанк пришёл новый автотестер который написал это, ваши ответы
Что это за ебанутая подсветка?
Потому он и автотестер, а не программист.
А как надо было по-твоему?
сеньёр автотестов
Звучит как "сеньор консультант в магазине".
Что не так? Вроде как по учебнику Effective Java. Только это петухотлин, там наллы проверять не надо в типах с вопросиками.
А что ты ожидаешь увидеть в кастомно переопределённом equals?
а ты нихуя не понимаешь в написании тестов.
именно так тесты и пишутся. У них особый стиль.
Эххх щас бы в тестах хуярить абстракции, чтобы потом писать тесты на тесты
Вообще что тесты, что CI/CD пайплайны в банках пишут именно так. Нет смысла делать функции, которые потом вызываются синхронно.
Конечно типо можно вынести все отдельную функцию и передавать в нее только eventType, чтобы дальше он либо луп запускал, либо еще чето. Такое тупо сложно поддерживать в будущем. Я сам DevOps и раньше тоже разбивал пайпы по классам, пока не поработал в СБЕР, где на проекте пульс пайплайн по типу пикрил, хорошо что в try(exception e) {} catch{} хоть обертывали шаги синхронные
Чел более чем адекватный, если ты этого не понимаешь или считаешь неверным, ты не на том месте сидишь
Что за джаваподобный язык?
Какой ужас. И хотя я сам не программист, но тут явно виден какой-то плохой тон. Что делает строчка other as ...? результат никуда не сохраняется, или там в языке этом своя магия? Почему логи на русском?
>Почему логи на русском?
Потому что ты живешь в России, европидараха.
Ну так и пишите на 1С тогда, или разрабатывайте свои языки на русском. Логи должны быть на английском, как и документация и все остальное.
Порридж, спок. То, что тебя с детства приучили дрочить на все иностранное, не значит, что оно хорошее.
Представляете, как дико выглядят наши коды для людей с англозычным сознанием? Примерно так.
Надеюсь ты когда-нибудь удостоишься чести работать в китайском компании, где все логи и комменты на китайском.
мимо
Зачем в компании, которая не планирует в ближайшие 5-10 лет превращать отдел разработки в интернациональный писать что-то на англюсике. В данном случае это каргокульт, а не необходимость
Это прав.
>все логи и комменты на китайском
На родном для команды языке мысли выражаются точнее и яснее.
Я знаю английский достаточно хорошо (лучше всех, с кем я работаю сейчас в РФ), и нет ничего более кринжового, чем видеть абсурдно неуместное словоприменение. Я понимаю, что там написано, потому что могу перевести дословно на русский и догадаться. Но некоторые выражения человек без знания русского просто не поймёт.
В РФ уровень английского в массах очень и очень низкий. Люди практикуются в среде таких же. Их никто не поправляет. Из-за этого в айти производятся тонны речекряков.
Да, айтишные термины вы видите в документации и применяете их уместно. Но если дело касается предметной области, начинается ужас.
При этом очевидно, что в вашей команде абсолютно никогда, с вероятностью строго равной нулю, не появится человек, который не будет знать русского.
Вы все отлично знаете и понимаете русский. Вы все ужасно пишите по английски. Вы плохо понимаете английский (как оригинальный, так и написанный малограмотными россиянами).
По сути дела применение английского - это гарантированное снижение производительности команды.
Думаю ОП имел в виду что должно быть что то такое (псевдокод)
propList = [ "eventType", "clientOrderId", "orderOwnerId", ...и т.д.];
for(prop : propList)
if(self[prop] != other[prop]
{
log.error("Значение {prop} не соответствует ОР"); return false;
}
Фьюить, конец.
propList = [ "eventType", "clientOrderId", "orderOwnerId", ...и т.д.];
for(prop : propList)
if(self[prop] != other[prop]
{
log.error("Значение {prop} не соответствует ОР"); return false;
}
Фьюить, конец.
Так работает(не работает) только в говноязыке типа жопаскрипта. В джаве придётся через рефлексию брать список полей класса со всеми вытекающими проблемами и подводными камнями.
Ну да, но даже для древнючего Си в свое время писали генераторы кода.
Причем я могу аргументировать почему в оп посте хуже. Там три раза написано одно слово, причем один раз текстом в кавычках. Это значит, что при копипасте которым эта портянка делается, может получиться
if(orderOwnerId != other.orderTradingAccountId)
В общем пикрил.
А кстати еще вопрос. Точно ли тест проверяет то, что написано? Тест проверяет равенство объектов, а в комменте написано что проверяет сохранение в БД.
Можно через делегаты сделать, запихать их в список.
Что не так на твоём пике?
Это же та самая читаемость кода за которую все топят.
> пик
Тоже такое пишу, когда фичу нужно было выкатить уже вчера
Это где? В омни где то?
А ничё что хтмл быстрее грузится чем жс? Ты типа предлагаешь в жс цикл сделать и захуярить разметку? Ясно всё с тобой
Это серверная разметка, она на сервере рендерится ещё. Только в случае цикла формошлёпский дизайнер нихуя не покажет.
Ну и вообще это богомерзкий web forms, который никак не умрёт и постоянно встречается в легаси.
