2016-11-06 9 views
-1

動的割り当て配列を削除しようとすると、自分のプログラムがクラッシュし続けます。私はプログラムをデバッグする場合、このエラーがアップします:動的配列を削除するとクラッシュします

#0 0x47a949 std::basic_ostream<char, std::char_traits<char> >& std::operator<< <char, std::char_traits<char>, std::allocator<char> >(std::basic_ostream<char, std::char_traits<char> >&, std::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)() (??:??) 
#1 0x48a940 std::cerr() (??:??) 
#2 0x722924 ??() (??:??) 
#3 0x4010fd __mingw_CRTStartup() (??:??) 
#4 0x7729cf34 strerror_s() (C:\WINDOWS\SysWoW64\msvcrt.dll:??) 
#5 0x775d0719 ??() (??:??) 
#6 0x775d06e4 ??() (??:??) 
#7 ?? ??() (??:??) 

これは私のコードです:

#include <iostream> 
#include <string> 
#include <stdlib.h> 

using namespace std; 

int main() 
{ 
    int numNames; 
    cout << "How many names do you want to enter?" << endl; 
    cin >> numNames; 
    std::string *names = new (nothrow) std::string[numNames]; 
    if (!names) 
    { 
     std::cout << "Could not allocate memory"; 
     exit(EXIT_FAILURE); 
    } 

    for (int i = 0; i <= numNames-1; i++) 
    { 
     cout << "Enter name #" << i+1 << endl; 
     cin >> names[i]; 
    } 

    for (int start = 0; start < numNames; start++) 
    { 
     int smallestName = start; 
     for (int currentName = start + 1; currentName < numNames; currentName++) 
     { 
      if (names[currentName] < names[smallestName]) 
      { 
       smallestName = currentName; 
      } 
     } 

     swap(names[start], names[smallestName]); 
    } 

    cout << endl << "Here is your sorted list: " << endl; 
    for (int i = 0; i <= numNames; i++) 
    { 
     cout << names[i] << endl; 
    } 

    delete[] names; 
     names = nullptr; 

    return 0; 
} 

私は両方の名前= 0で試してみました。およびnames = nulltptr;どちらも働いていませんでした。 私の問題を見つけるのを助けてくれることを願っています。 乾杯!

+3

このような問題を解決する適切なツールは、デバッガです。スタックオーバーフローを尋ねる前に、コードを一行ずつ進める必要があります。詳しいヘルプは、[小さなプログラムをデバッグする方法(Eric Lippert)](https://ericlippert.com/2014/03/05/how-to-debug-small-programs/)を参照してください。最低限、問題を再現する[最小、完全、および検証可能](http://stackoverflow.com/help/mcve)の例と、その問題を再現するためのデバッガ。ループの最後で –

+2

が...私は<= numNames ...が、私はそれをしようとすると、細かい動作 – HazemGomaa

+1

numNamesを<...する必要があり、何がありません、彼のデバッガは、この特定を解決するために、正確に間違ったツール@πάνταῥεῖある –

答えて

1

エラーは削除ステートメントのためではありません。 <=のためにループfor (int i = 0; i <= numNames; i++)で出力しているときに、メモリ内で利用できない要素にアクセスしているため、プログラムがクラッシュするためです。これを修正するにはi < numNames

+0

ありがとうございました! – Elhoej

関連する問題