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
258 78 0 0

Files

Class Bugs
org.opentrafficsim.graphs.Axis 2
org.opentrafficsim.graphs.ContourPlot 2
org.opentrafficsim.graphs.FundamentalDiagram$Sample 1
org.opentrafficsim.graphs.FundamentalDiagramLane$Sample 1
org.opentrafficsim.graphs.TrajectoryPlot 1
org.opentrafficsim.graphs.TrajectoryPlot$FixedSampleRateTrajectory 1
org.opentrafficsim.graphs.TrajectoryPlot$VariableSampleRateTrajectory 2
org.opentrafficsim.road.gtu.animation.DefaultCarAnimation 1
org.opentrafficsim.road.gtu.generator.LaneBasedGTUGenerator 1
org.opentrafficsim.road.gtu.generator.ListGTUGenerator 2
org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU 7
org.opentrafficsim.road.gtu.lane.perception.AbstractLanePerception 5
org.opentrafficsim.road.gtu.lane.perception.InfrastructureLaneChangeInfo 1
org.opentrafficsim.road.gtu.lane.perception.LaneStructure$Entry 1
org.opentrafficsim.road.gtu.lane.perception.categories.DefaultSimplePerception 1
org.opentrafficsim.road.gtu.lane.perception.categories.InfrastructurePerception 1
org.opentrafficsim.road.gtu.lane.perception.categories.NeighborsPerception 3
org.opentrafficsim.road.gtu.lane.plan.operational.LaneBasedOperationalPlan 1
org.opentrafficsim.road.gtu.lane.plan.operational.LaneOperationalPlanBuilder 3
org.opentrafficsim.road.gtu.lane.tactical.AbstractLaneBasedTacticalPlanner 1
org.opentrafficsim.road.gtu.lane.tactical.LaneBasedCFLCTacticalPlanner 3
org.opentrafficsim.road.gtu.lane.tactical.following.AbstractGTUFollowingModelMobil 1
org.opentrafficsim.road.gtu.lane.tactical.following.IDMPlusMulti 1
org.opentrafficsim.road.gtu.lane.tactical.lanechangemobil.AbstractLaneChangeModel 1
org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRS 3
org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRSFactory 1
org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo 1
org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo$GapAcceptanceInfo 2
org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo$GapInfo 2
org.opentrafficsim.road.gtu.strategical.od.ODMatrix 5
org.opentrafficsim.road.gtu.strategical.route.LaneBasedStrategicalRoutePlannerFactory 4
org.opentrafficsim.road.network.animation.DefaultSensorAnimation 1
org.opentrafficsim.road.network.animation.LaneAnimation 1
org.opentrafficsim.road.network.animation.ShoulderAnimation 1
org.opentrafficsim.road.network.animation.StripeAnimation 1
org.opentrafficsim.road.network.lane.CrossSectionLink 1
org.opentrafficsim.road.network.lane.Lane 5
org.opentrafficsim.road.network.lane.object.LaneBlockAnimation 1
org.opentrafficsim.road.network.lane.object.sensor.SensorAnimation 1
org.opentrafficsim.road.network.lane.object.sensor.SinkAnimation 1
org.opentrafficsim.road.network.lane.object.trafficlight.AbstractTrafficLightController 1
org.opentrafficsim.road.network.lane.object.trafficlight.TrafficLightAnimation 1
org.opentrafficsim.road.network.speed.SpeedLimitInfo 1

org.opentrafficsim.graphs.Axis

Bug Category Details Line Priority
org.opentrafficsim.graphs.Axis.getGranularities() may expose internal representation by returning Axis.granularities MALICIOUS_CODE EI_EXPOSE_REP 182 Medium
new org.opentrafficsim.graphs.Axis(DoubleScalarInterface, DoubleScalarInterface, double[], double, String, String, String) may expose internal representation by storing an externally mutable object into Axis.granularities MALICIOUS_CODE EI_EXPOSE_REP2 60 Medium

org.opentrafficsim.graphs.ContourPlot

