Java构造函数

Question

再会！

我创建了重载构造函数，如下所示：

public ContactsBean(String firstName, String lastName,
                String telNumber, String email) {
    this.id = count;
    count = count + 1;
    this.firstName = firstName;
    this.lastName = lastName;
    this.email = email;
    this.telNumber = telNumber;
}

public ContactsBean() {
    this.id = count;
    count = count + 1;
}

我想自动递增 id，因此我使用了此变量：

    private static int count;   
    private int id;

我的问题是，当我实例化 ContactsBean() contacts = new ContactsBean() 时，id 的值增加 2.. 2,4,6,8...等等

为什么？如何让ID号自动加1？

谢谢。

编辑：

行动：

private ContactsBean contacts = new ContactsBean();
private ContactsManager contactsManager = new ContactsManager();

    public String add() {
        contactsManager.addContacts(contacts);
        return SUCCESS;
    }

经理：

private ContactsDAO contactsDAO = ContactsDAO.getInstance();
private List<ContactsBean> contactsList = contactsDAO.getContactsList(); 

public void addContacts(ContactsBean contact) {
    contactsList.add(contact);
}

Answer 1

首先，DRY（不要重复自己）会更好：

    public ContactsBean(String firstName, String lastName,
                String telNumber, String email){
    this();
    this.firstName = firstName;
    this.lastName = lastName;
    this.email = email;
    this.telNumber = telNumber;
}

其次，代码中没有增加 2。请粘贴您的测试代码。

Answer 2

尝试删除第一组括号
ContactsBean() 联系人 = new ContactsBean();。
也就是说，尝试这个构造函数：

ContactsBean contacts = new ContactsBean();

Answer 3

我认为您正在其他类中创建两个 ContactsBean 对象，但您可能没有意识到这一点。你必须检查代码。

Answer 4

您是否尝试过在两个构造函数上设置断点来调试代码？

Eng的建议。 Fouad 是一个很好的建议，但它并不能解决你的问题。

另请注意，您的计数器不是线程安全的（不过，问题与它无关。在这种情况下，您的计数器的值将低于应有的值）

并且如果您确实需要跟踪实际有多少个对象创建，我不认为最好的方法是在 Java Bean 中使用静态属性......

Answer 5

这可能是由于复制构造函数调用了构造函数的无参数版本。

（或者我患有C++病？）

Answer 6

我的建议是不要尝试在构造函数中增加 contactID，而是从新创建的数据库对象中获取它，其中 ID 通过身份规范由数据库增加，或者因为您正在获取基于您的下一个 id 的联系人列表关闭 contactDAO.getContactsList().size()+1。

我还建议从： 更改

private ContactsDAO contactsDAO = ContactsDAO.getInstance();
private List<ContactsBean> contactsList = contactsDAO.getContactsList(); 

public void addContacts(ContactsBean contact) {
    contactsList.add(contact);
}

为：

private ContactsDAO contactsDAO = ContactsDAO.getInstance();
private List<ContactsBean> contactsList; 

public void addContacts(ContactsBean contact) {
    int id = getContactList().size()+1;
    contact.setId(id);
    contactsList.add(contact);
}

public List<ContactsBean> getContactList(){
     return contactsDAO.getContactsList();
}

或者，如果您能够从数据库中删除联系人，则此号码对于 ID 来说可能不准确。您可以基于以下内容创建查询：

select MAX(ID) from contacts

这将返回使用的最大 ID 号。

Answer 7

private static int count = 0;
private int id;
// ...
public ContactsBean(String firstName, String lastName,String telNumber, String email)
{
    this();
    this.firstName = firstName;
    this.lastName = lastName;
    this.email = email;
    this.telNumber = telNumber;
}
public ContactsBean()
{
    id = ++count;
}

Answer 8

我了解到 Struts2 在 Action 类中自动生成对象（bean）的实例，因此不需要实例化它......

我之前的代码是

private ContactsBean contacts = new ContactsBean();
private ContactsManager contactsManager = new ContactsManager();

    public String add() {
        contactsManager.addContacts(contacts);
        return SUCCESS;
    }
    //getters and setters

我将其更改为......

private ContactsBean contacts;
private ContactsManager contactsManager = new ContactsManager();

    public String add() {
        contactsManager.addContacts(contacts);
        return SUCCESS;
    }
//getters and setters

并且它可以工作......