Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.60/5: Рейтинг темы: голосов - 5, средняя оценка - 4.60
GraK
0 / 0 / 0
Регистрация: 22.01.2017
Сообщений: 34
Завершенные тесты: 1
1

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

26.06.2017, 15:05. Просмотров 824. Ответов 4
Метки нет (Все метки)

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

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;
    
}
0
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
26.06.2017, 15:05
Ответы с готовыми решениями:

Функция работает некорректно
написал функцию,возвращающий нажатую клавишу: char func2(){ if(_kbhit()){ char c;...

Некорректно работает функция
Функция Small_S должна находить короткое слово в строке str, а она почему-то выводит первое слово!...

Функция работает некорректно
#include &lt;iostream&gt; using namespace std; short fill(double*, short); const short size = 5; ...

Некорректно работает функция rand
в полном коде почему-то не работает корректно функция rand, вот пример: #include&lt;cstdlib&gt;...

Некорректно работает функция проверки на численность
Фрагмент кода: //... bool is_number() { //* while(cin.fail()) { cin.clear(); cin.sync();...

4
nmcf
6525 / 5749 / 2621
Регистрация: 14.04.2014
Сообщений: 24,493
26.06.2017, 15:38 2
После erase() итератор недействителен. Если о стране не более одной записи в массиве, то просто прерывай цикл после удаления.
1
Croessmah
++Ͻ
15216 / 8609 / 1637
Регистрация: 27.09.2012
Сообщений: 21,184
Записей в блоге: 2
Завершенные тесты: 1
26.06.2017, 15:39 3
Лучший ответ Сообщение было отмечено GraK как решение

Решение

C++
1
if (badValue(*itr, name)) itr = theCountry.erase(itr);
1
GraK
0 / 0 / 0
Регистрация: 22.01.2017
Сообщений: 34
Завершенные тесты: 1
26.06.2017, 23:33  [ТС] 4
Цитата Сообщение от Croessmah Посмотреть сообщение
C++
1
if (badValue(*itr, name)) itr = theCountry.erase(itr);
Помогите пожалуйста разобраться еще в одной проблеме. После записи в файле одна запись, а функция Display видит две, и после удаления проблемы: если записать две страны (Франция, Англия), а потом удалить Францию то Англии будет при вызове Display 3 штуки. Где я накосячил?
Заранее спасибо!)
0
Croessmah
++Ͻ
15216 / 8609 / 1637
Регистрация: 27.09.2012
Сообщений: 21,184
Записей в блоге: 2
Завершенные тесты: 1
27.06.2017, 10:23 5
Лучший ответ Сообщение было отмечено GraK как решение

Решение

C++
1
2
3
4
    while (file >> tmp->Name &&  file >> tmp->Capital && file >> tmp->Language && file >> tmp->Population)
    {       
        theCountry.push_back(*tmp);
    }
1
27.06.2017, 10:23
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
27.06.2017, 10:23

Некорректно работает функция добавления, указать ошибку
Здравствуйте, в моей программе некорректно работает добавление элемента, функция addBook. Также...

Функция send() работает некорректно после применения closesocket()
Доброго времени суток! Имеется простенький тестовый сервер: SOCKET s; SOCKET s1; SOCKET s2 =...

Некорректно работает функция рандомайз при заполнении массива
Задача - внести случайные числа в динамический массив. Проблема - почему-то при первом запуске...


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

Или воспользуйтесь поиском по форуму:
5
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2019, vBulletin Solutions, Inc.
Рейтинг@Mail.ru