свободный вывод strdup в цикле

Question

свободный вывод strdup в цикле

У меня есть цикл, который пытается прочитать данные из объекта списка qml, вот мой цикл

char * argvarry[(gcps.size() * 5) + 8];
argvarry[0] = "-of";
argvarry[1] = "GTiff";
argvarry[2] = "-a_nodata";
argvarry[3] = "'0 0 0'";
argvarry[4] = "-a_srs";
argvarry[5] = a_srs;

int argc = 6;

for (int i = 0;i < gcps.size(); i++) {
argvarry[argc] = "-gcp";argc++;//gcp_values
gcppoint_ *a = qobject_cast<gcppoint_ *>(gcps.at(i).value<QObject *>());

argvarry[argc]= strdup( const_cast<char*>(QString::number(a->row()).toStdString().c_str())); argc++;
argvarry[argc] = strdup(const_cast<char*>(QString::number(a->column()).toStdString().c_str())); argc++;
argvarry[argc] = strdup(const_cast<char*>(QString::number(a->lon()).toStdString().c_str())); argc++;
argvarry[argc] =strdup( const_cast<char*>(QString::number(a->lat()).toStdString().c_str())); argc++;
}

хорошо, я попробовал несколько методов, чтобы заставить вышеупомянутый код работать, но ни один не работал,
например, если я использую strdup это будет работать, но его нужно освободить, я попытался сохранить вывод strdup в переменной, а затем освободить его в цикле, но он освободил все переменные. вот моя попытка

for(int i=0;i<vl.size();i++) {
gcppoint_ *a = qobject_cast<gcppoint_ *>(vl.at(i).value<QObject *>());

char* srcX =strdup(const_cast<char*>(QString::number(a->row()).toStdString().c_str()));
char* srcY = strdup(const_cast<char*>(QString::number(a->column()).toStdString().c_str()));
char* dstX =strdup( const_cast<char*>(QString::number(a->lon()).toStdString().c_str()));
char* dstY = strdup(const_cast<char*>(QString::number(a->lat()).toStdString().c_str()));
qDebug() <<srcX<<" " <<srcY << " " <<dstX<< " " <<dstY;
argvarry[i]=srcX;

if(srcX)
free(srcX) //does not work it frees argvarry[i] too

}
//...do some thing with argvarry
free????

Я заметил что все argvarry[i] будет освобожден тоже, поэтому я не могу их больше использовать. ну как я могу заставить это работать, у меня слишком бесплатно strdup но я не могу справиться с этим. как я могу изменить этот цикл и когда я использовал argvarry я свободен strup результат?

Что делать, если я изменяю вышеуказанный код на аналогичный

char * srcX;
char * srcY;
char * dstX;
char * dstY;

for (int i = 0;i < gcps.size(); i++) {
argvarry[argc] = "-gcp";argc++;//gcp_values
gcppoint_ *a = qobject_cast<gcppoint_ *>(gcps.at(i).value<QObject *>());

srcX= strdup( const_cast<char*>(QString::number(a->row()).toStdString().c_str()));
srcY = strdup(const_cast<char*>(QString::number(a->column()).toStdString().c_str()));
dstX = strdup(const_cast<char*>(QString::number(a->lon()).toStdString().c_str()));
dstY =strdup( const_cast<char*>(QString::number(a->lat()).toStdString().c_str()));
argvarry[argc]=srcX;argc++;
argvarry[argc]=srcY;argc++;
argvarry[argc]=dstX;argc++;
argvarry[argc]=dstY;argc++;
}
///do things

//free(srcX);free(srcY)

0

c++strdup

Решение

Другие решения

Других решений пока нет …

Источник

Accepted Answer

  char* srcX = const_cast<char*>(QString::number(a->row()).toStdString().c_str());

Проблема с вышеупомянутым состоит в том, что toStdString () возвращает временный объект QString, и, поскольку объект QString является временным, он уничтожается в конце строки. Это означает, что srcX является висящим указателем, когда вы попытаетесь использовать его позже.

Если вам абсолютно необходим массив char * указатели, вам нужно убедиться, что данные, на которые они указывают, остаются действительными в течение всего времени жизни массива. strdup() это один из способов сделать это, но, как вы заметили, у него есть свои трудности, в частности, вам нужно будет вручную вызвать free () для строк после того, как вы покончили с ними, иначе вы потеряете память.

Другой подход — сначала создать список объектов std :: string, например так:

// Build up a list of std::strings
std::vector<std::string> strsList;
strsList.push_back("-of");
strsList.push_back("GTiff");
strsList.push_back("-a_nodata");
strsList.push_back("'0 0 0'");
strsList.push_back("-a_srs");
strsList.push_back("a_srs");
for (int i = 0;i < gcps.size(); i++) {
strsList.push_back("-gcp");
strsList.push_back(QString::number(a->row()).toStdString());
strsList.push_back(QString::number(a->column()).toStdString());
strsList.push_back(QString::number(a->lon()).toStdString());
strsList.push_back(QString::number(a->lat()).toStdString());
}

// Now create an array of pointers to the data in those std::strings
// This array will remain valid for as long as strsList remains valid and unmodified
char * argvarray[strsList.size()];
for (size_t i=0; i<strsList.size(); i++) argvarray[i] = const_cast<char *>(strsList[i].c_str());

1