Sign upLogin

prowide/prowide-core

View on GitHub
Star
src/test/java/com/prowidesoftware/swift/model/field/Field98KTest.java

Summary

Maintainability
A
1 hr
Test Coverage

Method testField98KString has 37 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    @Test
    public void testField98KString() {
        Field98K f;

        f = new Field98K("");
Severity: Minor
Found in src/test/java/com/prowidesoftware/swift/model/field/Field98KTest.java - About 1 hr to fix

    Define a constant instead of duplicating this literal "11111111" 3 times.
    Open

            assertEquals("11111111", f.getComponent3());
    Severity: Critical
    Found in src/test/java/com/prowidesoftware/swift/model/field/Field98KTest.java by sonar-java

    Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

    On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

    Noncompliant Code Example

    With the default threshold of 3:

    public void run() {
  prepare("action1");                              // Noncompliant - "action1" is duplicated 3 times
  execute("action1");
  release("action1");
}

@SuppressWarning("all")                            // Compliant - annotations are excluded
private void method1() { /* ... */ }
@SuppressWarning("all")
private void method2() { /* ... */ }

public String method3(String a) {
  System.out.println("'" + a + "'");               // Compliant - literal "'" has less than 5 characters and is excluded
  return "";                                       // Compliant - literal "" has less than 5 characters and is excluded
}

    Compliant Solution

    private static final String ACTION_1 = "action1";  // Compliant

public void run() {
  prepare(ACTION_1);                               // Compliant
  execute(ACTION_1);
  release(ACTION_1);
}

    Exceptions

    To prevent generating some false-positives, literals having less than 5 characters are excluded.

    There are no issues that match your filters.

    Category
    Status