Review

Detector:	MuDetect
Target:	project ' thomas-s-b-visualee ' version 410a80f
Misuse:	misuse ' 30 '
Tags:

Misuse Details

Details about the known misuse from the MUBench dataset.

Description:	Examiner.java calls 'scanner.next()' on 'java.util.Scanner scanner' without checking if there are more elements. Because the scanner is built from the JavaSource parameter that can be invalid (e.g., an empty source), this can lead to a runtime exception without a useful error message. This pull request adds an error message and a test.
Fix Description:	(see diff)
Violation Types:	missing/condition/value_or_state
In File:	de/strullerbaumann/visualee/examiner/Examiner.java
In Method:	jumpOverJavaToken(String, Scanner)
Code with Misuse:	`class Examiner { protected static String jumpOverJavaToken(String token, Scanner scanner) { String nextToken = token; while (isAJavaToken(nextToken)) { if (nextToken.startsWith("@") && nextToken.indexOf('(') > -1 && !nextToken.endsWith(")")) { nextToken = scanAfterClosedParenthesis(nextToken, scanner); } else { nextToken = scanner.next(); } } return nextToken; } }`

Potential Hits

Findings of the detector that identify an anomaly in the same file and method as the known misuse.

Hit	Rank	Confidence	Confidence String	Pattern Examples	Pattern Support	Pattern Violation	Target Environment Mapping
Yes	2	0.08333333333333333	(pattern support = 11 / 11)(pattern violations = 1 / 6)(overlap = 3.00 / 6.00)	thomas-s-b-visualee/410a80f/build/src/main/java/de/strullerbaumann/visualee/examiner/JavaSourceInspector.java#examine() thomas-s-b-visualee/410a80f/build/src/main/java/de/strullerbaumann/visualee/ui/graph/boundary/GraphCreator.java#generateGraphs(File, File, String) thomas-s-b-visualee/410a80f/build/src/main/java/de/strullerbaumann/visualee/ui/graph/boundary/GraphCreator.java#buildJSONLinks(DependencyFilter) thomas-s-b-visualee/410a80f/build/src/main/java/de/strullerbaumann/visualee/source/boundary/JavaSourceContainer.java#loadJavaFiles(String) thomas-s-b-visualee/410a80f/build/src/main/java/de/strullerbaumann/visualee/ui/graph/boundary/GraphConfigurator.java#setGraphConfigs(List)	11
No	26	0.027056277056277056	(pattern support = 10 / 11)(pattern violations = 1 / 28)(overlap = 5.00 / 6.00)	thomas-s-b-visualee/410a80f/build/src/main/java/de/strullerbaumann/visualee/examiner/jpa/ExaminerJPA.java#examineDetail(JavaSource, Scanner, String, DependencyType) thomas-s-b-visualee/410a80f/build/src/main/java/de/strullerbaumann/visualee/examiner/cdi/ExaminerInstance.java#examineDetail(JavaSource, Scanner, String, DependencyType) thomas-s-b-visualee/410a80f/build/src/main/java/de/strullerbaumann/visualee/source/entity/JavaSource.java#JavaSource(Path) thomas-s-b-visualee/410a80f/build/src/main/java/de/strullerbaumann/visualee/examiner/cdi/ExaminerInject.java#examineDetail(JavaSource, Scanner, String, DependencyType) thomas-s-b-visualee/410a80f/build/src/main/java/de/strullerbaumann/visualee/examiner/cdi/ExaminerEvent.java#examineDetail(JavaSource, Scanner, String, DependencyType)	10

Reviewer Name:	sven
Comment: