메서드에 문자열 매개 변수가 너무 많습니다.

Question

경우에 따라 코드에 대한 PMD 검사에 문제가 있습니다. 내 말은 - 너무 많은 문자열 매개 변수 :

철에게 "보다는 문자열 인수를 많이 사용하여, 그 값에 대한 컨테이너 개체를 사용하는 것이 좋습니다"

public void printPersons (final String name, final String surname, final String day, final String car, final String city, final String petName) 
{ 

} 

것은 당신이 그 상황을 피할 수있는 방법 너의 프로젝트에서?

Answer 1

모든 필드를 캡슐화하는 간단한 클래스를 정의하고이를 원래 메소드로 전달할 수 있습니다.

public class Person { 
    private final String name; 
    private final String surname; 
    private final String day; 
    private final String car; 
    private final String city; 
    private final String petName; 

    public Person(final String name, final String surname, final String day, final String car, final String city, final String petName) { 
    this.name = name; 
    this.surname = surname; 
    this.day = day; 
    this.car = car; 
    this.city = city; 
    this.petName = petName; 
    } 

    public String getName() { 
    return this.name; 
    } 

    public String getSurname() { 
    return this.surname; 
    } 

    public String getDay() { 
    return this.day; 
    } 

    public String getCar() { 
    return this.car; 
    } 

    public String getCity() { 
    return this.city; 
    } 

    public String getPetName() { 
    return this.petName; 
    } 
} 

그리고 당신의 메서드 호출은 지금된다 : 나를 위해

public void printPersons(Person person) {} 

Answer 2

은 컨테이너 개체 (사람이) 여기에 사용되어야한다는 것을 밖으로 질문입니다. Person 인스턴스의 생성을 내부 클래스 인 PersonBuilder가 사용하기 쉽게하기 위해 생성자 또는 설정자는 노출되지 않습니다.

public final class Person { 
    private final String name; 
    private final String surname; 
    private final String day; 
    private final String car; 

    // all the other attributes omitted 

    public static void main(final String[] args) { 
     // usage example 
     Person person = Person.builder().name("Name").surname("Surname") 
       .car("Tesla").day("Friday").build(); 

     person.toString(); 

    } 

    // private constructor, only used by the PersonBuilder 
    private Person(final PersonBuilder pb) { 
     this.name = pb.name; 
     this.surname = pb.surname; 
     this.day = pb.day; 
     this.car = pb.car; 
    } 

    public static PersonBuilder builder() { 
     return new PersonBuilder(); 
    } 

    public String toString() { 

     return "return a nicely formatted String for printing, logging etc."; 
    } 

    /** 
    * expose a nice, fluent API to construct Persons. 
    */ 
    public static final class PersonBuilder { 
     private String name; 
     private String surname; 
     private String day; 
     private String car; 

     // use the static method Person.builder() to get an instance. 
     private PersonBuilder() { 
     } 

     public PersonBuilder surname(final String surname) { 
      this.surname = surname; 
      return this; 
     } 

     public PersonBuilder name(final String name) { 
      this.name = name; 
      return this; 
     } 

     public PersonBuilder day(final String day) { 
      this.day = day; 
      return this; 
     } 

     public PersonBuilder car(final String car) { 
      this.car = car; 
      return this; 
     } 

     public Person build() { 
      return new Person(this); 
     } 
    } 

    public String getName() { 
     return name; 
    } 

    public String getSurname() { 
     return surname; 
    } 

    public String getDay() { 
     return day; 
    } 

    public String getCar() { 
     return car; 
    } 
}