Не выходит из цикла. Не переходит на след. узел.

@"Ламер"-программер:) · Регистрация: 17.04.2009

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

Доброго времени суток всем!
Как видно из названия проблема проста, но почему то не решаема для меня. Собственно проблема в присвоении.

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
        case 2://Вывести список всех записей
            {
                DListNode* ptr = NULL;
                if (first==NULL)
                {
                    cout<<"Cnucok nycT!\n";
                    break;
                }
                else
                {
                    ptr=first;
                    while (1)
                    {
                        cout<<ptr->denomination<<" ";
                        cout<<ptr->curse<<" ";
                        cout<<ptr->day<<"-";
                        cout<<ptr->month<<"-";
                        cout<<ptr->year<<"\n\n";
                        cin>>c;
                        if (ptr->next==NULL)
                        {
                            goto s1;
                        }
                        ptr=ptr->next;
                    }
                }
            }

@manfeese · 08.12.2009, 22:13

Выкладывай весь текст! Что это за "кусочки"

@"Ламер"-программер:) · 08.12.2009, 22:18 **[ТС]**

В общем надеюсь сильно бить не будете, в общем вот моё "чудовище" =)

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>
#include <string>
#include <time.h>
 
using namespace std;
 
class DListNode
{
public:
    string denomination;//Наименование купюры
    double curse;       //Стоимость в рублях
    int day, month, year;       //Дата - переменные
    DListNode* next;    //Ссылка на след. элемент
    DListNode* prev;    //Ссылка на пред. элемент
};
 
void main()
{
    //Указатели на элементы
    DListNode* first=NULL;
    DListNode* last=NULL;
    DListNode* tek=NULL;
 
    int chose;
    int k=1, n=0, i=0, j=0, q, w, e;
    double cost;//Для преобразования
    int numb;//Для преобразования
    string key, c;
 
    while (k==1)
    {s1://Главное меню программы
        cout<<"1 - Do6aBuTb HoB eJleM\n";
        cout<<"2 - BblBecTu cnucok\n";
        cout<<"3 - nouck kynIOpbI\n";
        cout<<"4 - y3HaTb B py6Jl9x\n";
        cout<<"5 - \n";
        cout<<"6 - \n";
        cout<<"7 - nepeBecTu B pybJLu\n";
        cout<<"8 - Bblxod\n\n";
        cout<<"BbinoJlHuTb: ";
        cin>>chose;
 
        switch(chose)
        {
 
        case 1://Добавить купюру
            {
                struct tm when;
                time_t now, result;
                string k;
                double c;
                int d;
 
s2:
                cout<<"BBeduTe kynIOpy: ";
                cin>>k;
                cout<<"BBeduTe cTouMocTb: ";
                cin>>c;
                cout<<"BBedute dHu: ";
                time( &now );
                when = *localtime( &now );
                cin>>d;
 
                when.tm_mday = when.tm_mday + d;
 
                DListNode* ptr=new DListNode;
                ptr->denomination=k;
                ptr->curse=c;
                if( (result = mktime( &when )) != (time_t)-1 )
                {
                    q=when.tm_mday;
                    w=when.tm_mon+1;
                    e=when.tm_year+1900;
                    ptr->day=q;
                    ptr->month=w;
                    ptr->year=e;
                }
                else
                {
                    perror("BBedeHo He KoppeKTHoe 4ucJlo!");
                    break;
                }
                ptr->next=NULL;
                last=ptr;
 
                if (first==NULL)
                {
                    first=ptr;
                    tek=ptr;
                    ptr->prev=NULL;
                }
 
                ptr->prev=tek;
                tek->next=ptr;
                tek=ptr;
                break;
            }
        case 2://Вывести список всех записей
            {
                DListNode* ptr = NULL;
                if (first==NULL)
                {
                    cout<<"Cnucok nycT!\n";
                    break;
                }
                else
                {
                    ptr=first;
                    while (1)
                    {
                        cout<<ptr->denomination<<" ";
                        cout<<ptr->curse<<" ";
                        cout<<ptr->day<<"-";
                        cout<<ptr->month<<"-";
                        cout<<ptr->year<<"\n\n";
                        cin>>c;
                        if (ptr->next==NULL)
                        {
                            break;
                        }
                        ptr=ptr->next;
                    }
                }
                break;
            }
        case 3://Поиск определённой купюры
            {
                DListNode* ptr=NULL;
                if (first==NULL)
                {   
                    cout<<"\neJLeMeHTbl oTcyTcTByIOT\n";
                    goto s1;
                }
                cout<<"BBeduTe eJLeMeHT dJL9 noucka: ";
                cin>>key;
                ptr=first;
                while (1)
                {
                    if (key==ptr->denomination)
                    {
                        cout<<"\ndaHHble EJleMeHTa:\n";
                        cout<<ptr->curse<<" ";
                        cout<<ptr->day<<"-";
                        cout<<ptr->month<<"-";
                        cout<<ptr->year<<" ";
                        cout<<ptr->denomination<<"\n";
                    }
                    if (ptr->next==NULL)
                        {
                            cout<<"\nEJleMeHT He HaudeH\n";
                            goto s1;
                        }
                    ptr=ptr->next;
                }
            }
 
        case 4://Конвертер
            {
                cout<<"\nBBeduTe cTouMocTb: ";
                cin>>cost;
                cout<<"\nBBeduTe koJlu4ecTBo: ";
                cin>>numb;
            }
        case 8://Выход из программы
            {
                k=0;
                break;
            }
        }
    }
}

