インスタンスが「削除された関数を参照しようとしました」というエラーを返しました

Question

過去10年または2年間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年代には、コンストラクタを明示的に一方から他方へCRecordsetオブジェクトのコピーを防ぐためにdeletedとしてマークされているコピーします。

ので、関数がポインタによって新しいオブジェクトを返し、それを使用して終了したときにオブジェクトを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?