С Новым годом! Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.86/7: Рейтинг темы: голосов - 7, средняя оценка - 4.86
 Аватар для Danil_2002
35 / 25 / 10
Регистрация: 16.02.2018
Сообщений: 152

Проблема с итератором

09.04.2019, 16:15. Показов 1481. Ответов 18
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Добрый день!

На днях столкнулся с трудностью. Есть текстовый файл с ионами. Моя функция должна перебрать этот файл и выбрать оттуда все ионы, которые начинаются с определенного символа.
Моя функция находит все нужные ионы, тут все норм. Но когда я переношу их из списка в массив(это важно) возникает проблема.
Ионов в списке трое, а в массив записывается только два , на третьем все останавливается и программа завершается.

C++ (Qt)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
void Ion::get_list_ion()
{
    list<string> listion;
    fstream list_ion_file;
    list_ion_file.open("C:\\Users\\bymi1\\OneDrive\\Рабочий стол\\Данил\\Разработка\\C++\\Nicolas-0.0.1\\ion_list.txt");
    string ion;
    while(getline(list_ion_file, ion))
    {
        if(ion[0] == simvol)
        {
            listion.push_back(ion);
        }
    }
    list<string>::iterator it;
    int start = 0;
    cout << listion.size();
    for (it = listion.begin(); it != listion.end(); it++)
    {
        array_ions[start] = *it; // проблема здесь на третьем повторении 
        start++;
        cout << *it << endl; на третьем повторении ничего не выводится 
    }
}
array_ions объявлен в хидере класса.

В чем моя ошибка и как ее исправить ?

Добавлено через 52 секунды
скорее всего что то с массивом не так
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
09.04.2019, 16:15
Ответы с готовыми решениями:

C++ | Проблема с итератором
У меня есть два класса - PhoneBook и Contact. PhoneBook содержит set&lt;Contact&gt;. Также есть метод ShowContacts() который выводит все...

Проблема с итератором
Для тренировки по теме map решил создать подобие словаря, где ключ - английское слово, а значение - его перевод на русский. Так вот на...

Проблема с итератором в <vector>
Всем привет. Помогите пожалуйста решить проблему: У меня существует класс class Book { string author,title,publishing; int...

18
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
09.04.2019, 16:22
Цитата Сообщение от Danil_2002 Посмотреть сообщение
array_ions объявлен в хидере класса.
Как объявлен?
0
Mental handicap
 Аватар для Azazel-San
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
09.04.2019, 16:24
Цитата Сообщение от Danil_2002 Посмотреть сообщение
C++
1
array_ions[start] = *it; // проблема здесь на третьем повторении
А ошибки какие-то есть?
Дебажить не пробовали?
0
 Аватар для Danil_2002
35 / 25 / 10
Регистрация: 16.02.2018
Сообщений: 152
09.04.2019, 16:38  [ТС]
Массив объявлен скорее всего не правильно. Ошибок компилятор не выдает.
Под массив я память выделял , не помогло. Выделял внутри функции, в которой собственно и ошибка.

class Ion
{
public:
Ion(char sim, string formula, int i);
// Удалить
string get_ion();
void get_list_ion();
int size_list_ion();
private:
int index;
string formula;
string array_ions[];
char simvol;
//void fixed_massiv();
};

Вообщем это с массивом что то не то. Потому что я до этого читал кое-какие статьи и писал небольшие примеры с использованием итераторов. все работало.
0
Mental handicap
 Аватар для Azazel-San
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
09.04.2019, 16:41
Цитата Сообщение от Danil_2002 Посмотреть сообщение
Ошибок компилятор не выдает.
А варнинги?
Цитата Сообщение от Danil_2002 Посмотреть сообщение
C++
1
string array_ions[];
Какой компилятор?
Это не должно компилится. Это flexible array member, унаследованный из Си, в С++ такого нету.
Но компиляторы вредные )
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
09.04.2019, 16:41
Цитата Сообщение от Danil_2002 Посмотреть сообщение
C++
1
string array_ions[];
И какой тут размер массива?
0
 Аватар для Danil_2002
35 / 25 / 10
Регистрация: 16.02.2018
Сообщений: 152
09.04.2019, 16:43  [ТС]
Но прикол то в том что первые два иона в этот массив записываются, даже не смотря на то что под него память не выделена.
0
Mental handicap
 Аватар для Azazel-San
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
09.04.2019, 16:44
Цитата Сообщение от Azazel-San Посмотреть сообщение
Это flexible array member, унаследованный из Си, в С++ такого нету.
Хотя не, он у вас не последним стоит.
Тогда такое не должно компилироваться.
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
09.04.2019, 16:44
Цитата Сообщение от Danil_2002 Посмотреть сообщение
Но прикол то в том что первые два иона в этот массив записываются, даже не смотря на то что под него память не выделена.
Знаешь, что такое UB?
0
 Аватар для Danil_2002
