FindBugs Bug Detector Report

The following document contains the results of FindBugs

FindBugs Version is 3.0.1

Threshold is medium

Effort is min

Summary

Classes Bugs Errors Missing Classes
109 64 0 0

Files

Class Bugs
loadfromxml.LoadXML 1
org.opentrafficsim.ahfe.AHFEAnimation$1 2
org.opentrafficsim.ahfe.AHFEAnimation$AHFEModel 3
org.opentrafficsim.ahfe.AHFESimulation$1 2
org.opentrafficsim.ahfe.AHFESimulation$AHFEModel 3
org.opentrafficsim.ahfe.AHFEUtil 1
org.opentrafficsim.ahfe.DistractionScript 3
org.opentrafficsim.ahfe.DistractionScript$DistractionFactorySupplier 1
org.opentrafficsim.demo.CircularLaneModel 1
org.opentrafficsim.demo.CircularRoadModel 1
org.opentrafficsim.demo.CircularRoadSwing 2
org.opentrafficsim.demo.CrossingTrafficLightsModel 1
org.opentrafficsim.demo.NetworksSwing 1
org.opentrafficsim.demo.SequentialLanes$SequentialModel 1
org.opentrafficsim.demo.ShortMerge$ShortMergeModel 2
org.opentrafficsim.demo.carFollowing.OSMModel 1
org.opentrafficsim.demo.conflict.BusStreetDemo$BusStreetModel 2
org.opentrafficsim.demo.conflict.BusStreetDemo$CharacteristicsGenerator 1
org.opentrafficsim.demo.conflict.TJunctionDemo$TJunctionModel 1
org.opentrafficsim.demo.conflict.TestNetworkDemo$TestNetworkModel 1
org.opentrafficsim.demo.conflict.TurboRoundaboutDemo$TurboRoundaboutModel 1
org.opentrafficsim.demo.conflictAndControl.DemoTrafcodAndTurbo$TrafCODModel 3
org.opentrafficsim.demo.geometry.shape.ShapeFileReader 5
org.opentrafficsim.demo.geometry.shape.ShapeModel 2
org.opentrafficsim.demo.sdm.SdmSimulation 1
org.opentrafficsim.demo.sdm.SdmStrategicalPlannerFactory$SdmPerception 1
org.opentrafficsim.demo.steering.SteeringSimulation 1
org.opentrafficsim.demo.trafficcontrol.TrafCODDemo$TrafCODModel 3
org.opentrafficsim.demo.trafficcontrol.TrafCODDemo2$TrafCODModel 3
strategies.LmrsStrategies 2
strategies.LmrsStrategies$LmrsStrategiesAnimation 1
strategies.LmrsStrategies$LmrsStrategiesModel 1
strategies.LmrsStrategies$LmrsStrategiesSimulation 1
strategies.LmrsStrategies$SigmaMeasurement 1
strategies.LmrsStrategies$VDes0Measurement 1
strategies.LmrsStrategies$VDesMeasurement 1
strategies.LmrsStrategies$VGainMeasurement 1
strategies.StrategiesDemo 2
strategies.StrategiesDemo$LmrsStrategiesPerceptionFactory 1
strategies.StrategiesDemo$SocioIDMFactory 1

loadfromxml.LoadXML

Bug Category Details Line Priority
Found reliance on default encoding in loadfromxml.LoadXML.main(String[]): new String(byte[]) I18N DM_DEFAULT_ENCODING 135 High

org.opentrafficsim.ahfe.AHFEAnimation$1

Bug Category Details Line Priority
Found reliance on default encoding in org.opentrafficsim.ahfe.AHFEAnimation$1.run(): new java.io.OutputStreamWriter(OutputStream) I18N DM_DEFAULT_ENCODING 337 High
org.opentrafficsim.ahfe.AHFEAnimation$1.run() invokes System.exit(...), which shuts down the entire virtual machine BAD_PRACTICE DM_EXIT 384 Medium

