[ packages] [ objects] [ functions] [ explanations]
The following document contains the results of a JavaNCSSmetric analysis.
[ packages] [ objects] [ functions] [ explanations]
| Package | Classes | Functions | NCSS | Javadocs | Javadoc Lines | Single Comment Lines | Multi Comment Lines |
|---|---|---|---|---|---|---|---|
| battlehex.ui.main | 12 | 61 | 571 | 48 | 227 | 7 | 348 |
| battlehex.model.map | 16 | 81 | 323 | 87 | 412 | 3 | 464 |
| battlehex.model.command | 12 | 28 | 241 | 32 | 156 | 3 | 348 |
| battlehex | 4 | 18 | 175 | 13 | 55 | 5 | 121 |
| battlehex.controller | 6 | 23 | 153 | 27 | 119 | 0 | 174 |
| battlehex.ui.startup | 2 | 12 | 75 | 7 | 39 | 0 | 58 |
| battlehex.model.player | 5 | 15 | 72 | 18 | 81 | 0 | 145 |
| battlehex.model | 4 | 7 | 23 | 10 | 52 | 0 | 116 |
| Classes total | Functions total | NCSS total | Javadocs | Javadoc Lines | Single Comment Lines | Multi Comment Lines |
|---|---|---|---|---|---|---|
| 61 | 245 | 1633 | 242 | 1141 | 18 | 1774 |
[ packages] [ objects] [ functions] [ explanations]
| Object | NCSS | Functions | Classes | Javadocs |
|---|---|---|---|---|
| battlehex.model.map.Territory | 92 | 18 | 1 | 15 |
| battlehex.ui.main.ControlDisplay | 69 | 7 | 2 | 3 |
| battlehex.ui.main.GridDisplayer | 68 | 6 | 1 | 3 |
| battlehex.ui.main.GameDisplayer | 62 | 6 | 1 | 3 |
| battlehex.ui.startup.GameTypeChoiceView | 57 | 8 | 6 | 2 |
| battlehex.ui.main.View | 53 | 7 | 4 | 2 |
| battlehex.controller.Controller | 52 | 11 | 1 | 10 |
| battlehex.ui.main.TerritoryInfoDisplayer | 50 | 4 | 1 | 4 |
| battlehex.BattleHex | 47 | 5 | 1 | 2 |
| battlehex.BattleHex | 47 | 5 | 1 | 2 |
| battlehex.model.map.SubTerritoryIterator | 47 | 7 | 0 | 2 |
| battlehex.GameFactory | 40 | 5 | 0 | 5 |
| battlehex.ui.main.TerritoryDisplayer | 40 | 4 | 0 | 4 |
| battlehex.ui.main.CurrentPlayerDisplayer | 37 | 7 | 0 | 6 |
| battlehex.ui.main.TileComponent | 36 | 3 | 0 | 3 |
| battlehex.model.map.Tile | 34 | 9 | 0 | 10 |
| battlehex.model.command.CastleBuy | 30 | 4 | 0 | 3 |
| battlehex.model.command.Move | 30 | 4 | 0 | 3 |
| battlehex.model.player.PlayerManager | 29 | 5 | 0 | 4 |
| battlehex.model.map.Money | 28 | 7 | 1 | 8 |
| battlehex.model.map.Surrounding | 27 | 4 | 0 | 5 |
| battlehex.controller.TerritorySelectedState | 23 | 3 | 0 | 4 |
| battlehex.controller.PlacementPendingState | 21 | 3 | 0 | 4 |
| battlehex.model.command.NewGame | 21 | 3 | 0 | 2 |
| battlehex.model.command.Game | 19 | 2 | 0 | 3 |
| battlehex.ui.main.PolygonDisplayer | 18 | 3 | 0 | 3 |
| battlehex.controller.BaseState | 17 | 3 | 0 | 4 |
| battlehex.model.command.EndTurn | 15 | 2 | 1 | 2 |
| battlehex.model.map.ITerritory | 14 | 13 | 0 | 14 |
| battlehex.model.player.CyclicIterator | 14 | 4 | 0 | 5 |
| Object | NCSS | Functions | Classes | Javadocs |
|---|---|---|---|---|
| battlehex.model.map.Territory | 92 | 18 | 1 | 15 |
| battlehex.model.map.ITerritory | 14 | 13 | 0 | 14 |
| battlehex.controller.Controller | 52 | 11 | 1 | 10 |
| battlehex.model.map.Tile | 34 | 9 | 0 | 10 |
| battlehex.ui.startup.GameTypeChoiceView | 57 | 8 | 6 | 2 |
| battlehex.model.map.ITile | 8 | 7 | 0 | 8 |
| battlehex.model.map.Money | 28 | 7 | 1 | 8 |
| battlehex.model.map.SubTerritoryIterator | 47 | 7 | 0 | 2 |
| battlehex.ui.main.ControlDisplay | 69 | 7 | 2 | 3 |
| battlehex.ui.main.CurrentPlayerDisplayer | 37 | 7 | 0 | 6 |
| battlehex.ui.main.PassiveLayout | 13 | 7 | 0 | 8 |
| battlehex.ui.main.View | 53 | 7 | 4 | 2 |
| battlehex.model.map.IMoney | 7 | 6 | 0 | 7 |
| battlehex.ui.main.GameDisplayer | 62 | 6 | 1 | 3 |
| battlehex.ui.main.GridDisplayer | 68 | 6 | 1 | 3 |
| battlehex.ui.main.IController | 7 | 6 | 0 | 7 |
| battlehex.BattleHex | 47 | 5 | 1 | 2 |
| battlehex.BattleHex | 47 | 5 | 1 | 2 |
| battlehex.GameFactory | 40 | 5 | 0 | 5 |
| battlehex.model.player.PlayerManager | 29 | 5 | 0 | 4 |
| battlehex.model.command.CastleBuy | 30 | 4 | 0 | 3 |
| battlehex.model.command.Move | 30 | 4 | 0 | 3 |
| battlehex.model.map.Surrounding | 27 | 4 | 0 | 5 |
| battlehex.model.player.CyclicIterator | 14 | 4 | 0 | 5 |
| battlehex.model.Turn | 11 | 4 | 0 | 4 |
| battlehex.ui.main.TerritoryDisplayer | 40 | 4 | 0 | 4 |
| battlehex.ui.main.TerritoryInfoDisplayer | 50 | 4 | 1 | 4 |
| battlehex.ui.startup.IGameFactory | 5 | 4 | 0 | 5 |
| battlehex.controller.BaseState | 17 | 3 | 0 | 4 |
| battlehex.controller.PlacementPendingState | 21 | 3 | 0 | 4 |
| NCSS average | Program NCSS | Classes average | Functions average | Javadocs average |
|---|---|---|---|---|
| 21.33 | 1,633.00 | 0.34 | 4.02 | 3.97 |
[ packages] [ objects] [ functions] [ explanations]
| Function | NCSS | CCN | Javadoc |
|---|---|---|---|
| battlehex.ui.main.TerritoryInfoDisplayer.format(ITerritory) | 23 | 1 | 0 |
| battlehex.BattleHex.main(String[]) | 20 | 8 | 1 |
| battlehex.BattleHex.main(String[]) | 20 | 8 | 1 |
| battlehex.ui.main.ControlDisplay.createBox(JComponent,JComponent,JComponent) | 18 | 1 | 0 |
| battlehex.ui.main.GridDisplayer.GridDisplayer(IController,IContext,IHexagonBuilder,JComponent) | 17 | 7 | 1 |
| battlehex.model.map.SubTerritoryIterator.extractNeighbours(Vector,ITile) | 16 | 4 | 0 |
| battlehex.model.command.Move.check(IContext) | 13 | 7 | 0 |
| battlehex.ui.main.GameDisplayer.GameDisplayer(JComponent,JComponent) | 13 | 5 | 1 |
| battlehex.ui.main.TerritoryDisplayer.TerritoryDisplayer(IController,IContext,IHexagonBuilder) | 13 | 7 | 1 |
| battlehex.ui.main.TileComponent.TileComponent(ITile,Polygon) | 13 | 5 | 1 |
| battlehex.controller.Controller.findTerritory(IGrid,Coordinate) | 12 | 9 | 0 |
| battlehex.model.command.CastleBuy.check(IContext) | 12 | 7 | 0 |
| battlehex.ui.main.GridDisplayer.addTiles() | 12 | 5 | 0 |
| battlehex.ui.startup.GameTypeChoiceView.GameTypeChoiceView(IGameFactory) | 12 | 3 | 1 |
| battlehex.controller.TerritorySelectedState.TerritorySelectedState(IConsumer,ITerritoryObserver,ITerritory) | 11 | 7 | 1 |
| battlehex.GameFactory.createClient(String,int) | 11 | 1 | 1 |
| battlehex.model.command.EndTurn.execute(IContext) | 11 | 3 | 1 |
| battlehex.model.command.IncomePayment.execute(IContext) | 11 | 3 | 1 |
| battlehex.ui.main.ControlDisplay.createSubPanel(JComponent,Border) | 11 | 1 | 0 |
| battlehex.ui.main.CurrentPlayerDisplayer.CurrentPlayerDisplayer(IContext,IHexagonBuilder) | 11 | 5 | 1 |
| battlehex.ui.main.GridDisplayer.update(IGrid) | 11 | 2 | 1 |
| battlehex.ui.main.TerritoryInfoDisplayer.TerritoryInfoDisplayer(IController,IContext) | 11 | 5 | 1 |
| battlehex.BattleHex.usage(String) | 10 | 1 | 0 |
| battlehex.controller.PlacementPendingState.PlacementPendingState(IConsumer,ITerritoryObserver,ITerritory) | 10 | 7 | 1 |
| battlehex.BattleHex.usage(String) | 10 | 1 | 0 |
| battlehex.model.command.CastleBuy.execute(IContext) | 10 | 3 | 1 |
| battlehex.model.command.Game.endTurn(IGameHandler) | 10 | 5 | 1 |
| battlehex.model.map.Territory.setCapital(ITile) | 10 | 5 | 0 |
| battlehex.model.map.Territory.merge(ITerritory) | 10 | 4 | 1 |
| battlehex.ui.main.TerritoryInfoDisplayer.select(IGrid,ITerritory) | 10 | 3 | 1 |
| Program NCSS: | NCSS average | CCN average | Javadocs average |
|---|---|---|---|
| 1,633.00 | 4.39 | 1.76 | 0.66 |
[ packages] [ objects] [ functions] [ explanations]
Statements for JavaNCSS are not statements as specified in the Java Language Specification but include all kinds of declarations too. Roughly spoken, NCSS is approximately equivalent to counting ';' and '{' characters in Java source files.
Actually, the NCSS counter gets incremented by one for each:
| Examples | Comment | |
|---|---|---|
| Package declaration | package java.lang; | |
| Import declaration | import java.awt.*; | |
| Class declaration | - public class Foo {
- public class Foo extends Bla { | |
| Interface declaration | public interface Able ; { | |
| Field declaration | - int a;
- int a, b, c = 5, d = 6; | No matter how many fields get actually declared through a comma separated list, and no matter if these fields get actually initialized, only one statement is counted. So "int a, b, c = 5, d = 6;" gets only +1 count, not four or even six (let me know if there is good reason to count it differently). |
| Method declaration | - public void cry();
- public void gib() throws DeadException { | |
| Constructor declaration | public Foo() { | |
| Constructor invocation | - this();
- super(); | |
| Statement | - i = 0;
- if (ok) - if (exit) { - if (3 == 4); - if (4 == 4) { ; } - } else { | expression, if, else, while, do, for, switch, break, continue, return, throw, synchronized, catch, finally |
| Label | fine : | normal, case, default |
Not counted are empty statements, empty blocks or semicolons after closing brackets. Of course, comments don't get counted too. Closing brackets also never get counted, the same applies to blocks in general.
In some cases consecutive semicolons are illegal according to the JLS but JavaNCSS still tolerates them (thought JavaNCSS is still more strict as 'javac'). Nevertheless they are never counted as two statements.
CCN is also know as McCabe Metric. There exists a much hyped theory behind it based on graph theory, but it all comes down to simply counting 'if', 'for', 'while' statements etc. in a method. Whenever the control flow of a method splits, the "CCN counter" gets incremented by one.
Each method has a minimum value of 1 per default. For each of the following Java keywords/statements this value gets incremented by one:
An ordinary return at the end of method (no matter if it's a function or a procedure) will not be counted.
Note that else, default, and finallydon't increment the CCN value any further. On the other hand, a simple method with a switchstatement and a huge block of casestatements can have a surprisingly high CCN value (still it has the same value when converting a switchblock to an equivalent sequence of ifstatements).
The first article about this trivial software metric is one of the most cited papers in computer science. Sometimes you just must be the first to point out some basic shit and you are history. It seems just like a wonder that nobody was able to package Lines of Code in a more scientific manner. Now it's too late to label someone's name on it :).
