Возвращаемый экземпляр вызывает ошибку «попытка ссылки на удаленную функцию»

Question

После работы с C# в течение последних десяти или двух лет мой C++ становится немного ржавым.

Я пишу класс базы данных и имеют проблемы со следующим методом:

CRecordset CAccessDatabaseReader::ExecuteSqlQuery(LPCTSTR pszSqlQuery) 
{ 
    CRecordset recordSet(&m_Database); 
    recordSet.Open(CRecordset::forwardOnly, pszSqlQuery); 
    return CRecordset(recordSet); 
} 

Компилятор жалуется на линии с return заявлением:

Ошибка C2280 «CRecordset :: CRecordset (const CRecordset &) ': попытка ссылки на удаленную функцию

Может кто-то помочь я точно понимаю, что здесь происходит?

Answer 1

CRecordset конструктор копирования был явно отмечен как deleted для предотвращения копирования CRecordset объектов от одного к другому.

Таким образом, функции придется вернуть новый объект по указателю и требует, чтобы вызывающий delete объекта после завершения его использования:

CRecordset* CAccessDatabaseReader::ExecuteSqlQuery(LPCTSTR pszSqlQuery) 
{ 
    CRecordset *recordSet = new CRecordset(&m_Database); 
    if (!recordSet->Open(CRecordset::forwardOnly, pszSqlQuery)) 
    { 
     delete recordSet; 
     return NULL; // or raise an exception 
    } 
    return recordSet; 
} 

CRecordset *rs = reader.ExecuteSqlQuery(TEXT("...")); 
if (rs) 
{ 
    ... 
    delete rs; 
} 

Или лучше:

std::unique_ptr<CRecordset> CAccessDatabaseReader::ExecuteSqlQuery(LPCTSTR pszSqlQuery) 
{ 
    std::unique_ptr<CRecordset> recordSet(new CRecordset(&m_Database)); 
    if (!recordSet->Open(CRecordset::forwardOnly, pszSqlQuery)) 
     recordSet.reset(); // or raise an exception 
    return recordSet; 
} 

std::unique_ptr<CRecordset> rs = reader.ExecuteSqlQuery(TEXT("...")); 
if (rs) 
{ 
    ... 
} 

Answer 2

CRecordset удалил свой конструктор копирования, поэтому вы не можете вернуть его по значению. Вы могли бы вернуть std::unique_ptr<CRecordset> или ссылку от ExecuteSqlQuery.

На удалении конструкторы копирования: https://stackoverflow.com/a/6077164/2449857

По возвращении из ссылки функций: Is the practice of returning a C++ reference variable, evil?