динамическое распределение строковых массивов C++

Я пытаюсь читать имена из файла, который мне дается. Количество имен, которые считываются, основано на пользовательском вводе. Это компилируется отлично, но когда я запускаю его, я получаю эту ошибку:динамическое распределение строковых массивов C++

«0xC0000005: место записи нарушения доступа 0xabababab».

Я провел некоторое исследование, и я думаю, что это, вероятно, потому, что я пишу над границами массива, но я не могу понять, почему это так.

Прежде, чем кто-нибудь спросит, мне сказали, что мне нужно использовать строку * для моего массива. Я не должен использовать векторы или массив символов. Спасибо.

int Capacity; 
    int numNames = 0; // wasn't in original question, but was in the code 
    cin >> Capacity; 

    string name; 
    string* arr = new string[Capacity]; 

    while(!fs.eof() && numRead < Capacity) 
    { 
     getline(fs , names); 
     arr[numRead] = names; // error thrown here 
     arr++; 
     numRead++; 
    }

источник

2014-02-03 Brian Thorpe

Вы делаете 'аранжировка ++' Может также просто изменить свой цикл, чтобы сделать: 'зЬй :: GetLine (фс, обр [numRead ++ ]); 'и избавиться от' имен' и всего остального. Не забудьте вызвать 'delete [] arr', когда закончите с ним. – Brandon

спасибо, что это был. –

также избавиться от '! Fs.eof()', это неправильная проверка. вместо этого завершите цикл, когда 'getline' терпит неудачу. –

Вы не можете увеличить начальный адрес массива (arr ++), делая это, теряя контроль над массивом. Индексация массива действительна только в том случае, если ссылка на массив указывает на первый элемент массива. Поэтому удалите строку кода arr ++, и, поскольку Дэвид сказал, что вы должны инициализировать numRead до нуля, в противном случае некоторые назначенные большие случайные числа также вызовут эту ошибку.

источник

2014-02-03 01:14:58 Varo

Вы не должны указывать обороты arr. Удалите эту строку кода. Вы используете индексирование массива, arr[numRead], для доступа к элементам массива.

Я также не вижу никаких доказательств того, что вы инициализировали numRead до 0 до начала цикла.

источник

2014-02-03 01:01:56

Используйте вектор Luke! Это было бы так: std :: vector arr; А затем в цикле добавьте имя к вектору: arr.push_back (name);

код вы пишете в C не C++ И пожалуйста, прекрати вниз вопросы голосования

источник

2014-02-03 01:23:40 Imaxd

динамическое распределение строковых массивов C++

ответ

Смежные вопросы