Bug Category Details Line Priority
org.opentrafficsim.graphs.ContourPlot.STANDARDDISTANCEGRANULARITIES should be package protected MALICIOUS_CODE MS_PKGPROTECT 96 Medium
org.opentrafficsim.graphs.ContourPlot.STANDARDTIMEGRANULARITIES should be package protected MALICIOUS_CODE MS_PKGPROTECT 90 Medium

org.opentrafficsim.graphs.FundamentalDiagram$Sample

Bug Category Details Line Priority
org.opentrafficsim.graphs.FundamentalDiagram$Sample is serializable and an inner class BAD_PRACTICE SE_INNER_CLASS 464-520 Medium

org.opentrafficsim.graphs.FundamentalDiagramLane$Sample

Bug Category Details Line Priority
org.opentrafficsim.graphs.FundamentalDiagramLane$Sample is serializable and an inner class BAD_PRACTICE SE_INNER_CLASS 532-568 Medium

org.opentrafficsim.graphs.TrajectoryPlot

Bug Category Details Line Priority
org.opentrafficsim.graphs.TrajectoryPlot.sample() makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 195 Medium

org.opentrafficsim.graphs.TrajectoryPlot$FixedSampleRateTrajectory

Bug Category Details Line Priority
org.opentrafficsim.graphs.TrajectoryPlot$FixedSampleRateTrajectory is serializable and an inner class BAD_PRACTICE SE_INNER_CLASS 737-889 Medium

org.opentrafficsim.graphs.TrajectoryPlot$VariableSampleRateTrajectory

Bug Category Details Line Priority
Class org.opentrafficsim.graphs.TrajectoryPlot$VariableSampleRateTrajectory defines non-transient non-serializable instance field samples BAD_PRACTICE SE_BAD_FIELD Not available High
org.opentrafficsim.graphs.TrajectoryPlot$VariableSampleRateTrajectory is serializable and an inner class BAD_PRACTICE SE_INNER_CLASS 561-667 Medium

org.opentrafficsim.road.gtu.animation.DefaultCarAnimation

Bug Category Details Line Priority
org.opentrafficsim.road.gtu.animation.DefaultCarAnimation is Serializable but its superclass doesn't define an accessible void constructor BAD_PRACTICE SE_NO_SUITABLE_CONSTRUCTOR 50-151 High

org.opentrafficsim.road.gtu.generator.LaneBasedGTUGenerator

Bug Category Details Line Priority
Class org.opentrafficsim.road.gtu.generator.LaneBasedGTUGenerator defines non-transient non-serializable instance field roomChecker BAD_PRACTICE SE_BAD_FIELD Not available Medium

org.opentrafficsim.road.gtu.generator.ListGTUGenerator

Bug Category Details Line Priority
Found reliance on default encoding in new org.opentrafficsim.road.gtu.generator.ListGTUGenerator(String, OTSDEVSSimulatorInterface, GTUType, Speed, Lane, Length, GTUDirectionality, GTUColorer, LaneBasedStrategicalPlanner, OTSNetwork, String): new java.io.FileReader(File) I18N DM_DEFAULT_ENCODING 120 High
Class org.opentrafficsim.road.gtu.generator.ListGTUGenerator defines non-transient non-serializable instance field reader BAD_PRACTICE SE_BAD_FIELD Not available High

org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU

Bug Category Details Line Priority
org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU.scheduleEnterLeaveTriggers() invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead PERFORMANCE DM_BOOLEAN_CTOR 671 Medium
Possible null pointer dereference of initialLongitudinalPositions in org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU.init(LaneBasedStrategicalPlanner, Set, Speed) CORRECTNESS NP_NULL_ON_SOME_PATH 130 High
Useless object stored in variable fractionalLanePositions of method org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU.executeLaneChangeFinalization(LateralDirectionality) STYLE UC_USELESS_OBJECT 326 Medium
org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU.changeLaneInstantaneously(LateralDirectionality) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 259 Medium
org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU.initLaneChange(LateralDirectionality) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 292 Medium
org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU.scheduleEnterLeaveTriggers() makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 662 Medium
org.opentrafficsim.road.gtu.lane.AbstractLaneBasedGTU.scheduleEnterLeaveTriggers() makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 548 Medium

