Программирование[9]

[sonac]

ну зависит ведь от традиций в тиме и отношения к подобной хуйне внутри (иногда бывает подход аля работает=похуй, а иногда вполне нормально взьебнуть за хуевый нейминг)

бай дефолт - я бы сначала лично поговорил с челом, мол так и так, вижу делаешь такую штуку, а ты не думал насчет того, что ее можно сделать еще и Х способом

[JuJeu]

Неоптимальность в чем заключается? 

[kijupsik]

JuJeu said 1 minute ago:

Неоптимальность в чем заключается? 

костыли и велосипеды

[JuJeu]

kijupsik написал 13 минут назад:

JuJeu написал 15 минут назад:

Неоптимальность в чем заключается? 

костыли и велосипеды

От кейса у кейсу сильно зависит. Я бы уже оставлял комментарии на код ревью и чтобы чел переписывал. Практика и на деле увидит в чем недостатки его решение. На длинной дистанции от этого Плюсов будет больше и для него и для команды. 

[GoldRobot]

Зависит от языка.

Хипстеры и хайперы на растах го и прочим попытаются вскрыть тебя и всю твою семью за унижение

С бэками питонщики плюсеры питонщиками ты нормально пообщаешься

У веперов горит жопа от такого, ибо это вся суть их проффесии

'SQL-программисты' в основном нормальные ребята, просто нужно научненько по фактикам расказать

Ну, а если это пишется на перле, то во первых земля те пухом, во вторых это видимо чувак со стажем 20 лет и ты чего то не догоняешь просто 

[Just.Doit]

kijupsik написал 7 часов назад:

короче ситуация. к нам взяли разработчика, он начал пилить новую фичу (средней сложности)

я из любопытства глянул в его ветку и он довольно, скажем, неоптимальную имплементацию там затеял. Вопрос такой, как в этой ситуации поступить - сказать ему сразу что можно проще и лучше и рассказать как, или дождаться пока он доведет до конца и уже тогда с ним обсуждать? Т.е. с одной стороны не хочется показаться душным уёбком а с другой не хочется потом челу говорить что надо половину того что он написал переделывать 

если чел новый - то задача тимлида/техлида (в первую очередь) и всей команды (во вторую) с ним притереться, для этого чем раньше будет презентовано решение и дана обратная связь - тем лучше

 

а ты уверен что неоптимальность имеет какое-то значение? 

 

если да, то пусть тимлид подойдет и спросит че как дела, как задача пишется? покажи че уже сделал? 

и дается обратная связь

и всё

 

вообще я не понял как ты посмотрел ветку если чел еще не дошел до пулреквеста

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

странно. обычно пилишь функционал, даже если 2-3 дня - делаешь локально 1-3 логических коммитов, потом все вместе пушишь и создаешь ПР

 

 

кстати душноты тут нет. ты просто на раннем этапе решил чекнуть и выявил что возможно есть что подсказать в какую сторону лучше смотреть

с подачи как сказал sonac это будет выглядеть норм

типа смотри, у нас есть такие штуки и еще можно вот так, посмотри/обрати-внимание может тебе в твоей задаче это пригодится

Изменено 6 марта 2020 пользователем Just.Doit

[JuJeu]

это будет выглядеть как полная душная хуйня, чел ещё не сделал пул реквест и к нему уже подваливает челик, который рассказывает про то как можно оптимальнее сделать. Эту нужно делать во время код ревью. Если это повторяется и он не использует советы, то уже тим лид должен периодически чекать его работу (зависит от размеров компании и от ожиданий от нового гребца).

[Just.Doit]

не согласен

[nobodies]

мне казалось что ПР для этого и нужны: мб у него такой подход "сделать лишь бы работало, а потом смотреть как можно улучшить" и к моменту когда будет отправлять на ревью все уже будет исправлено

если при собеседовании кандидата кажется что ты не заслуживаешь свое место, это говорит в тебе синдром самозванца или реально надо подкачаться собирать вещи и идти дворником/грузчиком работать?

[EbalKopal]

nobodies написал 6 минут назад:

мне казалось что ПР для этого и нужны: мб у него такой подход "сделать лишь бы работало, а потом смотреть как можно улучшить" и к моменту когда будет отправлять на ревью все уже будет исправлено

