Дублирование записей после завершения программы

@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
166
167
168
169
#include <iostream>
#include <string> 
#include <vector>
#include <cstdlib>
#include <fstream>
#include <cmath>
#define EARTH_RADIUS 6372795
 
 
using namespace std;
const char* filename="Place.txt";
 
 
struct Place
{
    string Name;
    double x;
    double y;
    
};
 
bool badValue(Place &tmp, string name)
{
    return tmp.Name == name;
} 
 
void Add (vector<Place> &a)
{    
    Place *tmp = new Place;
 
    cout << "Enter name place and koordinate x,y " << endl;
    cout << "Enter Name: ";
    getline(cin ,tmp->Name);
    cout << "Enter x: ";
    cin >> tmp->x;
    cout << "Enter y: ";
    cin >> tmp->y;     
    cout << "\n";    
    a.push_back( *tmp ); 
    delete tmp;
}
void Display(vector<Place> &a)
{
    if (a.empty())
    {
        cout << "Not found file!!!" << endl;
        return;
    }
    vector<Place>::iterator itr;
 
    for ( itr = a.begin(); itr != a.end(); itr++)
    {
        cout << "Place: ";
        cout << (*itr).Name << endl;
        cout << "x: ";
        cout << (*itr).x << endl;
        cout << "y: ";
        cout << (*itr).y << endl;                
        cout << "\n";
    }
}
void Delete(vector<Place> &a, string name)
{
    if (a.empty())
    {
        cout << "Not Found!!!" << endl;
        return;
    }
    vector<Place>::iterator itr;
 
    for (itr = a.begin(); itr != a.end();)
    {
        if (badValue(*itr, name)) a.erase(itr++);
        else ++itr;
    }
}
void Write(fstream &file, vector<Place> &a)
{
    if (!file || a.empty())
        return;
 
    vector<Place>::iterator itr;
 
    for (itr = a.begin(); itr != a.end(); itr++)
    {
        file << (*itr).Name << endl;
        file << (*itr).x << endl;
        file << (*itr).y << endl;
    }
}
 
void Read(fstream &file, vector<Place> &a)
{
    if ( !file )
        return;
 
    Place *tmp = new Place;
 
    while ( !file.eof() )
    {       
        file >> tmp->Name;
        file >> tmp->x;
        file >> tmp->y;
        
        a.push_back(*tmp);
    }
 
    delete tmp;
}     
 
int main()
{
        
    vector<Place> thePlace;
    string tmpName;   
    fstream my_file(filename, ios_base::in);
    char key = 'a';
    
    do {
        std::cout //<< " 1. Calculate\n"
            << " 2. Add\n"
            << " 3. Delete\n"
            << " 4. Show All\n"
            << " 0. Exit\n\n"
            << " Enter: "; 
            std::cin >> key;
            std::cin.ignore();
            switch(key) 
            {
                /*case '1':
                {
                    Read(my_file, thePlace);
                    calculateTheDistance(my_file, thePlace);
                }*/
                    break;
                case '2':
                {
                    fstream my_file1(filename, ios_base::app);
                    Add(thePlace);
                    Write(my_file1, thePlace);
                    my_file1.close();
                }
                    break;
                case '3':
                {
                    Delete(thePlace, tmpName);
                    fstream my_file1(filename, ios_base::out);      
                    Write(my_file1, thePlace);
                    my_file1.close();
                }
                    break;
                case '4':
                {
                    Read(my_file, thePlace);
                    Display(thePlace);
                    my_file.close();
                    system("pause");
                }
                default:
                
                    std::cout << "[Error] Function not found!" << std::endl << std::endl;
                    system("cls"); 
            }
      } 
      while (key != '0');
  
    return 0;
    
}

@anapshy · 18.08.2017, 14:07

GraK,
1) Забыл подчищать вектор [INLINE]your_vector.clear();,
из-за этого дублировалось.
2) Корявый вывод в файл - поправил.
3) Старайся использовать const переменные вместо #define, т.к. #define может быть удален препроцессором раньше, чем попадет в компилятор.
#define больше относится к языку С, чем к С++.
О данной проблеме есть куча статей в интернете, если интересно - почитай.
4) После вывода файла в консоль, он закрывался и нельзя было его прочитать повторно - исправил.
5) Зачем EARTH_RADIUS мне не совсем понятно, она же не используется. Ну да ладно.

