The following document contains the results of SpotBugs
SpotBugs Version is 3.1.12
Threshold is medium
Effort is default
Classes | Bugs | Errors | Missing Classes |
---|---|---|---|
184 | 132 | 0 | 0 |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in loadfromxml.LoadXML.main(String[]): new String(byte[]) | I18N | DM_DEFAULT_ENCODING | 134 | High |
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 | 339 | High |
org.opentrafficsim.ahfe.AHFEAnimation$1.run() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 386 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.ahfe.AHFEAnimation$AHFEModel.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 498 | Medium |
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 | 326 | High |
org.opentrafficsim.ahfe.AHFESimulation$1.run() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 373 | Medium |
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 |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Bad attempt to compute absolute value of signed random integer in org.opentrafficsim.ahfe.AHFEUtil.createDemand(OTSRoadNetwork, GTUColorer, OTSSimulatorInterface, int, String, Duration, Duration, double, Time, Frequency, Frequency, double, double, double, double) | CORRECTNESS | RV_ABSOLUTE_VALUE_OF_RANDOM_INT | 153 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.opentrafficsim.ahfe.AbstractDelayedPerceptionCategory.changeLane(LateralDirectionality) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 184 | Medium |
org.opentrafficsim.ahfe.AbstractDelayedPerceptionCategory.changeLane(LateralDirectionality) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 176 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
There is an apparent infinite recursive loop in org.opentrafficsim.ahfe.AbstractDelayedPerceptionCategory$DelayedInfoType.getId() | CORRECTNESS | IL_INFINITE_RECURSIVE_LOOP | 231 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.ahfe.AnticipationRelianceScript.setupSimulation(OTSSimulatorInterface) | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 344 | Medium |
Write to static field org.opentrafficsim.core.gtu.AbstractGTU.ALIGNED from instance method org.opentrafficsim.ahfe.AnticipationRelianceScript.setupSimulation(OTSSimulatorInterface) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 341 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.opentrafficsim.ahfe.AnticipationRelianceScript$CarFollowingTaskAR be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 587-605 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.opentrafficsim.ahfe.AnticipationRelianceScript$IdmPlusFactoryAR be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 742-770 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.opentrafficsim.ahfe.AnticipationRelianceScript$LaneChangeTaskAR be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 615-625 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.opentrafficsim.ahfe.AnticipationRelianceScript$SituationalAwarenessDataType be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 854-874 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.opentrafficsim.ahfe.AnticipationRelianceScript$TaskAnticipationRelianceDataType be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 801-822 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.opentrafficsim.ahfe.AnticipationRelianceScript$TaskDemandDataType be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 828-848 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.opentrafficsim.ahfe.AnticipationRelianceScript$TaskManagerAR be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 704-738 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.opentrafficsim.ahfe.AnticipationRelianceScript$TaskSaturationDataType be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 775-795 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.opentrafficsim.ahfe.DelayedNeighborsPerception$ErrorValue be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 512-545 | Medium |
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 |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.CircularRoadModel.constructModel() | STYLE | REC_CATCH_EXCEPTION | 228 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Useless object stored in variable lanes of method org.opentrafficsim.demo.CircularRoadSwing.addStatisticsTabs(OTSSimulatorInterface) | STYLE | UC_USELESS_OBJECT | 165 | Medium |
Useless object stored in variable positions of method org.opentrafficsim.demo.CircularRoadSwing.addStatisticsTabs(OTSSimulatorInterface) | STYLE | UC_USELESS_OBJECT | 166 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Useless object stored in variable trafficLights of method org.opentrafficsim.demo.CrossingTrafficLightsModel.constructModel() | STYLE | UC_USELESS_OBJECT | 193 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Integer remainder modulo 1 computed in org.opentrafficsim.demo.NetworksSwing.addStatisticsTabs(OTSSimulatorInterface) | STYLE | INT_BAD_REM_BY_1 | 151 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to lanesBC in org.opentrafficsim.demo.RampMeteringDemo.setupSimulation(OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 284 | Medium |
Dead store to lanesFB in org.opentrafficsim.demo.RampMeteringDemo.setupSimulation(OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 292 | Medium |
Useless object stored in variable detectors34 of method org.opentrafficsim.demo.RampMeteringDemo.setupSimulation(OTSSimulatorInterface) | STYLE | UC_USELESS_OBJECT | 312 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.opentrafficsim.demo.RampMeteringDemo$ControlledTacticalPlanner defines non-transient non-serializable instance field laneChangeSystem | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Useless object stored in variable links of method org.opentrafficsim.demo.SequentialLanes$SequentialModel.constructModel() | STYLE | UC_USELESS_OBJECT | 299 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.ShortMerge$ShortMergeModel.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 242 | 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 | 276 | High |
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 |
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 | 178 | Medium |
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.conflict.BusStreetDemo$BusStreetModel.constructModel() | STYLE | REC_CATCH_EXCEPTION | 223 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
new org.opentrafficsim.demo.conflict.BusStreetDemo$CharacteristicsGenerator(DEVSSimulatorInterface$TimeDoubleUnit, double[], OTSRoadNetwork) may expose internal representation by storing an externally mutable object into BusStreetDemo$CharacteristicsGenerator.probabilities | MALICIOUS_CODE | EI_EXPOSE_REP2 | 365 | Medium |
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 |
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 | 119 | Medium |
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 | 121 | Medium |
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 | 251 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.conflictAndControl.DemoTrafcodAndTurbo$TrafCODModel.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 157 | Medium |
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.conflictAndControl.DemoTrafcodAndTurbo$TrafCODModel.constructModel() | STYLE | REC_CATCH_EXCEPTION | 282 | Medium |
Useless object stored in variable sensors of method org.opentrafficsim.demo.conflictAndControl.DemoTrafcodAndTurbo$TrafCODModel.constructModel() | STYLE | UC_USELESS_OBJECT | 179 | Medium |
Useless object stored in variable trafficLights of method org.opentrafficsim.demo.conflictAndControl.DemoTrafcodAndTurbo$TrafCODModel.constructModel() | STYLE | UC_USELESS_OBJECT | 178 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to capacity in org.opentrafficsim.demo.geometry.shape.ShapeFileReader.readLinks(OTSRoadNetwork, String, Map, Map, OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 242 | Medium |
Dead store to length in org.opentrafficsim.demo.geometry.shape.ShapeFileReader.readLinks(OTSRoadNetwork, String, Map, Map, OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 230 | Medium |
Dead store to speed in org.opentrafficsim.demo.geometry.shape.ShapeFileReader.readLinks(OTSRoadNetwork, String, Map, Map, OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 240 | 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(OTSRoadNetwork, String, Map, Map, OTSSimulatorInterface) | STYLE | REC_CATCH_EXCEPTION | 283 | Medium |
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 | 57 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.geometry.shape.ShapeModel.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 55 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.Circuit$TestXMLModel.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 131 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.FourStop$TestXMLModel.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 131 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Format string should use %n rather than \n in org.opentrafficsim.demo.network.xml.Grid10X10.main(String[]) | BAD_PRACTICE | VA_FORMAT_STRING_USES_NEWLINE | 258 | Medium |
Format string should use %n rather than \n in org.opentrafficsim.demo.network.xml.Grid10X10.main(String[]) | BAD_PRACTICE | VA_FORMAT_STRING_USES_NEWLINE | 262 | Medium |
Format string should use %n rather than \n in org.opentrafficsim.demo.network.xml.Grid10X10.main(String[]) | BAD_PRACTICE | VA_FORMAT_STRING_USES_NEWLINE | 244 | Medium |
Format string should use %n rather than \n in org.opentrafficsim.demo.network.xml.Grid10X10.main(String[]) | BAD_PRACTICE | VA_FORMAT_STRING_USES_NEWLINE | 235 | Medium |
Format string should use %n rather than \n in org.opentrafficsim.demo.network.xml.Grid10X10.main(String[]) | BAD_PRACTICE | VA_FORMAT_STRING_USES_NEWLINE | 237 | Medium |
Format string should use %n rather than \n in org.opentrafficsim.demo.network.xml.Grid10X10.main(String[]) | BAD_PRACTICE | VA_FORMAT_STRING_USES_NEWLINE | 242 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to colorer in org.opentrafficsim.demo.network.xml.N201ODfactory.makeGeneratorsFromOD(OTSRoadNetwork, ODMatrix, OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 155 | High |
Dead store to northBound in org.opentrafficsim.demo.network.xml.N201ODfactory.getQuery(OTSRoadNetwork, Sampler) | STYLE | DLS_DEAD_LOCAL_STORE | 234 | Medium |
Dead store to route in org.opentrafficsim.demo.network.xml.N201ODfactory.makeGeneratorsFromOD(OTSRoadNetwork, ODMatrix, OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 167 | Medium |
Useless object stored in variable northBound of method org.opentrafficsim.demo.network.xml.N201ODfactory.getQuery(OTSRoadNetwork, Sampler) | STYLE | UC_USELESS_OBJECT | 234 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.NetworkTest$TestXMLModel.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 131 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserA58$TestXMLModelA58.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 143 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserA58$TestXMLModelA58.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 160 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserAimsun$TestAimsunModel.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 129 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserAmsterdam$TestXMLModelAmsterdam.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 166 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserAmsterdam$TestXMLModelAmsterdam.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 186 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserEindhoven$TestXMLModelEindhoven.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 161 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserEindhoven$TestXMLModelEindhoven.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 144 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserKPIs$TestXMLModelKPIs.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 153 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserKPIs$TestXMLModelKPIs.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 177 | Medium |
Class org.opentrafficsim.demo.network.xml.TestXMLParserKPIs$TestXMLModelKPIs defines non-transient non-serializable instance field meanSpeed | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Class org.opentrafficsim.demo.network.xml.TestXMLParserKPIs$TestXMLModelKPIs defines non-transient non-serializable instance field meanTravelTimePerDistance | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Class org.opentrafficsim.demo.network.xml.TestXMLParserKPIs$TestXMLModelKPIs defines non-transient non-serializable instance field meanTripLength | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Class org.opentrafficsim.demo.network.xml.TestXMLParserKPIs$TestXMLModelKPIs defines non-transient non-serializable instance field totalDelay | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Class org.opentrafficsim.demo.network.xml.TestXMLParserKPIs$TestXMLModelKPIs defines non-transient non-serializable instance field totalNumberOfStops | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Class org.opentrafficsim.demo.network.xml.TestXMLParserKPIs$TestXMLModelKPIs defines non-transient non-serializable instance field totalTravelDistance | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Class org.opentrafficsim.demo.network.xml.TestXMLParserKPIs$TestXMLModelKPIs defines non-transient non-serializable instance field totalTravelTime | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserN201$TestXMLModelN201.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 143 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserN201$TestXMLModelN201.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 160 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserReadXstream$TestXMLModelReadXStream.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 148 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserReadXstream$TestXMLModelReadXStream.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 133 | Medium |
Found reliance on default encoding in org.opentrafficsim.demo.network.xml.TestXMLParserReadXstream$TestXMLModelReadXStream.constructModel(): new String(byte[]) | I18N | DM_DEFAULT_ENCODING | 133 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserWriteXstream$TestXMLModelWriteXStream.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 136 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserWriteXstream$TestXMLModelWriteXStream.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 157 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.network.xml.TestXMLParserWriteXstream$TestXMLModelWriteXStream.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 148 | Medium |
Found reliance on default encoding in org.opentrafficsim.demo.network.xml.TestXMLParserWriteXstream$TestXMLModelWriteXStream.constructModel(): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 148 | High |
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 | 576 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should org.opentrafficsim.demo.sdm.SdmStrategicalPlannerFactory$SdmPerception be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 148-180 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Useless object stored in variable originLanes of method org.opentrafficsim.demo.steering.SteeringSimulation.setupSimulation(OTSSimulatorInterface) | STYLE | UC_USELESS_OBJECT | 172 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo1.demo(boolean) | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 117 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo1$TrafCODModel.constructModel() | STYLE | REC_CATCH_EXCEPTION | 276 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo2.demo(boolean) | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 111 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo2$TrafCODModel.constructModel() | STYLE | REC_CATCH_EXCEPTION | 245 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo2_Generators.demo(boolean) | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 101 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.trafficcontrol.TrafCODDemo2_Generators$TrafCODModel.constructModel() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 174 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.web.OTSDemoServer$MyResourceHandler.handle(String, Request, HttpServletRequest, HttpServletResponse) | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 326 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.web.OTSDemoServer$MyResourceHandler.handle(String, Request, HttpServletRequest, HttpServletResponse) | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 332 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Boxing/unboxing to parse a primitive org.opentrafficsim.demo.web.OTSDemoServer$XHRHandler.setParameters(OTSModelInterface, String) | PERFORMANCE | DM_BOXED_PRIMITIVE_FOR_PARSING | 718 | High |
Boxing/unboxing to parse a primitive org.opentrafficsim.demo.web.OTSDemoServer$XHRHandler.setParameters(OTSModelInterface, String) | PERFORMANCE | DM_BOXED_PRIMITIVE_FOR_PARSING | 713 | High |
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.web.OTSDemoServer$XHRHandler.setParameters(OTSModelInterface, String) | STYLE | REC_CATCH_EXCEPTION | 746 | Medium |
HTTP parameter written to Servlet output in org.opentrafficsim.demo.web.OTSDemoServer$XHRHandler.handle(String, Request, HttpServletRequest, HttpServletResponse) | SECURITY | XSS_REQUEST_PARAMETER_TO_SERVLET_WRITER | 465 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.web.OTSFederatedDemoServer$MyResourceHandler.handle(String, Request, HttpServletRequest, HttpServletResponse) | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 223 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.web.OTSFederatedDemoServer$MyResourceHandler.handle(String, Request, HttpServletRequest, HttpServletResponse) | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 229 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Boxing/unboxing to parse a primitive org.opentrafficsim.demo.web.OTSFederatedDemoServer$XHRHandler.setParameters(OTSModelInterface, String) | PERFORMANCE | DM_BOXED_PRIMITIVE_FOR_PARSING | 598 | High |
Boxing/unboxing to parse a primitive org.opentrafficsim.demo.web.OTSFederatedDemoServer$XHRHandler.setParameters(OTSModelInterface, String) | PERFORMANCE | DM_BOXED_PRIMITIVE_FOR_PARSING | 593 | High |
Exception is caught when Exception is not thrown in org.opentrafficsim.demo.web.OTSFederatedDemoServer$XHRHandler.setParameters(OTSModelInterface, String) | STYLE | REC_CATCH_EXCEPTION | 626 | Medium |
HTTP parameter written to Servlet output in org.opentrafficsim.demo.web.OTSFederatedDemoServer$XHRHandler.handle(String, Request, HttpServletRequest, HttpServletResponse) | SECURITY | XSS_REQUEST_PARAMETER_TO_SERVLET_WRITER | 345 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unread field: org.opentrafficsim.demo.web.SimpleLoadBalancer.messageCount | PERFORMANCE | URF_UNREAD_FIELD | 51 | Medium |
Unused field: org.opentrafficsim.demo.web.SimpleLoadBalancer.lbContext | PERFORMANCE | UUF_UNUSED_FIELD | Not available | Medium |
Unused field: org.opentrafficsim.demo.web.SimpleLoadBalancer.lbSocket | PERFORMANCE | UUF_UNUSED_FIELD | Not available | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
instanceof will always return true for all non-null values in org.opentrafficsim.demo.web.SuperDemoWebApplication.processSimRunControl(String, FM2SimRunControlMessage), since all org.sim0mq.message.types.NumberDuration are instances of Number | STYLE | BC_VACUOUS_INSTANCEOF | 308 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.web.SuperDemoWebApplication.init() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 143 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.demo.web.SuperDemoWebApplication.init() | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 149 | Medium |
org.opentrafficsim.demo.web.SuperDemoWebApplication.processKillFederate() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 526 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
strategies.LmrsStrategies.notify(EventInterface) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 1031 | Medium |
Return value of new LmrsStrategies$LmrsStrategiesSimulation(LmrsStrategies, OTSModelInterface) ignored, but method has no side effect | STYLE | RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT | 411 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should strategies.LmrsStrategies$LmrsStrategiesAnimation be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 500-513 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
strategies.LmrsStrategies$LmrsStrategiesModel.constructModel() makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 839 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should strategies.LmrsStrategies$LmrsStrategiesSimulation be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 472-474 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should strategies.LmrsStrategies$SigmaMeasurement be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 1048-1105 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should strategies.LmrsStrategies$VDes0Measurement be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 1267-1326 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should strategies.LmrsStrategies$VDesMeasurement be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 1194-1251 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should strategies.LmrsStrategies$VGainMeasurement be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 1121-1178 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Integral division result cast to double or float in strategies.StrategiesDemo.setupSimulation(OTSSimulatorInterface) | STYLE | ICAST_IDIV_CAST_TO_DOUBLE | 581 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should strategies.StrategiesDemo$LmrsStrategiesPerceptionFactory be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 692-712 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Should strategies.StrategiesDemo$SocioIDMFactory be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 672-687 | Medium |