Выход за пределы массива. Исправить код

@cin_cout · Регистрация: 06.10.2012

Студворк — интернет-сервис помощи студентам

Вот такой результат исполнения кода:
5 17 22 10.5 11.23 12.87 6.64 9.82 15.35 - ввел числа в массив с клавиатуры

5 5 17
17 22 22
22 44 10.5
10.5 54.5 11.23
11.23 65.73 12.87
12.87 78.6 6.64
6.64 85.24 9.82
9.82 95.06 15.35
terminate called after throwing an instance of 'Range_error'
what(): Range error: 9

При компилировании никаких замечаний. При исполнении вылезаем за пределы вектора.
Как мы можем вылезти за его размеры, если в цикле заданы рамки i<dist.size() ?
И как тогда определять "последующий" элемент вектора для сравнения с предыдущим, для прочих итераций (нахождение максимального, минимального значений, сортировки)?
Где в моей логике рассуждений ошибка?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#include <iostream>
#include "std_lib_facilities.h"
 
int main()
{
    vector<double> dist;
    double distance;
    while (cin>>distance)
        dist.push_back(distance); 
    double sumdist;
    
    for (int i=0;i<dist.size(); ++i) 
    {
        sumdist+=dist[i];
        cout << dist[i] << '\t' << sumdist << '\t' << dist[i+1] << '\n';
    }
        
    cout << " Среднее расстояние: " << sumdist/dist.size() << '\n';
 
    return 0;
 
}

alsav22 · 12.10.2012, 12:08

Сообщение от cin_cout

Как мы можем вылезти за его размеры, если в цикле заданы рамки i<dist.size() ?

А вот здесь выход за границу: dist[i+1]. В конце цикла, при значении i = i + 1, должен быть выход, а делается попытка cout << dist[i+1]

@cin_cout · 12.10.2012, 12:13 **[ТС]**

Сообщение от alsav22

А вот здесь выход за границу: dist[i+1]. В конце цикла, при этом значении i, должен быть выход, а делается попытка cout << dist[i+1]

А как этого избежать? Если, например, дальше я хочу сравнивать в цикле предыдущий элемент с последующим?

alsav22 · 12.10.2012, 12:23

Сообщение от cin_cout

А как этого избежать? Если, например, дальше я хочу сравнивать в цикле предыдущий элемент с последующим?

Сделать можно, но самый последний с чем будете сравнивать?

Добавлено через 2 минуты

C++
1
2
3
4
5
6
7
8
for (int i=0; ; ) 
{
     sumdist+=dist[i];
     cout << dist[i] << '\t' << sumdist << '\t';
     i++;
     if (i < dist.size()) cout  << dist[i+1] << '\n';
     else break;
}

DiffEreD · 12.10.2012, 12:37

cin_cout, вы используете не инициализированную переменную double sumdist.

alsav22 · 12.10.2012, 12:47

Сообщение от yuron_477

cin_cout, вы используете не инициализированную переменную double sumdist.

В 14 она, вроде бы, инициализируется (но нужно проверить). Для проядка можно в 10:

C++
1
double sumdist = 0;

DiffEreD · 12.10.2012, 12:55

Нельзя использовать не инициализированную переменную в выражении +=

Добавлено через 40 секунд

C++
1
2
3
4
5
6
7
8
9
10
11
int main()
{
    int a[] = {1, 2, 3};
    vector<int> v(a, a+3);
    int sum;
    for (int i = 0; i<v.size(); i++)
        sum+=v[i];
    cout<<sum<<endl; // что будет выведено?
    system("pause");
    return 0;
}

alsav22 · 12.10.2012, 13:06

Ошибку сделал. Нужно так:

C++
1
2
3
4
5
6
7
8
for (int i=0; ; ) 
{
     sumdist+=dist[i];
     cout << dist[i] << '\t' << sumdist << '\t';
     i++;
     if (i < dist.size()) cout  << dist[i] << '\n';
     else break;
}

Добавлено через 45 секунд

Сообщение от yuron_477

Нельзя использовать не инициализированную переменную в выражении +=

Согласен.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .

DiffEreD 1458 / 795 / 257 Регистрация: 21.06.2011 Сообщений: 1,740 Записей в блоге: 2
	12.10.2012, 12:37
	cin_cout, вы используете не инициализированную переменную double sumdist. 0