Diese Präsentation wurde erfolgreich gemeldet.
Wir verwenden Ihre LinkedIn Profilangaben und Informationen zu Ihren Aktivitäten, um Anzeigen zu personalisieren und Ihnen relevantere Inhalte anzuzeigen. Sie können Ihre Anzeigeneinstellungen jederzeit ändern.

Code review psyhology

3.396 Aufrufe

Veröffentlicht am

My XPDays presentation

Veröffentlicht in: Technologie

Code review psyhology

  1. 1. Психология Code Review иличто делать, если в вашей команде есть люди, которые "пишут код без багов" Алексей Резчиков 2011
  2. 2. Обо мне10 лет программирую заденьги4 года тим лидМного шишек
  3. 3. Просьба к аудитории Задавайте вопросы!
  4. 4. О чем я НЕ буду говорить сегодня что такое плохой код что такое хороший код как писать хороший код как поставлять качественный продукт почему надо проводить Code Review
  5. 5. О чем мы сегодня поговоримПроблемы связанные с CodeReviewПрактические рекомендаций как ихпреодолетьИстории из реальной жизни
  6. 6. Что такое 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)
  7. 7. Проблема №1
  8. 8. Шаг №2
  9. 9. Шаг №3 Стандарты кодирования Правила именования Стратегия развития проекта
  10. 10. Шаг №5Совместное владениекодом
  11. 11. Шаг №6 Code Implementation Review Deploy Test
  12. 12. Проблема №2
  13. 13. Автоматизируйте Redmine
  14. 14. ИнструментыgerritGerrit Code Review
  15. 15. Подходы к организацииИх очень многоВыбор за вамиБудьте гибкими
  16. 16. Определяем понятияРазделяйте Code Review и Solution Review
  17. 17. Не пишите код!!!
  18. 18. Немного практики
  19. 19. Ситуация №1Я пишу код без багов
  20. 20. Чем плохо? Совместное владение кодом не работает Подрывает командную работу Проблемы с тестированием
  21. 21. Что делать? Понять причину Объяснить что это миф «Вернуть в семью»
  22. 22. Ситуация №2Код должен быть среднего качестваили я отрефакторю потом
  23. 23. Чем плохо? Падение дисциплины Принцип разбитого окна
  24. 24. Что можно сделать? Показать что не бывает «потом» Бывает или хорошо или плохо
  25. 25. Ситуация №3Тут надо все отрефакторить!
  26. 26. Чем плохо? Преждевременная оптимизация Избыточная сложность
  27. 27. Что можно сделать? Примеры преждевременной оптимизации Избыточная сложность и принципы KISS иYAGNI То, что просто – легко тестируется
  28. 28. Ситуация №4"Тимлид не умеет писать транзакции"
  29. 29. Чем плохо? Хамство Нарушение субординации Плохой прецедент для командной работы
  30. 30. Что можно сделать? «Хирургическое вмешательство» Авторитет и субординация Арбитраж
  31. 31. Общие советыОбщие правила на видномместе, понятные всем, с которыми всесогласныОценивайте код, а не человекаОперируйте фактами, а не суждениямиСтарайтесь быть предельно точнымиЧеткий и прозрачный арбитраж
  32. 32. Вопросы@twincengray alex.mcgray@gmail.com http://mcgray.com.ua

×