35 / 25 / 10
Регистрация: 16.02.2018
Сообщений: 152
09.04.2019, 16:45  [ТС]
У меня C++98 или C98++ в общем 98 года поддерживается.
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
09.04.2019, 16:46
Цитата Сообщение от Danil_2002 Посмотреть сообщение
У меня C++98 или C98++ в общем 98 года поддерживается.
Компилятор (среда) какой?
0
Mental handicap
 Аватар для Azazel-San
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
09.04.2019, 16:47
Цитата Сообщение от Danil_2002 Посмотреть сообщение
У меня C++98 или C98++ в общем 98 года поддерживается.
Всеравно не должно компилироваться.
Учитывая что 98 стандарт поддерживают только gcc и clang, предположу что у вас один из них.
И таки это не должно компилироваться.
0
 Аватар для Danil_2002
35 / 25 / 10
Регистрация: 16.02.2018
Сообщений: 152
09.04.2019, 18:57  [ТС]
Ладно, с тонкостями компиляции программ я сам потом разберусь. И вообще моя проблема наверное по большей части заключается сейчас в том, что я не правильно спроектировал класс. Если бы я по нормальному все сделал, то не возникло бы такой проблемы. ООП тоже еще хромает.

Так вот. Есть класс Ion он принимает символ, формулу в виде строки и индекс в формуле. Задача этого класса считать этот ион из формулы и вернуть его. Например есть формула: Cu(OH)2 Этот класс принимает: ( Символ: C, формулу: Cu(OH)2 и индекс этого символа 0) Самая первая функция, (где у меня проблемы как раз) должна перебрать файл где хранятся все ионы и вернуть список ионов, которые начинаются на данный символ. В моем случае это: Cu, Cl,Cr. Дальше этот список передается в другую функцию, которая выбирает правильный ион. Читает посимвольно формулу и ион, если символы не совпадают, то удаляет ион. Так пока не останется один истинный ион, его мы и возвращаем.

Это кратенько о том, что я сейчас делаю, может это как то прояснит ситуацию.

Вот код, который я уже перелопатил и стер вторую функцию, но цель та же. Нужно передать эти три иона во вторую функцию, а та в свою очередь найдет нужный ион. Но передавать нужно именно массивом, а не списком. Как это лучше сделать?

C++ (Qt)
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
#include "ion.h"
 
Ion::Ion(char sim, string formul, int i)
{
    simvol = sim;
    formula = formul;
    index = i;
}
string* Ion::get_list_ion()
{
    list<string> listion;
    fstream list_ion_file;
    list_ion_file.open("C:\\Users\\bymi1\\OneDrive\\Рабочий стол\\Данил\\Разработка\\C++\\Nicolas-0.0.1\\ion_list.txt");
    string ion;
    while(getline(list_ion_file, ion))
    {
        if(ion[0] == simvol)
        {
            listion.push_back(ion);
        }
    }
    size_list = listion.size();
    cout << size_list;
    string array_ion[size_list];
    list<string>::iterator it;
    int start = 0;
    for (it = listion.begin(); it != listion.end(); it++)
    {
        array_ion[start] = *it;
        cout << *it << endl;
        start++;
    }
    return &array_ion[0];
}
string Ion::get_ion()
{
      string* pointer = get_list_ion();
      for(int start = 0; start < size_list; start++)
      {
          cout << "OK";
          string ion = *(pointer + start); 
 
      }
}
C++ (Qt)
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
#ifndef ION_H_INCLUDED
#define ION_H_INCLUDED
#include <list>
#include <string>
#include <fstream>
#include <iostream>
#include <iterator>
 
using namespace std;
class Ion
{
    public:
        Ion(char sim, string formula, int i);
        // Удалить
        string get_ion();
        string* get_list_ion();
        int size_list_ion();
    private:
        int index;
        int size_list;
        string formula;
        //string list<string> listions;
        char simvol;
};
 
#endif // ION_H_INCLUDED
Проблема get_ion. OK выводится один раз и программа виснет.

P.S Спасибо что прочитали до конца
0
Mental handicap
 Аватар для Azazel-San
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
09.04.2019, 20:02
Danil_2002, как в файле содержатся записи?
Приведи пример, что хранится в файле.
0
 Аватар для Danil_2002
35 / 25 / 10
Регистрация: 16.02.2018
Сообщений: 152
09.04.2019, 20:50  [ТС]
H
OH
Na
Al
Cl
Ba
Cr