org.opentrafficsim.ahfe.AHFEAnimation$AHFEModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.ahfe.AHFEAnimation$AHFEModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 496 Medium
Class org.opentrafficsim.ahfe.AHFEAnimation$AHFEModel defines non-transient non-serializable instance field sampler BAD_PRACTICE SE_BAD_FIELD Not available Medium
org.opentrafficsim.road.network.sampling.RoadSampler stored into non-transient field AHFEAnimation$AHFEModel.sampler BAD_PRACTICE SE_BAD_FIELD_STORE 492 Medium

org.opentrafficsim.ahfe.AHFESimulation$1

Bug Category Details Line Priority
Found reliance on default encoding in org.opentrafficsim.ahfe.AHFESimulation$1.run(): new java.io.OutputStreamWriter(OutputStream) I18N DM_DEFAULT_ENCODING 325 High
org.opentrafficsim.ahfe.AHFESimulation$1.run() invokes System.exit(...), which shuts down the entire virtual machine BAD_PRACTICE DM_EXIT 372 Medium

org.opentrafficsim.ahfe.AHFESimulation$AHFEModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.ahfe.AHFESimulation$AHFEModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 478 Medium
Class org.opentrafficsim.ahfe.AHFESimulation$AHFEModel defines non-transient non-serializable instance field sampler BAD_PRACTICE SE_BAD_FIELD Not available Medium
org.opentrafficsim.road.network.sampling.RoadSampler stored into non-transient field AHFESimulation$AHFEModel.sampler BAD_PRACTICE SE_BAD_FIELD_STORE 473 Medium

org.opentrafficsim.ahfe.AHFEUtil

Bug Category Details Line Priority
Bad attempt to compute absolute value of signed random integer in org.opentrafficsim.ahfe.AHFEUtil.createDemand(OTSNetwork, GTUColorer, OTSSimulatorInterface, int, String, Duration, Duration, double, Time, Frequency, Frequency, double, double, double, double) CORRECTNESS RV_ABSOLUTE_VALUE_OF_RANDOM_INT 157 High

org.opentrafficsim.ahfe.DistractionScript

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.ahfe.DistractionScript.setupSimulation(OTSSimulatorInterface) STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 206 Medium
Write to static field org.opentrafficsim.core.gtu.AbstractGTU.ALIGNED from instance method org.opentrafficsim.ahfe.DistractionScript.setupSimulation(OTSSimulatorInterface) STYLE ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD 202 High
Write to static field org.opentrafficsim.road.gtu.lane.plan.operational.LaneOperationalPlanBuilder.INSTANT_LANE_CHANGES from instance method org.opentrafficsim.ahfe.DistractionScript.setupSimulation(OTSSimulatorInterface) STYLE ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD 203 High

org.opentrafficsim.ahfe.DistractionScript$DistractionFactorySupplier

Bug Category Details Line Priority
Should org.opentrafficsim.ahfe.DistractionScript$DistractionFactorySupplier be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 316-373 Medium

org.opentrafficsim.demo.CircularLaneModel

Bug Category Details Line Priority
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.CircularLaneModel.constructModel() STYLE REC_CATCH_EXCEPTION 209 Medium

org.opentrafficsim.demo.CircularRoadModel

Bug Category Details Line Priority
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.CircularRoadModel.constructModel() STYLE REC_CATCH_EXCEPTION 222 Medium

org.opentrafficsim.demo.CircularRoadSwing

Bug Category Details Line Priority
Useless object stored in variable lanes of method org.opentrafficsim.demo.CircularRoadSwing.addStatisticsTabs(OTSSimulatorInterface) STYLE UC_USELESS_OBJECT 161 Medium
Useless object stored in variable positions of method org.opentrafficsim.demo.CircularRoadSwing.addStatisticsTabs(OTSSimulatorInterface) STYLE UC_USELESS_OBJECT 162 Medium

org.opentrafficsim.demo.CrossingTrafficLightsModel

Bug Category Details Line Priority
Useless object stored in variable trafficLights of method org.opentrafficsim.demo.CrossingTrafficLightsModel.constructModel() STYLE UC_USELESS_OBJECT 149 Medium

org.opentrafficsim.demo.NetworksSwing

Bug Category Details Line Priority
Integer remainder modulo 1 computed in org.opentrafficsim.demo.NetworksSwing.addStatisticsTabs(OTSSimulatorInterface) STYLE INT_BAD_REM_BY_1 150 High