org.opentrafficsim.road.gtu.lane.perception.AbstractLanePerception

Bug Category Details Line Priority
Call to org.opentrafficsim.core.network.LinkDirection.equals(org.opentrafficsim.core.network.Link) in org.opentrafficsim.road.gtu.lane.perception.AbstractLanePerception.buildDownstreamRecursive(Set, GTUType, Length, Length, Length, Length) CORRECTNESS EC_UNRELATED_CLASS_AND_INTERFACE 299 Medium
Potentially dangerous use of non-short-circuit logic in org.opentrafficsim.road.gtu.lane.perception.AbstractLanePerception.extendLateral(Set, GTUType, Length, Length, Length, boolean) STYLE NS_DANGEROUS_NON_SHORT_CIRCUIT 408 High
Useless object stored in variable minStart of method org.opentrafficsim.road.gtu.lane.perception.AbstractLanePerception.buildUpstreamRecursive(Set, GTUType, Length, Length, Length) STYLE UC_USELESS_OBJECT 435 Medium
Useless object stored in variable laneSet of method org.opentrafficsim.road.gtu.lane.perception.AbstractLanePerception.startBuild(LaneStructureRecord, double, GTUType, Length, Length, Length, Length) STYLE UC_USELESS_OBJECT 164 Medium
org.opentrafficsim.road.gtu.lane.perception.AbstractLanePerception.connectLaterally(Map, GTUType) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 510 Medium

org.opentrafficsim.road.gtu.lane.perception.InfrastructureLaneChangeInfo

Bug Category Details Line Priority
org.opentrafficsim.road.gtu.lane.perception.InfrastructureLaneChangeInfo defines compareTo(InfrastructureLaneChangeInfo) and uses Object.equals() BAD_PRACTICE EQ_COMPARETO_USE_OBJECT_EQUALS 78 Medium

org.opentrafficsim.road.gtu.lane.perception.LaneStructure$Entry

Bug Category Details Line Priority
Should org.opentrafficsim.road.gtu.lane.perception.LaneStructure$Entry be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 346-451 Medium

org.opentrafficsim.road.gtu.lane.perception.categories.DefaultSimplePerception

Bug Category Details Line Priority
Call to org.opentrafficsim.road.gtu.lane.LaneBasedGTU.equals(org.opentrafficsim.road.gtu.lane.perception.categories.DefaultSimplePerception) in org.opentrafficsim.road.gtu.lane.perception.categories.DefaultSimplePerception.parallel(Lane, Time) CORRECTNESS EC_UNRELATED_CLASS_AND_INTERFACE 1018 Medium

org.opentrafficsim.road.gtu.lane.perception.categories.InfrastructurePerception

Bug Category Details Line Priority
org.opentrafficsim.road.gtu.lane.perception.categories.InfrastructurePerception.updateInfrastructureLaneChangeInfo(RelativeLane) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 149 Medium

org.opentrafficsim.road.gtu.lane.perception.categories.NeighborsPerception

Bug Category Details Line Priority
Private method org.opentrafficsim.road.gtu.lane.perception.categories.NeighborsPerception.forwardHeadway(Length) is never called PERFORMANCE UPM_UNCALLED_PRIVATE_METHOD 743-744 Medium
org.opentrafficsim.road.gtu.lane.perception.categories.NeighborsPerception.getFirstDownstreamGTUs(LateralDirectionality, RelativePosition$TYPE, RelativePosition$TYPE) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 218 Medium
org.opentrafficsim.road.gtu.lane.perception.categories.NeighborsPerception.getFirstUpstreamGTUs(LateralDirectionality, RelativePosition$TYPE, RelativePosition$TYPE) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 280 Medium

org.opentrafficsim.road.gtu.lane.plan.operational.LaneBasedOperationalPlan

Bug Category Details Line Priority
org.opentrafficsim.road.gtu.lane.plan.operational.LaneBasedOperationalPlan doesn't override org.opentrafficsim.core.gtu.plan.operational.OperationalPlan.equals(Object) STYLE EQ_DOESNT_OVERRIDE_EQUALS 1 Medium

