1. ホーム
  2. c++

[解決済み] 無効なNullポインタ - C++

2022-02-14 02:28:22

質問

この部分のコードで無効なヌルポインターエラーが発生します。文字列が原因だと思うのですが、最近文字列について学んだばかりで、問題を見つけることができません。

string batchCipherFiles()
{
int count(0);
string text, file; 
ifstream inputFile;

cout << " How many files do you wish to encrypt?: ";
cin >> count;
for (int i(1) ; i <= count ; ++i)
{
    stringstream ss;
    ss << "unencrypted" << i << ".txt";
    file = ss.str();
    inputFile.open(file.c_str(), ios::in);
    if (inputFile.fail()) 
    {
        cout << "\n An error has occurred.";
    }
    else
    {
        while (!inputFile.eof())
        {
            getline(inputFile, text);
            cout << "\n " << file << " = " << text << "\n\n";
            int applyCeasarShift(string,string);        
            applyCeasarShift(text, file);
        }
        inputFile.close();
    }
}

return (0);
}

以下はxstringのデバッグ行です。

 #ifdef _DEBUG
_CRTIMP2_PURE void __CLRCALL_PURE_OR_CDECL _Debug_message(const wchar_t *message, const wchar_t *file, unsigned int line)
{   // report error and die
    if(::_CrtDbgReportW(_CRT_ASSERT, file, line, NULL, message)==1)
    {
        ::_CrtDbgBreak();
    }
}

よろしくお願いします。

EDIT: エラーは Return 文で発生し、xstring からの抽出では、黄色の矢印が "::_CrtDbgBreak();" を指しています。

EDIT 2:

xstring:
basic_string(const _Elem *_Ptr)
    : _Mybase()
    {   // construct from [_Ptr, <null>)
    _Tidy();
    assign(_Ptr);
    }

xutility:
template<class _Ty> inline
void _Debug_pointer(const _Ty *_First, _Dbfile_t _File, _Dbline_t _Line)
{   // test iterator for non-singularity, const pointers
if (_First == 0)
    _DEBUG_ERROR2("invalid null pointer", _File, _Line);
}

stdthrow:
#ifdef _DEBUG
_CRTIMP2_PURE void __CLRCALL_PURE_OR_CDECL _Debug_message(const wchar_t *message, const wchar_t *file, unsigned int line)
{   // report error and die
    if(::_CrtDbgReportW(_CRT_ASSERT, file, line, NULL, message)==1)
    {
        ::_CrtDbgBreak();
    }
}

これが正しいものであることを祈ります。

どのように解決するのですか?

問題は string の戻り値の型は batchCipherFiles しかし return (0); が返される場合: 返り値の型を変更するか、あるいは string .

の方がいいと思います。 return (0); に暗黙のうちに変換されます。 std::string((char*)0); クラッシュの原因となります。のドキュメントは std::string コンストラクタ の状態になります。

sが指すヌル文字で終端する文字列の内容で文字列を構成します。文字列の長さは、最初のヌル文字で決まります。 sはNULLポインタであってはならない。