org.opentrafficsim.demo.SequentialLanes$SequentialModel

Bug Category Details Line Priority
Useless object stored in variable links of method org.opentrafficsim.demo.SequentialLanes$SequentialModel.constructModel() STYLE UC_USELESS_OBJECT 295 Medium

org.opentrafficsim.demo.ShortMerge$ShortMergeModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.ShortMerge$ShortMergeModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 243 Medium
Bad attempt to compute absolute value of signed random integer in org.opentrafficsim.demo.ShortMerge$ShortMergeModel.addGenerator() CORRECTNESS RV_ABSOLUTE_VALUE_OF_RANDOM_INT 279 High

org.opentrafficsim.demo.carFollowing.OSMModel

Bug Category Details Line Priority
Class org.opentrafficsim.demo.carFollowing.OSMModel defines non-transient non-serializable instance field converter BAD_PRACTICE SE_BAD_FIELD Not available Medium

org.opentrafficsim.demo.conflict.BusStreetDemo$BusStreetModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.conflict.BusStreetDemo$BusStreetModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 177 Medium
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.conflict.BusStreetDemo$BusStreetModel.constructModel() STYLE REC_CATCH_EXCEPTION 224 Medium

org.opentrafficsim.demo.conflict.BusStreetDemo$CharacteristicsGenerator

Bug Category Details Line Priority
new org.opentrafficsim.demo.conflict.BusStreetDemo$CharacteristicsGenerator(DEVSSimulatorInterface$TimeDoubleUnit, double[], OTSNetwork) may expose internal representation by storing an externally mutable object into BusStreetDemo$CharacteristicsGenerator.probabilities MALICIOUS_CODE EI_EXPOSE_REP2 355 Medium

org.opentrafficsim.demo.conflict.TJunctionDemo$TJunctionModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.conflict.TJunctionDemo$TJunctionModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 121 Medium

org.opentrafficsim.demo.conflict.TestNetworkDemo$TestNetworkModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.conflict.TestNetworkDemo$TestNetworkModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 118 Medium

org.opentrafficsim.demo.conflict.TurboRoundaboutDemo$TurboRoundaboutModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.conflict.TurboRoundaboutDemo$TurboRoundaboutModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 122 Medium

org.opentrafficsim.demo.conflictAndControl.DemoTrafcodAndTurbo$TrafCODModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.conflictAndControl.DemoTrafcodAndTurbo$TrafCODModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 247 Medium
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.conflictAndControl.DemoTrafcodAndTurbo$TrafCODModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 154 Medium
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.conflictAndControl.DemoTrafcodAndTurbo$TrafCODModel.constructModel() STYLE REC_CATCH_EXCEPTION 278 Medium

org.opentrafficsim.demo.geometry.shape.ShapeFileReader

Bug Category Details Line Priority
Dead store to capacity in org.opentrafficsim.demo.geometry.shape.ShapeFileReader.readLinks(Network, String, Map, Map, OTSSimulatorInterface) STYLE DLS_DEAD_LOCAL_STORE 241 Medium
Dead store to length in org.opentrafficsim.demo.geometry.shape.ShapeFileReader.readLinks(Network, String, Map, Map, OTSSimulatorInterface) STYLE DLS_DEAD_LOCAL_STORE 229 Medium
Dead store to speed in org.opentrafficsim.demo.geometry.shape.ShapeFileReader.readLinks(Network, String, Map, Map, OTSSimulatorInterface) STYLE DLS_DEAD_LOCAL_STORE 239 Medium
Redundant nullcheck of nr, which is known to be non-null in org.opentrafficsim.demo.geometry.shape.ShapeFileReader.readNodes(Network, String, String, boolean, boolean) STYLE RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE 126 Medium
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.geometry.shape.ShapeFileReader.readLinks(Network, String, Map, Map, OTSSimulatorInterface) STYLE REC_CATCH_EXCEPTION 282 Medium

org.opentrafficsim.demo.geometry.shape.ShapeModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.geometry.shape.ShapeModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 56 Medium
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.geometry.shape.ShapeModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 54 Medium

org.opentrafficsim.demo.sdm.SdmSimulation