Добавлено через 3 минуты
Что то мне подсказывает что причина ещё при введении. Но где и почему - в голове не вяжется.

@manfeese · 08.12.2009, 22:28

Строки 88 и 90 поменяй местами и здесь же не задано значение для ptr->next, которое по идее должно пристваиваться указателю first

@"Ламер"-программер:) · 08.12.2009, 22:50 **[ТС]**

manfeese, спасибо, но не помогло. Может в 84 строке что-то не так? Я уже и знать не знаю что идёт не так. И чем больше становится мой код, тем хуже я его понимаю

@"Ламер"-программер:) · 09.12.2009, 18:32 **[ТС]**

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
                ptr->next=NULL;
                last=ptr;
 
                if (first==NULL)
                {
                    first=ptr;
                    tek=ptr;
                    ptr->prev=NULL;
                    ptr->next=NULL;
                }
                else
                {
                    ptr->next=ptr;
                    ptr->prev=tek;
                    ptr->next=NULL;
                    tek=ptr;
                }

Добавлено через 16 минут
омг, у меня всё вышло, спасибо всем xD

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
                ptr->next=NULL;
                last=ptr;
 
                if (first==NULL)
                {
                    first=ptr;
                    tek=ptr;
                    ptr->prev=NULL;
                }
                else
                {
                    ptr->prev=tek;
                    tek->next=ptr;
                    tek=ptr;
                }
                continue;

@manfeese · 09.12.2009, 20:44

Попробуй *Next и *Prev использовать не в самом классе, а как отдельно, как переменную данного класса

...А что за задание вообще???

@"Ламер"-программер:) · 09.12.2009, 21:21 **[ТС]**

Так, задание сделать двунаправленный список содержащий инфу о купюре: название, дата и стоимость.
В общем проблема решена, больше нету цикла, выводит не один элемент.У меня вопрос, если кого не затруднит - ответьте, как записать в файл переменную типа стринг. Или как можно в этом коде:

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
#include <iostream>
#include <stdio.h>
#include <string>
#include <time.h>
 
using namespace std;
 
class DListNode
{
public:
    char denomination[100];//Наименование купюры
    double curse;       //Стоимость в рублях
    int day, month, year;       //Дата - переменные
    DListNode* next;    //Ссылка на след. элемент
    DListNode* prev;    //Ссылка на пред. элемент
};
 
