Непонятки с итераторами

@UmKaM · Регистрация: 23.10.2012

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

Здравствуйте, господа программисты. Пытался написать вот эту задачку, но получается что-то нехорошее. Помогите, пожалуйста!

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
#include <iostream>
#include <list>
#include <iterator> 
#include <algorithm>
 
using namespace std;
 
list<int>::iterator find(int h, list<int> l)
{
    list<int>::iterator i=l.begin();
    for(i++; (*i)!=h && i!=l.end(); ++i);
    return i;
}
 
void spisker(list<int> l, list<int>::iterator z, int a1, int a2, int a3, int a4, int b1, int b2, int c)
{   //функция вычисляет очередной элемент последовательности.
    list<int>::iterator i=z;
    z--;
    int h=(*z)*(a1*(*i)+a2)+(*i)*a3+a4;
    if(h>b1 && h>b2)
    {
        h=h-c*(((h-b2)%c!=0)+(h-b2)/c);
    }
    z=find(h, l); //возвращает l.end()
    if((z!=l.end()) && (z!=l.begin())) //но какой-то неведомой мне магией проходит эту проверку и заходит в блок.
    {
        z--;
        i--;
        h=(*i);
        if(h==(*z)) //в итоге тут происходит обращение к "левой" памяти. 
            c=-1;
    }
    l.push_back(h);
}
 
int main()
{
    int a1,a2,a3,a4,b1,b2,c;
    a1=0;           //ввод значений упускаю, даже на этом примере не работает.
    a2=0;           
    a3=2;
    a4=3;
    b1=20;
    b2=5;
    c=7;           //cin >> a1 >> a2 >> a3 >> a4 >> b1 >> b2 >> c;
    list<int> l;
    int q, p=0;
    q=0;            //cin >> q;
    l.push_back(q);
    q=1;               //cin >> q;
    l.push_back(q);
    while(p==0) //происходит обращение к "левой" памяти уже на первом проходе цикла
    {
        list<int>::iterator z=l.end();
        z--;
        spisker(l, z, a1, a2, a3, a4, b1, b2, c);
        if(c==-1)
        {
            q=1;
            p=1;
            for(list<int>::iterator i=l.begin(); i!=z; i++)
                p++;
            for(list<int>::iterator i=z; i!=l.end(); i++)
                q++;
        }
    }
    cout << p << ' ' << q;
    return 0;
}

@Герц · 23.10.2012, 19:49

У всех контейнеров есть метод find( T item ), возвращающий итератор на запрошенный элемент.
Сделай typedef для list<int> и list<int>::iterator, а то читать нереально.

@UmKaM · 23.10.2012, 20:14 **[ТС]**

У всех контейнеров есть метод find( T item ), возвращающий итератор на запрошенный элемент.
Сделай typedef для list<int> и list<int>::iterator, а то читать нереально.

- сделал.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
#include <iostream>
#include <list>
#include <iterator>
#include <algorithm>
 
using namespace std;
 
typedef list<int>::iterator it;
typedef list<int> lis;
 
it find(int h, lis l)
{
    it i=l.begin();
    for(i++; (*i)!=h && i!=l.end(); ++i);
    return i;
}
 
void spisker(lis l, it z, int a1, int a2, int a3, int a4, int b1, int b2, int c)
{   //функция вычисляет очередной элемент последовательности.
    it i=z;
    z--;
    int h=(*z)*(a1*(*i)+a2)+(*i)*a3+a4;
    if(h>b1 && h>b2)
    {
        h=h-c*(((h-b2)%c!=0)+(h-b2)/c);
    }
    z=find(h, l); //возвращает l.end()
    if((z!=l.end()) && (z!=l.begin())) //но какой-то неведомой мне магией проходит эту проверку и заходит в блок.
    {
        z--;
        i--;
        h=(*i);
        if(h==(*z)) //в итоге тут происходит обращение к "левой" памяти.
            c=-1;
    }
    l.push_back(h);
}
 
int main()
{
    int a1,a2,a3,a4,b1,b2,c;
    a1=0;           //ввод значений упускаю, даже на этом примере не работает.
    a2=0;
    a3=2;
    a4=3;
    b1=20;
    b2=5;
    c=7;           //cin >> a1 >> a2 >> a3 >> a4 >> b1 >> b2 >> c;
    lis l;
    int q, p=0;
    q=0;            //cin >> q;
    l.push_back(q);
    q=1;               //cin >> q;
    l.push_back(q);
    while(p==0) //происходит обращение к "левой" памяти уже на первом проходе цикла
    {
        it z=l.end();
        z--;
        spisker(l, z, a1, a2, a3, a4, b1, b2, c);
        if(c==-1)
        {
            q=1;
            p=1;
            for(it i=l.begin(); i!=z; i++)
                p++;
            for(it i=z; i!=l.end(); i++)
                q++;
        }
    }
    cout << p << ' ' << q;
    return 0;
}

Добавлено через 12 минут
Герц, сделал.

Кликните здесь для просмотра всего текста

сорри за два поста, ещё не до конца всё понял

@I.M. · 23.10.2012, 20:16

C++
1
2
3
4
5
6
list<int>::iterator find(int h, list<int>& l)
{
    list<int>::iterator i=l.begin();
    for(; i!=l.end() && *i!=h; ++i);
    return i;
}

Но лучше так

C++
1
z = std::find(l.begin(), l.end(), h);

И вы в курсе, что l.begin() - это значащий элемент в отличие от l.end()?

@UmKaM · 23.10.2012, 20:18 **[ТС]**

I.M., узнал полчаса назад

Но, однако, косяка не вижу в упор. А этот кусок уже и вправду лишний (я его вычеркнуть забыл, он не используется).

@soon · 23.10.2012, 20:22

Не по теме:

Сообщение от Герц

У всех контейнеров есть метод find( T item ), возвращающий итератор на запрошенный элемент.

А что, std::vector уже не контейнер?

@I.M. · 23.10.2012, 20:25

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

А этот кусок уже и вправду лишний (я его вычеркнуть забыл, он не используется).

Т.е. проблема решена или нет?
Косяк был в том, что вы в функцию find передавали копию списка l. Соответственно, возвращался итератор на конец копии. Я же выше поправил вашу функцию

@UmKaM · 23.10.2012, 20:27 **[ТС]**

I.M., после фикса этот косяк исчез, но работать программа отказывается наотрез.

@I.M. · 23.10.2012, 20:32

UmKaM, у меня компилится и запускает. Что теперь не так?

@UmKaM · 23.10.2012, 20:41 **[ТС]**

I.M., компилится, запускается, но не работает. Она просто берёт и зависает. Точнее даже не зависает, а не хочет выходить из этого цикла (тобишь не изменяет значение c)

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
#include <iostream>
#include <list>
#include <iterator>
#include <algorithm>
 
using namespace std;
 
typedef list<int>::iterator it;
typedef list<int> lis;
 
void spisker(lis l, it z, int a1, int a2, int a3, int a4, int b1, int b2, int c)
{   //функция вычисляет очередной элемент последовательности.
    it i=z;
    z--;
    int h=(*z)*(a1*(*i)+a2)+(*i)*a3+a4;
    if(h>b1 && h>b2)
    {
        h=h-c*(((h-b2)%c!=0)+(h-b2)/c);
    }
    z=std::find(l.begin(), l.end(), h); 
    if((z!=l.end()) && (z!=l.begin())) //Теперь оно сюда не заходит. Вообще. Принципиально. 
    {
        z--;
        i--;
        h=(*i);
        if(h==(*z)) //в итоге тут происходит обращение к "левой" памяти.
            c=-1;
    }
    l.push_back(h);
}
 
int main()
{
    int a1,a2,a3,a4,b1,b2,c;
    a1=0;           //ввод значений упускаю, даже на этом примере не работает.
    a2=0;
    a3=2;
    a4=3;
    b1=20;
    b2=5;
    c=7;           //cin >> a1 >> a2 >> a3 >> a4 >> b1 >> b2 >> c;
    lis l;
    int q, p=0;
    q=0;            //cin >> q;
    l.push_back(q);
    q=1;               //cin >> q;
    l.push_back(q);
    while(p==0) 
    {
        it z=l.end();
        z--;
        spisker(l, z, a1, a2, a3, a4, b1, b2, c);
        if(c==-1)
        {
            q=1;
            p=1;
            for(it i=l.begin(); i!=z; i++)
                p++;
            for(it i=z; i!=l.end(); i++)
                q++;
        }
    }
    cout << p << ' ' << q;
    return 0;
}

@Герц · 23.10.2012, 20:53

А что, std::vector уже не контейнер?

Ок ок, есть std::find

Новые блоги и статьи Все статьи Все блоги /
Конвертировать закладки 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. На борту пять. . .	Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@Герц 545 / 344 / 12 Регистрация: 05.11.2010 Сообщений: 1,076 Записей в блоге: 1
	23.10.2012, 19:49
	У всех контейнеров есть метод find( T item ), возвращающий итератор на запрошенный элемент. Сделай typedef для list<int> и list<int>::iterator, а то читать нереально. 0

@UmKaM 0 / 0 / 0 Регистрация: 23.10.2012 Сообщений: 5
	23.10.2012, 20:18 [ТС]
	I.M., узнал полчаса назад Но, однако, косяка не вижу в упор. А этот кусок уже и вправду лишний (я его вычеркнуть забыл, он не используется). 0

@UmKaM 0 / 0 / 0 Регистрация: 23.10.2012 Сообщений: 5
	23.10.2012, 20:27 [ТС]
	I.M., после фикса этот косяк исчез, но работать программа отказывается наотрез. 0

@I.M. 576 / 559 / 47 Регистрация: 16.12.2011 Сообщений: 1,389
	23.10.2012, 20:32
	UmKaM, у меня компилится и запускает. Что теперь не так? 0