Нужна помощь в исправлении кода одной функции...

@kandrey58region · Регистрация: 11.03.2009

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

Плиз.. помогите исправить код функции 'void Opt()', мне необходимо было создать программу которая читатет файл 'd:\file.txt' , создает и копирует всё его содержимое в типизированный файл 'd:\out.dat'
Данная функция по теории должна работать так, вводишь число равное 500 если оптовая цена какого-либо товара равна 500 то нужно вывести на экран всю строку с этим товаром (название, оптовую и розничную цены)
Программа содержит меню:
1 Запись в файл
2 Вывод на экран
3 Очистка файла //очистка файла 'out.dat'
4 Поиск...
5 Выход

Меню Поиск ...
как раз и содержит данную функцию 'void Opt()'

Code
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
#include <iostream.h>
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <conio.h>
 
struct mon {
  char type[20];
  int opt, rozn;
  char comm[40];
};
int kol=0;
void WriteFile()
{
    FILE *fi,*fo;
     clrscr();
     if ((fi=fopen("D:\\file.txt", "r"))==0){
      cout << "Error open input file!!!" ;
      return;}
     if ((fo=fopen("D:\\out.dat","w+b"))==0){
      cout << "Error open output file!!!";
      return;}
     const int dl=80;
     char s[dl];
    mon a;
    while(fgets(s,dl,fi)){
    strncpy(a.type,s,19);
    a.type[19]='\0';
    a.opt=atoi(&s[20]);
    a.rozn=atoi(&s[25]);
    strncpy(a.comm,&s[30],40);
    fwrite(&a,sizeof(mon),1,fo);
    kol++;
    };
    fclose(fi);
    fclose(fo);
 
    return;
}
 
void out()
{
    FILE *fo;
    clrscr();
    if ((fo=fopen("D:\\out.dat","rb"))==0){
      cout << "Error open output file!!!";
      return ;}
    int i;
    cout<<"Input number of Zapisi ";
    cin >>i;//после ввода i не забываем нажать Enter
    if (i>kol) {
      cout << "Zapisi net!!!";
      return;}
 
    mon a;
    fseek(fo,sizeof(mon)*(i-1),SEEK_SET);
    fread(&a,sizeof(mon),1,fo);
    cout << "Tip: " << a.type << " opt.: " << a.opt << " rozn: " << a.rozn << endl;
    fclose(fo);
}
 
void ClearFile()
{
    FILE *fo;
    clrscr();
    if ((fo=fopen("D:\\out.dat","wb"))==0){
    cout << "Error open output file!!!"<<endl;
      return ;}
    fclose(fo);
}
 
[COLOR="Red"]void Opt()
{
  mon a;
  FILE *fo;
 
  if ((fo=fopen("D:\\out.dat","r"))==0){
    cout << "Error open output file!!!";
    return;}
  fseek(fo,0,SEEK_SET);
   int res;
   int p;
   do
   {
   res=fread(&a,sizeof(mon),1,fo);
    if (a.opt==p)
    {
      cout<<a.type<<" "<<a.opt<<" "<<a.rozn<<" "<<a.comm<<" "<<endl;
    }
   }
   while(res==1);
  fclose(fo);
   return;
}[/COLOR]
void Search()
{
 char menu_2;
 clrscr();
 int p;
 do
 {
   cout <<endl<< "1 Nazvanie" << endl <<"2 Optovaya" << endl << "3 Roznica" << endl << "4 Vozvrat v gl. menu"<< endl;
   menu_2=getch();
   switch(menu_2)
   {
    // case '1':
      // Nazv();
     //  break;
     case '2':
       cout << "Vvedite optovuy cenu: " << endl;
       cin >> p;
       Opt();
       break;
    // case '3':
      // Rozn();
     //  break;
      };
   if (menu_2=='4') break;
 }
 while(1);
clrscr();
}
 
