My XPDays presentation

1. Психология Code Review
или
что делать, если в вашей команде есть люди,
которые "пишут код без багов"
Алексей Резчиков
2011

2. Обо мне
10 лет программирую за
деньги
4 года тим лид
Много шишек

3. Просьба к аудитории
Задавайте вопросы!

4. О чем я НЕ буду говорить сегодня
что такое плохой код
что такое хороший код
как писать хороший код
как поставлять качественный продукт
почему надо проводить Code Review

5. О чем мы сегодня поговорим
Проблемы связанные с CodeReview
Практические рекомендаций как их
преодолеть
Истории из реальной жизни

6. Что такое Code Review?
Code review is systematic examination (often as
peer review) of computer source code. It is
intended to find and fix mistakes overlooked in the
initial development phase, improving both the
overall quality of software and the developers'
skills. Reviews are done in various forms such as
pair programming, informal walkthroughs, and
formal inspections.
(Wikipedia)

7. Проблема №1

9. Шаг №2

11. Шаг №3
Стандарты кодирования
Правила именования
Стратегия развития проекта

13. Шаг №5
Совместное владение
кодом

14. Шаг №6
Code
Implementation
Review
Deploy Test

15. Проблема №2

16. Автоматизируйте
Redmine

17. Инструменты
gerrit
Gerrit Code Review

18. Подходы к организации
Их очень много
Выбор за вами
Будьте гибкими

19. Определяем понятия
Разделяйте Code Review и Solution Review

20. Не пишите код!!!

21. Немного практики

23. Ситуация №1
Я пишу код без багов

24. Чем плохо?
Совместное владение кодом не работает
Подрывает командную работу
Проблемы с тестированием

25. Что делать?
Понять причину
Объяснить что это миф
«Вернуть в семью»

27. Ситуация №2
Код должен быть среднего качества
или я отрефакторю потом

28. Чем плохо?
Падение дисциплины
Принцип разбитого окна

29. Что можно сделать?
Показать что не бывает «потом»
Бывает или хорошо или плохо

31. Ситуация №3
Тут надо все отрефакторить!

32. Чем плохо?
Преждевременная оптимизация
Избыточная сложность

33. Что можно сделать?
Примеры преждевременной
оптимизации
Избыточная сложность и принципы
KISS иYAGNI
То, что просто – легко тестируется

34. Ситуация №4
"Тимлид не умеет писать транзакции"

35. Чем плохо?
Хамство
Нарушение субординации
Плохой прецедент для командной
работы

36. Что можно сделать?
«Хирургическое вмешательство»
Авторитет и субординация
Арбитраж

37. Общие советы
Общие правила на видном
месте, понятные всем, с которыми все
согласны
Оценивайте код, а не человека
Оперируйте фактами, а не суждениями
Старайтесь быть предельно точными
Четкий и прозрачный арбитраж

38. Вопросы
@twincengray
alex.mcgray@gmail.com
http://mcgray.com.ua