если при собеседовании кандидата кажется что ты не заслуживаешь свое место, это говорит в тебе синдром самозванца или реально надо подкачаться собирать вещи и идти дворником/грузчиком работать?

Fake it till you make it

[scarppy]

очевидно подойти и сразу помочь/подсказать/просто в разговоре начать говорить об этом

главное что бы это действительно было желания помочь, а не желания самоутвердится 

[suez]

Если он публично не шарил свои нароботки (PR, постил в канале слака или еще где) то отъебись от него. Вот вывалит кучу говна в пулл-реквесте, вот там и будешь пояснять. А сейчас дай человеку спокойно обосраться, если он думает что все знает и ни у кого не просит помощи.

[kijupsik]

Мнения разделились 

[Lorde - Sober ll]

напиши ему в личку, что код отстой и компания с ним прощается

[Rooster]

Если бы кто-то вторгся в мою личную веточку и посмотрел мою работу без моего ведома я бы пожаловался начальству на абьюз+сталкинг

Изменено 6 марта 2020 пользователем Rooster

[FeelYourDestiny]

Надо ждать пока он выкатит свою ветку. Может он понимает, что у него там велосипеды и он это исправит до того как поднимать ПР.

У меня бы подгорело, если бы ко мне доебались за мой не готовый код

[scarppy]

ППц вы прогеры социопаты

ваще на релаксычах можно подойти и просто за сигаретой/за обедом ненавязчива поспрашивать о его проекте

вот я такой чел, я могу делать что-то и быть убежденным, что я прав и потом очень обидно ошибаться

ну я не прогрер бтв

всегда надо людям помогать и бтв делать это максимально органично 

[GoldRobot]

Насоветовали блять. А потом мы увидим примерную пасту на "итебанное"

Цитата

 

Пришел в компанию новую перспективную, выглядит все круто, интересно.

Дали задание на неизвестном тебе проекте. Ничего фактически о нем не знаешь. На языке этом тоже писал мало, только знаком фактически.

Сидишь себе потихоньку погружаешься, изучаешь продукт, кодишь в свою веточку черновые решения, которые чисто mvp, не более.

Через день к тебе за обедом подсаживается типчик с бегающими крысиными глазками, которые давно работает в компании, и начинает спрашивать что делаешь и как сделал сейчас. На каждое предложение начинает проводить лекцию что "Я ничего не навязыю, но вот я бы сделал по другому" с ультра хуевыми предложками.

Еще черезь день в курилке подваливает какой-то вонючий уебан, который начинает как с духом в армейке за жизнь разговаривать, и навязывает свои кодо порядки, то звездочку ему нужно ставить окружив десятью пробелами, то однострочные ифы писать, и еще тонна "неоптимальных" мелочей.

В конце первой недели получил от менеджера выговор, мол к нему подошел Анонимный Разработчик и нажаловался что мой код говно, и меня нужно уволить как несоотведствующего стандартам компании.

 

[Kant]

если челик проебался где-то в архитектурной части, имхо лучше сразу ему сказать, что он долбоеб, потому что потом придется 90% кода выкинуть нахуй

 

если там всякая фигня в бизнеслогике, то жди ПР

[Professor1]

GoldRobot написал 54 минуты назад:

 

Цитата

Через день к тебе за обедом подсаживается типчик с бегающими крысиными глазками, которые давно работает в компании, и начинает спрашивать что делаешь и как сделал сейчас. На каждое предложение начинает проводить лекцию что "Я ничего не навязыю, но вот я бы сделал по другому" с ультра хуевыми предложками.

 

Ебать, как представил , самому душно стало, форточку открыл

Сейчас на втором спринте проходим в Яндекс Практикум.

HTML: видео, iframe, embed, API
Анимации и трансформации (СSS)
Разметка и вёрстка форм
Пути к файлам и организация файлов по БЭМ
Продвинутый Git 
Дебаггинг в вёрстке

В тренажере весь материал грамотно представлен, сложностей пока не возникает. 
В субботу дойду до 2ой проектной работы. 

15 марта, deadline. Надо будет всё сдать. А то в академ на месяц отправят и со след. потоком старт.

8 месяцев до вката осталось, буду держать в курсе.