pmd / pmd / 218

Committed 24 Oct 2025 01:23PM UTC coverage: 78.676% (-0.007%) from 78.683%

Build # 218

Build Type

push

github

Committed by

web-flow

Commit Message

[plsql] Excessive*/Ncss*Count/NPathComplexity include the metric (#6077)

Run Details

18267 of 24067 branches covered (75.9%)

Branch coverage included in aggregate %.

8 of 11 new or added lines in 5 files covered. (72.73%)

2 existing lines in 1 file now uncovered.

39803 of 49742 relevant lines covered (80.02%)

0.81 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

75.0

/pmd-plsql/src/main/java/net/sourceforge/pmd/lang/plsql/rule/design/NcssMethodCountRule.java

/*
 * BSD-style license; for more info see http://pmd.sourceforge.net/license.html
 */

package net.sourceforge.pmd.lang.plsql.rule.design;

import net.sourceforge.pmd.lang.plsql.ast.ExecutableCode;

/**
 * Non-commented source statement counter for methods.
 *
 * <p>Analogous to and cribbed from Java version of the rule.</p>
 */
public class NcssMethodCountRule extends AbstractNcssCountRule<ExecutableCode> {

    /**
     * Count the size of all non-constructor methods.
     */
    public NcssMethodCountRule() {
        super(ExecutableCode.class);
    }

    @Override
    protected int defaultReportLevel() {
        return 100;
    }

    @Override
    protected Object[] getViolationParameters(ExecutableCode node, int metric, int limit) {
        String name = node.getMethodName();
        return new Object[] {name == null ? "(unnamed)" : name, metric, limit};
    }

    /**
     * @deprecated Since 7.18.0. Use {@link #getViolationParameters(ExecutableCode, int, int)} instead.
     */
    @Deprecated
    protected Object[] getViolationParameters(ExecutableCode node, int metric) {
        return getViolationParameters(node, metric, -1);
    }



}

1	/*
2	* BSD-style license; for more info see http://pmd.sourceforge.net/license.html
3	*/
4
5	package net.sourceforge.pmd.lang.plsql.rule.design;
6
7	import net.sourceforge.pmd.lang.plsql.ast.ExecutableCode;
8
9	/**
10	* Non-commented source statement counter for methods.
11	*
12	* <p>Analogous to and cribbed from Java version of the rule.</p>
13	*/
14	public class NcssMethodCountRule extends AbstractNcssCountRule<ExecutableCode> {
15
16	/**
17	* Count the size of all non-constructor methods.
18	*/
19	public NcssMethodCountRule() {
20	super(ExecutableCode.class);	1✔
21	}	1✔
22
23	@Override
24	protected int defaultReportLevel() {
25	return 100;	1✔
26	}
27
28	@Override
29	protected Object[] getViolationParameters(ExecutableCode node, int metric, int limit) {
30	String name = node.getMethodName();	1✔
31	return new Object[] {name == null ? "(unnamed)" : name, metric, limit};	1!
32	}
33
34	/**
35	* @deprecated Since 7.18.0. Use {@link #getViolationParameters(ExecutableCode, int, int)} instead.
36	*/
37	@Deprecated
38	protected Object[] getViolationParameters(ExecutableCode node, int metric) {
NEW 39	return getViolationParameters(node, metric, -1);	×
40	}
41
42
43
44	}

pmd / pmd / 218

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous