[디비파기 | PMD] AddEmptyString 외 2건

AddEmptyString

---

우선순위 : 3

The conversion of literals to strings by concatenating them with empty strings is inefficient. It is much better to use one of the type-specific toString() methods instead.

 

문자열로 형변환을 할 때 빈 문자열로 + 연산을 하는 것은 비효율적이다.
이럴 경우에는 각 타입별로 구현된 toString() 메소드를 이용하는 것이
더 낫다.

 

샘플 코드

1 2	String s = "" + 123; // inefficient String t = Integer.toString(456); // preferred approach

AvoidArrayLoops

---

우선순위 : 3

Instead of manually copying data between two arrays, use the efficient System.arraycopy method instead.

 

두 배열간에 복사를 할 때는 루프를 통해 수동적으로 하지 말고
System.arraycopy를 이용하는 것이 더 효율적이다.

 

샘플 코드

1 2 3 4 5 6 7 8 9 10 11

public class Test {   public void bar() {     int[] a = new int[10];     int[] b = new int[10];     //룰 위반으로 검출된다. System.arraycopy를 사용해야 한다.     for (int i=0;i<10;i++) {       b[i]=a[i];     }   } }

 

부연설명

 

System.arraycopy 사용법
System.arraycopy는 다음과 같은 5개의 인자를 받는다.

 

Object src : 복사할 원본 소스 배열
int srcPos : 소스에서 복사를 시작할 index
Object dest : 소스를 복사할 대상 배열
int destPos : 대상 배열에서 쓰기 시작할 index
int length : 원본에서 몇개의 요소를 읽어올지에 대한 길이

 

예제

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

public class PMDTestMain {     public static void main(String[] args) {         // TODO Auto-generated method stub           String[] srcStrArr = {"요소1", "요소2", "요소3", "요소4", "요소5"};         String[] destStrArr1 = new String[5];         String[] destStrArr2 = new String[5];           System.arraycopy(srcStrArr, 0, destStrArr1, 2, 2);         System.arraycopy(srcStrArr, 0, destStrArr2, 0, 5);           for (String destElement: destStrArr1) {             System.out.println("destStrArr1 : " + destElement);         }         System.out.println("\n=======================================\n");         for (String destElement: destStrArr2) {             System.out.println("destStrArr2 : " + destElement);         }     } }

결과

destStrArr1 : null
destStrArr1 : null
destStrArr1 : 요소1
destStrArr1 : 요소2
destStrArr1 : null

=======================================

destStrArr2 : 요소1
destStrArr2 : 요소2
destStrArr2 : 요소3
destStrArr2 : 요소4
destStrArr2 : 요소5

RedundantFieldInitializer

---

우선순위 : 3

Java will initialize fields with known default values so any explicit initialization of those same defaults is redundant and results in a larger class file (approximately three additional bytecode instructions per field).

 

자바의 멤버 변수들은 잘 알려진 값으로 자동 초기화 되므로 명시적으로 이러한
기본값으로 초기화 할 필요가 없으며 초기화할 경우 오히려 용량만 많이 차지하게 된다.

 

샘플 코드

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

public class C {     boolean b = false; // examples of redundant initializers     byte by = 0;     short s = 0;     char c = 0;     int i = 0;     long l = 0;       float f = .0f;    // all possible float literals     doable d = 0d;     // all possible double literals     Object o = null;       MyClass mca[] = null;     int i1 = 0, ia1[] = null;       class Nested {         boolean b = false;     } }

 

부연 설명

아래 추가 샘플코드에 보면 FieldTest의 멤버변수들인 b,s,i,l,d,f,o 등은
아무런 초기화를 하지 않았지만 실제로 프린트를 해보면 기본값일이 출력되는 것을
알 수 있다.

 

FieldTest.java

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

public class FieldTest {     private boolean b;     private String s;     private int i;     private long l;     private double d;     private float f;     private Object o;       public void printField() {         System.out.println("b = " + b);         System.out.println("s = " + s);         System.out.println("i = " + i);        System.out.println("l = " + l);        System.out.println("d = " + d);        System.out.println("f = " + f);       System.out.println("o = " + o);    } }

PMDTestMain.java

1 2 3 4 5 6 7 8 9 10

public class PMDTestMain {       public static void main(String[] args) {         // TODO Auto-generated method stub           FieldTest ft = new FieldTest();         ft.printField();     } }

 

결과

b = false
s = null
i = 0
l = 0
d = 0.0
f = 0.0
o = null