Wrong confirmation ID

Email
Favorite
Favorited ×
Download
Embed
Private Content
Copy and paste this code into your blog or website Copy Customize… Close
We have emailed the verification/download link to "".
Login to your email and click the link to download the file directly.

To request the link at a different email address, update it here. Close
Validation messages. Success message. Fail message.

Check your bulk/spam folders if you can't find our mail.

+ Follow

Code review psyhology

by Oleksiy Rezchykov on Dec 17, 2011

1,530 views

My XPDays presentation

More…

My XPDays presentation

Less

2 Embeds 872

http://mcgray.com.ua	492
http://xpdays.com.ua	380

Statistics

Favorites: 2
Downloads: 1
Comments: 3
Embed Views: 872
Views on SlideShare: 658
Total Views: 1,530

1–3 of 3 previous next Post a comment

Oleksiy Rezchykov , Java Senior Software Engineer at Lohika Systems xpdays.com.ua/materials/code-review/ 4 months ago Reply
Are you sure you want to Yes No
Oleksiy Rezchykov , Java Senior Software Engineer at Lohika Systems Со дня на день будет на сайте конференции. 4 months ago Reply
Are you sure you want to Yes No
Volodymyr Chykhman Привет Алексей.
Не могли бы вы выложить видео запись вашего выступления на XP days на тему 'Code review psyhology'
Спасибо. 4 months ago Reply
Are you sure you want to Yes No

ISO 9001: Не гарантирует качество продукта на выходе, а только стандартизирует процесс
ISO 9001: Не гарантирует качество продукта на выходе, а только стандартизирует процесс
У нас ввели code-review -- А это как? -- Ну как, сидишь, читаешь код, ревёшь...
Sgt drill instructor
Консенсусопоранапрошлыйпроектилипростона Sun вслучае Java
Сложноубежитьлюдейнокаждыйздравыйчеловекможетпонятьпреимещустватакогоподходадлянего лично
визуализация“И кем бы ты ни был, чтоб ты не делал между зимой и летом - весна”
Sgt drill instructor
Static code analyzers CI
"Руками"CrucibleGerrit
Рекомендация - презентацияАлименков+Солнцев (виситунаснавнутреннемресурседляновыхкоманд)
Защитныйрефлекс, почему? Обижен, боитсякритики?Гуру? Значит collective code ownership - fail. Этообщийкод. Завтраэтоткодможетпереписатьсянетобой. Завтраегомогут "убрать". Надопониматьчтоделаеткод. Взамодействиенужнотестировать, регрессиюнужнотестировать
Защитныйрефлекс, почему? Обижен, боитсякритики?Гуру? Значит collective code ownership - fail. Этообщийкод. Завтраэтоткодможетпереписатьсянетобой. Завтраегомогут "убрать". Надопониматьчтоделаеткод. Взамодействиенужнотестировать, регрессиюнужнотестировать
Защитныйрефлекс, почему? Обижен, боитсякритики?Гуру? Значит collective code ownership - fail. Этообщийкод. Завтраэтоткодможетпереписатьсянетобой. Завтраегомогут "убрать". Надопониматьчтоделаеткод. Взамодействиенужнотестировать, регрессиюнужнотестировать
Защитныйрефлекс, почему? Обижен, боитсякритики?Гуру? Значит collective code ownership - fail. Этообщийкод. Завтраэтоткодможетпереписатьсянетобой. Завтраегомогут "убрать". Надопониматьчтоделаеткод. Взамодействиенужнотестировать, регрессиюнужнотестировать
Небываетпотом!!! КакуБулгакова "перваяипоследняясвежесть"
Небываетпотом!!! КакуБулгакова "перваяипоследняясвежесть"
Небываетпотом!!! КакуБулгакова "перваяипоследняясвежесть"
Небываетпотом!!! КакуБулгакова "перваяипоследняясвежесть"
Чтоговорятпропреждевременнуюоптимизацию... ОпирайтесьназдравыйсмыслВсечтосложноскореевсегораньшесломается, егосложнотестировать.
Чтоговорятпропреждевременнуюоптимизацию... ОпирайтесьназдравыйсмыслВсечтосложноскореевсегораньшесломается, егосложнотестировать.
Чтоговорятпропреждевременнуюоптимизацию... ОпирайтесьназдравыйсмыслВсечтосложноскореевсегораньшесломается, егосложнотестировать.
Чтоговорятпропреждевременнуюоптимизацию... ОпирайтесьназдравыйсмыслВсечтосложноскореевсегораньшесломается, егосложнотестировать.
Нехамить!!! Авторитет, субординация, хотьиформальная, людинасвоихместах.Такогонельзядопускать, еслитимлидневсостояниисправитьсястакойситуацией - еедолженразрулить ПМ. Ноглавное - профессионалыниприкакихобстоятельствахнепереходятналичности - унихвсегдаестьдругиеаргументы.
Нехамить!!! Авторитет, субординация, хотьиформальная, людинасвоихместах.Такогонельзядопускать, еслитимлидневсостояниисправитьсястакойситуацией - еедолженразрулить ПМ. Ноглавное - профессионалыниприкакихобстоятельствахнепереходятналичности - унихвсегдаестьдругиеаргументы.
Нехамить!!! Авторитет, субординация, хотьиформальная, людинасвоихместах.Такогонельзядопускать, еслитимлидневсостояниисправитьсястакойситуацией - еедолженразрулить ПМ. Ноглавное - профессионалыниприкакихобстоятельствахнепереходятналичности - унихвсегдаестьдругиеаргументы.

