2016-11-06 3 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

Правильный инструмент для решения таких проблем - ваш отладчик. Перед тем, как просить о переполнении стека, вы должны пропустить свой код по очереди *. Для получения дополнительной информации, пожалуйста, прочтите [Как отлаживать небольшие программы (Эрик Липперт)] (https://ericlippert.com/2014/03/05/how-to-debug-small-programs/). Как минимум, вы должны \ [изменить] ваш вопрос, чтобы включить пример [Минимальный, полный и проверенный] (http://stackoverflow.com/help/mcve), который воспроизводит вашу проблему, а также замечания, сделанные вами в отладчик. –

+2

В вашем последнем цикле ... i <= numNames .. должно быть .. i HazemGomaa

+1

Хорошо работает, когда я пытаюсь, какие значения вызывают его ошибку –

ответ

1

Ваша ошибка не из-за инструкции удаления. Это связано с тем, что когда вы выходите в цикле for (int i = 0; i <= numNames; i++) из-за <=, вы получаете доступ к элементу, который недоступен в памяти, следовательно, сбой программы. Чтобы исправить это, просто используйте i < numNames

+0

Большое спасибо за вашу помощь! – Elhoej