{"id":504,"date":"2006-09-19T21:21:43","date_gmt":"2006-09-19T13:21:43","guid":{"rendered":"http:\/\/www.zhangling.org\/blog\/20060919\/code-review-static-analysis.html"},"modified":"2008-04-25T00:47:53","modified_gmt":"2008-04-24T16:47:53","slug":"code-review-static-analysis","status":"publish","type":"post","link":"https:\/\/www.zhangling.org\/blog\/20060919\/code-review-static-analysis.html","title":{"rendered":"\u6211\u5bf9code review\u7684\u51e0\u70b9\u7406\u89e3"},"content":{"rendered":"<p>Code Review\u5c31\u662f\u901a\u8fc7\u5ba1\u67e5\u4ee3\u7801\uff0c\u53d1\u73b0\u4ee3\u7801\u4e2d\u53ef\u80fd\u5b58\u5728\u7684\u95ee\u9898\u5e76\u7ed9\u4e88\u7ea0\u6b63\uff0c\u8fd9\u4e9b\u95ee\u9898\u53ef\u80fd\u5305\u62ec\u8bbe\u8ba1\u4e0a\u7684\u3001\u5b9e\u73b0\u4e0a\u7684\u6216\u8005\u7f16\u7a0b\u98ce\u683c\u7b49\u591a\u65b9\u9762\u3002\u76ee\u524d\u505acode review\u4e00\u662f\u9760\u4eba\u8bfb\u4ee3\u7801\uff0c\u4e8c\u662f\u501f\u52a9\u4e8e\u4e00\u4e9b\u5de5\u5177\u3002\u4eba\u5de5\u9605\u8bfb\u6548\u7387\u6162\uff0c\u4f46\u6709\u6df1\u5ea6\uff0c\u53ef\u4ee5\u89e3\u51b3\u8bbe\u8ba1\u4e0a\u7684\u95ee\u9898\uff1b\u5de5\u5177\u5229\u7528\u6a21\u5f0f\u5339\u914d\u505a\u9759\u6001\u5206\u6790\u627e\u5230\u4e0d\u5408\u9002\u51fd\u6570\u8c03\u7528\uff0c\u901f\u5ea6\u5feb\u4f46\u7f3a\u4e4fcontext\u65e0\u6cd5\u8fdb\u884c\u6df1\u5c42\u5206\u6790\u3002\u76ee\u524dRational\u7684\u201cCode Review&#8221;\u63d2\u4ef6\u63d0\u4f9b\u4e86\u4e0a\u767e\u6761\u89c4\u5219\u5bf9Java\u4ee3\u7801\u8fdb\u884c\u5206\u6790\u3002<\/p>\n<p>\u6709\u5173\u4eba\u5de5review\u548c\u673a\u5668review\u7684\u5173\u7cfb\uff0c\u53ef\u4ee5\u8ba8\u8bba\u597d\u51e0\u5929\u3002\u4e24\u8005\u76f8\u540c\u4e4b\u5904\u90fd\u662f\u53d1\u73b0\u95ee\u9898\uff0c\u4e0d\u540c\u4e4b\u5904\u662f\u4eba\u5de5review\u4fa7\u91cd\u5185\u5728\u903b\u8f91\u548c\u5b9e\u73b0\u65b9\u6cd5\uff0c\u907f\u514d\u6846\u67b6\u6027\u8d28\u7684\u9519\u8bef\uff08\u5b8f\u89c2\uff0b\u90e8\u5206\u5fae\u89c2\uff09\uff1b\u673a\u5668review\u5173\u6ce8\u7ec6\u8282\u3001\u53ef\u91cd\u73b0\uff0c\u5bf9\u4e8e\u5178\u578b\u9519\u8bef\u7684\u5904\u7406\u5f88\u6709\u5e2e\u52a9\uff08\u5fae\u89c2\uff09\u3002<\/p>\n<p>\u6211\u559c\u6b22\u7528\u73b0\u5b9e\u751f\u6d3b\u7684\u4f8b\u5b50\u6765\u89e3\u91ca\u6280\u672f\uff0c\u5c24\u5176\u662f\u5728\u4ece\u53bb\u5e74build\u4e86\u88c5\u4fee\u7684domain knowledge\u540e\u3002\u57fa\u672c\u4e0a\u53ef\u4ee5\u8bf4\uff0c\u4eba\u5de5review\u597d\u6bd4\u623f\u5c4b\u8bbe\u8ba1\u3001\u673a\u5668review\u597d\u6bd4\u9009\u6750\u3002\u9009\u6750\u5f88\u91cd\u8981\uff0c\u88c5\u4fee\u8fc7\u7684\u4eba\u90fd\u77e5\u9053\u5f00\u5173\u9009\u201c\u897f\u8499\u201d\u3001\u201c\u6885\u5170\u65e5\u5170\u201d\uff0c\u6c34\u7ba1\u9009\u201c\u76ae\u5c14\u8428\u201d\u3001\u9f99\u5934\u9009\u201c\u9ad8\u4eea\u201d\uff0c\u6c34\u69fd\u9009\u201c\u7687\u51a0\u201d\u3001\u201c\u5f17\u5170\u5361\u201d\u3001\u6c34\u6ce5\u9009\u201c\u8c61&#8221;\u724c\u3001\u7535\u7ebf\u9009\u201c\u718a\u732b\u201c\u3001\u5237\u5b50\u9009\u201c\u5927\u5e08\u201d&#8230;\u201c\u6ca1\u6709\u597d\u6750\uff0c\u518d\u597d\u7684\u620f\u4e5f\u51fa\u4e0d\u6765\u201d\u3002\u9009\u6750\u5bb9\u6613\u4e4b\u5904\u5728\u4e8e\u4ece\u4e0d\u77e5\u9053\u53d8\u6210\u77e5\u9053\u95e8\u69db\u5f88\u4f4e\uff0c\u4e45\u800c\u4e45\u4e4b\u5c31\u4f1a\u6162\u6162\u719f\u8d77\u6765\uff0c\u53d8\u5f97\u6709\u7ecf\u9a8c\uff0c\u8c61\u6211\u73b0\u5728\u8fd9\u6837\u3002\u8bbe\u8ba1\u5219\u4e0d\u7136\uff0c\u5c31\u662f\u540c\u4e00\u5957\u623f\u5b50\uff0c\u4e24\u6b21\u8bbe\u8ba1\u4e5f\u662f\u4f1a\u6709\u5dee\u522b\uff0c\u800c\u4e14\u6ca1\u6cd5\u5b9a\u91cf\u8bc4\u4ef7 &#8211; \u4eba\u5bf9\u7f8e\u4e0e\u548c\u8c10\u7684\u8ffd\u6c42\u65e0\u6b62\u5883\u3002\u51fa\u597d\u7684\u8bbe\u8ba1\u9700\u8981\u65f6\u95f4\u548c\u7ecf\u9a8c\u78e8\u7ec3\uff0c\u4e0e\u4e4b\u76f8\u5173\u7684\u77e5\u8bc6\u7684\u4f20\u9012\u6bd4\u8d77checklist\u5f62\u5f0f\u7684\u9759\u6001\u5206\u6790\u89c4\u5219\u66f4\u96be\u3002\u540c\u6837\uff0c\u6ca1\u6709\u597d\u7684\u8bbe\u8ba1\uff08\u8b6c\u5982\u5728\u9a6c\u6876\u5bf9\u9762\u7684\u5899\u4e0a\u9576\u4e00\u9762\u955c\u5b50\u4f1a\u600e\u4e48\u6837\uff1f\u8fd9\u662f\u88c5\u4fee\u91cc\u5e38\u89c1\u7684\u4e00\u4e2abad practice\uff09\uff0c\u518d\u597d\u7684\u6750\u6599\u5806\u780c\u5728\u4e00\u8d77\u4e5f\u662f\u65e0\u7528\u7684\u3002<\/p>\n<p>\u6ca1\u6709\u7ecf\u8fc7\u9759\u6001\u5206\u6790\u7684\u7a0b\u5e8f\u53ef\u80fd\u8dd1\u8d77\u6765\u4e0d\u9519\uff0c\u800c\u5b9e\u9645\u4e0a\u53ef\u80fd\u50cf\u4e00\u5757\u5976\u916a\u4e00\u6837\uff0c\u6ee1\u8eab\u662f\u6f0f\u6d1e\uff0c\u95ee\u9898\u4e00\u51fa\u73b0\u5c31\u5f88\u4e25\u91cd\u800c\u4e14\u76f4\u63a5\u9762\u5bf9\u95ee\u9898\u7684\u662f\u53ef\u80fd\u662f\u6700\u7ec8\u7528\u6237\uff1b\u6ca1\u6709\u7ecf\u8fc7\u4eba\u5de5review\u7684\u4ee3\u7801\u8dd1\u8d77\u6765\u4e5f\u53ef\u80fd\u4e0d\u9519\uff0c\u53ea\u662f\u6f5c\u5728\u7684\u95ee\u9898\uff0c\u5982\u6027\u80fd\u3001\u6269\u5c55\u6027\u80fd\u4f1a\u5728\u9700\u6c42\u53d8\u66f4\u7684\u65f6\u5019\u663e\u73b0\u51fa\u6765\uff0c\u76f4\u63a5\u9762\u5bf9\u9ebb\u70e6\u7684\u53ef\u80fd\u662f\u7a0b\u5e8f\u5458\u3002<\/p>\n<p>\u4e0a\u5468\u9762\u8bd5\u5b9e\u4e60\u751f\u7684\u65f6\u5019\uff0c\u6211\u95ee\u4e86\u4e00\u4e2a\u95ee\u9898\uff0c.NET\u5e73\u53f0\u4e0a\u5b58\u5728\u591a\u79cd\u8bed\u8a00\uff0c\u5982\u679c\u4f60\u6765\u8bbe\u8ba1\u4e00\u4e2a\u9759\u6001\u5206\u6790\u5de5\u5177\u7684\u8bdd\uff0c\u8be5\u600e\u4e48\u505a\uff0c\u9700\u8981\u8003\u8651\u54ea\u4e9b\u56e0\u7d20\uff1f\uff08\u53c2\u8003\uff1a<a href=\"http:\/\/www.gotdotnet.com\/team\/FxCop\/\">FxCop<\/a>\uff09 \u5176\u5b9e\u73b0\u5728\u7684Java\u4e5f\u5f00\u59cb\u652f\u6301\u5176\u4ed6\u811a\u672c\u8bed\u8a00\uff0c\u4e3a\u4e86\u9002\u5e94JVM\u4e4b\u4e0a\u7684\u591a\u79cd\u8bed\u8a00\uff0c\u672a\u6765\u57fa\u4e8eJava\u5e73\u53f0\u7684\u4ee3\u7801\u5206\u6790\u5de5\u5177\u4e5f\u4f1a\u4ece\u6e90\u4ee3\u7801\u8f6c\u5411bytecode\u7684\u5206\u6790\uff0c\u90a3\u662fJVM\u552f\u4e00\u80fd\u542c\u61c2\u7684\u8bed\u8a00\u3002\n<\/p>\n<p><!--6d93e37850c57479e974855059e427d0-->\n<\/p>\n<p><!--65756d0bf77763da8387aa509b3ee0fe--><\/p>\n","protected":false},"excerpt":{"rendered":"<p>Code Review\u5c31\u662f\u901a\u8fc7\u5ba1\u67e5\u4ee3\u7801\uff0c\u53d1\u73b0\u4ee3\u7801\u4e2d\u53ef\u80fd\u5b58\u5728\u7684\u95ee\u9898\u5e76\u7ed9\u4e88\u7ea0\u6b63\uff0c\u8fd9\u4e9b\u95ee\u9898\u53ef\u80fd\u5305\u62ec\u8bbe\u8ba1\u4e0a\u7684\u3001\u5b9e\u73b0\u4e0a\u7684\u6216\u8005\u7f16\u7a0b\u98ce\u683c\u7b49\u591a\u65b9\u9762\u3002\u76ee\u524d\u505acode review\u4e00\u662f\u9760\u4eba\u8bfb\u4ee3\u7801\uff0c\u4e8c\u662f\u501f\u52a9\u4e8e\u4e00\u4e9b\u5de5\u5177\u3002\u4eba\u5de5\u9605\u8bfb\u6548\u7387\u6162\uff0c\u4f46\u6709\u6df1\u5ea6\uff0c\u53ef\u4ee5\u89e3\u51b3\u8bbe\u8ba1\u4e0a\u7684\u95ee\u9898\uff1b\u5de5\u5177\u5229\u7528\u6a21\u5f0f\u5339\u914d\u505a\u9759\u6001\u5206\u6790\u627e\u5230\u4e0d\u5408\u9002\u51fd\u6570\u8c03\u7528\uff0c\u901f\u5ea6\u5feb\u4f46\u7f3a\u4e4fcontext\u65e0\u6cd5\u8fdb\u884c\u6df1\u5c42\u5206\u6790\u3002\u76ee\u524dRational\u7684\u201cCode Review&#8221;\u63d2\u4ef6\u63d0\u4f9b\u4e86\u4e0a\u767e\u6761\u89c4\u5219\u5bf9Java\u4ee3\u7801\u8fdb\u884c\u5206\u6790\u3002 \u6709\u5173\u4eba\u5de5review\u548c\u673a\u5668review\u7684\u5173\u7cfb\uff0c\u53ef\u4ee5\u8ba8\u8bba\u597d\u51e0\u5929\u3002\u4e24\u8005\u76f8\u540c\u4e4b\u5904\u90fd\u662f\u53d1\u73b0\u95ee\u9898\uff0c\u4e0d\u540c\u4e4b\u5904\u662f\u4eba\u5de5review\u4fa7\u91cd\u5185\u5728\u903b\u8f91\u548c\u5b9e\u73b0\u65b9\u6cd5\uff0c\u907f\u514d\u6846\u67b6\u6027\u8d28\u7684\u9519\u8bef\uff08\u5b8f\u89c2\uff0b\u90e8\u5206\u5fae\u89c2\uff09\uff1b\u673a\u5668review\u5173\u6ce8\u7ec6\u8282\u3001\u53ef\u91cd\u73b0\uff0c\u5bf9\u4e8e\u5178\u578b\u9519\u8bef\u7684\u5904\u7406\u5f88\u6709\u5e2e\u52a9\uff08\u5fae\u89c2\uff09\u3002 \u6211\u559c\u6b22\u7528\u73b0\u5b9e\u751f\u6d3b\u7684\u4f8b\u5b50\u6765\u89e3\u91ca\u6280\u672f\uff0c\u5c24\u5176\u662f\u5728\u4ece\u53bb\u5e74build\u4e86\u88c5\u4fee\u7684domain knowledge\u540e\u3002\u57fa\u672c\u4e0a\u53ef\u4ee5\u8bf4\uff0c\u4eba\u5de5review\u597d\u6bd4\u623f\u5c4b\u8bbe\u8ba1\u3001\u673a\u5668review\u597d\u6bd4\u9009\u6750\u3002\u9009\u6750\u5f88\u91cd\u8981\uff0c\u88c5\u4fee\u8fc7\u7684\u4eba\u90fd\u77e5\u9053\u5f00\u5173\u9009\u201c\u897f\u8499\u201d\u3001\u201c\u6885\u5170\u65e5\u5170\u201d\uff0c\u6c34\u7ba1\u9009\u201c\u76ae\u5c14\u8428\u201d\u3001\u9f99\u5934\u9009\u201c\u9ad8\u4eea\u201d\uff0c\u6c34\u69fd\u9009\u201c\u7687\u51a0\u201d\u3001\u201c\u5f17\u5170\u5361\u201d\u3001\u6c34\u6ce5\u9009\u201c\u8c61&#8221;\u724c\u3001\u7535\u7ebf\u9009\u201c\u718a\u732b\u201c\u3001\u5237\u5b50\u9009\u201c\u5927\u5e08\u201d&#8230;\u201c\u6ca1\u6709\u597d\u6750\uff0c\u518d\u597d\u7684\u620f\u4e5f\u51fa\u4e0d\u6765\u201d\u3002\u9009\u6750\u5bb9\u6613\u4e4b\u5904\u5728\u4e8e\u4ece\u4e0d\u77e5\u9053\u53d8\u6210\u77e5\u9053\u95e8\u69db\u5f88\u4f4e\uff0c\u4e45\u800c\u4e45\u4e4b\u5c31\u4f1a\u6162\u6162\u719f\u8d77\u6765\uff0c\u53d8\u5f97\u6709\u7ecf\u9a8c\uff0c\u8c61\u6211\u73b0\u5728\u8fd9\u6837\u3002\u8bbe\u8ba1\u5219\u4e0d\u7136\uff0c\u5c31\u662f\u540c\u4e00\u5957\u623f\u5b50\uff0c\u4e24\u6b21\u8bbe\u8ba1\u4e5f\u662f\u4f1a\u6709\u5dee\u522b\uff0c\u800c\u4e14\u6ca1\u6cd5\u5b9a\u91cf\u8bc4\u4ef7 &#8211; \u4eba\u5bf9\u7f8e\u4e0e\u548c\u8c10\u7684\u8ffd\u6c42\u65e0\u6b62\u5883\u3002\u51fa\u597d\u7684\u8bbe\u8ba1\u9700\u8981\u65f6\u95f4\u548c\u7ecf\u9a8c\u78e8\u7ec3\uff0c\u4e0e\u4e4b\u76f8\u5173\u7684\u77e5\u8bc6\u7684\u4f20\u9012\u6bd4\u8d77checklist\u5f62\u5f0f\u7684\u9759\u6001\u5206\u6790\u89c4\u5219\u66f4\u96be\u3002\u540c\u6837\uff0c\u6ca1\u6709\u597d\u7684\u8bbe\u8ba1\uff08\u8b6c\u5982\u5728\u9a6c\u6876\u5bf9\u9762\u7684\u5899\u4e0a\u9576\u4e00\u9762\u955c\u5b50\u4f1a\u600e\u4e48\u6837\uff1f\u8fd9\u662f\u88c5\u4fee\u91cc\u5e38\u89c1\u7684\u4e00\u4e2abad practice\uff09\uff0c\u518d\u597d\u7684\u6750\u6599\u5806\u780c\u5728\u4e00\u8d77\u4e5f\u662f\u65e0\u7528\u7684\u3002 \u6ca1\u6709\u7ecf\u8fc7\u9759\u6001\u5206\u6790\u7684\u7a0b\u5e8f\u53ef\u80fd\u8dd1\u8d77\u6765\u4e0d\u9519\uff0c\u800c\u5b9e\u9645\u4e0a\u53ef\u80fd\u50cf\u4e00\u5757\u5976\u916a\u4e00\u6837\uff0c\u6ee1\u8eab\u662f\u6f0f\u6d1e\uff0c\u95ee\u9898\u4e00\u51fa\u73b0\u5c31\u5f88\u4e25\u91cd\u800c\u4e14\u76f4\u63a5\u9762\u5bf9\u95ee\u9898\u7684\u662f\u53ef\u80fd\u662f\u6700\u7ec8\u7528\u6237\uff1b\u6ca1\u6709\u7ecf\u8fc7\u4eba\u5de5review\u7684\u4ee3\u7801\u8dd1\u8d77\u6765\u4e5f\u53ef\u80fd\u4e0d\u9519\uff0c\u53ea\u662f\u6f5c\u5728\u7684\u95ee\u9898\uff0c\u5982\u6027\u80fd\u3001\u6269\u5c55\u6027\u80fd\u4f1a\u5728\u9700\u6c42\u53d8\u66f4\u7684\u65f6\u5019\u663e\u73b0\u51fa\u6765\uff0c\u76f4\u63a5\u9762\u5bf9\u9ebb\u70e6\u7684\u53ef\u80fd\u662f\u7a0b\u5e8f\u5458\u3002 \u4e0a\u5468\u9762\u8bd5\u5b9e\u4e60\u751f\u7684\u65f6\u5019\uff0c\u6211\u95ee\u4e86\u4e00\u4e2a\u95ee\u9898\uff0c.NET\u5e73\u53f0\u4e0a\u5b58\u5728\u591a\u79cd\u8bed\u8a00\uff0c\u5982\u679c\u4f60\u6765\u8bbe\u8ba1\u4e00\u4e2a\u9759\u6001\u5206\u6790\u5de5\u5177\u7684\u8bdd\uff0c\u8be5\u600e\u4e48\u505a\uff0c\u9700\u8981\u8003\u8651\u54ea\u4e9b\u56e0\u7d20\uff1f\uff08\u53c2\u8003\uff1aFxCop\uff09 \u5176\u5b9e\u73b0\u5728\u7684Java\u4e5f\u5f00\u59cb\u652f\u6301\u5176\u4ed6\u811a\u672c\u8bed\u8a00\uff0c\u4e3a\u4e86\u9002\u5e94JVM\u4e4b\u4e0a\u7684\u591a\u79cd\u8bed\u8a00\uff0c\u672a\u6765\u57fa\u4e8eJava\u5e73\u53f0\u7684\u4ee3\u7801\u5206\u6790\u5de5\u5177\u4e5f\u4f1a\u4ece\u6e90\u4ee3\u7801\u8f6c\u5411bytecode\u7684\u5206\u6790\uff0c\u90a3\u662fJVM\u552f\u4e00\u80fd\u542c\u61c2\u7684\u8bed\u8a00\u3002<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[5],"tags":[],"class_list":["post-504","post","type-post","status-publish","format-standard","hentry","category-tech"],"_links":{"self":[{"href":"https:\/\/www.zhangling.org\/blog\/wp-json\/wp\/v2\/posts\/504","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.zhangling.org\/blog\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/www.zhangling.org\/blog\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/www.zhangling.org\/blog\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/www.zhangling.org\/blog\/wp-json\/wp\/v2\/comments?post=504"}],"version-history":[{"count":0,"href":"https:\/\/www.zhangling.org\/blog\/wp-json\/wp\/v2\/posts\/504\/revisions"}],"wp:attachment":[{"href":"https:\/\/www.zhangling.org\/blog\/wp-json\/wp\/v2\/media?parent=504"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.zhangling.org\/blog\/wp-json\/wp\/v2\/categories?post=504"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.zhangling.org\/blog\/wp-json\/wp\/v2\/tags?post=504"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}