Bug Category Details Line Priority
Found reliance on default encoding in org.opentrafficsim.demo.sdm.SdmSimulation.onSimulationEnd(): new java.io.OutputStreamWriter(OutputStream) I18N DM_DEFAULT_ENCODING 473 High

org.opentrafficsim.demo.sdm.SdmStrategicalPlannerFactory$SdmPerception

Bug Category Details Line Priority
Should org.opentrafficsim.demo.sdm.SdmStrategicalPlannerFactory$SdmPerception be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 146-178 Medium

org.opentrafficsim.demo.steering.SteeringSimulation

Bug Category Details Line Priority
Useless object stored in variable originLanes of method org.opentrafficsim.demo.steering.SteeringSimulation.setupSimulation(OTSSimulatorInterface) STYLE UC_USELESS_OBJECT 149 Medium

org.opentrafficsim.demo.trafficcontrol.TrafCODDemo$TrafCODModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo$TrafCODModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 146 Medium
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo$TrafCODModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 193 Medium
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo$TrafCODModel.constructModel() STYLE REC_CATCH_EXCEPTION 215 Medium

org.opentrafficsim.demo.trafficcontrol.TrafCODDemo2$TrafCODModel

Bug Category Details Line Priority
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo2$TrafCODModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 190 Medium
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo2$TrafCODModel.constructModel() STYLE DMI_HARDCODED_ABSOLUTE_FILENAME 146 Medium
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo2$TrafCODModel.constructModel() STYLE REC_CATCH_EXCEPTION 212 Medium

strategies.LmrsStrategies

Bug Category Details Line Priority
strategies.LmrsStrategies.notify(EventInterface) invokes System.exit(...), which shuts down the entire virtual machine BAD_PRACTICE DM_EXIT 1004 Medium
Return value of new LmrsStrategies$LmrsStrategiesSimulation(LmrsStrategies, OTSModelInterface) ignored, but method has no side effect STYLE RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT 410 Medium

strategies.LmrsStrategies$LmrsStrategiesAnimation

Bug Category Details Line Priority
Should strategies.LmrsStrategies$LmrsStrategiesAnimation be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 499-512 Medium

strategies.LmrsStrategies$LmrsStrategiesModel

Bug Category Details Line Priority
strategies.LmrsStrategies$LmrsStrategiesModel.constructModel() makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 819 Medium

strategies.LmrsStrategies$LmrsStrategiesSimulation

Bug Category Details Line Priority
Should strategies.LmrsStrategies$LmrsStrategiesSimulation be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 471-473 Medium

strategies.LmrsStrategies$SigmaMeasurement

Bug Category Details Line Priority
Should strategies.LmrsStrategies$SigmaMeasurement be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 1021-1078 Medium

strategies.LmrsStrategies$VDes0Measurement

Bug Category Details Line Priority
Should strategies.LmrsStrategies$VDes0Measurement be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 1240-1299 Medium

strategies.LmrsStrategies$VDesMeasurement

Bug Category Details Line Priority
Should strategies.LmrsStrategies$VDesMeasurement be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 1167-1224 Medium

strategies.LmrsStrategies$VGainMeasurement

Bug Category Details Line Priority
Should strategies.LmrsStrategies$VGainMeasurement be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 1094-1151 Medium

strategies.StrategiesDemo

Bug Category Details Line Priority
Integral division result cast to double or float in strategies.StrategiesDemo.setupSimulation(OTSSimulatorInterface) STYLE ICAST_IDIV_CAST_TO_DOUBLE 562 Medium
Write to static field org.opentrafficsim.road.gtu.lane.plan.operational.LaneOperationalPlanBuilder.INSTANT_LANE_CHANGES from instance method strategies.StrategiesDemo.setupSimulation(OTSSimulatorInterface) STYLE ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD 492 Medium

strategies.StrategiesDemo$LmrsStrategiesPerceptionFactory

Bug Category Details Line Priority
Should strategies.StrategiesDemo$LmrsStrategiesPerceptionFactory be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 672-692 Medium

strategies.StrategiesDemo$SocioIDMFactory

Bug Category Details Line Priority
Should strategies.StrategiesDemo$SocioIDMFactory be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 652-667 Medium