Очистка потока

Nishen · Регистрация: 26.02.2015

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

Приветствую!

Есть метод класса, принимающий имя сотрудника (объект класса string) и его номер:

C++

void Employee::setData() {
    
    cout << "Введите имя: ";
    getline(cin, name);
    
    cout << "Введите ID сотрудника: ";
    cin >> ID;
 
}

Использую getline(), чтобы можно было ставить пробелы при вводе имени.

Далее в функции main() я делаю следующее:

C++

do {
 
    cout << endl << "Введите данные о работнике с номером " << i + 1 << endl;
    staff[i++].setData();
 
    cout << "Продолжить ввод данных? (y/n): ";
    cin >> answer;
 
} while (answer != 'n');

При этом из-за ввода ответа пользователем о продолжении работы, следующий ввод имени сотрудника работает некорректно (скриншот). Как исправить?

@sourcerer · 21.08.2016, 11:40

Nishen, тынц.

Nishen · 21.08.2016, 11:56 **[ТС]**

gru74ik, спасибо большое!

Добавлено через 5 минут
А почему мне не помогает вариант с cin.sync()?

Пытаюсь сделать вот так:

C++

do {
 
    cout << endl << "Введите данные о работнике с номером " << i + 1 << endl;
    staff[i++].setData();
 
    cout << "Продолжить ввод данных? (y/n): ";
    cin >> answer;
    cin.sync();
        
} while (answer != 'n');

@sourcerer · 21.08.2016, 11:59

Nishen, весь свой код скинь.

Nishen · 21.08.2016, 12:03 **[ТС]**

C++

//CHAPTER 7 - JOB_02
//main.cpp
 
#include <iostream>
#include <string>
 
using namespace std;
 
 
class Employee {
 
    private:
        string name;
        long ID;
 
    public:
        Employee();
        Employee(string n, long id);
        void setData();
        void setData(string n, long id);
        void setData(char n[], long id);
        void getData() const;
 
};
//-----------------------------------------------------------------------------
Employee::Employee() : name(""), ID(0) {
 
}
//-----------------------------------------------------------------------------
Employee::Employee(string n, long id) : name(n), ID(id) {
 
}
//-----------------------------------------------------------------------------
void Employee::setData() {
    
    cout << "Введите имя: ";
    getline(cin, name);
    
    cout << "Введите ID сотрудника: ";
    cin >> ID;
 
}
//-----------------------------------------------------------------------------
void Employee::setData(string n, long id) {
 
    name = n;
    ID = id;
 
}
//-----------------------------------------------------------------------------
void Employee::setData(char n[], long id) {
 
    int length = strlen(n);
 
    for (int i = 0; i < length; i++) {
 
        name[i] = n[i];
 
    }
 
    ID = id;
}
//-----------------------------------------------------------------------------
void Employee::getData() const {
 
    cout << "Сотрудник №" << ID << ". Имя: " << name;
 
}
 
 
 
int main() {
 
    setlocale(LC_ALL, "Russian");
 
    const int MAX = 100;
 
    Employee staff[MAX];
 
    int i = 0;
    char answer = 'y';
 
    do {
 
        cout << endl << "Введите данные о работнике с номером " << i + 1 << endl;
        staff[i++].setData();
 
        cout << "Продолжить ввод данных? (y/n): ";
        cin >> answer;
        cin.sync();
        
 
    } while (answer != 'n');
 
    for (int j = 0; j < i; j++) {
 
        staff[j].getData();
        cout << endl;
 
    }
 
    system("PAUSE");
    return 0;
 
}

@sourcerer · 21.08.2016, 12:13

Странно. У меня все работает нормально. Правда, я два заголовочника добавил:

C++

1 2	#include <cstdlib> // для system() #include <cstring> // для strlen()

Nishen · 21.08.2016, 12:16 **[ТС]**

gru74ik, а у Вас в цикле do-while используется функция sync()? Просто я сначала не тот вариант скопировал из IDE, потом исправил.

@sourcerer · 21.08.2016, 12:25

Nishen, у меня так:

C++

//CHAPTER 7 - JOB_02
//main.cpp
 
#include <iostream>
#include <string>
#include <cstdlib> // для system()
#include <cstring> // для strlen()
 
using namespace std;
 
 
class Employee {
 
    private:
        string name;
        long ID;
 
