Форум программистов, компьютерный форум CyberForum.ru

Функция удаления из базы данных (Найти ошибки) - C++

Восстановить пароль Регистрация
 
Delpix
delpix.ru
 Аватар для Delpix
19 / 19 / 1
Регистрация: 06.05.2011
Сообщений: 235
03.06.2011, 15:27     Функция удаления из базы данных (Найти ошибки) #1
У меня есть функция удаления строки из базы данных
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
void Delete(int n)
{
    int d,i;
    Abbase *db;
    system ("cls");
    db=new Abbase[n];
    Abbase *temp; 
    temp=new Abbase[n]; 
    d=-1;
    while(d<0||d>n)
    {
        cout<<"Введите № строки которую хотите удалить: "; cin>>d;
        d--;
        if(d<0||d>n) cout<<"Ошибка!!! Такой строки не существует..."<<endl;
    }
    ifstream fin("pacient.txt", ios::in);
    for(i=0;i<n;i++)
    {
        fin>>db[i].number>>db[i].family>>db[i].f_name>>db[i].l_name>>db[i].gr>>db[i].zno;
    }
    fin.close();
    for(i=0;i<d;i++) temp[i]=db[i];
    for(i=d+1;i<n;i++) temp[i-1]=db[i];
    delete []db;
    n--;
    ofstream fout("pacient.txt", ios::out);
    fout.clear();
    for(i=0;i<n;i++)
    {
        fout<<endl<<" "<<temp[i].number<<" "<<temp[i].family<<" "<<temp[i].f_name<<" "<<temp[i].l_name<<" "<<temp[i].gr<<" "<<temp[i].zno;
            }
    fout.close();
}
Функция удаляет, но после нескольких удалений подряд, в последней строки выводится бред
Скрин во вложенном файле.
Как это исправить?
Миниатюры
Функция удаления из базы данных (Найти ошибки)  
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Delpix
delpix.ru
 Аватар для Delpix
19 / 19 / 1
Регистрация: 06.05.2011
Сообщений: 235
03.06.2011, 17:34  [ТС]     Функция удаления из базы данных (Найти ошибки) #2
Помогите
Evg
Эксперт С++Автор FAQ
 Аватар для Evg
16827 / 5248 / 321
Регистрация: 30.03.2009
Сообщений: 14,132
Записей в блоге: 26
03.06.2011, 23:41     Функция удаления из базы данных (Найти ошибки) #3
По такому огрызку кода вряд ли кто тебе поможет, потому что ни понять логики, ни воспроизвести невозможно
bigredcat
364 / 311 / 3
Регистрация: 24.02.2011
Сообщений: 1,512
Записей в блоге: 1
04.06.2011, 06:40     Функция удаления из базы данных (Найти ошибки) #4
Цитата Сообщение от Delpix Посмотреть сообщение
Функция удаляет, но после нескольких удалений подряд, в последней строки выводится бред
Это вывод на консоль, а что в файле?

В вашей функции Delete утечка памяти:
C++
1
2
temp=new Abbase[n]; // это есть
delete[] temp; // а вот этого не вижу
Теперь думаем, что будет при больших объемах данных и многочисленных вызовах.
Delpix
delpix.ru
 Аватар для Delpix
19 / 19 / 1
Регистрация: 06.05.2011
Сообщений: 235
04.06.2011, 09:30  [ТС]     Функция удаления из базы данных (Найти ошибки) #5
Цитата Сообщение от bigredcat Посмотреть сообщение
Это вывод на консоль, а что в файле?

В вашей функции Delete утечка памяти:
C++
1
2
temp=new Abbase[n]; // это есть
delete[] temp; // а вот этого не вижу
Теперь думаем, что будет при больших объемах данных и многочисленных вызовах.
Вот я поставил (хоть и не уверен что туда):
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
void Delet(int n)
{
    int d,i;
    Abbase *db;
    system ("cls");
    db=new Abbase[n];
    Abbase *temp; 
    temp=new Abbase[n]; 
    d=-1;
    while(d<0||d>n)
    {
        cout<<"Введите № строки которую хотите удалить: "; cin>>d;
        d--;
        if(d<0||d>n) cout<<"Ошибка!!! Такой строки не существует."<<endl;
    }
    ifstream fin("pacient.txt", ios::in);
    for(i=0;i<n;i++)
    {
        fin>>db[i].number>>db[i].family>>db[i].f_name>>db[i].l_name>>db[i].gr>>db[i].zno>>db[i].phone;
    }
    fin.close();
    for(i=0;i<d;i++) temp[i]=db[i];
    for(i=d+1;i<n;i++) temp[i-1]=db[i];
    delete db;
    n--;
    ofstream fout("pacient.txt", ios::out);
    fout.clear();
    
    for(i=0;i<n;i++)
    {
        fout<<endl<<" "<<temp[i].number<<" "<<temp[i].family<<" "<<temp[i].f_name<<" "<<temp[i].l_name<<" "<<temp[i].gr<<" "<<temp[i].zno<<" "<<temp[i].phone;
    }
    delete[] temp;
    fout.close();
}
Но проблемы это не решило, после второго удаления выводит бред

Добавлено через 33 минуты
Вот весь мой код, первый содержит структуру и функции, второй главный, там меню
Всю программу портит delet, после удаления и сортировка отказывается работатьэ


biblioteka.h
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
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
#include <iostream>
#include <fstream>
#include <conio.h>
#include <stdio.h>
#include <tchar.h>
using namespace std;
struct Abbase
{
    char family[20];//фамилия 
    char f_name[20];//имя
    char l_name[20];//отчество
    int number;//таб. номер
    int gr;//год рождения
    double zno;//Общий бал ВНО
    int phone;// номер телефона
};
char r='+'; 
void Newadd();             
void Open();        
void Search();          
void Delet(int n);  
void Navigation();  
void Sorting(int n);
 
