Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.75/8: Рейтинг темы: голосов - 8, средняя оценка - 4.75
cofo
0 / 0 / 0
Регистрация: 17.02.2014
Сообщений: 17
1

Метод getline(cin, m) не срабатывает без cin.ignore() / Ревью кода

31.10.2016, 17:39. Просмотров 1680. Ответов 2
Метки нет (Все метки)

Почему в моем случае getline(cin, m) не срабатывает без cin.ignore() ? Если по коду есть какие-нибудь замечания, то просьба написать что не так и почему

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
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
#include <iostream>
#include <string>
 
using namespace std;
 
class Transport { 
protected:
    int balance, price, mileage;
public:
    Transport(int balance, int price, int mileage) {
        this->balance = balance;
        this->price = price;
        this->mileage = mileage;
    }
    virtual void get() = 0; // метод вывода на экран
    virtual void check() = 0; // метод проверки на необходимость прохождения техосмотра
    void money() {
        if (balance < price) {
            cout << "У вас недостаточно средств для покупки данного транспортного средства" << endl;
        }
        else {
            cout << "Вам по карману данное транспортное средство" << endl;
        }
    }
 
    virtual ~Transport() {}
};
 
class Train : public Transport {
private:
    int vagon, seat;
public:
    Train(int balance, int price, int mileage, int vagon):Transport(balance, price, mileage) {
        this->vagon = vagon; // количество вагонов
        seat = 50; // количество мест в одном вагоне
    }
 
    void get() {
        cout << "Ваш баланс: " << balance << endl;
        cout << "Стоимость поезда: " << price << endl;
        cout << "Пробег поезда: " << mileage << endl;
        cout << "Количество вагонов: " << vagon << endl;
        cout << "Количество сидячих мест в одном вагоне: " << seat << endl;
        cout << "Количество сидячих мест в поезде: " << vagon * seat << endl;
    }
 
    void depo() {
        if (vagon <= 25) {
            cout << "Поезд поместится в депо" << endl;
        }
        else {
            cout << "Поезд не поместится в депо" << endl;
        }
    }
 
    void check() {
        if (mileage > 5000000) {
            cout << "Требуется прохождение техосмотра" << endl;
        }
        else {
            cout << "Прохождение техосмотра не требуется" << endl;
        }
    }
 
    ~Train() {}
};
 
class Auto : public Transport {
protected:
    string m;
    int g, hp, t;
public:
    Auto(int balance, int price, int mileage, string m, int g, int hp):Transport(balance, price, mileage) {
        this->m = m;
        this->g = g;
        this->hp = hp;
    }
 
    virtual int tax() = 0; // метод вычисления налога
 
    void check() {
        if (mileage > 100000) {
            cout << "Требуется прохождение техосмотра" << endl;
        }
        else {
            cout << "Прохождение техосмотра не требуется" << endl;
        }
    }
    ~Auto() {}
};
 
class Car : public Auto {
public:
    Car(int balance, int price, int mileage, string m, int g, int hp):Auto(balance, price, mileage, m, g, hp) {}
 
    int tax() {
        if (hp <= 100) 
        {
            t = hp * 12;
        }
        else if (hp > 100 && hp <= 250) 
        {
            t = hp * 25;
        }
        else if (hp > 250) 
        {
            t = hp * 50;
        }
        return t;
    }
 
    void get() {
        cout << "Ваш баланс: " << balance << endl;
        cout << "Стоимость легкового автомобиля: " << price << endl;
        cout << "Пробег легкового автомобиля: " << mileage << endl;
        cout << "Марка и модель легкового автомобиля: " << m << endl;
        cout << "Год выпуска легкового автомобиля: " << g << endl;
        cout << "Мощность двигателя легкового автомобиля (л.с.): " << hp << endl;
        cout << "Налог за год на данный легковой автомобиль: " << t << endl;
    }
    ~Car() {}
};
 
class Bus : public Auto {
public:
    Bus(int balance, int price, int mileage, string m, int g, int hp):Auto(balance, price, mileage, m, g, hp) {}
 
    int tax() {
        if (hp <= 100)
        {
            t = hp * 19;
        }
        else if (hp > 100 && hp <= 200)
        {
            t = hp * 30;
        }
 
        else if (hp > 200)
        {
            t = hp * 60;
        }
        return t;
    }
 
