phoedos/pmd
1
1
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
pmd/pmd_rules_plsql_design.html
PMD CI (pmd-bot) 6724632e9a Update documentation 
https://github.com/pmd/pmd/actions/runs/988706398
null
2021-07-01 04:22:06 +00:00

2444 lines
99 KiB
HTML
Raw Blame History

This file contains invisible Unicode characters

This file contains invisible Unicode characters that are indistinguishable to humans but may be processed differently by a computer. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.

This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.

<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8">
<meta http-equiv="X-UA-Compatible" content="IE=edge">
<meta name="viewport" content="width=device-width, initial-scale=1">
<meta name="description" content="Rules that help you discover design issues.">
<meta name="keywords" content=" Design, CyclomaticComplexity, ExcessiveMethodLength, ExcessiveObjectLength, ExcessivePackageBodyLength, ExcessivePackageSpecificationLength, ExcessiveParameterList, ExcessiveTypeLength, NcssMethodCount, NcssObjectCount, NPathComplexity, TooManyFields, TooManyMethods">
<title>Design | PMD Source Code Analyzer</title>
<link rel="stylesheet" href="css/syntax.css">
<link rel="stylesheet" type="text/css" href="https://maxcdn.bootstrapcdn.com/font-awesome/4.5.0/css/font-awesome.min.css">
<!--<link rel="stylesheet" type="text/css" href="css/bootstrap.min.css">-->
<link rel="stylesheet" href="css/modern-business.css">
<link rel="stylesheet" href="css/lavish-bootstrap.css">
<link rel="stylesheet" href="css/customstyles.css">
<link rel="stylesheet" href="css/theme-blue.css">
<link rel="stylesheet" href="css/pmd-customstyles.css">
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/2.1.4/jquery.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery-cookie/1.4.1/jquery.cookie.min.js"></script>
<script src="js/jquery.navgoco.min.js"></script>
<script src="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.4/js/bootstrap.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/anchor-js/2.0.0/anchor.min.js"></script>
<script src="js/toc.js"></script>
<script src="js/customscripts.js"></script>
<link rel="shortcut icon" href="images/favicon.ico" type="image/x-icon">
<link rel="icon" href="images/favicon.ico" type="image/x-icon">
<!-- HTML5 Shim and Respond.js IE8 support of HTML5 elements and media queries -->
<!-- WARNING: Respond.js doesn't work if you view the page via file:// -->
<!--[if lt IE 9]>
<script src="https://oss.maxcdn.com/libs/html5shiv/3.7.0/html5shiv.js"></script>
<script src="https://oss.maxcdn.com/libs/respond.js/1.4.2/respond.min.js"></script>
<![endif]-->
<link rel="alternate" type="application/rss+xml" title="" href="https://pmd.github.io/pmd/feed.xml">
<script>
$(document).ready(function() {
// Initialize navgoco with default options
$("#mysidebar").navgoco({
caretHtml: '',
accordion: true,
openClass: 'active', // open
save: false, // leave false or nav highlighting doesn't work right
cookie: {
name: 'navgoco',
expires: false,
path: '/'
},
slide: {
duration: 400,
easing: 'swing'
}
});
$("#collapseAll").click(function(e) {
e.preventDefault();
$("#mysidebar").navgoco('toggle', false);
});
$("#expandAll").click(function(e) {
e.preventDefault();
$("#mysidebar").navgoco('toggle', true);
});
});
</script>
<script>
$(function () {
$('[data-toggle="tooltip"]').tooltip()
})
</script>
<script>
$(document).ready(function() {
$("#tg-sb-link").click(function() {
$("#tg-sb-sidebar").toggle();
$("#tg-sb-content").toggleClass('col-md-9');
$("#tg-sb-content").toggleClass('col-md-12');
$("#tg-sb-icon").toggleClass('fa-toggle-on');
$("#tg-sb-icon").toggleClass('fa-toggle-off');
});
});
</script>
</head>
<body>
<!-- Content is offset by the height of the topnav bar. -->
<!-- There's already a padding-top rule in modern-business.css, but it apparently doesn't work on Firefox 60 and Chrome 67 -->
<div id="topbar-content-offset">
<!-- Navigation -->
<nav class="navbar navbar-inverse navbar-fixed-top">
<div class="container topnavlinks">
<div class="navbar-header">
<button type="button" class="navbar-toggle" data-toggle="collapse" data-target="#bs-example-navbar-collapse-1">
<span class="sr-only">Toggle navigation</span>
<span class="icon-bar"></span>
<span class="icon-bar"></span>
<span class="icon-bar"></span>
</button>
<a class="fa fa-home fa-lg navbar-brand" href="index.html">&nbsp;<span class="projectTitle"> PMD Source Code Analyzer Project</span></a>
</div>
<div class="collapse navbar-collapse" id="bs-example-navbar-collapse-1">
<ul class="nav navbar-nav navbar-right">
<!-- toggle sidebar button -->
<li><a id="tg-sb-link" href="#"><i id="tg-sb-icon" class="fa fa-toggle-on"></i> Nav</a></li>
<!-- entries without drop-downs appear here -->
<li><a href="https://github.com/pmd/pmd/releases/latest" target="_blank">Download</a></li>
<li><a href="https://github.com/pmd/pmd" target="_blank">Fork us on github</a></li>
<!-- entries with drop-downs appear here -->
<!-- conditional logic to control which topnav appears for the audience defined in the configuration file.-->
<!--comment out this block if you want to hide search-->
<li>
<!--start search-->
<div id="search-demo-container">
<input type="text" id="search-input" placeholder="search...">
<ul id="results-container"></ul>
</div>
<script src="js/jekyll-search.js" type="text/javascript"></script>
<script type="text/javascript">
SimpleJekyllSearch.init({
searchInput: document.getElementById('search-input'),
resultsContainer: document.getElementById('results-container'),
dataSource: 'search.json',
searchResultTemplate: '<li><a href="{url}" title="Design">{title}</a></li>',
noResultsText: 'No results found.',
limit: 10,
fuzzy: true,
})
</script>
<!--end search-->
</li>
</ul>
</div>
</div>
<!-- /.container -->
</nav>
<!-- Page Content -->
<div class="container">
<div class="col-lg-12">&nbsp;</div>
<!-- Content Row -->
<div class="row">
<!-- Sidebar Column -->
<div class="col-md-3" id="tg-sb-sidebar">
<ul id="mysidebar" class="nav">
<li class="sidebarTitle">PMD 6.37.0-SNAPSHOT</li>
<li>
<a href="#">About</a>
<ul>
<li><a href="index.html">Home</a></li>
<li><a href="pmd_release_notes.html">Release notes</a></li>
<li><a href="pmd_next_major_development.html">PMD 7.0.0 development</a></li>
<li><a href="pmd_about_help.html">Getting help</a></li>
</ul>
</li>
<li>
<a href="#">User Documentation</a>
<ul>
<li><a href="pmd_userdocs_installation.html">Installation and basic CLI usage</a></li>
<li><a href="pmd_userdocs_making_rulesets.html">Making rulesets</a></li>
<li><a href="pmd_userdocs_configuring_rules.html">Configuring rules</a></li>
<li><a href="pmd_userdocs_best_practices.html">Best practices</a></li>
<li><a href="pmd_userdocs_suppressing_warnings.html">Suppressing warnings</a></li>
<li><a href="pmd_userdocs_incremental_analysis.html">Incremental analysis</a></li>
<li><a href="pmd_userdocs_cli_reference.html">PMD CLI reference</a></li>
<li><a href="pmd_userdocs_report_formats.html">PMD Report formats</a></li>
<li class="subfolders">
<a href="#">CPD reference</a>
<ul>
<li><a href="pmd_userdocs_cpd.html">Copy-paste detection</a></li>
<li><a href="pmd_userdocs_cpd_report_formats.html">CPD Report formats</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">Extending PMD</a>
<ul>
<li><a href="pmd_userdocs_extending_writing_rules_intro.html">Introduction to writing rules</a></li>
<li><a href="pmd_userdocs_extending_your_first_rule.html">Your first rule</a></li>
<li><a href="pmd_userdocs_extending_writing_xpath_rules.html">XPath rules</a></li>
<li><a href="pmd_userdocs_extending_writing_java_rules.html">Java rules</a></li>
<li><a href="pmd_userdocs_extending_designer_reference.html">Rule designer reference</a></li>
<li><a href="pmd_userdocs_extending_defining_properties.html">Defining rule properties</a></li>
<li><a href="pmd_userdocs_extending_metrics_howto.html">Using and defining code metrics</a></li>
<li><a href="pmd_userdocs_extending_rule_guidelines.html">Rule guidelines</a></li>
<li><a href="pmd_userdocs_extending_testing.html">Testing your rules</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">Tools / Integrations</a>
<ul>
<li><a href="pmd_userdocs_tools_maven.html">Maven PMD Plugin</a></li>
<li><a href="pmd_userdocs_tools_gradle.html">Gradle</a></li>
<li><a href="pmd_userdocs_tools_ant.html">Ant</a></li>
<li><a href="pmd_userdocs_tools_java_api.html">PMD Java API</a></li>
<li><a href="pmd_userdocs_tools_ci.html">CI integrations</a></li>
<li><a href="pmd_userdocs_tools.html">Other Tools / Integrations</a></li>
</ul>
</li>
</ul>
</li>
<li>
<a href="#">Rule Reference</a>
<ul>
<li class="subfolders">
<a href="#">Apex Rules</a>
<ul>
<li><a href="pmd_rules_apex.html">Index</a></li>
<li><a href="pmd_rules_apex_bestpractices.html">Best Practices</a></li>
<li><a href="pmd_rules_apex_codestyle.html">Code Style</a></li>
<li><a href="pmd_rules_apex_design.html">Design</a></li>
<li><a href="pmd_rules_apex_documentation.html">Documentation</a></li>
<li><a href="pmd_rules_apex_errorprone.html">Error Prone</a></li>
<li><a href="pmd_rules_apex_performance.html">Performance</a></li>
<li><a href="pmd_rules_apex_security.html">Security</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">Ecmascript Rules</a>
<ul>
<li><a href="pmd_rules_ecmascript.html">Index</a></li>
<li><a href="pmd_rules_ecmascript_bestpractices.html">Best Practices</a></li>
<li><a href="pmd_rules_ecmascript_codestyle.html">Code Style</a></li>
<li><a href="pmd_rules_ecmascript_errorprone.html">Error Prone</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">Java Rules</a>
<ul>
<li><a href="pmd_rules_java.html">Index</a></li>
<li><a href="pmd_rules_java_bestpractices.html">Best Practices</a></li>
<li><a href="pmd_rules_java_codestyle.html">Code Style</a></li>
<li><a href="pmd_rules_java_design.html">Design</a></li>
<li><a href="pmd_rules_java_documentation.html">Documentation</a></li>
<li><a href="pmd_rules_java_errorprone.html">Error Prone</a></li>
<li><a href="pmd_rules_java_multithreading.html">Multithreading</a></li>
<li><a href="pmd_rules_java_performance.html">Performance</a></li>
<li><a href="pmd_rules_java_security.html">Security</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">Java Server Pages Rules</a>
<ul>
<li><a href="pmd_rules_jsp.html">Index</a></li>
<li><a href="pmd_rules_jsp_bestpractices.html">Best Practices</a></li>
<li><a href="pmd_rules_jsp_codestyle.html">Code Style</a></li>
<li><a href="pmd_rules_jsp_design.html">Design</a></li>
<li><a href="pmd_rules_jsp_errorprone.html">Error Prone</a></li>
<li><a href="pmd_rules_jsp_security.html">Security</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">Maven POM Rules</a>
<ul>
<li><a href="pmd_rules_pom.html">Index</a></li>
<li><a href="pmd_rules_pom_errorprone.html">Error Prone</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">Modelica Rules</a>
<ul>
<li><a href="pmd_rules_modelica.html">Index</a></li>
<li><a href="pmd_rules_modelica_bestpractices.html">Best Practices</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">PLSQL Rules</a>
<ul>
<li><a href="pmd_rules_plsql.html">Index</a></li>
<li><a href="pmd_rules_plsql_bestpractices.html">Best Practices</a></li>
<li><a href="pmd_rules_plsql_codestyle.html">Code Style</a></li>
<li class="active"><a href="pmd_rules_plsql_design.html">Design</a></li>
<li><a href="pmd_rules_plsql_errorprone.html">Error Prone</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">Salesforce VisualForce Rules</a>
<ul>
<li><a href="pmd_rules_vf.html">Index</a></li>
<li><a href="pmd_rules_vf_security.html">Security</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">VM Rules</a>
<ul>
<li><a href="pmd_rules_vm.html">Index</a></li>
<li><a href="pmd_rules_vm_bestpractices.html">Best Practices</a></li>
<li><a href="pmd_rules_vm_design.html">Design</a></li>
<li><a href="pmd_rules_vm_errorprone.html">Error Prone</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">XML Rules</a>
<ul>
<li><a href="pmd_rules_xml.html">Index</a></li>
<li><a href="pmd_rules_xml_errorprone.html">Error Prone</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">XSL Rules</a>
<ul>
<li><a href="pmd_rules_xsl.html">Index</a></li>
<li><a href="pmd_rules_xsl_codestyle.html">Code Style</a></li>
<li><a href="pmd_rules_xsl_performance.html">Performance</a></li>
</ul>
</li>
</ul>
</li>
<li>
<a href="#">Language Specific Documentation</a>
<ul>
<li><a href="pmd_languages_jsp.html">JSP Support</a></li>
<li><a href="pmd_java_metrics_index.html">Java code metrics</a></li>
<li><a href="pmd_apex_metrics_index.html">Apex code metrics</a></li>
<li><a href="pmd_languages_plsql.html">PLSQL</a></li>
</ul>
</li>
<li>
<a href="#">Developer Documentation</a>
<ul>
<li><a href="pmd_devdocs_development.html">Developer resources</a></li>
<li><a href="pmd_devdocs_building.html">Building PMD from source</a></li>
<li><a href="https://github.com/pmd/pmd/blob/master/CONTRIBUTING.md" target="_blank">Contributing</a></li>
<li><a href="pmd_devdocs_writing_documentation.html">Writing documentation</a></li>
<li><a href="pmd_devdocs_roadmap.html">Roadmap</a></li>
<li><a href="pmd_devdocs_how_pmd_works.html">How PMD works</a></li>
<li><a href="pmd_devdocs_pmdtester.html">Pmdtester</a></li>
<li><a href="pmd_devdocs_rule_deprecation_policy.html">Rule Deprecation Policy</a></li>
<li class="subfolders">
<a href="#">Major contributions</a>
<ul>
<li><a href="pmd_devdocs_major_adding_new_language.html">Adding a new language</a></li>
<li><a href="pmd_devdocs_major_adding_new_cpd_language.html">Adding a new CPD language</a></li>
<li><a href="pmd_devdocs_major_adding_new_metrics_framework.html">Adding metrics support to a language</a></li>
</ul>
</li>
<li class="subfolders">
<a href="#">Experimental features</a>
<ul>
<li><a href="pmd_devdocs_experimental_ast_dump.html">Creating (XML) dump of the AST</a></li>
</ul>
</li>
</ul>
</li>
<li>
<a href="#">Project documentation</a>
<ul>
<li class="subfolders">
<a href="#">Trivia about PMD</a>
<ul>
<li><a href="pmd_projectdocs_trivia_news.html">PMD in the press</a></li>
<li><a href="pmd_projectdocs_trivia_products.html">Products & books related to PMD</a></li>
<li><a href="pmd_projectdocs_trivia_similarprojects.html">Similar projects</a></li>
<li><a href="pmd_projectdocs_trivia_meaning.html">What does 'PMD' mean?</a></li>
</ul>
</li>
<li><a href="pmd_projectdocs_faq.html">FAQ</a></li>
<li><a href="license.html">License</a></li>
<li><a href="pmd_projectdocs_credits.html">Credits</a></li>
<li><a href="pmd_release_notes_old.html">Old release notes</a></li>
<li class="subfolders">
<a href="#">Project management</a>
<ul>
<li><a href="pmd_projectdocs_committers_infrastructure.html">Infrastructure</a></li>
<li><a href="pmd_projectdocs_committers_releasing.html">Release process</a></li>
<li><a href="pmd_projectdocs_committers_merging_pull_requests.html">Merging pull requests</a></li>
<li><a href="pmd_projectdocs_committers_main_landing_page.html">Main Landing page</a></li>
</ul>
</li>
</ul>
</li>
<!-- if you aren't using the accordion, uncomment this block:
<p class="external">
<a href="#" id="collapseAll">Collapse All</a> | <a href="#" id="expandAll">Expand All</a>
</p>
-->
</ul>
<!-- this highlights the active parent class in the navgoco sidebar. this is critical so that the parent expands when you're viewing a page. This must appear below the sidebar code above. Otherwise, if placed inside customscripts.js, the script runs before the sidebar code runs and the class never gets inserted.-->
<script>$("li.active").parents('li').toggleClass("active");</script>
</div>
<!-- Content Column -->
<div class="col-md-9" id="tg-sb-content">
<div class="post-header">
<h1 class="post-title-main">Design</h1>
</div>
<div class="post-content">
<div class="summary">Rules that help you discover design issues.</div>
<!-- this handles the automatic toc. use ## for subheads to auto-generate the on-page minitoc. if you use html tags, you must supply an ID for the heading element in order for it to appear in the minitoc. -->
<script>
$( document ).ready(function() {
// Handler for .ready() called.
$('#toc').toc({ minimumHeaders: 0, listType: 'ul', showSpeed: 0, headers: 'h2,h3,h4' });
});
</script>
<div id="toc"></div>
<a target="_blank" href="https://github.com/pmd/pmd/blob/master/docs/../pmd-plsql/src/main/resources/category/plsql/design.xml" class="btn btn-default githubEditButton" role="button"><i class="fa fa-github fa-lg"></i> Edit me</a>
<!-- DO NOT EDIT THIS FILE. This file is generated from file ../pmd-plsql/src/main/resources/category/plsql/design.xml. -->
<h2 id="cyclomaticcomplexity">CyclomaticComplexity</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>Complexity directly affects maintenance costs is determined by the number of decision points in a method
plus one for the method entry. The decision points include ‘if’, ‘while’, ‘for’, and ‘case labels’ calls.
Generally, numbers ranging from 1-4 denote low complexity, 5-7 denote moderate complexity, 8-10 denote
high complexity, and 11+ is very high complexity.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/CyclomaticComplexityRule.java">net.sourceforge.pmd.lang.plsql.rule.design.CyclomaticComplexityRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="c1">-- Cyclomatic Complexity of 25</span>
<span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span> <span class="n">PACKAGE</span> <span class="n">BODY</span> <span class="n">pkg_pmd_working_sequence</span> <span class="k">AS</span>
<span class="mi">1</span> <span class="k">PROCEDURE</span> <span class="n">ty_logger</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="mi">2</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">3</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">4</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">5</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="mi">6</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">7</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">8</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">9</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">10</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="mi">11</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">12</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">13</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">14</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="mi">15</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="mi">16</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">17</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">18</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">19</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">20</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="mi">21</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">22</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">23</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">24</span> <span class="n">IF</span> <span class="k">true</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'IF/THEN l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="mi">25</span> <span class="n">ELSIF</span> <span class="k">false</span>
<span class="k">THEN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">ELSE</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="s1">'ELSIF l_Integer='</span><span class="o">||</span><span class="n">l_integer</span><span class="p">);</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="k">END</span><span class="p">;</span>
<span class="k">END</span><span class="p">;</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>showMethodsComplexity</td>
<td>true</td>
<td>Add method average violations to the report</td>
<td>no</td>
</tr>
<tr>
<td>showClassesComplexity</td>
<td>true</td>
<td>Add class average violations to the report</td>
<td>no</td>
</tr>
<tr>
<td>reportLevel</td>
<td>10</td>
<td>Cyclomatic Complexity reporting threshold</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/CyclomaticComplexity"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/CyclomaticComplexity"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"showMethodsComplexity"</span> <span class="na">value=</span><span class="s">"true"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"showClassesComplexity"</span> <span class="na">value=</span><span class="s">"true"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"reportLevel"</span> <span class="na">value=</span><span class="s">"10"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="excessivemethodlength">ExcessiveMethodLength</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>When methods are excessively long this usually indicates that the method is doing more than its
name/signature might suggest. They also become challenging for others to digest since excessive
scrolling causes readers to lose focus.
Try to reduce the method length by creating helper methods and removing any copy/pasted code.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/ExcessiveMethodLengthRule.java">net.sourceforge.pmd.lang.plsql.rule.design.ExcessiveMethodLengthRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span>
<span class="k">PROCEDURE</span> <span class="n">doSomething</span> <span class="k">BEGIN</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="nv">"Hello world!"</span><span class="p">);</span>
<span class="n">DBMS_OUTPUT</span><span class="p">.</span><span class="n">PUT_LINE</span><span class="p">(</span><span class="nv">"Hello world!"</span><span class="p">);</span>
<span class="c1">-- 98 copies omitted for brevity.</span>
<span class="k">END</span><span class="p">;</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>topscore</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Top score value</td>
<td>no</td>
</tr>
<tr>
<td>minimum</td>
<td>100.0</td>
<td>Minimum reporting threshold</td>
<td>no</td>
</tr>
<tr>
<td>sigma</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Sigma value</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessiveMethodLength"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessiveMethodLength"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"minimum"</span> <span class="na">value=</span><span class="s">"100.0"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="excessiveobjectlength">ExcessiveObjectLength</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>Excessive object line lengths are usually indications that the object may be burdened with excessive
responsibilities that could be provided by other objects. In breaking these methods
apart the code becomes more managable and ripe for reuse.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/ExcessiveObjectLengthRule.java">net.sourceforge.pmd.lang.plsql.rule.design.ExcessiveObjectLengthRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span>
<span class="n">PACKAGE</span> <span class="n">BODY</span> <span class="n">Foo</span> <span class="k">AS</span>
<span class="k">PROCEDURE</span> <span class="n">bar1</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">bar1</span><span class="p">;</span>
<span class="k">PROCEDURE</span> <span class="n">bar2</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">bar2</span><span class="p">;</span>
<span class="k">PROCEDURE</span> <span class="n">bar3</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">bar3</span><span class="p">;</span>
<span class="k">PROCEDURE</span> <span class="n">barN</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">barn</span><span class="p">;</span>
<span class="k">END</span><span class="p">;</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>topscore</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Top score value</td>
<td>no</td>
</tr>
<tr>
<td>minimum</td>
<td>1000.0</td>
<td>Minimum reporting threshold</td>
<td>no</td>
</tr>
<tr>
<td>sigma</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Sigma value</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessiveObjectLength"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessiveObjectLength"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"minimum"</span> <span class="na">value=</span><span class="s">"1000.0"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="excessivepackagebodylength">ExcessivePackageBodyLength</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>Excessive class file lengths are usually indications that the class may be burdened with excessive
responsibilities that could be provided by external classes or functions. In breaking these methods
apart the code becomes more managable and ripe for reuse.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/ExcessivePackageBodyLengthRule.java">net.sourceforge.pmd.lang.plsql.rule.design.ExcessivePackageBodyLengthRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span>
<span class="n">PACKAGE</span> <span class="n">BODY</span> <span class="n">Foo</span> <span class="k">AS</span>
<span class="k">PROCEDURE</span> <span class="n">bar1</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">bar1</span><span class="p">;</span>
<span class="k">PROCEDURE</span> <span class="n">bar2</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">bar2</span><span class="p">;</span>
<span class="k">PROCEDURE</span> <span class="n">bar3</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">bar3</span><span class="p">;</span>
<span class="k">PROCEDURE</span> <span class="n">barN</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">barn</span><span class="p">;</span>
<span class="k">END</span><span class="p">;</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>topscore</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Top score value</td>
<td>no</td>
</tr>
<tr>
<td>minimum</td>
<td>1000.0</td>
<td>Minimum reporting threshold</td>
<td>no</td>
</tr>
<tr>
<td>sigma</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Sigma value</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessivePackageBodyLength"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessivePackageBodyLength"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"minimum"</span> <span class="na">value=</span><span class="s">"1000.0"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="excessivepackagespecificationlength">ExcessivePackageSpecificationLength</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>Excessive class file lengths are usually indications that the class may be burdened with excessive
responsibilities that could be provided by external classes or functions. In breaking these methods
apart the code becomes more managable and ripe for reuse.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/ExcessivePackageSpecificationLengthRule.java">net.sourceforge.pmd.lang.plsql.rule.design.ExcessivePackageSpecificationLengthRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span>
<span class="n">PACKAGE</span> <span class="n">Foo</span> <span class="k">AS</span>
<span class="k">PROCEDURE</span> <span class="n">bar1</span><span class="p">;</span>
<span class="k">PROCEDURE</span> <span class="n">bar2</span><span class="p">;</span>
<span class="k">PROCEDURE</span> <span class="n">bar3</span><span class="p">;</span>
<span class="p">...</span>
<span class="k">PROCEDURE</span> <span class="n">barN</span><span class="p">;</span>
<span class="k">END</span><span class="p">;</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>topscore</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Top score value</td>
<td>no</td>
</tr>
<tr>
<td>minimum</td>
<td>1000.0</td>
<td>Minimum reporting threshold</td>
<td>no</td>
</tr>
<tr>
<td>sigma</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Sigma value</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessivePackageSpecificationLength"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessivePackageSpecificationLength"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"minimum"</span> <span class="na">value=</span><span class="s">"1000.0"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="excessiveparameterlist">ExcessiveParameterList</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>Methods with numerous parameters are a challenge to maintain, especially if most of them share the
same datatype. These situations usually denote the need for new objects to wrap the numerous parameters.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/ExcessiveParameterListRule.java">net.sourceforge.pmd.lang.plsql.rule.design.ExcessiveParameterListRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span>
<span class="k">PROCEDURE</span> <span class="n">addPerson</span><span class="p">(</span> <span class="c1">-- too many arguments liable to be mixed up</span>
<span class="n">birthYear</span> <span class="n">pls_integer</span><span class="p">,</span> <span class="n">birthMonth</span> <span class="n">pls_integer</span><span class="p">,</span> <span class="n">birthDate</span> <span class="n">pls_integer</span><span class="p">,</span> <span class="n">height</span> <span class="n">pls_integer</span><span class="p">,</span> <span class="n">weight</span> <span class="n">pls_integer</span><span class="p">,</span> <span class="n">ssn</span> <span class="n">pls_integer</span><span class="p">)</span> <span class="err">{</span>
<span class="p">.</span> <span class="p">.</span> <span class="p">.</span> <span class="p">.</span>
<span class="k">END</span> <span class="n">ADDPERSON</span><span class="p">;</span>
<span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span>
<span class="k">PROCEDURE</span> <span class="n">addPerson</span><span class="p">(</span> <span class="c1">-- preferred approach</span>
<span class="n">birthdate</span> <span class="nb">DATE</span><span class="p">,</span> <span class="n">measurements</span> <span class="n">BodyMeasurements</span> <span class="p">,</span> <span class="n">ssn</span> <span class="nb">INTEGER</span><span class="p">)</span> <span class="k">BEGIN</span>
<span class="p">.</span> <span class="p">.</span> <span class="p">.</span> <span class="p">.</span>
<span class="k">END</span><span class="p">;</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>topscore</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Top score value</td>
<td>no</td>
</tr>
<tr>
<td>minimum</td>
<td>10.0</td>
<td>Minimum reporting threshold</td>
<td>no</td>
</tr>
<tr>
<td>sigma</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Sigma value</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessiveParameterList"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessiveParameterList"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"minimum"</span> <span class="na">value=</span><span class="s">"10.0"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="excessivetypelength">ExcessiveTypeLength</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>Excessive class file lengths are usually indications that the class may be burdened with excessive
responsibilities that could be provided by external classes or functions. In breaking these methods
apart the code becomes more managable and ripe for reuse.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/ExcessiveTypeLengthRule.java">net.sourceforge.pmd.lang.plsql.rule.design.ExcessiveTypeLengthRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span>
<span class="k">TYPE</span> <span class="n">BODY</span> <span class="n">Foo</span> <span class="k">AS</span>
<span class="n">MEMBER</span> <span class="k">PROCEDURE</span> <span class="n">bar1</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">bar1</span><span class="p">;</span>
<span class="n">MEMBER</span> <span class="k">PROCEDURE</span> <span class="n">bar2</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">bar2</span><span class="p">;</span>
<span class="n">MEMBER</span> <span class="k">PROCEDURE</span> <span class="n">bar3</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">bar3</span><span class="p">;</span>
<span class="n">MEMBER</span> <span class="k">PROCEDURE</span> <span class="n">barN</span> <span class="k">IS</span> <span class="k">BEGIN</span>
<span class="c1">-- 1000 lines of code</span>
<span class="k">END</span> <span class="n">barn</span><span class="p">;</span>
<span class="k">END</span><span class="p">;</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>topscore</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Top score value</td>
<td>no</td>
</tr>
<tr>
<td>minimum</td>
<td>1000.0</td>
<td>Minimum reporting threshold</td>
<td>no</td>
</tr>
<tr>
<td>sigma</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Sigma value</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessiveTypeLength"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/ExcessiveTypeLength"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"minimum"</span> <span class="na">value=</span><span class="s">"1000.0"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="ncssmethodcount">NcssMethodCount</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>This rule uses the NCSS (Non-Commenting Source Statements) algorithm to determine the number of lines
of code for a given method. NCSS ignores comments, and counts actual statements. Using this algorithm,
lines of code that are split are counted as one.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/NcssMethodCountRule.java">net.sourceforge.pmd.lang.plsql.rule.design.NcssMethodCountRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span> <span class="n">PACKAGE</span> <span class="n">BODY</span> <span class="k">AS</span>
<span class="k">FUNCTION</span> <span class="n">methd</span> <span class="k">RETURN</span> <span class="nb">INTEGER</span> <span class="k">IS</span>
<span class="k">BEGIN</span>
<span class="k">RETURN</span> <span class="mi">1</span><span class="p">;;</span>
<span class="k">END</span><span class="p">;</span>
<span class="k">END</span><span class="p">;</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>topscore</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Top score value</td>
<td>no</td>
</tr>
<tr>
<td>minimum</td>
<td>100.0</td>
<td>Minimum reporting threshold</td>
<td>no</td>
</tr>
<tr>
<td>sigma</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Sigma value</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/NcssMethodCount"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/NcssMethodCount"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"minimum"</span> <span class="na">value=</span><span class="s">"100.0"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="ncssobjectcount">NcssObjectCount</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>This rule uses the NCSS (Non-Commenting Source Statements) algorithm to determine the number of lines
of code for a given Oracle object. NCSS ignores comments, and counts actual statements. Using this algorithm,
lines of code that are split are counted as one.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/NcssObjectCountRule.java">net.sourceforge.pmd.lang.plsql.rule.design.NcssObjectCountRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span> <span class="n">PACKAGE</span> <span class="n">pkg_</span>
<span class="k">PROCEDURE</span> <span class="n">Foo</span> <span class="k">IS</span>
<span class="k">BEGIN</span>
<span class="c1">--this class only has 6 NCSS lines</span>
<span class="n">super</span><span class="p">();</span>
<span class="n">super</span><span class="p">();</span>
<span class="k">END</span><span class="p">;</span>
<span class="err">}</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>topscore</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Top score value</td>
<td>no</td>
</tr>
<tr>
<td>minimum</td>
<td>1500.0</td>
<td>Minimum reporting threshold</td>
<td>no</td>
</tr>
<tr>
<td>sigma</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Sigma value</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/NcssObjectCount"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/NcssObjectCount"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"minimum"</span> <span class="na">value=</span><span class="s">"1500.0"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="npathcomplexity">NPathComplexity</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>The NPath complexity of a method is the number of acyclic execution paths through that method.
A threshold of 200 is generally considered the point where measures should be taken to reduce
complexity and increase readability.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/NPathComplexityRule.java">net.sourceforge.pmd.lang.plsql.rule.design.NPathComplexityRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span>
<span class="k">PROCEDURE</span> <span class="n">bar</span> <span class="k">AS</span> <span class="k">BEGIN</span> <span class="c1">-- this is something more complex than it needs to be,</span>
<span class="n">if</span> <span class="p">(</span><span class="n">y</span><span class="p">)</span> <span class="k">THEN</span> <span class="c1">-- it should be broken down into smaller methods or functions</span>
<span class="k">for</span> <span class="n">j</span> <span class="k">IN</span> <span class="mi">0</span> <span class="p">..</span> <span class="n">j</span><span class="o">-</span><span class="mi">1</span> <span class="n">LOOP</span>
<span class="n">if</span> <span class="p">(</span><span class="n">j</span> <span class="o">&gt;</span> <span class="n">r</span><span class="p">)</span> <span class="k">THEN</span>
<span class="n">doSomething</span><span class="p">;</span>
<span class="n">while</span> <span class="p">(</span><span class="n">f</span> <span class="o">&lt;</span> <span class="mi">5</span> <span class="p">)</span> <span class="n">LOOP</span>
<span class="n">anotherThing</span><span class="p">;</span>
<span class="n">f</span> <span class="p">:</span><span class="o">=</span> <span class="n">f</span> <span class="o">-</span> <span class="mi">27</span><span class="p">;</span>
<span class="k">END</span> <span class="n">LOOP</span><span class="p">;</span>
<span class="k">else</span>
<span class="n">tryThis</span><span class="p">();</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="k">END</span> <span class="n">LOOP</span><span class="p">;</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="n">if</span> <span class="p">(</span> <span class="n">r</span> <span class="o">-</span> <span class="n">n</span> <span class="o">&gt;</span> <span class="mi">45</span><span class="p">)</span> <span class="k">THEN</span>
<span class="n">while</span> <span class="p">(</span><span class="n">doMagic</span><span class="p">)</span> <span class="n">LOOP</span>
<span class="n">findRabbits</span><span class="p">;</span>
<span class="k">END</span> <span class="n">LOOP</span><span class="p">;</span>
<span class="k">END</span> <span class="n">IF</span><span class="p">;</span>
<span class="k">BEGIN</span>
<span class="n">doSomethingDangerous</span><span class="p">();</span>
<span class="n">EXCEPTION</span> <span class="k">WHEN</span> <span class="n">FooException</span> <span class="k">THEN</span>
<span class="n">makeAmends</span><span class="p">;</span>
<span class="k">BEGIN</span>
<span class="n">dontDoItAgain</span><span class="p">;</span>
<span class="n">EXCEPTION</span>
<span class="k">WHEN</span> <span class="n">OTHERS</span> <span class="k">THEN</span>
<span class="n">log_problem</span><span class="p">;</span>
<span class="k">END</span><span class="p">;</span>
<span class="k">END</span><span class="p">;</span>
<span class="k">END</span><span class="p">;</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>topscore</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Top score value</td>
<td>no</td>
</tr>
<tr>
<td>minimum</td>
<td>200.0</td>
<td>Minimum reporting threshold</td>
<td>no</td>
</tr>
<tr>
<td>sigma</td>
<td> </td>
<td><span style="border-radius: 0.25em; color: #fff; padding: 0.2em 0.6em 0.3em; display: inline; background-color: #d9534f; font-size: 75%;">Deprecated</span> Sigma value</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/NPathComplexity"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/NPathComplexity"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"minimum"</span> <span class="na">value=</span><span class="s">"200.0"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="toomanyfields">TooManyFields</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>Classes that have too many fields can become unwieldy and could be redesigned to have fewer fields,
possibly through grouping related fields in new objects. For example, a class with individual
city/state/zip fields could park them within a single Address field.</p>
<p><strong>This rule is defined by the following Java class:</strong> <a href="https://github.com/pmd/pmd/blob/master/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/TooManyFieldsRule.java">net.sourceforge.pmd.lang.plsql.rule.design.TooManyFieldsRule</a></p>
<p><strong>Example(s):</strong></p>
<div class="language-sql highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="k">CREATE</span> <span class="k">OR</span> <span class="k">REPLACE</span> <span class="n">PACKAGE</span> <span class="n">pkg_too_many_fields</span> <span class="k">AS</span>
<span class="n">C_CHAR_A</span> <span class="n">CONSTANT</span> <span class="nb">CHAR</span><span class="p">(</span><span class="mi">1</span> <span class="nb">CHAR</span><span class="p">)</span> <span class="p">:</span><span class="o">=</span> <span class="s1">'A'</span><span class="p">;</span>
<span class="n">C_CHAR_B</span> <span class="n">CONSTANT</span> <span class="nb">CHAR</span><span class="p">(</span><span class="mi">1</span> <span class="nb">CHAR</span><span class="p">)</span> <span class="p">:</span><span class="o">=</span> <span class="s1">'B'</span><span class="p">;</span>
<span class="p">...</span>
<span class="n">C_CHAR_Z</span> <span class="n">CONSTANT</span> <span class="nb">CHAR</span><span class="p">(</span><span class="mi">1</span> <span class="nb">CHAR</span><span class="p">)</span> <span class="p">:</span><span class="o">=</span> <span class="s1">'Z'</span><span class="p">;</span>
<span class="k">END</span> <span class="n">pkg_too_many_fields</span><span class="p">;</span>
</code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>maxfields</td>
<td>15</td>
<td>Max allowable fields</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/TooManyFields"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/TooManyFields"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"maxfields"</span> <span class="na">value=</span><span class="s">"15"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<h2 id="toomanymethods">TooManyMethods</h2>
<p><strong>Since:</strong> PMD 5.1</p>
<p><strong>Priority:</strong> Medium (3)</p>
<p>A package or type with too many methods is probably a good suspect for refactoring, in order to reduce its complexity and find a way to
have more fine grained objects.</p>
<p><strong>This rule is defined by the following XPath expression:</strong></p>
<div class="language-xpath highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="o">//</span><span class="kr">node()</span><span class="w">
</span><span class="p">[</span><span class="w"> </span><span class="p">(</span><span class="w">
</span><span class="nf">local-name</span><span class="p">(.)</span><span class="w"> </span><span class="err">=</span><span class="w"> </span><span class="s1">'PackageSpecification'</span><span class="w">
</span><span class="ow">or</span><span class="w">
</span><span class="nf">local-name</span><span class="p">(.)</span><span class="w"> </span><span class="err">=</span><span class="w"> </span><span class="s1">'TypeSpecification'</span><span class="w">
</span><span class="p">)</span><span class="w">
</span><span class="ow">and</span><span class="w">
</span><span class="p">(</span><span class="w">
</span><span class="nf">count</span><span class="p">(</span><span class="o">/</span><span class="kn">descendant::</span><span class="nt">ProgramUnit</span><span class="p">[</span><span class="w">
</span><span class="ow">not</span><span class="w"> </span><span class="p">(</span><span class="w">
</span><span class="nf">starts-with</span><span class="p">(</span><span class="na">@Name</span><span class="p">,</span><span class="s1">'get'</span><span class="p">)</span><span class="w">
</span><span class="ow">or</span><span class="w">
</span><span class="nf">starts-with</span><span class="p">(</span><span class="na">@Name</span><span class="p">,</span><span class="s1">'set'</span><span class="p">)</span><span class="w">
</span><span class="ow">or</span><span class="w">
</span><span class="nf">starts-with</span><span class="p">(</span><span class="na">@Name</span><span class="p">,</span><span class="s1">'is'</span><span class="p">)</span><span class="w">
</span><span class="p">)</span><span class="w">
</span><span class="p">]</span><span class="w">
</span><span class="p">)</span><span class="w">
</span><span class="o">+</span><span class="w">
</span><span class="nf">count</span><span class="p">(</span><span class="o">/</span><span class="kn">descendant::</span><span class="nt">TypeMethod</span><span class="p">[</span><span class="w">
</span><span class="ow">not</span><span class="w"> </span><span class="p">(</span><span class="w">
</span><span class="nf">starts-with</span><span class="p">(</span><span class="na">@Name</span><span class="p">,</span><span class="s1">'get'</span><span class="p">)</span><span class="w">
</span><span class="ow">or</span><span class="w">
</span><span class="nf">starts-with</span><span class="p">(</span><span class="na">@Name</span><span class="p">,</span><span class="s1">'set'</span><span class="p">)</span><span class="w">
</span><span class="ow">or</span><span class="w">
</span><span class="nf">starts-with</span><span class="p">(</span><span class="na">@Name</span><span class="p">,</span><span class="s1">'is'</span><span class="p">)</span><span class="w">
</span><span class="p">)</span><span class="w">
</span><span class="p">]</span><span class="w">
</span><span class="p">)</span><span class="w">
</span><span class="p">)</span><span class="w"> </span><span class="o">&gt;</span><span class="w"> </span><span class="nv">$maxmethods</span><span class="w">
</span><span class="p">]</span><span class="w">
</span></code></pre></div></div>
<p><strong>This rule has the following properties:</strong></p>
<table>
<thead>
<tr>
<th>Name</th>
<th>Default Value</th>
<th>Description</th>
<th>Multivalued</th>
</tr>
</thead>
<tbody>
<tr>
<td>maxmethods</td>
<td>1</td>
<td>The method count reporting threshold</td>
<td>no</td>
</tr>
</tbody>
</table>
<p><strong>Use this rule with the default properties by just referencing it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/TooManyMethods"</span> <span class="nt">/&gt;</span>
</code></pre></div></div>
<p><strong>Use this rule and customize it:</strong></p>
<div class="language-xml highlighter-rouge"><div class="highlight"><pre class="highlight"><code><span class="nt">&lt;rule</span> <span class="na">ref=</span><span class="s">"category/plsql/design.xml/TooManyMethods"</span><span class="nt">&gt;</span>
<span class="nt">&lt;properties&gt;</span>
<span class="nt">&lt;property</span> <span class="na">name=</span><span class="s">"maxmethods"</span> <span class="na">value=</span><span class="s">"1"</span> <span class="nt">/&gt;</span>
<span class="nt">&lt;/properties&gt;</span>
<span class="nt">&lt;/rule&gt;</span>
</code></pre></div></div>
<div class="tags">
</div>
</div>
<hr class="shaded"/>
<footer>
<div class="row">
<div class="col-lg-12 footer">
&copy;2021 PMD Open Source Project. All rights reserved. <br />
Site last generated: Jul 1, 2021 <br />
<p><img src="images/pmd-logo-small.png" alt="Company logo"/></p>
</div>
</div>
</footer>
</div>
<!-- /.row -->
</div>
<!-- /.container -->
</div>
</div>
</body>
</html>
Reference in New Issue View Git Blame Copy Permalink