void main()
{
    //Указатели на элементы
    DListNode* first=NULL;
    DListNode* last=NULL;
    DListNode* tek=NULL;
 
    int chose;
    int k=1, q, w, e;
    double cost, rez;//Для преобразования
    int numb;//Для преобразования
    string key, c;
 
    while (k==1)
    {//Главное меню программы
        cout<<"1 - Do6aBuTb HoB eJleM\n";
        cout<<"2 - BblBecTu cnucok\n";
        cout<<"3 - nouck kynIOpbI\n";
        cout<<"4 - y3HaTb cTouMocTb\n";
        cout<<"5 - \n";
        cout<<"6 - \n";
        cout<<"7 - \n";
        cout<<"8 - Bblxod\n\n";
        cout<<"BbinoJlHuTb: ";
        cin>>chose;
 
        switch(chose)
        {
 
        case 1://Добавить купюру
            {
                struct tm when;
                time_t now, result;
                char kyn[100];
                double c;
                int d;
 
                cout<<"BBeduTe kynIOpy: ";
                cin>>kyn;
                cout<<"BBeduTe cTouMocTb: ";
                cin>>c;
                cout<<"BBedute dHu: ";
                time( &now );
                when = *localtime( &now );
                cin>>d;
 
                when.tm_mday = when.tm_mday + d;
 
                DListNode* ptr=new DListNode;
                ptr->denomination=kyn;
                ptr->curse=c;
 
                if( (result = mktime( &when )) != (time_t)-1 )
                {
                    q=when.tm_mday;
                    w=when.tm_mon+1;
                    e=when.tm_year+1900;
                    ptr->day=q;
                    ptr->month=w;
                    ptr->year=e;
                }
                else
                {
                    perror("BBedeHo He KoppeKTHoe 4ucJlo!");
                    break;
                }
 
                ptr->next=NULL;
                last=ptr;
 
                if (first==NULL)
                {
                    first=ptr;
                    tek=ptr;
                    ptr->prev=NULL;
                }
                else
                {
                    ptr->prev=tek;
                    tek->next=ptr;
                    tek=ptr;
                }
                continue;
            }
        case 2://Вывести список всех записей
            {
                DListNode* ptr = NULL;
                if (first==NULL)
                {
                    cout<<"Cnucok nycT!\n";
                    break;
                }
                else
                {
                    ptr=first;
                    while(true)
                    {
                        cout<<ptr->denomination<<" ";
                        cout<<ptr->curse<<" ";
                        cout<<ptr->day<<"-";
                        cout<<ptr->month<<"-";
                        cout<<ptr->year<<"\n\n";
                        if (ptr->next==NULL)
                        {
                            break;
                        }
                        ptr=ptr->next;
                    }
                }
                break;
            }
        case 3://Поиск определённой купюры
            {
                DListNode* ptr=NULL;
                if (first==NULL)
                {   
                    cout<<"\n eJLeMeHTbl oTcyTcTByIOT\n";
                    break;
                }
                cout<<"BBeduTe eJLeMeHT dJL9 noucka: ";
                cin>>key;
                ptr=first;
                while (true)
                {
                    if (key==ptr->denomination)
                    {
                        cout<<"\n daHHble EJleMeHTa:\n";
                        cout<<ptr->curse<<" ";
                        cout<<ptr->day<<"-";
                        cout<<ptr->month<<"-";
                        cout<<ptr->year<<" ";
                        cout<<ptr->denomination<<"\n";
                    }
                    if (ptr->next==NULL)
                        {
                            cout<<"\nEJleMeHT He HaudeH\n";
                            break;
                        }
                    ptr=ptr->next;
                }
                break;
            }
 
        case 4://Конвертер
            {
                cout<<"\nBBeduTe cTouMocTb: ";
                cin>>cost;
                cout<<"\nBBeduTe koJlu4ecTBo: ";
                cin>>numb;
                rez=cost*numb;
                cout<<"Pe3yJlbTaT: "<<rez;
                cout<<"\n\n";
                continue;
            }
 
        case 5://Удаление элемента из начала
            {
                
            }
        case 8://Выход из программы
            {
                k=0;
                break;
            }
        }
    }
}

