Re[17]: 31 летних программистов съедает дракон
От: baxton_ulf США  
Дата: 04.06.23 13:02
Оценка: +1
Здравствуйте, Pauel, Вы писали:

P>Из тех, с кем я учился, те самые 90% уже не разработчики.


практика показывает, что цифра 75% работает гораздо лучше
(если чё, то моя реплика о полной фуфлёжности приводимых выше цифр)

P>Из моих студентов уже слишком много нелинейных — тл, архитекторы,девменеджеры итд.


вам бы договориться о терминологии. "нелинейный" — это как? звучит как "не программист уровня мидл или джун".
ну да, таких после 40 трудно найти

P>По конторе — к 40 сеньоров остается ничтжное количество.


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

P>Хотелось бы взглянуть на ваши источники.


источник под названием: что вижу, то пою.
Re: Не люблю ревьюить чужой код
От: baxton_ulf США  
Дата: 04.06.23 13:13
Оценка:
Здравствуйте, Философ, Вы писали:

Ф>Во-первых, мне тяжело что-либо ревьюить код, если я не вник в суть задачи. Я при ревью спотыкаюсь об вопросы типа, решает ли код подставленную задачу и решает ли он её полностью. Т.е. мне прежде чем ревьюить нужно сначала прочитать задачу, её предысторию (связанные обсуждения) и только потом я могу приступать к коду. Для меня это настоящая работа.


если при ревью что то не ясно, то надо задавать вопрос в комментах, а не жать на аппрув. попроси автора написать подробнее, что он/она там сделали и зачем

Ф>Во-вторых, я спотыкаюсь о то, что я раньше аппрувил задачи с косяками. Такой опыт заставляет немного нервничать, когда курсор зависает над кнопкой Approve.


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

Ф>Как у вас с этим дело обстоит?


нормально обстоит делаю довольно много их (1 — 3 в день) если что то не ясно или выглядит подозрительно не апрувлю пока не объяснят или не исправят. иногда для ревью создается целый митинг и на нем очень тщательно все проверяем
Re[2]: Не люблю ревьюить чужой код
От: RedUser Россия  
Дата: 04.06.23 13:38
Оценка:
_>нормально обстоит делаю довольно много их (1 — 3 в день)

А какой в средней объём изменений в строчках?
Сколько комментариев на одно ревью?
Re[18]: 31 летних программистов съедает дракон
От: Pauel Беларусь http://blogs.rsdn.org/ikemefula
Дата: 04.06.23 13:39
Оценка:
Здравствуйте, baxton_ulf, Вы писали:

P>>Из моих студентов уже слишком много нелинейных — тл, архитекторы,девменеджеры итд.


_>вам бы договориться о терминологии. "нелинейный" — это как? звучит как "не программист уровня мидл или джун".


Линейный — джун, мидл, сеньор. После линейных идут менеджерские разновидности, например, тот же архитектор или r&d, тимлид, дев менеджер, департмент, деливери, пм итд.

_>и да и нет. у нас, например джунов не нанимают.


Сеньоры пилят в том числе и мелочевку и рутинные задачи? Ну круто. Только зачем?
Кстати, у меня в скайпе подпись: "команда сеньоров без джунов это команда мидлов"

_>полагаю сеньоров не надо столько же сколько нужно "линейных" (откуда этот термин вообще взялся?)


Рутинного кода на любом проекте в районе 80-90%
У вас или сеньоры — лиды будут этим заняты, или двуны разгружают сеньоров и постепенно вырастают

_>источник под названием: что вижу, то пою.


Демографика на SO куда более релевантный источник
Отредактировано 04.06.2023 13:42 Pauel . Предыдущая версия .
Re[3]: Не люблю ревьюить чужой код
От: baxton_ulf США  
Дата: 04.06.23 17:18
Оценка:
Здравствуйте, RedUser, Вы писали:

_>>нормально обстоит делаю довольно много их (1 — 3 в день)


RU>А какой в средней объём изменений в строчках?


я продвигаю идею маленьких ПРов, если приходит простыня с изменениями за неделю (месяц и тд) я прошу разбить на части.

RU>Сколько комментариев на одно ревью?


когда как, может и ни одного. особенно если где-то уже все обсудили. а может и много. но если уж совсем много получается, то это признак того, что сделано совсем уж плохо
Re[19]: 31 летних программистов съедает дракон
От: baxton_ulf США  
Дата: 04.06.23 17:27
Оценка:
Здравствуйте, Pauel, Вы писали:

P>Здравствуйте, baxton_ulf, Вы писали:


P>>>Из моих студентов уже слишком много нелинейных — тл, архитекторы,девменеджеры итд.


_>>вам бы договориться о терминологии. "нелинейный" — это как? звучит как "не программист уровня мидл или джун".


