NullPointerException 문제

Question

오늘이 복습 테스트에서는 Dress와 TestClass의 두 클래스를 만들어야했습니다. 나는이 클래스들을 끝내었지만 프로그램을 실행하려고 할 때 NullPointerException 메시지가 발생했습니다. 여기 내 클래스는 다음과 같습니다

클래스 드레스 :

public class Dress { 
    String colors []; 
    int sizes []; 

    public Dress (String colors [], int sizes []){ 
     this.colors = new String [colors.length]; 
     this.sizes = new int [sizes.length] ; 
     this.colors = colors; 
     this.sizes = sizes; 
    } 

    public boolean search (String color){ 
     for (int i =0; i<colors.length;i++) 
      if (colors [i].equals(color)) 
       return true; 
     return false; 
    } 
    public boolean search (int size){ 
     for (int i =0; i<sizes.length;i++) 
      if (sizes [i] == size) 
       return true; 
     return false; 
    } 
} 

클래스 테스트 : 참고로

public class Tests { 
    public static void main (String args []){ 
     String color[] = {"Pink","Blue","Red"}; 
     int size[] = {8,9,7}; 
     Dress d = new Dress (color, size); 
     System.out.println(d.search("Pink")); 
     System.out.println(d.search(8)); 
    } 
} 

Answer 1

- 그것은 개인 데이터 멤버로 변경 가능한 참조를 할당하는 나쁜 생각이다 :

this.colors = new String [colors.length]; // The new reference is discarded after reassignment on next line 
this.colors = colors; // The program that passes this reference can modify it; changes will be visible to your class instance. 

참고 문헌을 확인하고 상태를 변경하는 사람은 개인 상태와 상관없이 인스턴스 데이터 멤버를 변경합니다.

여기 (하나의 명확성을 위해)을 할 수있는 올바른 방법이다 :

public Dress(String [] colors) { 
    if (colors == null) throw new IllegalArgumentException("colors cannot be null"); 
    this.colors = new String[colors.length]; 
    // Copy the values from the parameter array into the new, private array. 
    System.arraycopy(colors, 0, this.colors, 0, this.colors.length); 
} 

당신은 항상 개인, 가변 데이터의 방어 복사본을 만들어야합니다.