{"id":8764,"date":"2024-05-27T14:01:01","date_gmt":"2024-05-27T06:01:01","guid":{"rendered":""},"modified":"2024-05-27T14:01:01","modified_gmt":"2024-05-27T06:01:01","slug":"pmml java_\u521d\u5b66\u8005\u600e\u6837\u770b\u61c2\u4ee3\u7801","status":"publish","type":"post","link":"https:\/\/mushiming.com\/8764.html","title":{"rendered":"pmml java_\u521d\u5b66\u8005\u600e\u6837\u770b\u61c2\u4ee3\u7801"},"content":{"rendered":"<p>\u672c\u6587\u7ed3\u5408\u5b9e\u4f8b\uff0c\u8bb2\u8bc9\u5982\u4f55\u5199\u51fa\u7b26\u5408PMD\u7f16\u7a0b\u89c4\u8303\u7684java\u4ee3\u7801\u3002<\/p>\n<p>\u9605\u8bfb\u524d\u9700\u8981\u6709\u4e00\u4e2a\u5171\u8bc6\uff1a<span style=\"color:#f33b45;\">\u4f18\u96c5\u7684\u4ee3\u7801\u4e00\u5b9a\u662f\u6700\u8ba9\u4eba\u559c\u6b22\u7684\u3002<\/span><\/p>\n<p>\u4e0b\u9762\u9644\u4e0a\u4ee3\u7801\u89c4\u8303\u7684pmd.xml\u6587\u4ef6\u7684\u4e0b\u8f7d\u5730\u5740\uff1ahttps:\/\/github.com\/jitongming\/pmdForDev<\/p>\n<hr>\n<p id=\"main-toc\"><strong>\u76ee\u5f55<\/strong><\/p>\n<p id=\"%E7%AC%AC%E4%B8%80%E7%B1%BB%EF%BC%9Abasic-toc\">\u7b2c\u4e00\u7c7b\uff1abasic<\/p>\n<p id=\"%E7%AC%AC%E4%BA%8C%E7%B1%BB%EF%BC%9Aunnecessary-toc\">\u7b2c\u4e8c\u7c7b\uff1aunnecessary<\/p>\n<p id=\"%E7%AC%AC%E4%B8%89%E7%B1%BB%EF%BC%9Abraces-toc\">\u7b2c\u4e09\u7c7b\uff1abraces<\/p>\n<p id=\"%E7%AC%AC%E5%9B%9B%E7%B1%BB%EF%BC%9Acodesize-toc\">\u7b2c\u56db\u7c7b\uff1acodesize<\/p>\n<p id=\"%E7%AC%AC%E4%BA%94%E7%B1%BB%EF%BC%9Acontroversial-toc\">\u7b2c\u4e94\u7c7b\uff1acontroversial<\/p>\n<p id=\"%E7%AC%AC%E5%85%AD%E7%B1%BB%EF%BC%9Acoupling-toc\">\u7b2c\u516d\u7c7b\uff1acoupling<\/p>\n<p id=\"%E7%AC%AC%E4%B8%83%E7%B1%BB%EF%BC%9Adesign-toc\">\u7b2c\u4e03\u7c7b\uff1adesign<\/p>\n<p id=\"%E7%AC%AC%E5%85%AB%E7%B1%BB%EF%BC%9Aempty-toc\">\u7b2c\u516b\u7c7b\uff1aempty<\/p>\n<p id=\"%E7%AC%AC%E4%B9%9D%E7%B1%BB%EF%BC%9Afinalizers-toc\">\u7b2c\u4e5d\u7c7b\uff1afinalizers<\/p>\n<p id=\"%E7%AC%AC%E5%8D%81%E7%B1%BB%EF%BC%9Aimports-toc\">\u7b2c\u5341\u7c7b\uff1aimports<\/p>\n<p id=\"%E7%AC%AC%E5%8D%81%E4%B8%80%E7%B1%BB%EF%BC%9Aj2ee-toc\">\u7b2c\u5341\u4e00\u7c7b\uff1aj2ee<\/p>\n<p id=\"%E7%AC%AC%E5%8D%81%E4%BA%8C%E7%B1%BB%EF%BC%9Alogging-java-toc\">\u7b2c\u5341\u4e8c\u7c7b\uff1alogging-java<\/p>\n<p id=\"%E7%AC%AC%E5%8D%81%E4%B8%89%E7%B1%BB%EF%BC%9Anaming-toc\">\u7b2c\u5341\u4e09\u7c7b\uff1anaming<\/p>\n<p id=\"%E7%AC%AC%E5%8D%81%E5%9B%9B%E7%B1%BB%EF%BC%9Aoptimizations-toc\">\u7b2c\u5341\u56db\u7c7b\uff1aoptimizations<\/p>\n<p id=\"%E7%AC%AC%E5%8D%81%E4%BA%94%E7%B1%BB%EF%BC%9Astrictexception-toc\">\u7b2c\u5341\u4e94\u7c7b\uff1astrictexception<\/p>\n<p id=\"%E7%AC%AC%E5%8D%81%E5%85%AD%E7%B1%BB%EF%BC%9Astrings-toc\">\u7b2c\u5341\u516d\u7c7b\uff1astrings<\/p>\n<p id=\"%E7%AC%AC%E5%8D%81%E4%B8%83%E7%B1%BB%EF%BC%9Aunusedcode-toc\">\u7b2c\u5341\u4e03\u7c7b\uff1aunusedcode<\/p>\n<hr id=\"hr-toc\">\n<p>------------------------------\u6211\u662f\u53ef\u7231\u7684\u5206\u5272\u7ebf-----------------------------------<\/p>\n<h2 id=\"%E7%AC%AC%E4%B8%80%E7%B1%BB%EF%BC%9Abasic\">\u7b2c\u4e00\u7c7b\uff1abasic<\/h2>\n<p><strong>1\uff09ReturnFromFinallyBlock\u3010\u4e0d\u8981\u5728finally\u4ee3\u7801\u5757\u4e2d\u4f7f\u7528return\u3011<\/strong><\/p>\n<ul>\n<li>Avoid returning from a finally block, this can discard exceptions.<\/li>\n<li>\u5f53\u5728finally\u5757\u4e2d\u4f7f\u7528return\u65f6\uff0c\u7f16\u8bd1\u5668\u4e0d\u4f1a\u518d\u5bf9try\u3001catch\u5757\u4e2d\u7684\u975e\u8fd0\u884c\u65f6\u5f02\u5e38\u8fdb\u884c\u68c0\u67e5\uff0cJVM\u4e0d\u4f1a\u518d\u53bb\u6355\u83b7try\u5757\u3001catch\u5757\u4e2d\u7684\u5f02\u5e38\uff0c\u7a0b\u5e8f\u7684\u8f93\u51fa\u4ee5finally\u5757\u4e3a\u51c6\uff0c\u5373finally\u5757\u7684\u8fd4\u56de\u503c\u6216\u8005finally\u5757\u4e2d\u629b\u51fa\u7684\u5f02\u5e38\u3002<\/li>\n<\/ul>\n<pre class=\"has\"><code class=\"language-java\">public void test(){\n    try{\n        \/\/......do something \n    }catch(Exception e){\n        \/\/......do something\n    }finally{\n        return \"\u5728\u8fd9\u8fdb\u884c\u8fd4\u56de\"; \/\/\u907f\u514d\u5728\u8fd9\u91cc\u5199return\n    }\n}<\/code><\/pre>\n<p><strong>2\uff09UnconditionalIfStatement\u3010if\u8bed\u53e5\u5224\u65ad\u6761\u4ef6\uff0c\u4e0d\u80fd\u6c38\u8fdc\u6210\u7acb\uff0c\u5fc5\u5b9a\u662ftrue\u6216false\u3011<\/strong><\/p>\n<ul>\n<li>Do not use \"if\" statements whose conditionals are always true or always false.<\/li>\n<li>\u5f53\u5728if\u4ee3\u7801\u5757\u6267\u884c\u524d\uff0c\u4f1a\u5148\u68c0\u67e5\u5224\u65ad\u6761\u4ef6\u662f\u5426\u6210\u7acb\uff0c\u82e5\u6761\u4ef6\u5fc5\u7136\u6210\u7acb\uff0c\u90a3\u6ca1\u6709\u5fc5\u8981\u7528if\u8bed\u53e5\u3002<\/li>\n<\/ul>\n<pre class=\"has\"><code class=\"language-java\">public void test(){\n    if(0&lt;1){ \/\/\u907f\u514d\u4ee5\u7edd\u5bf9\u6210\u7acb\u7684\u6761\u4ef6\u4f5c\u4e3a\u5224\u65ad\u4f9d\u636e\n        \/\/....do something\n    }\n}<\/code><\/pre>\n<p>&nbsp;<\/p>\n<h2 id=\"%E7%AC%AC%E4%BA%8C%E7%B1%BB%EF%BC%9Aunnecessary\">\u7b2c\u4e8c\u7c7b\uff1aunnecessary<\/h2>\n<p><strong>1\uff09UnnecessaryConversionTemporary\u3010\u4e0d\u8981\u6d6a\u8d39\u4e00\u4e2aobject\u8fdb\u884c\u57fa\u7840\u6570\u503c\u7684\u5b57\u7b26\u4e32\u8f6c\u6362\uff0c\u4f7f\u7528\u5305\u88c5\u7c7b\u76f4\u63a5\u5904\u7406\u3011<\/strong><\/p>\n<ul>\n<li>Avoid the use temporary objects when converting primitives to Strings. Use the static conversion methodson the wrapper classes instead.<\/li>\n<li>\u4e0d\u8981\u7533\u8bf7\u4e00\u4e2a\u4e34\u65f6\u7684\u5bf9\u8c61\u53bb\u5c06\u201c\u57fa\u672c\u7c7b\u578b\u6570\u503c\u201d\u8f6c\u6362\u4e3a\u201c\u5b57\u7b26\u4e32\u201d\uff0c\u76f4\u63a5\u4f7f\u7528\u5305\u88c5\u7c7b\u5904\u7406\u3002\u521b\u5efa\u4e00\u4e2a\u5bf9\u8c61\u7684\u5f00\u9500\u662f\u6bd4\u8f83\u5927\u7684\uff0c\u5c3d\u91cf\u4e0d\u8981\u521b\u5efa\u4e0d\u5fc5\u8981\u7684\u5bf9\u8c61\u3002<\/li>\n<\/ul>\n<pre class=\"has\"><code class=\"language-java\">String a = new Integer(x).toString();\/\/1.\u907f\u514d\u5199\u6cd5\uff1a\u6d6a\u8d39\u4e86\u4e00\u4e2aobject\n\nString a= Integer.toString(x);\/\/2.\u63a8\u8350\u5199\u6cd5<\/code><\/pre>\n<p>&nbsp;<\/p>\n<p><strong>2\uff09UnnecessaryFinalModifier\u3010\u5220\u9664\u4e0d\u5fc5\u8981\u7684final\u4fee\u9970\u7b26\u3011<\/strong><\/p>\n<ul>\n<li>When a class has the final modifier, all the methods are automatically final and do not need to betagged as such.<\/li>\n<li>\u5982\u679cclass\u5df2\u7ecf\u7528final\u4fee\u9970\u8fc7\u4e86,\u6240\u6709\u65b9\u6cd5\u5c06\u81ea\u52a8\u9ed8\u8ba4\u4e3afinal\u578b\u7684\uff0c\u4e0d\u9700\u8981\u518d\u591a\u6b64\u4e00\u4e3e\u7ed9\u65b9\u6cd5\u52a0final\u4fee\u9970\u7b26\u3002<\/li>\n<\/ul>\n<pre class=\"has\"><code class=\"language-java\">public final class Test{\n\n    public final String method1(){\/\/1.\u907f\u514d\u5199\u6cd5\n        return \"test\";\n    }\n\n    public String method2(){\/\/\u63a8\u8350\u5199\u6cd5\n        return \"test\";\n    }\n}<\/code><\/pre>\n<h2 id=\"%E7%AC%AC%E4%B8%89%E7%B1%BB%EF%BC%9Abraces\">\u7b2c\u4e09\u7c7b\uff1abraces<\/h2>\n<p><strong>1\uff09ForLoopsMustUseBraces\u3010for\u5faa\u73af\u5fc5\u987b\u7528{}\u5305\u542b\u8d77\u6765\u3011<\/strong><\/p>\n<ul>\n<li>Avoid using 'for' statements without using curly braces. If the code formatting or indentation is lost then it becomes difficult to separate the code being controlled from the rest.<\/li>\n<li>for\u5faa\u73af\u5fc5\u987b\u4f7f\u7528\u62ec\u53f7\uff1a\u907f\u514d\u5728for\u5faa\u73af\u65f6\u4e0d\u4f7f\u7528{}\u3002\u52a0\u4e0a{}\u7684\u597d\u5904\uff1a\u4fbf\u4e8e\u4ee3\u7801\u7684\u7406\u89e3\u548c\u7ef4\u62a4\u3002<\/li>\n<\/ul>\n<pre class=\"has\"><code class=\"language-java\">int a=0;\nfor(int i=0;i&lt;100;i++) \/\/1.\u907f\u514d\u5199\u6cd5\n    a+=i;\n\nfor(int i=0;i&lt;100;i++) {\/\/2.\u63a8\u8350\u5199\u6cd5\n    a+=i;\n}\n<\/code><\/pre>\n<p><strong>2\uff09IfElseStmtsMustUseBraces\u3010if..else..\u8bed\u53e5\u5fc5\u987b\u7528{}\u5305\u542b\u8d77\u6765\u3011<\/strong><\/p>\n<ul>\n<li>Avoid using if..else statements without using surrounding braces. If the code formatting or indentation is lost then it becomes difficult to separate the code being controlled from the rest.<\/li>\n<li>if\u2026else\u2026\u5757\u5fc5\u987b\u4f7f\u7528\u62ec\u53f7\uff1a\u907f\u514d\u4f7f\u7528if\u2026else\u2026\u5757\u65f6\u4e0d\u4f7f\u7528{}\u3002\u597d\u5904\u540c\u4e0a\u3002<\/li>\n<\/ul>\n<pre class=\"has\"><code class=\"language-java\">int a=1;\nint b=2;\n\nif(1==a) \/\/1.\u907f\u514d\u5199\u6cd5\n   b=a;     \nelse\n   b+=a;\n\n\nif(1==a){\/\/2.\u63a8\u8350\u5199\u6cd5\n   b=a;     \n}else{\n   b+=a;\n}<\/code>&lt;<\/pre>\n","protected":false},"excerpt":{"rendered":"pmml java_\u521d\u5b66\u8005\u600e\u6837\u770b\u61c2\u4ee3\u7801\u672c\u6587\u7ed3\u5408\u5b9e\u4f8b\uff0c\u8bb2\u8bc9\u5982\u4f55\u5199\u51fa\u7b26\u5408PMD\u7f16\u7a0b\u89c4\u8303\u7684java\u4ee3\u7801","protected":false},"author":1,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[],"tags":[],"_links":{"self":[{"href":"https:\/\/mushiming.com\/wp-json\/wp\/v2\/posts\/8764"}],"collection":[{"href":"https:\/\/mushiming.com\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/mushiming.com\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/mushiming.com\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/mushiming.com\/wp-json\/wp\/v2\/comments?post=8764"}],"version-history":[{"count":0,"href":"https:\/\/mushiming.com\/wp-json\/wp\/v2\/posts\/8764\/revisions"}],"wp:attachment":[{"href":"https:\/\/mushiming.com\/wp-json\/wp\/v2\/media?parent=8764"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/mushiming.com\/wp-json\/wp\/v2\/categories?post=8764"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/mushiming.com\/wp-json\/wp\/v2\/tags?post=8764"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}