Size: a a a

2020 September 13

PM

Pavieł Michajłaŭ in ctodailychat
они уже сами подтвердили
источник

С

Слава in ctodailychat
Сергей Аксёнов
Сегодня ловили полдня баг. Временно запретили пользователям менять ники, чтобы накатить на них другой уникальный индекс (был общий, стал с разделением по проектам). Но ники продолжали меняться. Оказалось, что кроме общепонятного \iFunny\Domain\User\Info\NicknameUpdater кто-то (точнее сейчас уже понятно кто, по коммиту видно) реализовал переименование через прямой запрос к \iFunny\Storage\Repository\MongoUserRepository. Стоимость этого решения - уже примерно 30 человеко-часов and counting, думаю дойдёт до 50.

Можете рассказывать эту историю джунам, которые будут спрашивать вас, нафига нужны эти мудрёные SOLID и DDD.
А на чём проект написан? Find usages не делали сперва?
источник

IV

Igor V in ctodailychat
Слава
А на чём проект написан? Find usages не делали сперва?
источник

С

Слава in ctodailychat
Мда
источник

IV

Igor V in ctodailychat
Сергей Аксёнов
Сегодня ловили полдня баг. Временно запретили пользователям менять ники, чтобы накатить на них другой уникальный индекс (был общий, стал с разделением по проектам). Но ники продолжали меняться. Оказалось, что кроме общепонятного \iFunny\Domain\User\Info\NicknameUpdater кто-то (точнее сейчас уже понятно кто, по коммиту видно) реализовал переименование через прямой запрос к \iFunny\Storage\Repository\MongoUserRepository. Стоимость этого решения - уже примерно 30 человеко-часов and counting, думаю дойдёт до 50.

Можете рассказывать эту историю джунам, которые будут спрашивать вас, нафига нужны эти мудрёные SOLID и DDD.
Сергей, расскажи потом как изменили процесс, что такого избегать подобных ситуаций в дальнейшем.

Расскажу как мы боремся с подобным у себя. У нас ML платформа на которой ресерчеры гоняют свои модели (произвольный код), а затем идут в прод. Хоть ребята умеют неплохо программировать, но они не программисты. Поэтому мы прикрутили валидатор на базе AST, чтобы отлавливать типичные ошибки: изменение глобального стейта, использование internal api, переопределение переменных окружения, использование абсолютных путей  вместо относительных, неправильные импорты и т.д.
источник

SS

Slava Savitskiy in ctodailychat
Жираф Жирафович
Закладки делать предлагают
или я вот вспомнил, бывает покупают личные данные, чтобы на всяких фриланс биржах лучше труд галерщиков продавать
источник

NB

Nikita Bayev in ctodailychat
А тут никого нет с Apple?
Хочу понять куда писать, чтобы пофиксили отображение страны при звонках с казахстанских номеров. У нас постоянно Россия выводится, хотя +77 за Казахстаном закреплён, а за Россией другие коды.
источник

СА

Сергей Аксёнов... in ctodailychat
Igor V
Сергей, расскажи потом как изменили процесс, что такого избегать подобных ситуаций в дальнейшем.

Расскажу как мы боремся с подобным у себя. У нас ML платформа на которой ресерчеры гоняют свои модели (произвольный код), а затем идут в прод. Хоть ребята умеют неплохо программировать, но они не программисты. Поэтому мы прикрутили валидатор на базе AST, чтобы отлавливать типичные ошибки: изменение глобального стейта, использование internal api, переопределение переменных окружения, использование абсолютных путей  вместо относительных, неправильные импорты и т.д.
Я пока сам думаю, что тут можно сделать. Кодбаза большая, даже линтер об неё спотыкается, а уж статанализ просто захлёбывается. Пока вот кроме перечитывания Clean Code с объяснением на примерах - ничего не придумал.
источник

СА

Сергей Аксёнов... in ctodailychat
Слава
А на чём проект написан? Find usages не делали сперва?
PHP, полмиллиона строк, не считая комментариев и пустых. Код довольно хороший, код-ревью дотошно делаем, но вот проскочило. Я уже выяснил, кто и когда эту заплатку с переименованием писал, всё как обычно: надо было срочно, продакты хотели фичей, "потом отрефакторим". Отрефакторили, чего уж)
источник