Работать с переменной denimination не как со стрингом а как с чаром. Код выше написанный с чаром. Просто это нужно что бы записывать в файл, иначе говоря стринг в файл не запишешь.

@manfeese · 09.12.2009, 21:45

C++
1
2
3
4
5
6
7
8
.....
FILE* f;
char s[] = "строка";
f=fopen("C:\\1.txt","w");
fwrite(s,sizeof(s),1,f);   // Неформатированный ввод в файл
fprintf(f,"%s",s);  // форматированный ввод
fclose(f);
.....

Добавлено через 4 минуты
Если надо записать запись структуры в файл, то можно использовать неформатированный ввод

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
.....
struct MyStruct{
.....
char s[100];
....
}
 
FILE* f;
MyStruct D;
D.s="строка";
...
f=fopen("C:\\1.txt","wb");
fwrite(D,sizeof(D),1,f);
fclose(f);
.....

@"Ламер"-программер:) · 09.12.2009, 22:53 **[ТС]**

Спасибо за структурный ввод. Но я наверное очень не точно выразился, извиняюсь. Я хотел сказать, что если работать с чаром, а не со стрингом, то вот эта строка:

C++
1
ptr->denomination=kyn //denomination - чар, kyn - чар;

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

Не по теме:

manfeese, примного благодарен за ответы! =)

@manfeese · 09.12.2009, 23:51

ну наверное циклом:

C++
1
2
for(int i = 0;i<100;i++)
   ptr->denomination[i]=kyn[i];

Добавлено через 4 минуты
или используй следующую функцию из string.h

C++
1
memcpy(ptr->denomination,kyn,100);

@"Ламер"-программер:) · 10.12.2009, 01:36 **[ТС]**

Спасибо, manfeese ! =)
а я нашёл ещё кое какую функцию, она вроде должна записывать. c_str() - переводит в стринг всё что угодно, вроде.

C++
1
2
3
4
5
6
7
8
9
                    file = fopen(name, "a"); 
                    fputs( temp->denomination.c_str(), file );
                    fputs( temp->curse.c_str(), file );
                    fputs( temp->day.c_str(), file );
                    fputs(tere.c_str(), file);
                    fputs( temp->month.c_str(), file );
                    fputs(tere.c_str(), file);
                    fputs( temp->year.c_str(), file );
                    fputs(space.c_str(), file);

Добавлено через 22 минуты
хм, странно, что-то не работает совсем, ругаетсо и пишет что-то вроде:error C2228: left of '.c_str' must have class/struct/union type
Нужно что то дописать в код?

C++
1
temp->denomination.c_str()

я так понял левее temp?

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@manfeese 133 / 132 / 29 Регистрация: 04.01.2009 Сообщений: 415
	08.12.2009, 22:13
	Выкладывай весь текст! Что это за "кусочки" 0

@manfeese 133 / 132 / 29 Регистрация: 04.01.2009 Сообщений: 415
	08.12.2009, 22:28
	Строки 88 и 90 поменяй местами и здесь же не задано значение для ptr->next, которое по идее должно пристваиваться указателю first 1

@"Ламер"-программер:) 5 / 5 / 0 Регистрация: 17.04.2009 Сообщений: 69
	08.12.2009, 22:50 [ТС]
	manfeese, спасибо, но не помогло. Может в 84 строке что-то не так? Я уже и знать не знаю что идёт не так. И чем больше становится мой код, тем хуже я его понимаю 0

@manfeese 133 / 132 / 29 Регистрация: 04.01.2009 Сообщений: 415
	09.12.2009, 20:44
	Попробуй Next и Prev использовать не в самом классе, а как отдельно, как переменную данного класса ...А что за задание вообще??? 0