org.opentrafficsim.road.gtu.lane.plan.operational.LaneOperationalPlanBuilder

Bug Category Details Line Priority
Dead store to fromFirst in org.opentrafficsim.road.gtu.lane.plan.operational.LaneOperationalPlanBuilder.buildAccelerationLaneChangePlan(LaneBasedGTU, List, LateralDirectionality, DirectedPoint, Time, Speed, Acceleration, Duration, LaneOperationalPlanBuilder$LaneChange) STYLE DLS_DEAD_LOCAL_STORE 388 Medium
Dead store to toFirst in org.opentrafficsim.road.gtu.lane.plan.operational.LaneOperationalPlanBuilder.buildAccelerationLaneChangePlan(LaneBasedGTU, List, LateralDirectionality, DirectedPoint, Time, Speed, Acceleration, Duration, LaneOperationalPlanBuilder$LaneChange) STYLE DLS_DEAD_LOCAL_STORE 389 Medium
Dead store to path in org.opentrafficsim.road.gtu.lane.plan.operational.LaneOperationalPlanBuilder.buildAccelerationPlan(LaneBasedGTU, List, Length, Time, Speed, Acceleration, Duration) STYLE DLS_DEAD_LOCAL_STORE 320 Medium

org.opentrafficsim.road.gtu.lane.tactical.AbstractLaneBasedTacticalPlanner

Bug Category Details Line Priority
org.opentrafficsim.road.gtu.lane.tactical.AbstractLaneBasedTacticalPlanner.buildPlanFromSimplePlan(LaneBasedGTU, Time, BehavioralCharacteristics, SimpleOperationalPlan, LaneOperationalPlanBuilder$LaneChange) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 639 Medium

org.opentrafficsim.road.gtu.lane.tactical.LaneBasedCFLCTacticalPlanner

Bug Category Details Line Priority
Dead store to currentLaneFine in org.opentrafficsim.road.gtu.lane.tactical.LaneBasedCFLCTacticalPlanner.generateOperationalPlan(Time, DirectedPoint) STYLE DLS_DEAD_LOCAL_STORE 148 Medium
Dead store to maximumReverseHeadway in org.opentrafficsim.road.gtu.lane.tactical.LaneBasedCFLCTacticalPlanner.generateOperationalPlan(Time, DirectedPoint) STYLE DLS_DEAD_LOCAL_STORE 125 Medium
Dead store to now in org.opentrafficsim.road.gtu.lane.tactical.LaneBasedCFLCTacticalPlanner.generateOperationalPlan(Time, DirectedPoint) STYLE DLS_DEAD_LOCAL_STORE 126 Medium

org.opentrafficsim.road.gtu.lane.tactical.following.AbstractGTUFollowingModelMobil

Bug Category Details Line Priority
Dead store to maximumSIDeceleration in org.opentrafficsim.road.gtu.lane.tactical.following.AbstractGTUFollowingModelMobil.minimumHeadway(Speed, Speed, Length, Length, Speed, Speed) STYLE DLS_DEAD_LOCAL_STORE 235 Medium

org.opentrafficsim.road.gtu.lane.tactical.following.IDMPlusMulti

Bug Category Details Line Priority
org.opentrafficsim.road.gtu.lane.tactical.following.IDMPlusMulti.combineInteractionTerm(Acceleration, BehavioralCharacteristics, Speed, Speed, Length, SortedMap) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 54 Medium

org.opentrafficsim.road.gtu.lane.tactical.lanechangemobil.AbstractLaneChangeModel

Bug Category Details Line Priority
org.opentrafficsim.road.gtu.lane.tactical.lanechangemobil.AbstractLaneChangeModel.computeLaneChangeAndAcceleration(LaneBasedGTU, Collection, Collection, Collection, Speed, Acceleration, Acceleration, Acceleration) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 58 Medium

org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRS

Bug Category Details Line Priority
Class org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRS defines non-transient non-serializable instance field lmrsData BAD_PRACTICE SE_BAD_FIELD Not available Medium
Class org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRS defines non-transient non-serializable instance field mandatoryIncentives BAD_PRACTICE SE_BAD_FIELD Not available Medium
Class org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRS defines non-transient non-serializable instance field voluntaryIncentives BAD_PRACTICE SE_BAD_FIELD Not available Medium

org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRSFactory

Bug Category Details Line Priority
Class org.opentrafficsim.road.gtu.lane.tactical.lmrs.LMRSFactory defines non-transient non-serializable instance field carFollowingModelFactory BAD_PRACTICE SE_BAD_FIELD Not available Medium

org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo

Bug Category Details Line Priority
Possible null pointer dereference of initiatedLaneChange in org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo.generateOperationalPlan(Time, DirectedPoint) CORRECTNESS NP_NULL_ON_SOME_PATH 257 Medium

org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo$GapAcceptanceInfo

Bug Category Details Line Priority
org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo$GapAcceptanceInfo is serializable and an inner class BAD_PRACTICE SE_INNER_CLASS 974-999 Medium
Should org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo$GapAcceptanceInfo be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 974-999 Medium

org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo$GapInfo

Bug Category Details Line Priority
org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo$GapInfo is serializable and an inner class BAD_PRACTICE SE_INNER_CLASS 897-940 Medium
Should org.opentrafficsim.road.gtu.lane.tactical.toledo.Toledo$GapInfo be a _static_ inner class? PERFORMANCE SIC_INNER_SHOULD_BE_STATIC 897-940 Medium

org.opentrafficsim.road.gtu.strategical.od.ODMatrix

Bug Category Details Line Priority
Dead store to e in org.opentrafficsim.road.gtu.strategical.od.ODMatrix.main(String[]) STYLE DLS_DEAD_LOCAL_STORE 510 High
Null passed for non-null parameter of new java.util.ArrayList(Collection) in new org.opentrafficsim.road.gtu.strategical.od.ODMatrix(String, List, List, Categorization, DurationVector, Interpolation) CORRECTNESS NP_NULL_PARAM_DEREF 91 Medium
Null passed for non-null parameter of new java.util.ArrayList(Collection) in new org.opentrafficsim.road.gtu.strategical.od.ODMatrix(String, List, List, Categorization, DurationVector, Interpolation) CORRECTNESS NP_NULL_PARAM_DEREF 90 Medium
Class org.opentrafficsim.road.gtu.strategical.od.ODMatrix defines non-transient non-serializable instance field demandData BAD_PRACTICE SE_BAD_FIELD Not available High
org.opentrafficsim.road.gtu.strategical.od.ODMatrix.print() makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 376 Medium

org.opentrafficsim.road.gtu.strategical.route.LaneBasedStrategicalRoutePlannerFactory

Bug Category Details Line Priority
Class org.opentrafficsim.road.gtu.strategical.route.LaneBasedStrategicalRoutePlannerFactory defines non-transient non-serializable instance field behavioralCharacteristicsFactory BAD_PRACTICE SE_BAD_FIELD Not available Medium
Class org.opentrafficsim.road.gtu.strategical.route.LaneBasedStrategicalRoutePlannerFactory defines non-transient non-serializable instance field routeGenerator BAD_PRACTICE SE_BAD_FIELD Not available Medium
org.opentrafficsim.core.gtu.behavioralcharacteristics.BehavioralCharacteristicsFactoryDefault stored into non-transient field LaneBasedStrategicalRoutePlannerFactory.behavioralCharacteristicsFactory BAD_PRACTICE SE_BAD_FIELD_STORE 58 High
org.opentrafficsim.core.gtu.behavioralcharacteristics.BehavioralCharacteristicsFactoryDefault stored into non-transient field LaneBasedStrategicalRoutePlannerFactory.behavioralCharacteristicsFactory BAD_PRACTICE SE_BAD_FIELD_STORE 86 High

org.opentrafficsim.road.network.animation.DefaultSensorAnimation

Bug Category Details Line Priority
org.opentrafficsim.road.network.animation.DefaultSensorAnimation is Serializable but its superclass doesn't define an accessible void constructor BAD_PRACTICE SE_NO_SUITABLE_CONSTRUCTOR 50-68 High