    void get() {
        cout << "Ваш баланс: " << balance << endl;
        cout << "Стоимость автобуса: " << price << endl;
        cout << "Пробег автобуса: " << mileage << endl;
        cout << "Марка и модель автобуса: " << m << endl;
        cout << "Год выпуска автобуса: " << g << endl;
        cout << "Мощность двигателя автобуса (л.с.): " << hp << endl;
        cout << "Налог за год на данный автобус: " << t << endl;
    }
    ~Bus() {}
};
 
 
int main() {
    setlocale(LC_ALL, "RUS");
    int balance, price, mileage, vagon, g, hp;
    string m;
 
    char i = ' ';
    while (i != '4')
    {
        cout << "\nВведите 1 - чтобы выбрать поезд\n " << "\nВведите 2 - чтобы выбрать легковой автомобиль \n " << "\nВведите 3 - чтобы выбрать автобус\n " << "\nВведите 4 - для выхода \n " << endl;
        cin >> i;
 
        switch(i) {
        case '1': {
            cout << endl << "Конструктор класса Train" << endl;
            cout << "Введите свой баланс: ";
            cin >> balance;
            cout << "Введите стоимость поезда: ";
            cin >> price;
            cout << "Введите пробег поезда: ";
            cin >> mileage;
            cout << "Введите число вагонов поезда: ";
            cin >> vagon;
            cout << endl;
 
            Train train_obj(balance, price, mileage, vagon);
            Transport *transport = &train_obj;
            //Train *train = &train_obj;
 
            transport->get();
            //train->depo();
            train_obj.depo();
            transport->check();
            //transport->money();
            train_obj.money();
 
            break;
        }
 
        case '2': {
            cout << endl << "Конструктор класса Car" << endl;
            cout << "Введите свой баланс: ";
            cin >> balance;
            cout << "Введите стоимость легкового автомобиля: ";
            cin >> price;
            cout << "Введите пробег легкового автомобиля: ";
            cin >> mileage;
            cout << "Введите марку и модель легкового автомобиля: ";
            cin.ignore();
            getline(cin, m);
            cout << "Введите год выпуска легкового автомобиля: ";
            cin >> g;
            cout << "Введите мощность двигателя легкового автомобиля (л.с.): ";
            cin >> hp;
            cout << endl;;
 
            Auto *car = new Car(balance, price, mileage, m, g, hp);
 
            car->tax();
            car->get();
            car->check();
            car->money();
            delete car;
            break;
        }
 
        case '3': {
            cout << endl << "Конструктор класса Bus" << endl;
            cout << "Введите свой баланс: ";
            cin >> balance;
            cout << "Введите стоимость автобуса: ";
            cin >> price;
            cout << "Введите пробег автобуса: ";
            cin >> mileage;
            cout << "Введите марку и модель автобуса: ";
            cin.ignore();
            getline(cin, m);
            cout << "Введите год выпуска автобуса: ";
            cin >> g;
            cout << "Введите мощность двигателя автобуса (л.с.): ";
            cin >> hp;
            cout << endl;;
 
            Auto *bus = new Bus(balance, price, mileage, m, g, hp);
 
            bus->tax();
            bus->get();
            bus->check();
            bus->money();
            delete bus;
            break;
        }
 
        case '4': {
            break;
        }
 
        default: {
                cout << "Неправильный ввод" << endl;
                 }
        }
    }
 
   return 0;
}
0
QA
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
31.10.2016, 17:39
Ответы с готовыми решениями:

Объясните работу методов cin.getline и cin.ignore
Фрагмент программы ниже. Что делают cin.getline и cin.ignore (12-13 строки) void...

Как работает cin.peek, cin,get, cin.ignore, cin.clear?
Здравствуйте, товарищи и не товарищи!:) Я только начал изучать C++, а уже использую вещи, которые...

cin.getline и cin.ignore
Есть кусок кода в котором не могу разобраться. Если cin.ignore() есть, то теряется первый вводимый...

Друзья getline и cin.ignore
Постараюсь как можно более точно обрисовать проблему. Имеется string * v1 = new string; string *...

2
_stanislav
(80 / 20 || 50 / 50) = x
1274 / 976 / 380
Регистрация: 16.08.2014
Сообщений: 3,915
Записей в блоге: 1
Завершенные тесты: 2
31.10.2016, 17:44 2
cofo, чем короче код, тем больше ответов, лень берет свое, тяжело читать чужие каракули, даже свои тяжело.
тебе нужно почитать про флаги состояний потока.
0
MrGluck
Форумчанин
Эксперт CЭксперт С++
8144 / 4995 / 1436
Регистрация: 29.11.2010
Сообщений: 13,460
31.10.2016, 17:50 3
Код тяжело читать - это главный его недостаток.
Разделите мухи от котлетобъявление методов от их определения.
Ветки кода в операторе switch-case разделите по функциям.

По проблеме - после ввода с потока istream (объект cin), в потоке остается символ '\n', который кушается при следующем чтении getline. Данная проблема не нова и уже даже решение с cin.ignore() стало классикой и на него не смотрят как на костыль (коим он является).
Вообщем, добавьте коду структурированности и успокойтесь.
1
31.10.2016, 17:50
Answers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
31.10.2016, 17:50

Заказываю контрольные, курсовые, дипломные и любые другие студенческие работы здесь.

Не работает getline(cin,s), istream.ignore()
Использую microsoft visual c++ 6.0. не работает: string s; getline(cin,s);//дважды приходиться...

Для чего в код вводится cin.ignore после того, как cin>> уже сработало?
Доброго времени суток! Просматриваю простой код. Вводится строка типа string: фунты и дюймы Затем...

Проблема с cin.ignore() и cin.clear()
Есть проблема. Добавлено через 11 минут // ConsoleApplication6.cpp: определяет точку входа...

Продемонстрировать различие между вводом с помощью cin.Get и cin.Getline с тремя параметрами
Нужна помощь. Задача продемонстрировать различие между вводом с помощью cin.get и cin.getline с...


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

Или воспользуйтесь поиском по форуму:
3
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2020, vBulletin Solutions, Inc.