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

Ошибка переполнения буфера - C++

Восстановить пароль Регистрация
 
ozzy_b
2 / 2 / 0
Регистрация: 02.10.2012
Сообщений: 169
05.12.2012, 13:15     Ошибка переполнения буфера #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
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
#include <iostream>
#include <stdio.h>
#include <math.h>
#include <conio.h>
using namespace std;
const int m=1;
struct good
{
    char name[30];
    char type_count[30];
    char sort[30];
    int count;
    int price;
};
void inp(good []);
void write(good []);
void outdata(good []);
void redag(good []);
 
int main()
{
    good goods[m];
    int key;
    cout<<"Enter command:\n1)Write information in file('1')\n2)Viev information('2')\n3)Edit information('3')\n";
    cin>>key;
    switch (key)
    {
        case (1): inp(goods); break;
        case (2): outdata(goods); break;
        case (3): redag(goods); break;
    }
    getch();
}
void inp(good goods[m])
{
     cout<<"\n************************\n\tInput\n************************"<<endl;
     
        for(int i=0; i<m; i++)
        {
            cout<<"Input name: "<<i+1<<" ";
            cin>>goods[i].name;
            cout<<"Input type count: "<<i+1<<" ";
            cin>>goods[i].type_count;
            cout<<"Input sort: "<<i+1<<" ";
            cin>>goods[i].sort;
            cout<<"Input count: "<<i+1<<" ";
            cin>>goods[i].count;
            cout<<"Input price: "<<i+1<<" ";
            cin>>goods[i].price;
        }
        cout<<endl;
        write(goods);
}
void write(good goods [m])
{
    FILE *f;
    f=fopen("D:\\Foods.txt", "ab");
    for(int i=0; i<m; i++)
    {
        fprintf (f, "%s\t", goods[i].name);
        fprintf (f, "%s\t", goods[i].type_count);
        fprintf (f, "%s\t", goods[i].sort);
        fprintf (f, "%d\t", goods[i].count);
        fprintf (f, "%d\t\t", goods[i].price);
    }
    fclose(f);
}
 
void outdata(good goods [10])
{
    
    FILE *f;
    f=fopen("D:\\Food.txt", "rb");
    int i=0;
    int count=0;
    while(!feof(f))
    {
        fscanf (f, "%s", &goods[i].name);
        fscanf (f, "%s", &goods[i].type_count);
        fscanf (f, "%s", &goods[i].sort);
        fscanf (f, "%d", &goods[i].count);
        fscanf (f, "%d", &goods[i].price);
        i++;
        count++;
    }
    fclose(f);
    for(int i=0; i<(count-1); i++)
    {
        cout<<"Name "<<i+1<<"\t"<<goods[i].name<<endl;
        cout<<"Type Count "<<i+1<<"\t"<<goods[i].type_count<<endl;
        cout<<"Sort "<<i+1<<"\t"<<goods[i].sort<<endl;
        cout<<"count "<<i+1<<"\t"<<goods[i].count<<endl;
        cout<<"price "<<i+1<<"\t"<<goods[i].price<<"\n"<<endl;
    }   
}
 
void redag(good goods [m])
{
    int k;
    outdata(goods);
    cout<<"Enter strint to edit: ";
    cin>>k;
}
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
vxg
Модератор
 Аватар для vxg
2663 / 1674 / 157
Регистрация: 13.01.2012
Сообщений: 6,251
05.12.2012, 13:37     Ошибка переполнения буфера #2
может данные плохие? в частности - поскольку вы читаете через %s строки должны быть без пробелов.
taras atavin
Ушёл с форума.
 Аватар для taras atavin
3569 / 1752 / 91
Регистрация: 24.11.2009
Сообщений: 27,619
05.12.2012, 13:57     Ошибка переполнения буфера #3
Цитата Сообщение от ozzy_b Посмотреть сообщение
void inp(good goods[m])
Функция должна быть универсальнее, а не привязывать размер массива к глобальной константе. Поэтому
C++
1
void inp(good *goods, size_t m)
и вызывать с двумя константами. Если только массив не ограничен по смыслу задачи. Например, если в массиве хранятся координаты вектора реального пространства, то у него ровно 3 элемента.
Цитата Сообщение от ozzy_b Посмотреть сообщение
cout<<"Input name: "<<i+1<<" ";
* * * * * * cin>>goods[i].name;
* * * * * * cout<<"Input type count: "<<i+1<<" ";
* * * * * * cin>>goods[i].type_count;
* * * * * * cout<<"Input sort: "<<i+1<<" ";
* * * * * * cin>>goods[i].sort;
* * * * * * cout<<"Input count: "<<i+1<<" ";
* * * * * * cin>>goods[i].count;
* * * * * * cout<<"Input price: "<<i+1<<" ";
* * * * * * cin>>goods[i].price;
Лучше занеси i+1 во временную переменную, это будет быстрей.
Цитата Сообщение от ozzy_b Посмотреть сообщение
char name[30];
,
Цитата Сообщение от ozzy_b Посмотреть сообщение
cin>>goods[i].name;
. А ты уверен, что 30-ти символов хватит на строку и терминальный ноль? Я бы не стал полагаться ни на то, что поток не даст ввести больше, ни на то, что пользователь не введёт больше.
Avazart
 Аватар для Avazart
6901 / 5141 / 252
Регистрация: 10.12.2010
Сообщений: 22,604
Записей в блоге: 17
05.12.2012, 14:06     Ошибка переполнения буфера #4
Почему бы не перегрузить операторы << и >> относительно потоков и работать одинаково что с потоком консоли что с потоком файла ?
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
05.12.2012, 14:11     Ошибка переполнения буфера #5
Цитата Сообщение от ozzy_b Посмотреть сообщение
В чем дело?
В функции outdata() название файла другое.
Yandex
Объявления
05.12.2012, 14:11     Ошибка переполнения буфера
Ответ Создать тему
Опции темы

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