P>Линейный — джун, мидл, сеньор. После линейных идут менеджерские разновидности, например, тот же архитектор или r&d, тимлид, дев менеджер, департмент, деливери, пм итд.


ясно. значит я все еще "линейный"
менеджеры и архитекторы — это отдельные виды. у них у самих то же самое деление — джун, мидл, сеньор. у нас возможно мигрировать между этими видами.


_>>и да и нет. у нас, например джунов не нанимают.


P>Сеньоры пилят в том числе и мелочевку и рутинные задачи? Ну круто. Только зачем?


сеньеры или мидлы, кому что перепадет тот то и пилит.
в терминах этого топика у нас "очень высокая цена ошибки", потому и нет джунов

P>Кстати, у меня в скайпе подпись: "команда сеньоров без джунов это команда мидлов"


я не против

_>>полагаю сеньоров не надо столько же сколько нужно "линейных" (откуда этот термин вообще взялся?)


P>Рутинного кода на любом проекте в районе 80-90%

P>У вас или сеньоры — лиды будут этим заняты, или двуны разгружают сеньоров и постепенно вырастают

ну примерно так, только мидлы разгружают

_>>источник под названием: что вижу, то пою.


P>Демографика на SO куда более релевантный источник


а можно линк?
Re[2]: Не люблю ревьюить чужой код
От: Alekzander  
Дата: 04.06.23 19:34
Оценка:
Здравствуйте, johny5, Вы писали:

J>отсутствие pragma warning off


Чем не угодили pragma warning off? Это хорошая вещь, помогающая держать treat warnings as errors включенным, чтобы в нужных местах сказать железному мозгу, что человек обо всём подумал, и это осознанное решение.
Re[20]: 31 летних программистов съедает дракон
От: Pauel Беларусь http://blogs.rsdn.org/ikemefula
Дата: 04.06.23 19:40
Оценка: +1
Здравствуйте, baxton_ulf, Вы писали:

P>>Демографика на SO куда более релевантный источник


_>а можно линк?


https://survey.stackoverflow.co/2022/#developer-profile-demographics
Re[17]: 31 летних программистов съедает дракон
От: CreatorCray  
Дата: 04.06.23 22:40
Оценка: +1
Здравствуйте, Pauel, Вы писали:

P>Из моих студентов уже слишком много нелинейных

Что такое "линейный"? Механический кодер, без малейшего просвета своих мыслей?

P>По конторе — к 40 сеньоров остается ничтжное количество.

Настоящий senior способен взять фичу на этапе "есть намётки что мы хотим получить", задизайнить и реализовать, включая cross-team взаимодействие.
Это и архитект и частично менеджер в одном флаконе.
... << RSDN@Home 1.3.110 alpha 5 rev. 62>>
Забанили по IP, значит пора закрыть эту страницу.
Всем пока
Re[3]: Не люблю ревьюить чужой код
От: CreatorCray  
Дата: 04.06.23 22:40
Оценка:
Здравствуйте, Alekzander, Вы писали:

A>держать treat warnings as errors включенным

Это как по мне обязательно и безальтернативно.
... << RSDN@Home 1.3.110 alpha 5 rev. 62>>
Забанили по IP, значит пора закрыть эту страницу.
Всем пока
Re[15]: Не люблю ревьюить чужой код
От: Артём Австралия жж
Дата: 04.06.23 23:48
Оценка:
Здравствуйте, CreatorCray, Вы писали:

Аё>>открываю свой веб код в айпада

CC>А я то тут каким боком, Артёмка

А ты, когда про рокет саенс сипипи команд написал- ты не про "что вижу, то и пою"? Ты же яблочко видишь изнутри. Так вот я тебе про "особенность" яблочка, когда вроде бы ну что можно испогагить- бери из хромиума, но нет, везде свой "штрижок". Рокет саентисты <.>.
Re[19]: 31 летних программистов съедает дракон
От: RedUser Россия  
Дата: 05.06.23 01:01
Оценка:
P>Рутинного кода на любом проекте в районе 80-90%
P>У вас или сеньоры — лиды будут этим заняты, или двуны разгружают сеньоров и постепенно вырастают

Что для сеньора рутинная задача, для джуна — совсем не факт.
Сеньор, может быть, быстренько сделал бы это сам.
А для джуна нужна чёткая постановка задачи.
Иногда с указанием, как именно это делать, чтобы он не начал делать что-нибудь не то.
Потом несколько итераций merge request-а, где сеньору надо будет расписывать для джуна, почему делать вот так не надо.
В процессе джун ещё будет дёргать сеньора с вопросами, как вот это или то лучше сделать.
Разгружает ли это сеньора — ну не знаю, не похоже.
Отредактировано 05.06.2023 1:03 RedUser . Предыдущая версия .
Re[16]: Не люблю ревьюить чужой код
От: Marty Пират https://www.youtube.com/channel/UChp5PpQ6T4-93HbNF-8vSYg
Дата: 05.06.23 01:09
Оценка:
Здравствуйте, Артём, Вы писали:

Аё>А ты, когда про рокет саенс сипипи команд написал- ты не про "что вижу, то и пою"? Ты же яблочко видишь изнутри. Так вот я тебе про "особенность" яблочка, когда вроде бы ну что можно испогагить- бери из хромиума, но нет, везде свой "штрижок". Рокет саентисты <.>.


А кстати, хромиум разве не яблочный?
Маньяк Робокряк колесит по городу
Re[4]: Не люблю ревьюить чужой код
От: Alekzander  
Дата: 05.06.23 01:45
Оценка: +1
Здравствуйте, CreatorCray, Вы писали:

A>>держать treat warnings as errors включенным

CC>Это как по мне обязательно и безальтернативно.

Соглы. Для этого и нужен #pragma disable, который позволяет игнорировать ворнинги точечно, а не все сразу. Навскидку пара примеров: хедер может придти из источника, над которым нет власти, а это библиотека, которую нельзя не использовать. Ну и deprecated, когда в нём почему-либо есть нужда.
Re[5]: Не люблю ревьюить чужой код
От: johny5 Новая Зеландия
Дата: 05.06.23 02:51
Оценка:
Здравствуйте, Alekzander, Вы писали:

A>Здравствуйте, CreatorCray, Вы писали:


A>>>держать treat warnings as errors включенным

CC>>Это как по мне обязательно и безальтернативно.

A>Соглы. Для этого и нужен #pragma disable, который позволяет игнорировать ворнинги точечно, а не все сразу. Навскидку пара примеров: хедер может придти из источника, над которым нет власти, а это библиотека, которую нельзя не использовать. Ну и deprecated, когда в нём почему-либо есть нужда.


Ну я не говорю что это абсолютное зло а то, за чем можно поглядывать в чужом коде и требовать обоснования использования. Впрочем это вершина айсберга того что должно быть вообще проверено во время ревью.
Re[17]: Не люблю ревьюить чужой код
От: Артём Австралия жж
Дата: 05.06.23 04:04
Оценка:
Здравствуйте, Marty, Вы писали:

M>А кстати, хромиум разве не яблочный?


Да, это совметная работа Гугл, Эппл и МС насколько я понимаю. Но API почему-то работают немножко по-разному между Хром / Эдж и Сафари.
Re[18]: Не люблю ревьюить чужой код
От: RedUser Россия  
Дата: 05.06.23 04:55
Оценка:
Аё>Да, это совметная работа Гугл, Эппл и МС насколько я понимаю. Но API почему-то работают немножко по-разному между Хром / Эдж и Сафари.

Google сделал свой форк (и Edge его использует):
https://ru.wikipedia.org/wiki/WebKit#Форк_движка_(Blink)
Отредактировано 05.06.2023 4:57 RedUser . Предыдущая версия .
Re[18]: 31 летних программистов съедает дракон
От: Pauel Беларусь http://blogs.rsdn.org/ikemefula
Дата: 05.06.23 05:33
Оценка:
Здравствуйте, CreatorCray, Вы писали:

P>>Из моих студентов уже слишком много нелинейных

CC>Что такое "линейный"? Механический кодер, без малейшего просвета своих мыслей?

До сеньора включительно. Это сотруднтки, которые выполняют основную работу, коей под 80-90%

CC>Это и архитект и частично менеджер в одном флаконе.


Архитектор и менеджер в одном флаконе это точно выше сеньора.
Re[16]: Не люблю ревьюить чужой код
От: CreatorCray  
Дата: 05.06.23 08:08
Оценка: +1 :)
Здравствуйте, Артём, Вы писали:

Аё>А ты, когда про рокет саенс сипипи команд написал

Артёмка, спикай на каком нить уж одном езыге
А то твои экзесисы нихрена не компренде
... << RSDN@Home 1.3.110 alpha 5 rev. 62>>
Забанили по IP, значит пора закрыть эту страницу.
Всем пока
Re[19]: Не люблю ревьюить чужой код
От: Артём Австралия жж
Дата: 05.06.23 10:43
Оценка:
Здравствуйте, RedUser, Вы писали:

Аё>>Да, это совметная работа Гугл, Эппл и МС насколько я понимаю. Но API почему-то работают немножко по-разному между Хром / Эдж и Сафари.


RU>Google сделал свой форк (и Edge его использует):

RU>https://ru.wikipedia.org/wiki/WebKit#Форк_движка_(Blink)

Blink — свободный движок для отображения веб-страниц, разработанный американской компанией Google Inc. на основе кода WebCore из WebKit для браузера Chromium.


Сафари не использует Хромиум? Ну м.б. в этом причина странных отличий Сафари.
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.