尝试将元素添加到动态数组

发布于 2025-02-01 12:54:18 字数 519 浏览 3 评论 0原文

我尝试将一个从类类型添加到一个空的动态数组中，但是什么也不会发生，也没有在添加一个元素后更改计数器。这是功能

void Bank::addCustomer(const Customer& newCustomer) {

    Customer* temp = new Customer[getCustomerCount()+1]; 

    for (int i = 0; i < getCustomerCount() + 1 ; ++i) {
        if (getCustomerCount() != 0) {
            temp[i] = customers[i];
        }
    }

    ++customerCount;
    setCustomerCount(customerCount);
    delete[] customers; 
    customers = temp;

    customers[customerCount] = newCustomer; 
    //log
}

原文

i've tried to add an element from class type to an empty dynamic array, but nothing happens, nor it changes the counter after adding one element.
This is the function

void Bank::addCustomer(const Customer& newCustomer) {

    Customer* temp = new Customer[getCustomerCount()+1]; 

    for (int i = 0; i < getCustomerCount() + 1 ; ++i) {
        if (getCustomerCount() != 0) {
            temp[i] = customers[i];
        }
    }

    ++customerCount;
    setCustomerCount(customerCount);
    delete[] customers; 
    customers = temp;

    customers[customerCount] = newCustomer; 
    //log
}

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

翻了热茶 2025-02-08 12:54:18

如果您的循环不为空，则超过旧数组的边界。 newcustomer的分配超出了新数组的界限。

而是尝试一下：

void Bank::addCustomer(const Customer& newCustomer) {

    int count = getCustomerCount(); 
    Customer* temp = new Customer[count + 1];

    for (int i = 0; i < count; ++i) {
        temp[i] = customers[i];
    }
    temp[count] = newCustomer; 

    delete[] customers; 
    customers = temp;

    ++customerCount;

    //log
}

Your loop is exceeding the bounds of the old array if it is not empty. And your assignment of the newCustomer is exceeding the bounds of the new array.

Try this instead:

void Bank::addCustomer(const Customer& newCustomer) {

    int count = getCustomerCount(); 
    Customer* temp = new Customer[count + 1];

    for (int i = 0; i < count; ++i) {
        temp[i] = customers[i];
    }
    temp[count] = newCustomer; 

    delete[] customers; 
    customers = temp;

    ++customerCount;

    //log
}

回复收藏 0 原文

┈┾☆殇 2025-02-08 12:54:18

我猜这就是您的意思：

void addCustomer(const Customer& newCustomer) {
    int old_customer_count = getCustomerCount();
    int new_customer_count = old_customer_count + 1;

    Customer* temp = new Customer[new_customer_count];
    for (int i = 0; i < new_customer_count; ++i) {
        temp[i] = customers[i];
    }

    setCustomerCount(new_customer_count);
    delete[] customers;
    customers = temp;
    customers[old_customer_count] = newCustomer;
}

但是正如某人所说，在实际代码中，您应该为客户收集std :: vector。

I'm guessing this is what you mean:

void addCustomer(const Customer& newCustomer) {
    int old_customer_count = getCustomerCount();
    int new_customer_count = old_customer_count + 1;

    Customer* temp = new Customer[new_customer_count];
    for (int i = 0; i < new_customer_count; ++i) {
        temp[i] = customers[i];
    }

    setCustomerCount(new_customer_count);
    delete[] customers;
    customers = temp;
    customers[old_customer_count] = newCustomer;
}

but as someone said, in real code you should an std::vector for the customers collection.

回复收藏 0 原文