Ошибка: abort() has been called

@kverg · Регистрация: 21.01.2024

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

Есть вот такой вот код, который должен искать количество символом в текстовом файле.

Вылетает ошибка при компиляции,
также завершает с вот таким вот кодом: завершил работу с кодом -2147483645.

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
#include <iostream>
#include <fstream>
using namespace std;
int much;
void find(const char simbol, string inside) {
    for (int i = 0; i <= inside.size(); i++) {
        if (inside.at(i) == simbol)
        {
            much = much + 1;
        }
    }
}
int main()
{
    string way;
    string file;
    char sy;
    cout << "Write way to file ";
    cin >> way;
    ifstream myfile(way);
    if (myfile.is_open()) {
        cout << "file has opened";
    }
    else cout << "file hasnt opened";
    cout << "\n\n";
    cout << "Choose symbol to find ";
    cin >> sy;
    while (!myfile.eof()) {
        char c;
        myfile >> c;
        file = file + c;
    }
    find(sy, file);
    cout << '\n';
    cout << sy << " has finded for " << much << " times";
}

@systy257 · 21.01.2024, 21:37

C++
1
for (int i = 0; i <= inside.size(); i++) {

В этой строчке вас должно что-то засмущать. Чисел от 0 до inside.size() существует inside.size() + 1. А символов в строке всего inside.size(). Что-то здесь не сходится

. Вы обращаетсь к символу за пределами строки, если берете inside.at(inside.size()). Вот программа и валится.

Алексей1153 · 21.01.2024, 21:52

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

else cout << "file hasnt opened";

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

while (!myfile.eof()) {

тут тоже есть, чему засмущать

@Tanya2007 · 21.01.2024, 22:27

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

C++
1
for (int i = 0; i <= inside.size(); i++)

Выход за границы массива, идти надо до

C++
1
i < inside.size();

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

C++
1
2
3
4
5
while (!myfile.eof()) {
char c;
myfile >> c;
file = file + c;
}

Эта конструкция не верна.

правильней будет так:

C++
1
2
3
4
5
char c;
while (myfile >> c) 
{
file += c;
}

+ если в вашем файле есть пробелы - оператор извлечения (>>) прочитает лишь до первого пробела, в таком случае лучше будет использовать функцию get().

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

C++
1
2
3
4
5
while (!myfile.eof()) {
char c;
myfile >> c;
file = file + c;
}

поясню почему не верна, вы читаете посимвольно, представим, что последний символ файла прочитан, указатель чтения файла установлен за последним символом, т.е. флаг eofbit потока еще не установлен, а значит на следующей итерации цикла !myfile.eof() вернет вам true. И в теле цикла, при очередном чтении из потока myfile >> c;, поток установится в eofbit, а в с останется мусор, который запишется в file.

@TheCalligrapher · 22.01.2024, 02:38

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

Вылетает ошибка при компиляции,

Где вы процитировали эту ошибку в своем вопросе?

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

также завершает с вот таким вот кодом: завершил работу с кодом -2147483645.

Не понял. Кто "завершил работу"?

@pechka_ne_sed · 22.01.2024, 19:56

kverg, проблемы вашего кода настолько просты и банальны, что Вам лучше разобраться в них самостоятельно, я
лишь укажу, что может помочь.
1. Чисто архитектурные проблемы. (на работу кода не влияют, но очееень важны, например, на собеседовании)
Вместо того, чтобы просто взять и вернуть значение из функции find Вы зачем-то заводите глобальную переменную much и записываете в нее результат. Не говоря уже о том, что от использования глобальных/статических переменных лучше по возможности воздерживаться, при многократном использовании этой функции Вам придется обнулять эту переменную.
Путь до файла называется не "way" а "path".
От using namespace std лучше отказаться.
2. Выход за границы строки (замечание Tanya2007).
Как вы знаете, индексация массивов начинается с 0. std::string - тоже не исключение.
Следовательно, индексы объекта std::string размером n, таковы: 0..n-1.
А итерация до n включительно (<= n) вызовет данное поведение.
Остальное было подмечено Tanya2007.

@kverg · 22.01.2024, 23:10 **[ТС]**

Всем спасибо за помощь, в следующий раз буду внимательней, учту пробелы в английском и донесении своей мысли до собеседника, ещё раз спасибо!!!

Новые блоги и статьи Все статьи Все блоги /
Перемещение выделенных строк ТЧ из одного документа в другой Maks 30.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .	Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .	Автоматическое создание документа при проведении другого документа Maks 29.03.2026 Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .	Настройка движения справочника по регистру сведений Maks 29.03.2026 Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .
Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .	Сумматор с применением элементов трёх состояний. Hrethgir 26.03.2026 Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .	Автозаполнение реквизитов при создании документа Maks 26.03.2026 Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .	Команды формы и диалоговое окно Maks 26.03.2026 1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .

@pechka_ne_sed 27 / 24 / 4 Регистрация: 20.11.2023 Сообщений: 131
	22.01.2024, 19:56
	kverg, проблемы вашего кода настолько просты и банальны, что Вам лучше разобраться в них самостоятельно, я лишь укажу, что может помочь. 1. Чисто архитектурные проблемы. (на работу кода не влияют, но очееень важны, например, на собеседовании) Вместо того, чтобы просто взять и вернуть значение из функции `find` Вы зачем-то заводите глобальную переменную `much` и записываете в нее результат. Не говоря уже о том, что от использования глобальных/статических переменных лучше по возможности воздерживаться, при многократном использовании этой функции Вам придется обнулять эту переменную. Путь до файла называется не "way" а "path". От `using namespace std` лучше отказаться. 2. Выход за границы строки (замечание Tanya2007). Как вы знаете, индексация массивов начинается с 0. `std::string` - тоже не исключение. Следовательно, индексы объекта `std::string` размером n, таковы: 0..n-1. А итерация до n включительно (<= n) вызовет данное поведение. Остальное было подмечено Tanya2007. 3

@kverg 0 / 0 / 0 Регистрация: 21.01.2024 Сообщений: 2
	22.01.2024, 23:10 [ТС]
	Всем спасибо за помощь, в следующий раз буду внимательней, учту пробелы в английском и донесении своей мысли до собеседника, ещё раз спасибо!!! 0