void main()
{
 char menu;
 int i;
 kol=0;
 clrscr();
 do
 {
   cout <<endl<< "1 Zapis' v file" << endl <<"2 Vivod na ekran" << endl << "3 Ochistka file" << endl << "4  Search..."<< endl << "5 Vichod" << endl;
   menu=getch();
   switch(menu)
   {
     case '1':
       WriteFile();
       break;
     case '2':
       out();
       break;
     case '3':
       ClearFile();
       break;
     case '4':
       Search();
       break;
         };
   if (menu=='5') break;
 }
 while(1);
}

текстовый файл выглядит так

Code
1
2
3
4
Huindai             350  400  plohoi                                  l
Samsung             500  550  norma                                   l
Sony                300  400  rtye                                    l
Acer                200  350  hgfak                                   l

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

@Gravity · 12.04.2009, 20:58

Code
1
2
3
4
5
6
7
8
9
10
11
   int res;
   int p;
   do
   {
   res=fread(&a,sizeof(mon),1,fo);
    if (a.opt==p)
    {
      cout<<a.type<<" "<<a.opt<<" "<<a.rozn<<" "<<a.comm<<" "<<endl;
    }
   }
   while(res==1);

Переменная p не определена, поэтому неизвестно с чем сравнивается a.opt.

@Humanitis · 12.04.2009, 21:00

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

int p;
do {
res=fread(&a,sizeof(mon),1,fo);
if (a.opt==p)
{ cout<<a.type<<" "<<a.opt<<" "<<a.rozn<<" "<<a.comm<<" "<<endl;
}
}

а p то где-то надо инициализировать

Evg · 12.04.2009, 21:04

Переменная res так же не определена, а потому цикл наверняка окажется бесконечный, а файл при этом ограниченного размера, после чего начинается работа с залипшими данными от предыдущих итераций цикла

Добавлено через 1 минуту 20 секунд
Пардон, про res не заметил. Снимаю свою претензию

@kandrey58region · 12.04.2009, 21:33 **[ТС]**

просто у меня p раньше иницилизировалась в меню_2
но даже если иницилизировать её в функции 'void Opt()' то никакого результата не получается

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
   int res;
   int p;
   cout << "Vvedite optovuy cenu: " << endl;
   cin >> p;
   do
   {
   res=fread(&a,sizeof(mon),1,fo);
    if (a.opt==p)
    {
      cout<<a.type<<" "<<a.opt<<" "<<a.rozn<<" "<<a.comm<<" "<<endl;
    }
   }
   while(res==1);

помогите пожалуйста довести эту функцию до ума

Evg · 12.04.2009, 21:45

Закомментируй строку 'if (a.opt==p)' и смотри, что печатает. Увидишь, что у тебя из файла реально программа читает и глазами хотябы оцени правильно это или нет

@Gravity · 12.04.2009, 22:08

Подозреваю, что проблема как раз во fread. Как вариант,

C
1
2
3
4
5
6
char line[80];
while(fgets(line, sizeof(line), fo) != NULL) {
   sscanf(line, "%s %d %d %s", a.type, &a.opt, &a.rozn, a.comm);
   if(a.opt == p)
      printf("%s", line);
}

Evg · 12.04.2009, 22:38

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

Подозреваю, что проблема как раз во fread. Как вариант,

C
1
2
3
4
5
6
char line[80];
while(fgets(line, sizeof(line), fo) != NULL) {
   sscanf(line, "%s %d %d %s", a.type, &a.opt, &a.rozn, a.comm);
   if(a.opt == p)
      printf("%s", line);
}

Я исходники особо не смотрел, но этот файл он формирует в процедуре WriteFile через fwrite. Т.е. с этой точки зрения вроде бы как всё правильно (чтение и запись симметричны)

accept · 12.04.2009, 23:18

внутри функции int p; убрать, перенести в void Opt()

Code
1
void Opt(int p)

функцию вызывать через Opt(p)

Code
1
2
3
4
5
       case '2':
       cout << "Vvedite optovuy cenu: " << endl;
       cin >> p;
       Opt();
       break;