P.s. где вносил изменения обозначил как // <--

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
#include <iostream>
#include <string> 
#include <vector>
#include <fstream>
#include <cstdlib>
#include <cmath>
using namespace std;
 
const std::string filename = "Place.txt";  // <--
const unsigned EARTH_RADIUS = 6372795;  // <--
 
struct Place
{
    string Name;
    double x;
    double y;
};
 
bool badValue(Place &tmp, string name)
{
    return tmp.Name == name;
}
 
void Add(vector<Place> &a)
{
    Place *tmp = new Place;
    cout << "Enter name place and koordinate x,y " << endl;
    cout << "Enter Name: "; getline(cin, tmp->Name);
    cout << "Enter x: ";    cin >> tmp->x;
    cout << "Enter y: ";    cin >> tmp->y;
    cout << "\n";
    a.push_back(*tmp);
    delete tmp;
}
 
void Display(vector<Place> &a)
{
    if (a.empty())
    {
        cout << "Not found file!!!" << endl;
        return;
    }
    for (vector<Place>::iterator itr = a.begin(); itr != a.end(); itr++)  // <--
    {
        cout << "Place: " << (*itr).Name << endl
            << "x: " << (*itr).x << endl
            << "y: " << (*itr).y << endl << std::endl;
    }
}
 
void Delete(vector<Place> &a, string name)
{
    if (a.empty())
    {
        cout << "Not Found!!!" << endl;
        return;
    }
 
    for (vector<Place>::iterator itr = a.begin(); itr != a.end();) // <--
    {
        if (badValue(*itr, name)) a.erase(itr++);
        else ++itr;
    }
}
 
void Write(fstream &file, vector<Place> &a)
{
    if (!file || a.empty())
        return;
 
    for (vector<Place>::iterator itr = a.begin(); itr != a.end(); itr++)
    {
        file << std::endl << (*itr).Name << " " // <--
            << (*itr).x << " "
            << (*itr).y;
    }
}
 
void Read(fstream &file, vector<Place> &a)
{
    if (!file)
        return;
 
    Place *tmp = new Place();
 
    while (!file.eof())
    {
        file >> tmp->Name;
        file >> tmp->x;
        file >> tmp->y;
        a.push_back(*tmp);
    }
 
    delete tmp;
}
 
int main()
{
 
    vector<Place> thePlace;
    string tmpName;
    char key = 'a';
 
    do {
        std::cout //<< " 1. Calculate\n"
            << " 2. Add\n"
            << " 3. Delete\n"
            << " 4. Show All\n"
            << " 0. Exit\n"
            << "\n Enter: ";
        std::cin >> key;
        std::cin.ignore();
        switch (key)
        {
            /*case '1':
            {
            Read(my_file, thePlace);
            calculateTheDistance(my_file, thePlace);
            break;
            }*/
        case '2':
        {
            fstream my_file1(filename, ios_base::app);
            Add(thePlace);
            Write(my_file1, thePlace);
            my_file1.close();
            break;
        }
        case '3':
        {
            Delete(thePlace, tmpName);
            fstream my_file1(filename, ios_base::out);
            Write(my_file1, thePlace);
            my_file1.close();
            break;
        }
        case '4':
        {
            fstream my_file(filename, ios_base::in);  // <--
            Read(my_file, thePlace);
            Display(thePlace);
            my_file.close();
            system("pause");
            break;
        }
        default:
            std::cout << "[Error] Function not found!" << std::endl << std::endl;
        }
        system("cls"); // <--
        if(!thePlace.empty()) thePlace.clear(); // <--
    } while (key != '0');
    
    return 0;
}

Новые блоги и статьи Все статьи Все блоги /
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Нашел на реддите интересную статью под названием «Кто-нибудь знает, где получить бесплатный компьютер или. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .