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

bnk>Спокойствие, только спокойствие
+100500

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

bnk>Упрощение кода, обкладывание автоматическими тестами, автоматические commit policy
bnk>навроде "функция не больше экрана", "код не коммитится без ссылки тикет".
bnk>Постараться чтобы всякая "хрень" не разрасталось, ну или по крайней мере, не так быстро.
Это действие — когда твой авторитет в коллективе вырастет — примерно через 5-6 месяцев работы над проектом (где-то раньше, где-то и позже).

bnk>Нужно поднимать общий уровень команды, почаще проводить код ревью, обсуждения проекта вообще,
bnk>посылать людей на курсы, ну и т.п.
+100500
Это верно, но здесь также следует действовать без фанатизма
Сразу же ломать устоявшуюся систему — не нужно.

bnk>А вообще, сложно это.. даже не то чтобы даже просто переписать, а переписать так,
bnk>чтобы "апач" твой код понял, почувствовал приемущества твоего подхода для себя, и смог этот код развивать.
Для этого, нужно не просто предоставить этот подход, но и суметь "на пальцах" разъяснить этому "апачу" все его приемущества...
Здесь есть такой момент: пока сам "чуешь нутром" приемущества твоего подхода, но не в состояниии (по каким-либо причинам)
разъяснить это коллегам из твоей команды — подожди, созрей сам, и тогда — действуй!

bnk>И поскольку это "КМК", то еще желательно переписывать так, чтобы твои шефы/коллеги увидели в этом переписывании не наезд,
bnk>а возможность роста, конечно же если хочешь задержаться или расти в этой компании. В общем думать о людях.
+100500

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

bnk>Спокойствие, только спокойствие
+100500

bnk>Главное, никаких революций, иначе сам же можешь пасть их жертвой. А вот постепенное улучшение работает.
Это действие верно в любом варианте, но только для "начального этапа" (aka "испытательный срок" порядка трёх месяцев).

bnk>Упрощение кода, обкладывание автоматическими тестами, автоматические commit policy
bnk>навроде "функция не больше экрана", "код не коммитится без ссылки тикет".
bnk>Постараться чтобы всякая "хрень" не разрасталось, ну или по крайней мере, не так быстро.
Это действие — когда твой авторитет в коллективе вырастет — примерно через 5-6 месяцев работы над проектом (где-то раньше, где-то и позже).

bnk>Нужно поднимать общий уровень команды, почаще проводить код ревью, обсуждения проекта вообще,
bnk>посылать людей на курсы, ну и т.п.
+100500
Это верно, но здесь также следует действовать без фанатизма
Сразу же ломать устоявшуюся систему — не нужно.

bnk>А вообще, сложно это.. даже не то чтобы даже просто переписать, а переписать так,
bnk>чтобы "апач" твой код понял, почувствовал приемущества твоего подхода для себя, и смог этот код развивать.
Для этого, нужно не просто предоставить этот подход, но и суметь "на пальцах" разъяснить этому "апачу" все его приемущества...
Здесь есть такой момент: пока сам "чуешь нутром" приемущества твоего подхода, но не в состояниии (по каким-либо причинам)
разъяснить это коллегам из твоей команды — подожди, созрей сам, и тогда — действуй!

bnk>И поскольку это "КМК", то еще желательно переписывать так, чтобы твои шефы/коллеги увидели в этом переписывании не наезд,
bnk>а возможность роста, конечно же если хочешь задержаться или расти в этой компании. В общем думать о людях.
+100500