void Newadd()
{
    Abbase z2;
    system ("cls");
    fstream fout("pacient.txt", ios::app);
    cout <<"Табельный номер:\n>> ";
    cin >>z2.number;
    cout <<"Фамилия:\n>> ";
    cin >>z2.family;
    cout <<"Имя:\n>> ";
    cin >>z2.f_name;
    cout <<"Отчество:\n>> ";
    cin >>z2.l_name;
    cout <<"Год рождения:\n>> ";
    cin >>z2.gr;
    cout <<"Введите общий бал ВНТ:\n>> ";
    cin >>z2.zno;
    cout<<"Введите номер телефона:\n>> ";
    cin >>z2.phone;
    cout <<"__________________________________"<<endl;
    cout <<"Студент добавлен в базу данных."<<endl;
    cout <<"__________________________________"<<endl;
    fout<<endl<<z2.number<<" "<<z2.family<<" "<<z2.f_name<<" "<<z2.l_name<<" "<<z2.gr<<" "<<z2.zno<<" "<<"0"<<z2.phone;
    fout.close();
    Navigation();           
}
void Open()
{
Abbase z3;
system ("cls");
ifstream fin("pacient.txt", ios::in);
if (!fin)
        cout <<"База пуста!\n";
    else
    {
        int n=1;
        cout <<"|#|Номер|Фамилия|Имя|Отчество|Год рождения|Бал ВНО|Номер телефона|\n";
        while (!fin.eof())
        {
            
            fin>>z3.number>>z3.family>>z3.f_name>>z3.l_name>>z3.gr>>z3.zno>>z3.phone;
            cout<<n<<" "<<z3.number<<" "<<z3.family<<" "<<z3.f_name<<" "<<z3.l_name<<" "<<z3.gr<<" "<<z3.zno<<" "<<"0"<<z3.phone<<endl;
            n++;
        }
        //fin.close();
    }
    Navigation();
}
void Search()
{
    system ("cls");
    bool fl=0;
    int q,i=0;
    Abbase z4;
    ifstream fin("pacient.txt", ios::in);
    cout<<"Введите табельный  номер абитуриента: ";
    cin>>q;
    cout <<"\nНайдены совпадения: \n";
    cout <<"\n|#|Номер|Фамилия|Имя|Отчество|Год рождения|Бал ВНО|Номер телефона|\n";
    while(!fin.eof())
    {
        fin>>z4.number>>z4.family>>z4.f_name>>z4.l_name>>z4.gr>>z4.zno>>z4.phone;
        if(z4.number==q) 
        {
            cout<<"\n";
            cout<<"|"<<z4.number;
            cout<<"|"<<z4.family;
            cout<<'|'<<z4.f_name;
            cout<<'|'<<z4.l_name;
            cout<<"|"<<z4.gr;
            cout<<"|"<<z4.zno;
            cout<<"0"<<z4.phone<<"|"<<endl;
            cout<<endl; 
            fl=1;
        }
        i++;
    }
      if(!fl) 
        {
        cout<<"Студент с таким номером не найден"<<endl;
        Navigation();
        return;
        }
        fin.close();
        Navigation();
}
void Delet(int n)
{
    int d,i;
    Abbase *db;
    system ("cls");
    db=new Abbase[n];
    Abbase *temp; 
    temp=new Abbase[n]; 
    d=-1;
    while(d<0||d>n)
    {
        cout<<"Введите № строки которую хотите удалить: "; cin>>d;
        d--;
        if(d<0||d>n) cout<<"Ошибка!!! Такой строки не существует."<<endl;
    }
    ifstream fin("pacient.txt", ios::in);
    for(i=0;i<n;i++)
    {
        fin>>db[i].number>>db[i].family>>db[i].f_name>>db[i].l_name>>db[i].gr>>db[i].zno>>db[i].phone;
    }
    fin.close();
    for(i=0;i<d;i++) temp[i]=db[i];
    for(i=d+1;i<n;i++) temp[i-1]=db[i];
    delete db;
    n--;
    ofstream fout("pacient.txt", ios::out);
    fout.clear();
    
    for(i=0;i<n;i++)
    {
        fout<<endl<<" "<<temp[i].number<<" "<<temp[i].family<<" "<<temp[i].f_name<<" "<<temp[i].l_name<<" "<<temp[i].gr<<" "<<temp[i].zno<<" "<<temp[i].phone;
    }
    delete[] temp;
    fout.close();
}
void Sorting(int n)
{
    int i;
    bool fl=1;
    Abbase *z6;
    z6=new Abbase[n];
    Abbase *temp; 
    temp=new Abbase[n];
    ifstream fin("pacient.txt", ios::in);
    for(i=0;i<n;i++)
    {
        fin>>z6[i].number>>z6[i].family>>z6[i].f_name>>z6[i].l_name>>z6[i].gr>>z6[i].zno>>z6[i].phone;
    }
    fin.close();
 
    while(fl)
    {
        fl=0;
        for (i=0;i<n-1;i++)
        {
            if (z6[i].gr>z6[i+1].gr)
            {
            temp[i]=z6[i];
            z6[i]=z6[i+1];
            z6[i+1]=temp[i];
            fl=1;
            }
 
        }
 
    }
 
    delete []temp;
    ofstream fout("pacient.txt", ios::out);
    fout.clear();
    for(i=0;i<n;i++)
    {
        fout<<endl<<z6[i].number<<" "<<z6[i].family<<" "<<z6[i].f_name<<" "<<z6[i].l_name<<" "<<z6[i].gr<<" "<<z6[i].zno<<" "<<z6[i].phone;
    }
    fout.close();
    Open();
}
void Navigation()
{
        cout <<"\n";
        cout <<"___________Навигация___________";
        cout <<"\n1:Вернуться в главное меню\n";
        cout <<"0: Выход\n\n";
        cout <<"Введите номер выбора:\n>> ";
        int ch;
        ch=-1;
        while (ch==-1)
        {
        cin >>ch;
        switch(ch)
        {
        case 1:
            break;
        case 0:
            {
                exit(1);
                break;
            }
        default:
            {
                ch=-1;
                cout <<"Неверный выбор, повторите еще раз:\n>> ";
            }
        }
    }
        
    }
main.cpp
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
#include <iostream>
#include <iomanip> 
#include <fstream>
#include "biblioteka.h"
using namespace std;
void main()
{
    int n=0;
    int m=0;
    Abbase z1;
    setlocale(LC_ALL,"Russian");
    ifstream fin("pacient.txt", ios::in);
    while(!fin.eof())
    {
        fin>>z1.number>>z1.family>>z1.f_name>>z1.l_name>>z1.gr>>z1.zno>>z1.phone;
        n++;
    }
        while(m!=6)
        {
        system ("cls");
        cout <<"Abbase 0.47 beta 5\n"<<endl;
        cout <<"МЕНЮ:\n"<<endl;
        cout<<"1) Просмотр "<<endl;
        cout<<"2) Добавить "<<endl;
        cout<<"3) Поиск    "<<endl;
        cout<<"4) Удалить  "<<endl;
        cout<<"5) Cортировать по году рождения"<<endl;
        cout<<"0) Выход    "<<endl;
        //while(!fin.eof())
        cout<<"-> ";
        cin>>m; cin.get();
        switch (m){
            case 1: {
                Open(); 
                m=1;
                break;
                    }           
            case 2: {
                Newadd();
                n++;
                system("pause");
                break;
                    }
            case 3: {
                Search(); 
                m=1;
                break;
                    }
            case 4: {
                Delet(n); 
                system("pause");
                break;
                    }
            case 5: {
                system ("cls");
                Sorting(n);
                m=1;
                break;
                    }
            case 0:
                {
                exit(1);
                break;
                }
        default: 
            {
            cout<<"Ошибка! Повторите ввод : "<<endl;
            break;
            }
        }
    }
}
Evg
Эксперт С++Автор FAQ
 Аватар для Evg
16827 / 5248 / 321
Регистрация: 30.03.2009
Сообщений: 14,132
Записей в блоге: 26
04.06.2011, 09:35     Функция удаления из базы данных (Найти ошибки) #6
Цитата Сообщение от Delpix Посмотреть сообщение
Но проблемы это не решило
Оно и не должно было решить. Кстати, теперь при "delete db" ты потерял скобки.

Цитата Сообщение от Delpix Посмотреть сообщение
после второго удаления выводит бред
После того, как из main вызываешь Delete, надо сделать "n--", потому как у тебя записей стало на одну меньше. Последняя строка на скриншоте говорит в пользу того, что у тебя именно этот косяк (т.е. чтение одной лишней записи, в то время как их на одну меньше). Если не поможет, то покажи исходный файл базы данных и файл, полученный после удаления одной из записей
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
04.06.2011, 10:34     Функция удаления из базы данных (Найти ошибки)
Еще ссылки по теме:

Не выполняется функция удаления C++
Функция удаления текста в скобках [2], непосредственно функция + 12кб вложений C++
C++ Программа формирования данных об автомобилях. Найти ошибки

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

Или воспользуйтесь поиском по форуму:
bigredcat
364 / 311 / 3
Регистрация: 24.02.2011
Сообщений: 1,512
Записей в блоге: 1
04.06.2011, 10:34     Функция удаления из базы данных (Найти ошибки) #7
Цитата Сообщение от Delpix Посмотреть сообщение
Вот я поставил (хоть и не уверен что туда)
Правильно поставил.
Цитата Сообщение от Delpix Посмотреть сообщение
Но проблемы это не решило, после второго удаления выводит бред
Это решало другую потенциальную проблему. Конечно, программа учебная, но нужно учиться и за памятью следить.

Ну, а описанная вами проблема, уверен решена выше.
Yandex
Объявления
04.06.2011, 10:34     Функция удаления из базы данных (Найти ошибки)
Ответ Создать тему
Опции темы

Текущее время: 06:16. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru