Sign upLogin

prasadtalasila/BITS-Darshini

View on GitHub
Star
src/main/java/in/ac/bits/protocolanalyzer/analyzer/TimeSeriesAnalyzer.java

Summary

Maintainability
A
0 mins
Test Coverage

Rename "standardDeviation" which hides the field declared at line 29.
Open

        double standardDeviation = Math.sqrt(StatUtils.populationVariance(
Severity: Major
Found in src/main/java/in/ac/bits/protocolanalyzer/analyzer/TimeSeriesAnalyzer.java by sonar-java

Overriding or shadowing a variable declared in an outer scope can strongly impact the readability, and therefore the maintainability, of a piece of code. Further, it could lead maintainers to introduce bugs because they think they're using one variable but are really using another.

Noncompliant Code Example

class Foo {
  public int myField;

  public void doSomething() {
    int myField = 0;
    ...
  }
}

See

Extra separation in import group before 'com.google.common.math.DoubleMath'
Open

import com.google.common.math.DoubleMath;
Severity: Minor
Found in src/main/java/in/ac/bits/protocolanalyzer/analyzer/TimeSeriesAnalyzer.java by checkstyle

Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Extra separation in import group before 'lombok.Getter'
Open

import lombok.Getter;
Severity: Minor
Found in src/main/java/in/ac/bits/protocolanalyzer/analyzer/TimeSeriesAnalyzer.java by checkstyle

Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Wrong lexicographical order for 'com.google.common.math.DoubleMath' import. Should be before 'org.springframework.stereotype.Service'.
Open

import com.google.common.math.DoubleMath;
Severity: Minor
Found in src/main/java/in/ac/bits/protocolanalyzer/analyzer/TimeSeriesAnalyzer.java by checkstyle

Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Extra separation in import group before 'org.apache.commons.math3.stat.StatUtils'
Open

import org.apache.commons.math3.stat.StatUtils;
Severity: Minor
Found in src/main/java/in/ac/bits/protocolanalyzer/analyzer/TimeSeriesAnalyzer.java by checkstyle

Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

Summary javadoc is missing.
Open

/**
Severity: Minor
Found in src/main/java/in/ac/bits/protocolanalyzer/analyzer/TimeSeriesAnalyzer.java by checkstyle

Checks thatJavadoc summary sentence does not contain phrases that are not recommended to use.Summaries that contain only the {@inheritDoc} tag are skipped. Check alsoviolate Javadoc that does not contain first sentence.

This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

There are no issues that match your filters.

Category
Status