Code
1
2
3
4
5
6
7
8
9
    do
   {
   res=fread(&a,sizeof(mon),1,fo);
    if (a.opt==p)
    {
      cout<<a.type<<" "<<a.opt<<" "<<a.rozn<<" "<<a.comm<<" "<<endl;
    }
   }
   while(res==1);

это можно заменить

C++
1
2
3
4
5
6
7
8
9
10
11
    while ((res = fread(&a, sizeof(mon), 1, fo)) == 1);
        if (a.opt == p)
            cout << a.type
                 << " "
                 << a.opt
                 << " "
                 << a.rozn
                 << " "
                 << a.comm
                 << " "
                 << endl;

Добавлено через 4 минуты 42 секунды
fo - file out
обычно пишут когда выводишь, а когда вводишь пишут
fi - file in
когда открываешь файл на чтение, указатель на него помещай в fi, а когда открываешь файл на запись - в fo

@kandrey58region · 12.04.2009, 23:23 **[ТС]**

всем спасибо огромное за помощь и объяснение...
теперь появились новые проблема, почему то при использовании данной функции у меня из такого текстового файла

Code
1
2
3
4
Huindai             350  400  plohoi                                  l
Samsung             500  550  norma                                   l
Sony                300  400  rtye                                    l
Acer                200  350  hgfak                                   l

1) в конце каждой записи стоит смайлик (улыбающаяся рожица)...
2) последняя запись с монитором Acer выводится два раза, остальные записи выводятся по одному разу как положено...
как думаете с чем могут быть связаны данные проблемы

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
#include <iostream.h>
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <conio.h>
 
struct mon {
  char type[20];
  int opt, rozn;
  char comm[40];
};
int kol=0;
int p=0;
int v=0;
void WriteFile()
{
    FILE *fi,*fo;
     clrscr();
     if ((fi=fopen("D:\\file.txt", "r"))==0){
      cout << "Error open input file!!!" ;
      return;}
     if ((fo=fopen("D:\\out.dat","w+b"))==0){
      cout << "Error open output file!!!";
      return;}
     const int dl=80;
     char s[dl];
    mon a;
    while(fgets(s,dl,fi)){
    strncpy(a.type,s,19);
    a.type[19]='\0';
    a.opt=atoi(&s[20]);
    a.rozn=atoi(&s[25]);
    strncpy(a.comm,&s[30],40);
    fwrite(&a,sizeof(mon),1,fo);
    kol++;
    };
    fclose(fi);
    fclose(fo);
 
    return;
}
 
void out()
{
    FILE *fo;
    clrscr();
    if ((fo=fopen("D:\\out.dat","rb"))==0){
      cout << "Error open output file!!!";
      return ;}
    int i;
    cout<<"Input number of Zapisi ";
    cin >>i;//после ввода i не забываем нажать Enter
    if (i>kol) {
      cout << "Zapisi net!!!";
      return;}
 
    mon a;
    fseek(fo,sizeof(mon)*(i-1),SEEK_SET);
    fread(&a,sizeof(mon),1,fo);
    cout << "Tip: " << a.type << " opt.: " << a.opt << " rozn: " << a.rozn << endl;
    fclose(fo);
}
 
void ClearFile()
{
    FILE *fo;
    clrscr();
    if ((fo=fopen("D:\\out.dat","wb"))==0){
    cout << "Error open output file!!!"<<endl;
      return ;}
    fclose(fo);
}
 
void Opt()
{
  mon a;
  FILE *fo;
 
  if ((fo=fopen("D:\\out.dat","r"))==0){
    cout << "Error open output file!!!";
    return;}
  fseek(fo,0,SEEK_SET);
   int res;
  cout << "Vvedite optovuy cenu: " << endl;
  cin >> p;
   do
   {
   res=fread(&a,sizeof(mon),1,fo);
    if (a.opt==p)
    {
      cout<<a.type<<" "<<a.opt<<" "<<a.rozn<<" "<<a.comm<<" "<<endl;
    }
   }
   while(res==1);
  fclose(fo);
   return;
}
void Rozn()
{
  mon a;
  FILE *fo;
  if ((fo=fopen("D:\\out.dat","r"))==0){
    cout << "Error open output file!!!";
    return;}
  fseek(fo,0,SEEK_SET);
   int res1;
  cout << "Vvedite roznichnuy cenu: " << endl;
  cin >> v;
   do
   {
   res1=fread(&a,sizeof(mon),1,fo);
    if (a.rozn==v)
    {
      cout<<a.type<<" "<<a.opt<<" "<<a.rozn<<" "<<a.comm<<" "<<endl;
    }
   }
   while(res1==1);
  fclose(fo);
   return;
}
void Search()
{
 char menu_2;
 do
 {
   cout <<endl<< "1 Optovaya" << endl << "2 Roznica" << endl << "3 Vozvrat v gl. menu"<< endl;
   menu_2=getch();
   switch(menu_2)
   {
     case '1':
       Opt();
       break;
     case '2':
       Rozn();
       break;
      };
   if (menu_2=='3') break;
 }
 while(1);
clrscr();
}
 
void main()
{
 char menu;
 int i;
 kol=0;
 clrscr();
 do
 {
   cout <<endl<< "1 Zapis' v file" << endl <<"2 Vivod na ekran" << endl << "3 Ochistka file" << endl << "4 Search..."<< endl << "5 Vichod" << endl;
   menu=getch();
   switch(menu)
   {
     case '1':
       WriteFile();
       break;
     case '2':
       out();
       break;
     case '3':
       ClearFile();
       break;
     case '4':
       Search();
       break;
         };
   if (menu=='5') break;
 }
 while(1);
}

accept · 13.04.2009, 00:13

Code
1
2
    cout << "Vvedite optovuy cenu: " << endl;
    cin >> p;

надо p создать, если вводишь внутри, то создавай внутри int p;
когда у тебя формат ввода вынесен в отдельную функцию, её можно менять, это гораздо проще, чем лазить по всей функции, где всего лишь используется результат этого ввода, это выглядело бы как

Code
1
    Opt(EnterOptFloor());

или

Code
1
2
3
4
    int floor;    
    
    if ((floor = EnterOptFloor()) > 0)
        Opt(floor);

сразу тогда можно встраивать проверку введённых данных в ту функцию ввода
а так, тебе приходится вводом заниматься в той функции, которая к нему вообще отношения не имеет

@kandrey58region · 13.04.2009, 00:32 **[ТС]**

объясни пожалуйста поподробней как это решает мою проблему с этими смайликами и последней записью, которые выводится два раза... а то если честно я даже не понял куда нужно добавить то что ты написал...

accept · 13.04.2009, 01:00

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
void Opt(void)
{
    FILE *fi;
    int res, p;
    mon a;
  
    if ((fi = fopen("D:\\out.dat", "rb")) == 0) {
        cout << "Error open output file!!!";
        return;
    }
    fseek(fi, 0, SEEK_SET);
    cout << "Vvedite optovuy cenu: " << endl;
    cin >> p;
    while ((res = fread(&a, sizeof(mon), 1, fi)) == 1);
        if (a.opt == p)
            cout << a.type
                 << " "
                 << a.opt
                 << " "
                 << a.rozn
                 << " "
                 << a.comm
                 << " "
                 << endl;
    fclose(fi);
}

юзай такую

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

1) в конце каждой записи стоит смайлик (улыбающаяся рожица)...

в конце каждой записи где ?

Добавлено через 2 минуты 42 секунды
ещё двоичный режим для файла b

@kandrey58region · 13.04.2009, 01:33 **[ТС]**

прога то у меня работает, самое обидное ведь она и раньше работало, просто очистка экрана 'clrscr' стирала мой результат и выводила опять меню