2 Favorites

Ivan Verhun 5 months ago
Elzenar Tags code review 5 months ago

Code review psyhology — Presentation Transcript

Психология Code Review иличто делать, если в вашей команде есть люди, которые "пишут код без багов" Алексей Резчиков 2011
Обо мне10 лет программирую заденьги4 года тим лидМного шишек
Просьба к аудитории Задавайте вопросы!
О чем я НЕ буду говорить сегодня что такое плохой код что такое хороший код как писать хороший код как поставлять качественный продукт почему надо проводить Code Review
О чем мы сегодня поговоримПроблемы связанные с CodeReviewПрактические рекомендаций как ихпреодолетьИстории из реальной жизни
Что такое Code Review?Code review is systematic examination (often aspeer review) of computer source code. It isintended to find and fix mistakes overlooked in theinitial development phase, improving both theoverall quality of software and the developersskills. Reviews are done in various forms such aspair programming, informal walkthroughs, andformal inspections.(Wikipedia)
Проблема №1
Шаг №2
Шаг №3 Стандарты кодирования Правила именования Стратегия развития проекта
Шаг №5Совместное владениекодом
Шаг №6 Code Implementation Review Deploy Test
Проблема №2
Автоматизируйте Redmine
ИнструментыgerritGerrit Code Review
Подходы к организацииИх очень многоВыбор за вамиБудьте гибкими
Определяем понятияРазделяйте Code Review и Solution Review
Не пишите код!!!
Немного практики
Ситуация №1Я пишу код без багов
Чем плохо? Совместное владение кодом не работает Подрывает командную работу Проблемы с тестированием
Что делать? Понять причину Объяснить что это миф «Вернуть в семью»
Ситуация №2Код должен быть среднего качестваили я отрефакторю потом
Чем плохо? Падение дисциплины Принцип разбитого окна
Что можно сделать? Показать что не бывает «потом» Бывает или хорошо или плохо
Ситуация №3Тут надо все отрефакторить!
Чем плохо? Преждевременная оптимизация Избыточная сложность
Что можно сделать? Примеры преждевременной оптимизации Избыточная сложность и принципы KISS иYAGNI То, что просто – легко тестируется
Ситуация №4"Тимлид не умеет писать транзакции"
Чем плохо? Хамство Нарушение субординации Плохой прецедент для командной работы
Что можно сделать? «Хирургическое вмешательство» Авторитет и субординация Арбитраж
Общие советыОбщие правила на видномместе, понятные всем, с которыми всесогласныОценивайте код, а не человекаОперируйте фактами, а не суждениямиСтарайтесь быть предельно точнымиЧеткий и прозрачный арбитраж
Вопросы@twincengray alex.mcgray@gmail.com http://mcgray.com.ua

Code review psyhology

by Oleksiy Rezchykov on Dec 17, 2011

Accessibility

Categories

Tags

Upload Details

Usage Rights

2 Embeds 872

Statistics

Code review psyhology — Presentation Transcript