The following document contains the results of FindBugs
FindBugs Version is 3.0.1
Threshold is medium
Effort is min
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.opentrafficsim.road.network.factory.xml.demand.XmlOdParser defines non-transient non-serializable instance field demand | BAD_PRACTICE | SE_BAD_FIELD | Not available | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to rg in org.opentrafficsim.road.network.factory.xml.old.FillTag.makeFill(FillTag, XmlNetworkLaneParserOld, LinkTag, DEVSSimulatorInterface$TimeDoubleUnit) | STYLE | DLS_DEAD_LOCAL_STORE | 224 | High |
Dead store of org.opentrafficsim.road.gtu.lane.LaneBasedIndividualGTU.class in org.opentrafficsim.road.network.factory.xml.old.FillTag.makeFill(FillTag, XmlNetworkLaneParserOld, LinkTag, DEVSSimulatorInterface$TimeDoubleUnit) | CORRECTNESS | DLS_DEAD_STORE_OF_CLASS_LITERAL | 217 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to generator in org.opentrafficsim.road.network.factory.xml.old.GeneratorTag.makeGenerator(GeneratorTag, XmlNetworkLaneParserOld, LinkTag, OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 362 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Class org.opentrafficsim.road.network.factory.xml.old.LinkTag defines non-transient non-serializable instance field bezierTag | BAD_PRACTICE | SE_BAD_FIELD | Not available | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Useless object stored in variable cseList of method org.opentrafficsim.road.network.factory.xml.old.Links.applyRoadTypeToLink(LinkTag, XmlNetworkLaneParserOld, OTSSimulatorInterface) | STYLE | UC_USELESS_OBJECT | 286 | Medium |
Useless object stored in variable lanes of method org.opentrafficsim.road.network.factory.xml.old.Links.applyRoadTypeToLink(LinkTag, XmlNetworkLaneParserOld, OTSSimulatorInterface) | STYLE | UC_USELESS_OBJECT | 287 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unwritten field: org.opentrafficsim.road.network.factory.xml.old.ListGeneratorTag.gtuColorer | CORRECTNESS | UWF_UNWRITTEN_FIELD | 143 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Write to static field org.opentrafficsim.road.network.factory.xml.old.GTUColorerTag.defaultColorer from instance method new org.opentrafficsim.road.network.factory.xml.old.XmlNetworkLaneParserOld(OTSSimulatorInterface, GTUColorer) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 172 | High |
Useless object stored in variable templates of method org.opentrafficsim.road.network.factory.xml.old.XmlNetworkLaneParserOld.fixOD(OTSRoadNetwork) | STYLE | UC_USELESS_OBJECT | 562 | Medium |
org.opentrafficsim.road.network.factory.xml.old.XmlNetworkLaneParserOld.build(InputStream, OTSRoadNetwork, boolean) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 370 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of networkGtuType, which is known to be non-null in org.opentrafficsim.road.network.factory.xml.parser.DefinitionsParser.parseGtuTypes(DEFINITIONS, OTSRoadNetwork, boolean) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 88 | Medium |
Redundant nullcheck of networkLaneType, which is known to be non-null in org.opentrafficsim.road.network.factory.xml.parser.DefinitionsParser.parseLaneTypes(DEFINITIONS, OTSRoadNetwork, boolean) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 181 | Medium |
Redundant nullcheck of networkLinkType, which is known to be non-null in org.opentrafficsim.road.network.factory.xml.parser.DefinitionsParser.parseLinkTypes(DEFINITIONS, OTSRoadNetwork, boolean, Map) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 132 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to categoryFactors in org.opentrafficsim.road.network.factory.xml.parser.DemandParser.parseDemand(OTSRoadNetwork, OTSSimulatorInterface, List, Map, Map, Map, Map) | STYLE | DLS_DEAD_LOCAL_STORE | 151 | Medium |
Useless object stored in variable categoryFactors of method org.opentrafficsim.road.network.factory.xml.parser.DemandParser.parseDemand(OTSRoadNetwork, OTSSimulatorInterface, List, Map, Map, Map, Map) | STYLE | UC_USELESS_OBJECT | 199 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.opentrafficsim.road.network.factory.xml.parser.GeneratorSinkParser.parseGenerators(OTSRoadNetwork, NETWORKDEMAND, Map, OTSSimulatorInterface, Map) | STYLE | REC_CATCH_EXCEPTION | 210 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of nodeTo, which is known to be non-null in org.opentrafficsim.road.network.factory.xml.parser.NetworkParser.parseShortestRoutes(OTSRoadNetwork, NETWORK) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 811 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Dead store to animation in org.opentrafficsim.road.network.factory.xml.parser.XmlNetworkLaneParser.build(InputStream, OTSRoadNetwork, OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 271 | Medium |
Dead store to generators in org.opentrafficsim.road.network.factory.xml.parser.XmlNetworkLaneParser.build(InputStream, OTSRoadNetwork, OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 241 | Medium |
Dead store to modelParameters in org.opentrafficsim.road.network.factory.xml.parser.XmlNetworkLaneParser.build(InputStream, OTSRoadNetwork, OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 269 | Medium |
Dead store to scenario in org.opentrafficsim.road.network.factory.xml.parser.XmlNetworkLaneParser.build(InputStream, OTSRoadNetwork, OTSSimulatorInterface) | STYLE | DLS_DEAD_LOCAL_STORE | 270 | Medium |
Hard coded reference to an absolute pathname in org.opentrafficsim.road.network.factory.xml.parser.XmlNetworkLaneParser.main(String[]) | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 285 | Medium |