    public:
        Employee();
        Employee(string n, long id);
        void setData();
        void setData(string n, long id);
        void setData(char n[], long id);
        void getData() const;
 
};
//-----------------------------------------------------------------------------
Employee::Employee() : name(""), ID(0) {
 
}
//-----------------------------------------------------------------------------
Employee::Employee(string n, long id) : name(n), ID(id) {
 
}
//-----------------------------------------------------------------------------
void Employee::setData() {
 
    cout << "Введите имя: ";
    getline(cin, name);
 
    cout << "Введите ID сотрудника: ";
    cin >> ID;
 
}
//-----------------------------------------------------------------------------
void Employee::setData(string n, long id) {
 
    name = n;
    ID = id;
 
}
//-----------------------------------------------------------------------------
void Employee::setData(char n[], long id) {
 
    int length = strlen(n);
 
    for (int i = 0; i < length; i++) {
 
        name[i] = n[i];
 
    }
 
    ID = id;
}
//-----------------------------------------------------------------------------
void Employee::getData() const {
 
    cout << "Сотрудник №" << ID << ". Имя: " << name;
 
}
 
 
 
int main() {
 
    setlocale(LC_ALL, "Russian");
 
    const int MAX = 100;
 
    Employee staff[MAX];
 
    int i = 0;
    char answer = 'y';
 
    do {
 
        cout << endl << "Введите данные о работнике с номером " << i + 1 << endl;
        staff[i++].setData();
        cout << "Продолжить ввод данных? (y/n): ";
        cin >> answer;
        cin.sync();
 
    } while (answer != 'n');
 
    for (int j = 0; j < i; j++) {
 
        staff[j].getData();
        cout << endl;
 
    }
 
    system("PAUSE");
    return 0;
 
}

Добавлено через 5 минут
Nishen, а вообще, почему бы sync() не запихнуть прямо в функцию-член setData()?

C++

void Employee::setData() {
    cin.sync();
    cout << "Введите имя: ";
    getline(cin, name);
 
    cout << "Введите ID сотрудника: ";
    cin >> ID;
}

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

Nishen · 21.08.2016, 12:27 **[ТС]**

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

почему бы sync() не запихнуть прямо в функцию-член setData

Да, спасибо за замечание. Но просто я использовал его изначально в метода setData(), он там не работал, я попробовал в main использовать его и так там и оставил. Странно, данный вариант решения проблемы на работает.

nd2 · 23.08.2016, 07:23

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

А почему мне не помогает вариант с cin.sync()?

ОС какая?

@HelicopterK52 · 23.08.2016, 07:54

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

Странно, данный вариант решения проблемы на работает.

Поведение sync зависит от реализации платформы. Он не обязан очищать поток.
ЕМНИП, в Windows с Visual Studio будет очищать, под Unix'ами не будет (во всех ли юниксах так

).

@sourcerer · 23.08.2016, 08:53

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

в Windows с Visual Studio будет очищать

Code::Blocks 16.01 + MinGW 4.9.2 + Windows 7 - очищает.

Добавлено через 14 минут
QtCreator 3.3.0 + MinGW 4.9.1 + Windows 7 - очищает.

Nishen · 23.08.2016, 10:15 **[ТС]**

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

ОС какая?

Windows 10 x64 Pro

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

Windows с Visual Studio

Какая раз мой вариант, но не очищает.

nd2 · 23.08.2016, 11:00

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

Какая раз мой вариант, но не очищает.

В Windows должно работать. Студия какая?

Добавлено через 31 минуту

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

Windows 10 x64 Pro

Сейчас проверил, в 10-ке не работает. Вывод: не использовать, для таких целей, sync(). Другие есть способы:
https://cppstudy.wordpress.com... et-and-co/

Nishen · 23.08.2016, 11:02 **[ТС]**

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

Студия какая?

2015

@hoggy · 23.08.2016, 11:27

C++

void clear_input()
{
    std::wcin.clear(),
    std::wcin.ignore(std::numeric_limits<std::streamsize>::max(), L'\n');
    assert(std::wcin);
}

nd2 · 23.08.2016, 11:44

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

std::wcin.ignore(std::numeric_limits<std::streamsize>::max(), L'\n');

Лучше так:

C++

1	std::wcin.ignore((std::numeric_limits<std::streamsize>::max)(), L'\n');

Nishen · 23.08.2016, 11:48 **[ТС]**

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

