diff --git a/src/etc/changelog.xsl b/src/etc/changelog.xsl index 6a5904fb39..142d72e8f6 100644 --- a/src/etc/changelog.xsl +++ b/src/etc/changelog.xsl @@ -1,8 +1,7 @@ - + - + - - + +
-

+

-

Classes

- +

Classes

+
@@ -317,21 +317,21 @@ -
- + +
- - + + - + - +

Package

@@ -340,7 +340,7 @@
-

Classes

+

Classes

@@ -349,22 +349,22 @@
- - + +
- - + + - - + + -

Class .

+

Class .

@@ -373,7 +373,7 @@
-

Methods

+

Methods

@@ -381,27 +381,27 @@
- - + +
- - - - - + +
- - https://ant.apache.org - -

Source Code Coverage

+ + + - + -
+ + https://ant.apache.org + +

Source Code Coverage

Designed for use with Sitraka JProbe and Ant.Designed for use with Sitraka JProbe and Ant.
+
@@ -471,7 +471,7 @@ - stylesheet.css + stylesheet.css diff --git a/src/etc/jdepend-frames.xsl b/src/etc/jdepend-frames.xsl index d35444b1e9..442dc33207 100644 --- a/src/etc/jdepend-frames.xsl +++ b/src/etc/jdepend-frames.xsl @@ -1,8 +1,8 @@ - + -

JDepend Analysis

+

JDepend Analysis

Designed for use with JDepend and Ant.


-

Summary

+

Summary

[summary] [packages] @@ -150,7 +151,7 @@
-

Packages

+

Packages

[summary] [packages] @@ -223,7 +224,7 @@ - + diff --git a/src/etc/junit-noframes-saxon.xsl b/src/etc/junit-noframes-saxon.xsl index ce7efb8e0a..508d53b5c3 100644 --- a/src/etc/junit-noframes-saxon.xsl +++ b/src/etc/junit-noframes-saxon.xsl @@ -1,8 +1,6 @@ - + @@ -160,7 +158,6 @@ - @@ -239,8 +236,7 @@ - -

Package

+

Package

-

Cycles

+

Cycles

