在构造函数中有很多参数

Question

在构造函数中包含很多参数是错误的吗？像10到15个参数？因为我正在设计一个类,其中构造函数将具有许多参数,例如,一个Person类.

我的示例person类在其构造函数中有6个参数:

public class Person {
    private String fName;
    private String lName;
    private String mInitial;
    private int age;
    private String contactNumber;
    private String emailAddress;

    public Person(String fName, String lName, String mInitial, int age, String contactNumber, String emailAddress) {
       //insert rest of code here 
    }
}

那是错的吗？为构造函数创建大量参数？

然后我打算创建一个名为class的类Employee,然后将它扩展到person类,然后它也会有一个很长的构造函数.

令我担心的是实践,这是好还是什么？还有其他建议吗？

Answer 1

1. 通常,如果您发现自己有太多参数,那意味着您没有足够的低级别类.在你的情况下,你可能有一个class Name { /* fname, lname, initial, */ },也许是一个class Contact { /* email, phone */ }

2. 扩展类时,让构造函数将基数作为一个参数,然后添加新的额外参数.(你可能意味着员工会扩展Person,而不是反过来,所以public Employee (Person person, Company company, String employeeId) { super(person); this.company = company; this.employeeId = employeeId; }

好问题!

Answer 2

而不是使用telescoping构造函数模式,使用构建器模式

public class Person {
    private final String fName;
    private final String lName;
    private final String mInitial;
    private final int age;
    private final String contactNumber;
    private final String emailAddress;

    public Person(PersonBuilder builder) {
       //insert rest of code here 
       fName = builder.fName;
       ...
    }

    public static class PersonBuilder {
        private String fName;
        private String lName;
        private String mInitial;
        private int age;
        private String contactNumber;
        private String emailAddress;
        // setter methods
        public PersonBuilder setFirstName(String name) {
             fName = name;
             return this;
        }
        ...
        // build method
        public Person build() {
            return new Person(this);
        }

    }
}

...

Person p = new PersonBuilder()
              .setFirstName("")
              // set all the setter methods
              .build();

Answer 3

你可以分解Person成Name和Contact.

public class ComposedPerson {
    private Name name;
    private int age;
    private Contact contact;

    public ComposedPerson(Name name, int age, Contact contact) {
        this.name = name;
        this.age = age;
        this.contact = contact;
    }

    public static void main(String... args) {
        Name name = new Name("John");
        Contact contact = new Contact("12345", "john@doe.com");
        ComposedPerson person = new ComposedPerson(name, 45, contact);
   }

样品Name.看看我如何使用伸缩构造函数来允许可选参数.

public class Name {
    private String fName;
    private String lName;
    private String mInitial;

    public Name(String fName) {
        this(fName, null, null);
    }

    public Name(String fName, String lName) {
        this(fName, lName, null);
    }

    public Name(String fName, String lName, String mInitial) {
        this.fName = fName;
        this.lName = lName;
        this.mInitial = mInitial;
    }
} 

Answer 4

是的，在任何类型的函数中都有很多参数都不好。根据一本名为 Code Complete 2 的书，最大参数应该在 7 左右。

这是因为它会降低代码的可读性、可维护性和可用性。想象一下其他开发人员在同一个项目上工作，如何遵循你的代码？

有很多不同的方法来处理这个问题，例如工厂模式。这取决于您如何设计应用程序。

但是在你的代码中，我认为没有。参数还是可以接受的（6个参数）

如果您的对象需要如此多的参数来实例化，那么您需要重新考虑如何设计您的代码。例如，是否可以将某些属性包装到一个单独的类中？某些不需要的属性可以作为参数传递吗？即，从另一个类中获取值。等等...