org.opentrafficsim.road.network.animation.LaneAnimation

Bug Category Details Line Priority
org.opentrafficsim.road.network.animation.LaneAnimation is Serializable but its superclass doesn't define an accessible void constructor BAD_PRACTICE SE_NO_SUITABLE_CONSTRUCTOR 51-86 High

org.opentrafficsim.road.network.animation.ShoulderAnimation

Bug Category Details Line Priority
org.opentrafficsim.road.network.animation.ShoulderAnimation is Serializable but its superclass doesn't define an accessible void constructor BAD_PRACTICE SE_NO_SUITABLE_CONSTRUCTOR 44-60 High

org.opentrafficsim.road.network.animation.StripeAnimation

Bug Category Details Line Priority
org.opentrafficsim.road.network.animation.StripeAnimation is Serializable but its superclass doesn't define an accessible void constructor BAD_PRACTICE SE_NO_SUITABLE_CONSTRUCTOR 65-244 High

org.opentrafficsim.road.network.lane.CrossSectionLink

Bug Category Details Line Priority
org.opentrafficsim.road.network.lane.CrossSectionLink doesn't override org.opentrafficsim.core.network.OTSLink.equals(Object) STYLE EQ_DOESNT_OVERRIDE_EQUALS 1 Medium

org.opentrafficsim.road.network.lane.Lane

Bug Category Details Line Priority
org.opentrafficsim.road.network.lane.Lane.checkDirectionality() makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 1378 Medium
org.opentrafficsim.road.network.lane.Lane.clone(CrossSectionLink, OTSSimulatorInterface, boolean) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 1487 Medium
org.opentrafficsim.road.network.lane.Lane.clone(CrossSectionLink, OTSSimulatorInterface, boolean) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 1473 Medium
org.opentrafficsim.road.network.lane.Lane.getLaneBasedObjectMap() makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 848 Medium
org.opentrafficsim.road.network.lane.Lane.getObjectAhead(Length, GTUDirectionality) makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 1020 Medium

org.opentrafficsim.road.network.lane.object.LaneBlockAnimation

Bug Category Details Line Priority
org.opentrafficsim.road.network.lane.object.LaneBlockAnimation is Serializable but its superclass doesn't define an accessible void constructor BAD_PRACTICE SE_NO_SUITABLE_CONSTRUCTOR 37-85 High

org.opentrafficsim.road.network.lane.object.sensor.SensorAnimation

Bug Category Details Line Priority
org.opentrafficsim.road.network.lane.object.sensor.SensorAnimation is Serializable but its superclass doesn't define an accessible void constructor BAD_PRACTICE SE_NO_SUITABLE_CONSTRUCTOR 52-71 High

org.opentrafficsim.road.network.lane.object.sensor.SinkAnimation

Bug Category Details Line Priority
org.opentrafficsim.road.network.lane.object.sensor.SinkAnimation is Serializable but its superclass doesn't define an accessible void constructor BAD_PRACTICE SE_NO_SUITABLE_CONSTRUCTOR 46-63 High

org.opentrafficsim.road.network.lane.object.trafficlight.AbstractTrafficLightController

Bug Category Details Line Priority
Unread public/protected field: org.opentrafficsim.road.network.lane.object.trafficlight.AbstractTrafficLightController.currentColor STYLE URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD 38 Medium

org.opentrafficsim.road.network.lane.object.trafficlight.TrafficLightAnimation

Bug Category Details Line Priority
org.opentrafficsim.road.network.lane.object.trafficlight.TrafficLightAnimation is Serializable but its superclass doesn't define an accessible void constructor BAD_PRACTICE SE_NO_SUITABLE_CONSTRUCTOR 45-86 High

org.opentrafficsim.road.network.speed.SpeedLimitInfo

Bug Category Details Line Priority
org.opentrafficsim.road.network.speed.SpeedLimitInfo.toString() makes inefficient use of keySet iterator instead of entrySet iterator PERFORMANCE WMI_WRONG_MAP_ITERATOR 127 Medium