SZ

Sergey Zhuk in ctodailychat
прочитал в статье на тинькофф журнале:
> Еще у нас на работе есть такое понятие, как «касса взаимопомощи»: специалистам, которые проработали 2—3 года без нареканий, мы спокойно выдаем до трех месячных окладов по договору займа. Обычно ставка для сотрудников — это ставка ЦБ + 1%, чтобы опять же налоговая не цеплялась.

у кого-то на работе есть что-то подобное? насколько востребовано?
источник

SS

Slava Savitskiy in ctodailychat
Sergey Zhuk
прочитал в статье на тинькофф журнале:
> Еще у нас на работе есть такое понятие, как «касса взаимопомощи»: специалистам, которые проработали 2—3 года без нареканий, мы спокойно выдаем до трех месячных окладов по договору займа. Обычно ставка для сотрудников — это ставка ЦБ + 1%, чтобы опять же налоговая не цеплялась.

у кого-то на работе есть что-то подобное? насколько востребовано?
у нас было такое в ИСП РАН, люди пользовались
источник

SS

Slava Savitskiy in ctodailychat
как вариант привязки сотрудника к месту работы
источник

СА

Сергей Аксёнов... in ctodailychat
Sergey Zhuk
прочитал в статье на тинькофф журнале:
> Еще у нас на работе есть такое понятие, как «касса взаимопомощи»: специалистам, которые проработали 2—3 года без нареканий, мы спокойно выдаем до трех месячных окладов по договору займа. Обычно ставка для сотрудников — это ставка ЦБ + 1%, чтобы опять же налоговая не цеплялась.

у кого-то на работе есть что-то подобное? насколько востребовано?
У моих родителей в 1980-е такое было (они работали вместе в оборонном НИИ). В XXI веке первый раз про такое слышу.
источник

С

Слава in ctodailychat
Но это не касса взаимопомощи. В кассу люди вносят свои и добровольно
источник

С

Слава in ctodailychat
А это всё же именно займ от банка
источник

O

Oleg in ctodailychat
Sergey Zhuk
прочитал в статье на тинькофф журнале:
> Еще у нас на работе есть такое понятие, как «касса взаимопомощи»: специалистам, которые проработали 2—3 года без нареканий, мы спокойно выдаем до трех месячных окладов по договору займа. Обычно ставка для сотрудников — это ставка ЦБ + 1%, чтобы опять же налоговая не цеплялась.

у кого-то на работе есть что-то подобное? насколько востребовано?
Мы выдаем в индивидуальном порядке без процентов
источник

SS

Slava Savitskiy in ctodailychat
Oleg
Мы выдаем в индивидуальном порядке без процентов
ваша фамилия не Тинькоф случайно? 😀
источник

SZ

Sergey Zhuk in ctodailychat
Slava Savitskiy
ваша фамилия не Тинькоф случайно? 😀
источник

O

Oleg in ctodailychat
Slava Savitskiy
ваша фамилия не Тинькоф случайно? 😀
Нет) если серьезно у людей есть разные ситуации и мы стараемся поддерживать своих сотрудников
источник

A

Alex in ctodailychat
Сергей Аксёнов
Сегодня ловили полдня баг. Временно запретили пользователям менять ники, чтобы накатить на них другой уникальный индекс (был общий, стал с разделением по проектам). Но ники продолжали меняться. Оказалось, что кроме общепонятного \iFunny\Domain\User\Info\NicknameUpdater кто-то (точнее сейчас уже понятно кто, по коммиту видно) реализовал переименование через прямой запрос к \iFunny\Storage\Repository\MongoUserRepository. Стоимость этого решения - уже примерно 30 человеко-часов and counting, думаю дойдёт до 50.

Можете рассказывать эту историю джунам, которые будут спрашивать вас, нафига нужны эти мудрёные SOLID и DDD.
я ниче не понял... "\iFunny\Domain\User\Info\NicknameUpdater" это что? путь к файлу на диске?
источник