Лучше так:

Чем Ваша конструкция отличается от конструкции hoggy? Я вижу пару скобок, но что это даёт?

nd2 · 23.08.2016, 11:51

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

Я вижу пару скобок, но что это даёт?

Добавь в код #include <Windows.h> и посмотри, что будет без пары скобок.

@hoggy · 23.08.2016, 12:05

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

Лучше так:

синтаксический мусор

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

Добавь в код #include <Windows.h> и посмотри, что будет без пары скобок.

C++

1 2	#define _NOMINMAX #include <windows.h>

Nishen 1352 / 851 / 365 Регистрация: 26.02.2015 Сообщений: 3,799
	23.08.2016, 11:48 [ТС]	18
	Сообщение от nd2 Лучше так: Чем Ваша конструкция отличается от конструкции hoggy? Я вижу пару скобок, но что это даёт? 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	23.08.2016, 11:51	19
	Сообщение от Nishen Я вижу пару скобок, но что это даёт? Добавь в код `#include <Windows.h>` и посмотри, что будет без пары скобок. 0

@sourcerer Модератор 5284 / 2371 / 342 Регистрация: 20.02.2013 Сообщений: 5,770 Записей в блоге: 20
	21.08.2016, 11:40	2
	Сообщение было отмечено Nishen как решение Решение Nishen, тынц. 1

@sourcerer Модератор 5284 / 2371 / 342 Регистрация: 20.02.2013 Сообщений: 5,770 Записей в блоге: 20
	21.08.2016, 11:59	4
	Nishen, весь свой код скинь. 0

Nishen 1352 / 851 / 365 Регистрация: 26.02.2015 Сообщений: 3,799
	21.08.2016, 12:16 [ТС]	7
	gru74ik, а у Вас в цикле do-while используется функция sync()? Просто я сначала не тот вариант скопировал из IDE, потом исправил. 0

Nishen 1352 / 851 / 365 Регистрация: 26.02.2015 Сообщений: 3,799
	21.08.2016, 12:27 [ТС]	9
	Сообщение от gru74ik почему бы sync() не запихнуть прямо в функцию-член setData Да, спасибо за замечание. Но просто я использовал его изначально в метода setData(), он там не работал, я попробовал в main использовать его и так там и оставил. Странно, данный вариант решения проблемы на работает. 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	23.08.2016, 07:23	10
	Сообщение от Nishen А почему мне не помогает вариант с cin.sync()? ОС какая? 1

@HelicopterK52 829 / 253 / 34 Регистрация: 27.07.2016 Сообщений: 497 Записей в блоге: 1
	23.08.2016, 07:54	11
	Сообщение от Nishen Странно, данный вариант решения проблемы на работает. Поведение sync зависит от реализации платформы. Он не обязан очищать поток. ЕМНИП, в Windows с Visual Studio будет очищать, под Unix'ами не будет (во всех ли юниксах так ). 1

@sourcerer Модератор 5284 / 2371 / 342 Регистрация: 20.02.2013 Сообщений: 5,770 Записей в блоге: 20
	23.08.2016, 08:53	12
	Сообщение от HelicopterK52 в Windows с Visual Studio будет очищать Code::Blocks 16.01 + MinGW 4.9.2 + Windows 7 - очищает. Добавлено через 14 минут QtCreator 3.3.0 + MinGW 4.9.1 + Windows 7 - очищает. 0

Nishen 1352 / 851 / 365 Регистрация: 26.02.2015 Сообщений: 3,799
	23.08.2016, 10:15 [ТС]	13
	Сообщение от nd2 ОС какая? Windows 10 x64 Pro Сообщение от HelicopterK52 Windows с Visual Studio Какая раз мой вариант, но не очищает. 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	23.08.2016, 11:00	14
	Сообщение от Nishen Какая раз мой вариант, но не очищает. В Windows должно работать. Студия какая? Добавлено через 31 минуту Сообщение от Nishen Windows 10 x64 Pro Сейчас проверил, в 10-ке не работает. Вывод: не использовать, для таких целей, sync(). Другие есть способы: https://cppstudy.wordpress.com... et-and-co/ 1

Nishen 1352 / 851 / 365 Регистрация: 26.02.2015 Сообщений: 3,799
	23.08.2016, 11:02 [ТС]	15
	Сообщение от nd2 Студия какая? 2015 0

Очистка потока

Решение