C ++数组/指针泄漏内存

Question

When I run function handleRequestWithParams(..) few times I am run out of RAM memory on my microcontroller (ESP8266).
I am not sure that I should delete "paramsNames", "paramsValues" and maybe "values", or maybe I should create this arrays in other way?

struct RequestStructure {
    int paramsCount;
    String* paramsNames;
};

void Requests::handleRequestWithParams(RequestStructure requestStructure) {
    const int PARAMS_COUNT = requestStructure.paramsCount;

    String* paramsNames = requestStructure.paramsNames;
    String* paramsValues = readParamsValues(paramsNames, PARAMS_COUNT);

    _server.send(200, TYPE_TEXT, response(paramsNames[0], paramsValues[0]);

    //delete paramsNames;
    //delete paramsValues;
}

String* Requests::readParamsValues(String* paramsNames, int count) {
    String* values = new String[count];
    for (int i = 0; i < count; i++) {
        values[i] = server.arg(paramsNames[i].c_str());
    }
    return values;
}

Answer 1

Every piece of memory you acquire with new or new T[] must be deleted using delete or delete[] accordingly.

Otherwise you are leaking memory.

---

That said, I'd strongly discourage to deal with new / delete yourself.
Use either std::vector , or the classes from Dynamic memory management of the c++ standard library instead.

These provide clear semantics how ownership of dynamic storage will be transferred between the distributed pieces of your code workflow, and manage the deletion automatically if the reference ceases to exist.