[summary] [packages] @@ -242,7 +243,7 @@ - @@ -697,14 +697,12 @@ h6 {

Classes

-

-

Explanations

+

Explanations

[summary] [packages] @@ -252,19 +253,19 @@

The following explanations are for quick reference and are lifted directly from the original JDepend documentation.

-

Number of Classes

+

Number of Classes

The number of concrete and abstract classes (and interfaces) in the package is an indicator of the extensibility of the package.

-

Afferent Couplings

+

Afferent Couplings

The number of other packages that depend upon classes within the package is an indicator of the package's responsibility.

-

Efferent Couplings

+

Efferent Couplings

The number of other packages that the classes in the package depend upon is an indicator of the package's independence.

-

Abstractness

+

Abstractness

The ratio of the number of abstract classes (and interfaces) in the analyzed package to the total number of classes in the analyzed package.

The range for this metric is 0 to 1, with A=0 indicating a completely concrete package and A=1 indicating a completely abstract package.

-

Instability

+

Instability

The ratio of efferent coupling (Ce) to total coupling (Ce / (Ce + Ca)). This metric is an indicator of the package's resilience to change.

The range for this metric is 0 to 1, with I=0 indicating a completely stable package and I=1 indicating a completely instable package.

-

Distance

+

Distance

The perpendicular distance of a package from the idealized line A + I = 1. This metric is an indicator of the package's balance between abstractness and stability.

A package squarely on the main sequence is optimally balanced with respect to its abstractness and stability. Ideal packages are either completely abstract and stable (x=0, y=1) or completely concrete and instable (x=1, y=0).

The range for this metric is 0 to 1, with D=0 indicating a package that is coincident with the main sequence and D=1 indicating a package that is as far from the main sequence as possible.

diff --git a/src/etc/junit-frames-saxon.xsl b/src/etc/junit-frames-saxon.xsl index 1b2084ed0e..5bcaca0054 100644 --- a/src/etc/junit-frames-saxon.xsl +++ b/src/etc/junit-frames-saxon.xsl @@ -1,8 +1,6 @@ - + @@ -675,7 +673,7 @@ - + @@ -731,14 +729,12 @@

Classes

-

-

@@ -878,9 +874,8 @@
- - + + diff --git a/src/etc/junit-frames-xalan1.xsl b/src/etc/junit-frames-xalan1.xsl index bc41a7cd80..163bc2986c 100644 --- a/src/etc/junit-frames-xalan1.xsl +++ b/src/etc/junit-frames-xalan1.xsl @@ -1,9 +1,9 @@ @@ -548,14 +548,12 @@ h6 {

Classes

-

-

diff --git a/src/etc/junit-frames.xsl b/src/etc/junit-frames.xsl index 732e040366..29a9b03ebb 100644 --- a/src/etc/junit-frames.xsl +++ b/src/etc/junit-frames.xsl @@ -1,9 +1,9 @@ + xmlns="http://www.w3.org/1999/xhtml" + xmlns:redirect="http://xml.apache.org/xalan/redirect" + xmlns:string="xalan://java.lang.String" + extension-element-prefixes="redirect">
-

@@ -844,8 +842,7 @@ h6 {
- + @@ -854,7 +851,7 @@ h6 { - Failure
@@ -258,8 +254,7 @@ - -

TestCase

+

TestCase

@@ -350,7 +345,7 @@ -

+

diff --git a/src/etc/junit-noframes.xsl b/src/etc/junit-noframes.xsl index 0d03eb3a82..1a03f88088 100644 --- a/src/etc/junit-noframes.xsl +++ b/src/etc/junit-noframes.xsl @@ -1,7 +1,7 @@ + xmlns="http://www.w3.org/1999/xhtml" + xmlns:string="xalan://java.lang.String"> @@ -124,7 +124,6 @@ - @@ -203,8 +202,7 @@ - -

Package

+

Package

@@ -222,8 +220,7 @@ - -

TestCase

+

TestCase

@@ -314,7 +311,7 @@ -

+

diff --git a/src/etc/log.xsl b/src/etc/log.xsl index 9fe60fb104..6ec1de99e9 100644 --- a/src/etc/log.xsl +++ b/src/etc/log.xsl @@ -1,5 +1,6 @@ - + - -

Error details

+

Error details

diff --git a/src/etc/maudit-frames.xsl b/src/etc/maudit-frames.xsl index aa36b38762..22654b8de5 100644 --- a/src/etc/maudit-frames.xsl +++ b/src/etc/maudit-frames.xsl @@ -1,8 +1,8 @@ + xmlns="http://www.w3.org/1999/xhtml" + xmlns:redirect="http://xml.apache.org/xalan/redirect" + extension-element-prefixes="redirect"> - - + + - - + + -

Class .

+

Class .

-

Violations

+

Violations

@@ -198,37 +198,37 @@
- - + +
- - + + - - + +
-

+

Classes

- +
-
- - +
+ + @@ -360,7 +360,7 @@ - + @@ -376,13 +376,13 @@ - - + + - - + + open('package-frame.html','classListFrame')

Package

@@ -393,7 +393,7 @@ -

Classes

+

Classes

@@ -402,8 +402,8 @@
- - + +
@@ -428,7 +428,7 @@ - stylesheet.css + stylesheet.css diff --git a/src/etc/mmetrics-frames.xsl b/src/etc/mmetrics-frames.xsl index 02616bd400..c97f4959ca 100644 --- a/src/etc/mmetrics-frames.xsl +++ b/src/etc/mmetrics-frames.xsl @@ -1,10 +1,10 @@ + xmlns="http://www.w3.org/1999/xhtml" + xmlns:exsl="http://exslt.org/common" + xmlns:redirect="http://xml.apache.org/xalan/redirect" + exclude-result-prefixes="exsl" + extension-element-prefixes="redirect"> @@ -171,34 +171,36 @@
LCOM | NOC - -

Cyclomatic Complexity - V(G)

+

Cyclomatic Complexity - V(G)

+

This metric was introduced in the 1970s to measure the amount of control flow complexity or branching complexity in a module such as a subroutine. It gives the number of paths that may be taken through the code, and was initially developed to give some measure of the cost of producing a test case for the module by executing each path. -

+

+

Methods with a high cyclomatic complexity tend to be more difficult to understand and maintain. In general the more complex the methods of an application, the more difficult it will be to test it, and this will adversely affect its reliability. -

+

+

V(G) is a measure of the control flow complexity of a method or constructor. It counts the number of branches in the body of the method, defined as: +

  • while statements;
  • if statements;
  • for statements.
- +

The metric can also be configured to count each case of a switch statement as well. - - -

Lines of Code - LOC

- +

+

Lines of Code - LOC

+

This is perhaps the simplest of all the metrics to define and compute. Counting lines has a long history as a software metric dating from before the rise of structured programming, and it is still in widespread use today. @@ -207,159 +209,163 @@ reusability and its maintainability. There are a variety of ways that the size can be calculated. These include counting all the lines of code, the number of statements, the blank lines of code, the lines of commentary, and the lines consisting only of syntax such as block delimiters. -

+

+

This metric can also be used for sizing other constructs as well, for example, the overall size of a Java class or package can be measured by counting the number of source lines it consists of. -

+

+

LOC can be used to determine the size of a compilation unit (source file), class or interface, method, constructor, or field. It can be configured to ignore: +

  • blank lines;
  • lines consisting only of comments;
  • lines consisting only of opening and closing braces.
- -
-

Depth of Inheritance Hierarchy - DIT

- +

Depth of Inheritance Hierarchy - DIT

+

This metric calculates how far down the inheritance hierarchy a class is declared. In Java all classes have java.lang.Object as their ultimate superclass, which is defined to have a depth of 1. So a class that immediately extends java.lang.Object has a metric value of 2; any of its subclasses will have a value of 3, and so on. -

+

+

A class that is deep within the tree inherits more methods and state variables, thereby increasing its complexity and making it difficult to predict its behavior. It can be harder to understand a system with many inheritance layers. -

+

+

DIT is defined for classes and interfaces: +

  • all interface types have a depth of 1;
  • the class java.lang.Object has a depth of 1;
  • all other classes have a depth of 1 + the depth of their super class.
- -
-

Number of Attributes - NOA

- +

Number of Attributes - NOA

+

The number of distinct state variables in a class serves as one measure of its complexity. The more state a class represents the more difficult it is to maintain invariants for it. It also hinders comprehensibility and reuse. -

+

+

In Java, state can be exposed to subclasses through protected fields, which entails that the subclass also be aware of and maintain any invariants. This interference with the class's data encapsulation can be a source of defects and hidden dependencies between the state variables. -

+

+

NOA is defined for classes and interfaces. It counts the number of fields declared in the class or interface. - - -

Number of Remote Methods - NRM

- +

+

Number of Remote Methods - NRM

+

NRM is defined for classes. A remote method call is defined as an invocation of a method that is not declared in any of: +

  • the class itself;
  • a class or interface that the class extends or implements;
  • a class or method that extends the class.
- +

The value is the count of all the remote method calls in all of the methods and constructors of the class. - - -

Number of Local Methods - NLM

- +

+

Number of Local Methods - NLM

+

NLM is defined for classes and interfaces. A local method is defined as a method that is declared in the class or interface. NLM can be configured to include the local methods of all of the class's superclasses. Methods with public, protected, package and private visibility can be independently counted by setting configuration parameters. - - -

Weighted Methods per Class - WMC

- +

+

Weighted Methods per Class - WMC

+

If the number of methods in a class can be determined during the design and modeling phase of a project, it can be used as a predictor of how much time and effort is needed to develop, debug and maintain it. This metric can be further refined by incorporating a weighting for the complexity of each method. The usual weighting is given by the cyclomatic complexity of the method. -

+

+

The subclasses of a class inherit all of its public and protected methods, and possibly its package methods as well, so the number of methods a class has directly impacts the complexity of its subclasses. Classes with large numbers of methods are often specific to a particular application, reducing the ability to reuse them. -

+

+

The definition of WMC is based upon NLM, and it provides the same configuration parameters for counting inherited methods and of varying visibility. The main difference is that NLM always counts each method as 1, whereas WMC will weight each method. There are two weighting schemes: +

  • V(G) the cyclomatic complexity of the method is used as its weight. Methods from class files are given a V(G) of 1.
  • the arity, or the number of parameters of the method are used to determine the weight.
- -
-

Response For Class - RFC

- +

Response For Class - RFC

+

The response set of a class is the set of all methods that can be invoked as a result of a message sent to an object of the class. This includes methods in the class's inheritance hierarchy and methods that can be invoked on other objects. The Response For Class metric is defined to be size of the response set for the class. A class which provides a larger response set is considered to be more complex than one with a smaller response set. -

+

+

One reason for this is that if a method call on a class can result in a large number of different method calls on the target and other classes, then it can be harder to test the behavior of the class and debug problems. It will typically require a deeper understanding of the potential interactions that objects of the class can have with the rest of the system. -

+

+

RFC is defined as the sum of NLM and NRM for the class. The local methods include all of the public, protected, package and private methods, but not methods declared only in a superclass. - - -

Data Abstraction Coupling - DAC

- +

+

Data Abstraction Coupling - DAC

+

DAC is defined for classes and interfaces. It counts the number of reference types that are used in the field declarations of the class or interface. The component types of arrays are also counted. Any field with a type that is either a supertype or a subtype of the class is not counted. - - -

Fan Out - FANOUT

- +

+

Fan Out - FANOUT

+

FANOUT is defined for classes and interfaces, constructors and methods. It counts the number of reference types that are used in: +

  • field declarations;
  • formal parameters and return types;
  • throws declarations;
  • local variables.
- +

The component types of arrays are also counted. Any type that is either a supertype or a subtype of the class is not counted. - - -

Coupling Between Objects - CBO

- +

+

Coupling Between Objects - CBO

+

When one object or class uses another object or class they are said to be coupled. One major source of coupling is that between a superclass and a subclass. A coupling is also introduced when a method or field in another class is accessed, or when an object of another class is passed into or out of a method invocation. Coupling Between Objects is a measure of the non-inheritance coupling between two objects. -

+

+

A high value of coupling reduces the modularity of the class and makes reuse more difficult. The more independent a class is the more likely it is that it will be possible to reuse it in another part of the system. When a @@ -367,34 +373,37 @@ class is coupled to another class it becomes sensitive to changes in that class, thereby making maintenance for difficult. In addition, a class that is overly dependent on other classes can be difficult to understand and test in isolation. -

+

+

CBO is defined for classes and interfaces, constructors and methods. It counts the number of reference types that are used in: +

  • field declarations
  • formal parameters and return types
  • throws declarations
  • local variables
- +

It also counts: +

  • types from which field and method selections are made
- +

The component types of arrays are also counted. Any type that is either a supertype or a subtype of the class is not counted. - - -

Lack of Cohesion Of Methods - LCOM

- +

+

Lack of Cohesion Of Methods - LCOM

+

The cohesion of a class is the degree to which its methods are related to each other. It is determined by examining the pattern of state variable accesses within the set of methods. If all the methods access the same state variables then they have high cohesion; if they access disjoint sets of variables then the cohesion is low. An extreme example of low cohesion would be if none of the methods accessed any of the state variables. - +

+

If a class exhibits low method cohesion it indicates that the design of the class has probably been partitioned incorrectly, and could benefit by being split into more classes with individually higher cohesion. On the other @@ -402,41 +411,47 @@ hand, a high value of cohesion (a low lack of cohesion) implies that the class is well designed. A cohesive class will tend to provide a high degree of encapsulation, whereas a lack of cohesion decreases encapsulation and increases complexity. -

+

+

Another form of cohesion that is useful for Java programs is cohesion between nested and enclosing classes. A nested class that has very low cohesion with its enclosing class would probably better designed as a peer class rather than a nested class. -

+

+

LCOM is defined for classes. Operationally, LCOM takes each pair of methods in the class and determines the set of fields they each access. If they have disjoint sets of field accesses increase the count P by one. If they share at least one field access then increase Q by one. After considering each pair of methods, LCOM = (P > Q) ? (P - Q) : 0 -

+

+

Indirect access to fields via local methods can be considered by setting a metric configuration parameter. - - -

Number Of Classes - NOC

- +

+

Number Of Classes - NOC

+

The overall size of the system can be estimated by calculating the number of classes it contains. A large system with more classes is more complex than a smaller one because the number of potential interactions between objects is higher. This reduces the comprehensibility of the system which in turn makes it harder to test, debug and maintain. -

+

+

If the number of classes in the system can be projected during the initial design phase of the project it can serve as a base for estimating the total effort and cost of developing, debugging and maintaining the system. -

+

+

The NOC metric can also usefully be applied at the package and class level as well as the total system. -

+

+

NOCL is defined for class and interfaces. It counts the number of classes or interfaces that are declared. This is usually 1, but nested class declarations will increase this number. +

@@ -502,64 +517,63 @@ will increase this number. - - + + - - + + -

Class .

+

Class .

-

Methods

+

Methods

- - + +
- - + + - - + +
-

+

-

Classes

- - - - +

Classes

+
+ + - -
- - + +
+ + @@ -668,20 +682,23 @@ will increase this number. - - + - +

+ +
+
+

Note: Metrics evaluate the quality of software by analyzing the program source and quantifying various kind of complexity. Complexity is a common source of problems and defects in software. High complexity makes it more difficult to develop, understand, maintain, extend, test and debug a program. -

+

+

The primary use of metrics is to focus your attention on those parts of code that potentially are complexity hot spots. Once the complex areas your program have been uncovered, you can take remedial actions. For additional information about metrics and their meaning, please consult Metamata Metrics manual. -

Packages

@@ -699,12 +716,12 @@ will increase this number. - - + + - + @@ -716,25 +733,28 @@ will increase this number. - - + - +

+ +
+
+

Note: Metrics evaluate the quality of software by analyzing the program source and quantifying various kind of complexity. Complexity is a common source of problems and defects in software. High complexity makes it more difficult to develop, understand, maintain, extend, test and debug a program. -

+

+

The primary use of metrics is to focus your attention on those parts of code that potentially are complexity hot spots. Once the complex areas your program have been uncovered, you can take remedial actions. For additional information about metrics and their meaning, please consult Metamata Metrics manual. -

-

Classes

+

Classes

@@ -746,7 +766,7 @@ will increase this number. - + @@ -771,7 +791,7 @@ will increase this number. - stylesheet.css + stylesheet.css diff --git a/src/etc/tagdiff.xsl b/src/etc/tagdiff.xsl index 42c9569e0c..af1152a23a 100644 --- a/src/etc/tagdiff.xsl +++ b/src/etc/tagdiff.xsl @@ -16,9 +16,8 @@ limitations under the License. --> - + @@ -60,14 +59,11 @@ -

- -

+

Tagdiff between and

Designed for use with Ant.


-
diff --git a/src/etc/testcases/taskdefs/optional/junitreport-with-include/junit-frames.xsl b/src/etc/testcases/taskdefs/optional/junitreport-with-include/junit-frames.xsl index 135ac8310e..63433523c6 100644 --- a/src/etc/testcases/taskdefs/optional/junitreport-with-include/junit-frames.xsl +++ b/src/etc/testcases/taskdefs/optional/junitreport-with-include/junit-frames.xsl @@ -1,9 +1,9 @@ + xmlns="http://www.w3.org/1999/xhtml" + xmlns:redirect="http://xml.apache.org/xalan/redirect" + xmlns:stringutils="xalan://org.apache.tools.ant.util.StringUtils" + extension-element-prefixes="redirect"> @@ -596,7 +596,7 @@ h6 {
@@ -112,7 +108,7 @@ - entries - (back to top) + (back to top)
- + @@ -651,14 +651,12 @@ h6 {

Classes

-

-

@@ -785,8 +783,7 @@ h6 { - - + @@ -795,7 +792,7 @@ h6 { - + Failure diff --git a/src/etc/testcases/taskdefs/optional/junitreport-with-include/junit-import.xsl b/src/etc/testcases/taskdefs/optional/junitreport-with-include/junit-import.xsl index ccfccb83c3..dca158145e 100644 --- a/src/etc/testcases/taskdefs/optional/junitreport-with-include/junit-import.xsl +++ b/src/etc/testcases/taskdefs/optional/junitreport-with-include/junit-import.xsl @@ -1,9 +1,5 @@ - + @@ -651,15 +651,13 @@ h6 {

Classes

-

-

-
+
@@ -787,8 +785,7 @@ h6 { - - + @@ -797,7 +794,7 @@ h6 { - + Failure diff --git a/src/etc/testcases/taskdefs/optional/xalan-redirect-in.xsl b/src/etc/testcases/taskdefs/optional/xalan-redirect-in.xsl index ef0be5f6fa..525c198ba8 100644 --- a/src/etc/testcases/taskdefs/optional/xalan-redirect-in.xsl +++ b/src/etc/testcases/taskdefs/optional/xalan-redirect-in.xsl @@ -15,10 +15,9 @@ See the License for the specific language governing permissions and limitations under the License. --> - + - + diff --git a/src/etc/testcases/taskdefs/style/printParams.xsl b/src/etc/testcases/taskdefs/style/printParams.xsl index 2eaa0e6ef7..4ad83c941c 100644 --- a/src/etc/testcases/taskdefs/style/printParams.xsl +++ b/src/etc/testcases/taskdefs/style/printParams.xsl @@ -16,10 +16,7 @@ limitations under the License. --> - + set default value diff --git a/src/tests/antunit/junit-frames.xsl b/src/tests/antunit/junit-frames.xsl index bbb0c76438..131277889a 100644 --- a/src/tests/antunit/junit-frames.xsl +++ b/src/tests/antunit/junit-frames.xsl @@ -1,9 +1,9 @@ + xmlns="http://www.w3.org/1999/xhtml" + xmlns:redirect="http://xml.apache.org/xalan/redirect" + xmlns:stringutils="xalan://org.apache.tools.ant.util.StringUtils" + extension-element-prefixes="redirect"> @@ -650,14 +650,12 @@ h6 {

Projects

-

-

@@ -786,8 +784,7 @@ h6 { - - + @@ -796,7 +793,7 @@ h6 {
- + Failure diff --git a/src/tests/antunit/junit-noframes.xsl b/src/tests/antunit/junit-noframes.xsl index 2f5e4fb685..f27591ff46 100644 --- a/src/tests/antunit/junit-noframes.xsl +++ b/src/tests/antunit/junit-noframes.xsl @@ -1,9 +1,9 @@ + xmlns="http://www.w3.org/1999/xhtml" + xmlns:stringutils="xalan://org.apache.tools.ant.util.StringUtils"> + doctype-public="-//W3C//DTD HTML 4.01 Transitional//EN" /> - @@ -202,8 +201,7 @@ under the License. - -

Directory

+

Directory

@@ -221,8 +219,7 @@ under the License. - -

Project

+

Project

@@ -314,7 +311,7 @@ under the License. -

Unit Test Results

+

Unit Test Results

diff --git a/src/tests/antunit/taskdefs/xslt/printParams-invalid.xsl b/src/tests/antunit/taskdefs/xslt/printParams-invalid.xsl index e045a5387b..7e8f6c6190 100644 --- a/src/tests/antunit/taskdefs/xslt/printParams-invalid.xsl +++ b/src/tests/antunit/taskdefs/xslt/printParams-invalid.xsl @@ -16,10 +16,7 @@ limitations under the License. --> - + set default value