Zu viele String-Parameter in Methode

Question

Manchmal habe ich das Problem mit einem PMD-Check über meinen Code. Ich meine - zu viel String-Parameter:

„Anstatt eine Menge von String-Argumente verwenden, sollten Sie für diese Werte ein Containerobjekt mit“

fe:

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

} 

Wie Sie können diese Situation vermeiden in Deinen Projekten?

Answer 1

Sie können eine einfache Klasse definieren, die alle diese Felder einkapselt und diese an Ihre ursprüngliche Methode übergibt.

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; 
    } 
} 

Und Ihr Methodenaufruf wird jetzt:

public void printPersons(Person person) {} 

Answer 2

Für mich ist es außer Frage, dass ein Container-Objekt (Person) sollte hier verwendet werden. Um die Konstruktion von Personeninstanzen zu erleichtern, wird ein innerer Klassen-PersonBuilder verwendet, kein Konstruktor oder Setter wird verfügbar gemacht.

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; 
    } 
}