Некорректно работает функция Delete

@GraK · Регистрация: 22.01.2017

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

Слепил прожку. Которая должна записать в файл информацию о стране. А потом функция Делит должна по названию страны удалить всю информацию о ней. Но что-то делаю не так. Посмотрите пожалуйста. Заранее буду благодарен.

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
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
#include <iostream>
#include <string> 
#include <vector>
#include <fstream>
#include <sstream>
#include "stdlib.h"
 
using namespace std;
const char* filename="CountryInfo.txt";
//const char end_of_tag = ';';
//const char end_of_struct = '^';
 
 
struct CountryInfo
{
    string Name;
    string Capital;
    string Language;
    int Population;
    
};
 
bool badValue(CountryInfo &tmp, string name)
{
    return tmp.Name == name;
} 
 
void Add (vector<CountryInfo> &theCountry)
{    
    CountryInfo *tmp = new CountryInfo;
 
    cout << "Numb " << theCountry.size()<< endl;
    cout << "Enter Coutry: ";
    getline(cin ,tmp->Name);
    cout << "Enter Capital: ";
    getline(cin, tmp->Capital);
    cout << "Enter Official Language: ";
    getline(cin, tmp->Language);
    cout << "Enter Population:  ";
    cin >> tmp->Population;
 
    cout << "\n";
    cin.ignore();
 
    theCountry.push_back( *tmp );
 
    delete tmp;
}
void Display(vector<CountryInfo> &theCountry)
{
    if (theCountry.empty())
    {
        cout << "Not found!!!" << endl;
        return;
    }
    vector<CountryInfo>::iterator itr;
 
    for ( itr = theCountry.begin(); itr != theCountry.end(); itr++)
    {
        cout << "Country ";
        cout << (*itr).Name << endl;
        cout << "Capital ";
        cout << (*itr).Capital << endl;
        cout << "Official Language: ";
        cout << (*itr).Language << endl;
        cout << "Population ";
        cout << (*itr).Population << endl;
        
        cout << "\n";
    }
}
void Delete(vector<CountryInfo> &theCountry, string name)
{
    if (theCountry.empty())
    {
        cout << "Not Found!!!" << endl;
        return;
    }
    vector<CountryInfo>::iterator itr;
    cout << "Enter name" << endl;
    cin >>name;
 
    for (itr = theCountry.begin(); itr != theCountry.end();)
    {
        if (badValue(*itr, name)) theCountry.erase(itr++);
        else ++itr;
    }
}
void Write(fstream &file, vector<CountryInfo> &theCountry)
{
    if (!file || theCountry.empty())
        return;
 
    vector<CountryInfo>::iterator itr;
 
    for (itr = theCountry.begin(); itr != theCountry.end(); itr++)
    {
        file << (*itr).Name << endl;
        file << (*itr).Capital << endl;
        file << (*itr).Language << endl;
        file << (*itr).Population << endl<<endl;
    }
}
 
void Read(fstream &file, vector<CountryInfo> &theCountry)
{
    if ( !file )
        return;
 
    CountryInfo *tmp = new CountryInfo;
 
    while ( !file.eof() )
    {       
        file >> tmp->Name;
        file >> tmp->Capital;
        file >> tmp->Language;
        file >> tmp->Population;
 
        theCountry.push_back(*tmp);
    }
 
    delete tmp;
}
 
 
 
int main()
{
    vector<CountryInfo> theCountry;
    string tmpName;
    fstream my_file(filename, ios_base::in);
    int n=0;
    
    cout << "What you want to do with this file?"<< endl;
    cout << "Enter 1 - Add\t 2 - Delete\t 3 - Show All: ";
    cin >> n;
    cin.ignore();
    if (n==1)
    {
        fstream my_file1(filename, ios_base::app);
        Add(theCountry);
        Write(my_file1, theCountry);
        my_file1.close();       
    }   
    if (n==2)
    {
        fstream my_file(filename, ios_base::in);
        Read(my_file, theCountry);
        fstream my_file1(filename, ios_base::out);
        Delete(theCountry, tmpName);        
        Write(my_file1, theCountry);
        my_file1.close();
        
    }
    if (n==3)
    {
        Read(my_file, theCountry);
        Display(theCountry);
        my_file.close();
    }
     
       
    return 0;
    
}

@nmcf · 26.06.2017, 15:38

После erase() итератор недействителен. Если о стране не более одной записи в массиве, то просто прерывай цикл после удаления.

Croessmah · 26.06.2017, 15:39

C++
1
if (badValue(*itr, name)) itr = theCountry.erase(itr);

@GraK · 26.06.2017, 23:33 **[ТС]**

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

C++
1
if (badValue(*itr, name)) itr = theCountry.erase(itr);

Помогите пожалуйста разобраться еще в одной проблеме. После записи в файле одна запись, а функция Display видит две, и после удаления проблемы: если записать две страны (Франция, Англия), а потом удалить Францию то Англии будет при вызове Display 3 штуки. Где я накосячил?
Заранее спасибо!)

Croessmah · 27.06.2017, 10:23

C++
1
2
3
4
    while (file >> tmp->Name &&  file >> tmp->Capital && file >> tmp->Language && file >> tmp->Population)
    {       
        theCountry.push_back(*tmp);
    }

Новые блоги и статьи Все статьи Все блоги /
Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.	Фиксация колонок в отчете СКД Maks 14.04.2026 Фиксация колонок в СКД отчета типа Таблица. Задача: зафиксировать три левых колонки в отчете. Процедура ПриКомпоновкеРезультата(ДокументРезультат, ДанныеРасшифровки, СтандартнаяОбработка) / / . . .	Настройки VS Code Loafer 13.04.2026 { "cmake. configureOnOpen": false, "diffEditor. ignoreTrimWhitespace": true, "editor. guides. bracketPairs": "active", "extensions. ignoreRecommendations": true, . . .	Оптимизация кода на разграничение прав доступа к элементам формы Maks 13.04.2026 Алгоритм из решения ниже реализован на нетиповом документе, разработанного в конфигурации КА2. Задачи, как таковой, поставлено не было, проделанное ниже исключительно моя инициатива. Было так:. . .
Контроль заполнения и очистка дат в зависимости от значения перечислений Maks 12.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: реализовать контроль корректности заполнения дат назначения. . .	Архитектура слоя интернета для сервера-слоя. Hrethgir 11.04.2026 В продолжение https:/ / www. cyberforum. ru/ blogs/ 223907/ 10860. html Знаешь что я подумал? Раз мы все источники пишем в голове ветки, то ничего не мешает добавить в голову такой источник, который сам. . .	Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: при выборе сотрудника (справочник Сотрудники) в ТЧ документа. . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .

Некорректно работает функция Delete

Решение

Решение

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	26.06.2017, 15:38
	После erase() итератор недействителен. Если о стране не более одной записи в массиве, то просто прерывай цикл после удаления. 1