Code
1
2
3
4
Huindai             350  400  plohoi                                  l
Samsung             500  550  norma                                   l
Sony                300  400  rtye                                    l
Acer                200  350  hgfak                                   l

т.е. при запросе оптовая цена ==500 на экран выводится

Code
1
Samsung             500  550  norma                                   :)

только смайлик черно-белый, палочка на конце каждой строки не влияет ни на что пробовал её убрать все равно тоже самое получается...

accept · 13.04.2009, 04:10

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
void WriteFile()
{
    FILE *fi,*fo;
 
    clrscr();        
        
        ...
 
        while (fgets(s, dl, fi)) {
        strncpy(a.type, s, 19);
            a.type[19] = '\0';
        a.opt = atoi(&s[20]);
            a.rozn = atoi(&s[25]);
            strncpy(a.comm, &s[30], 40);
        fwrite(&a, sizeof(mon), 1, fo);
            kol++;
    };

докуда strncpy(a.comm, &s[30], 40); будет читать ?

Добавлено через 9 минут 46 секунд

C++
1
2
        strncpy(a.comm, &s[30], 39);
        a.comm[39] = '\0';

@kandrey58region · 13.04.2009, 09:49 **[ТС]**

accept, спасибо проблема со смайликами решилась...
теперь осталась одна проблема с последней записью которая выводиться на экран дважды...

Evg · 13.04.2009, 12:53

C++
1
2
3
4
5
6
7
8
9
   do
   {
   res=fread(&a,sizeof(mon),1,fo);
    if (a.opt==p)
    {
      cout<<a.type<<" "<<a.opt<<" "<<a.rozn<<" "<<a.comm<<" "<<endl;
    }
   }
   while(res==1);

Проблема всё там же. Когда файл у тебя закончился, то fread вернул 0. Но вне зависимости от этого ты всё равно делаешь проверку a.opt с дальнейшей печатью. В тот момент, когда fread вернул 0, он из файла ничего не прочилал, а в переменной 'a' отсались старые значения с предыдущей итерации цикла. В посте #9 тебе уже написали готовое решение, как побороть эту проблему

accept · 13.04.2009, 23:51

https://www.cyberforum.ru/post136670.html
цикл и правильное название для файлового указателя

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение Это мой обзор планшета X220 с точки зрения школьника. Недавно я решила попытаться уменьшить свой. . .	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 - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .

Evg 21281 / 8305 / 637 Регистрация: 30.03.2009 Сообщений: 22,660 Записей в блоге: 30
	12.04.2009, 21:04
	Переменная res так же не определена, а потому цикл наверняка окажется бесконечный, а файл при этом ограниченного размера, после чего начинается работа с залипшими данными от предыдущих итераций цикла Добавлено через 1 минуту 20 секунд Пардон, про res не заметил. Снимаю свою претензию 1

Evg 21281 / 8305 / 637 Регистрация: 30.03.2009 Сообщений: 22,660 Записей в блоге: 30
	12.04.2009, 21:45
	Закомментируй строку 'if (a.opt==p)' и смотри, что печатает. Увидишь, что у тебя из файла реально программа читает и глазами хотябы оцени правильно это или нет 1

@kandrey58region 1 / 1 / 0 Регистрация: 11.03.2009 Сообщений: 107
	13.04.2009, 00:32 [ТС]
	объясни пожалуйста поподробней как это решает мою проблему с этими смайликами и последней записью, которые выводится два раза... а то если честно я даже не понял куда нужно добавить то что ты написал... 0

@kandrey58region 1 / 1 / 0 Регистрация: 11.03.2009 Сообщений: 107
	13.04.2009, 09:49 [ТС]
	accept, спасибо проблема со смайликами решилась... теперь осталась одна проблема с последней записью которая выводиться на экран дважды... 0

accept 4866 / 3288 / 468 Регистрация: 10.12.2008 Сообщений: 10,570
	13.04.2009, 23:51
	https://www.cyberforum.ru/post136670.html цикл и правильное название для файлового указателя 0