Добавлено через 1 минуту
Одна строка - один ион. Но читается все ок. Мне нужно передать считанные ионы на обработку другой функции. С помощью массива только как правильно. Как я сделал не работает.
0
Mental handicap
 Аватар для Azazel-San
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
09.04.2019, 20:55
Цитата Сообщение от Danil_2002 Посмотреть сообщение
Этот класс принимает: ( Символ: C, формулу: Cu(OH)2
А формулы откуда берете?
0
 Аватар для Danil_2002
35 / 25 / 10
Регистрация: 16.02.2018
Сообщений: 152
09.04.2019, 21:09  [ТС]
Формулу я сам ввожу. Дело в том что я пишу небольшую химическую программу, которая должна составить структурную формулу вводимого вещества. И я хочу для начала написать что то типа терминала, который принимает текст с точки зрения компьютера
и разбивает его на готовые ионы для дальнейшей работы программы.

Вот код еще одного класса, который обрабатывает формулу по символам.

C++ (Qt)
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
#include "enumeration.h"
//#include "seach_skobka.h"
//#include "global_file.h"
 
using namespace std;
 
struct Skobka_struct
{
    int border[2]; // Начальный и конечный индекс скобки
    int koif; // Коэффицент скобки
};
 
void Enumeration::set_formula(string formul)
{
    formula = formul;
    seach();
}
void Enumeration::seach()
{
    for(int start = 0; start < formula.size(); start++)
    {
        char simvol = formula[start];
        cout << simvol << endl;
        char s = seach_array_skobka(simvol);
        if (s != '0') // Значит скобка
        {
            Seach_skobka skobka_class(formula, s, start);
            int* pointer_array_info = skobka_class.get_skobka();
            int info_array[3];
            for (int start = 0; start < 3; start++)
            {
                info_array[start] = *(pointer_array_info + start);
                cout << info_array[start] << endl;
            }
            Skobka_struct skobka_struct;
            skobka_struct.border[0] = info_array[0];
            skobka_struct.border[1] = info_array[1];
            skobka_struct.koif = info_array[2];
        }
        else if (isalpha(simvol) && isupper(simvol)) // Значит начало иона
        {
            Ion ion(simvol, formula, start);
            ion.get_ion();
            //string* pointer = ion.get_list_ion();
            //int size_list = ion.size_list_ion(); // Размер списка ионов
        }
        else {continue;}
    }
}
0
Mental handicap
 Аватар для Azazel-San
1246 / 624 / 171
Регистрация: 24.11.2015
Сообщений: 2,429
09.04.2019, 21:23
Лучший ответ Сообщение было отмечено Danil_2002 как решение

Решение

Danil_2002, ага, ну удачи вам
Цитата Сообщение от Danil_2002 Посмотреть сообщение
C++
1
return &array_ion[0];
Здесь у вас грубая ошибка, вы пытаетесь возвратить указатель на локальную или как еще ее называют автоматическую переменную. Она перестает существовать после выхода из функции, соответственно вы и ловите в рантайме SEGFAULT при обращении к такому указателю. Каков выход?
Их несколько, можно вернуть ссылку на массив. Приведу упрощенный пример:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <iostream>
#include <string>
 
auto& get_string_arr() {
    static std::string str_arr[4]{"Hello", ",", "world", "!"};
    return str_arr;
}
 
int main() {
    auto& str_arr = get_string_arr();
    for (auto&& str : str_arr) {
        std::cout << str;
    }
}
1
 Аватар для Danil_2002
35 / 25 / 10
Регистрация: 16.02.2018
Сообщений: 152
09.04.2019, 21:25  [ТС]
Блин, точно! Я забыл что локальные переменные перестают существовать. Большое спасибо
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
09.04.2019, 21:25
Помогаю со студенческими работами здесь

Работа с итератором
Есть задание: Описать функцию с двумя параметрами: контейнер-список целых элементов и контейнер-вектор указателей на целые. Функция...

ошибка с итератором
Помогите, пожалуйста, не могу понять зачем вычитать нужно еденицу из deq1.begin() #include &lt;iostream&gt; #include &lt;string&gt; ...

Что не так с итератором ?
Есть задача - для списка целочисленных значений реализовать итератор с поддержкой константной корректности. (т.е с возможностью итерации по...

Сравнение указателя с итератором
Здравствуйте, уважаемые знатоки! Продолжая изучать STL и итераторы наткнулся на такую вот аналогию: #include &lt;iostream&gt; ...

Задача с чтением из потока итератором
Здравствуйте! Делаю задачу из книги Липмана, есть входная последовательность чисел через istream_iterator и 2 выходные через...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
19
Ответ Создать тему
Новые блоги и статьи
Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR
ФедосеевПавел 06.01.2026
Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .
Модель микоризы: классовый агентный подход 2
anaschu 06.01.2026
репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .
Расчёт токов в цепи постоянного тока
igorrr37 05.01.2026
/ * Дана цепь постоянного тока с сопротивлениями и напряжениями. Надо найти токи в ветвях. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и решает её. Последовательность действий:. . .
Новый CodeBlocs. Версия 25.03
palva 04.01.2026
Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .
Модель микоризы: классовый агентный подход
anaschu 02.01.2026
Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
Programma_Boinc 28.12.2025
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / **********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / **********/ gallery/ bKBkQFf Пост отсюда. . .
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
Programma_Boinc 26.12.2025
Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru