Создать динамическую струтуру-массив с типом string и int

@proninyaroslav · Регистрация: 29.01.2013

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

Здравствуйте. Учу C++ по книге Прата и делаю его задачи. И тут попалась вот такая задача: создать динамическую струтуру-массив с типом стринг и int. Но дело в том, что когда я с помощью getline() присваиваю значение структуре, он не даёт даже ввести строку и сразу переходит к вводу значения int. То есть вывод получается таким:

Кликните здесь для просмотра всего текста

Сколько автомобилей поместить в каталог? 2
Автомобиль #1

Введите производителя: Укажите год выпуска: 1952
Автомобиль #2

Введите производителя: Укажите год выпуска: 1951
Ваша коллекция:

1952
1951

Вот сам код:

C++

#include <iostream>
#include <string>
 
using namespace std;
 
struct car
{
    string name;
    int age;
};
 
int main()
{
    int number;
    cout << "Сколько автомобилей поместить в каталог? ";
    cin >> number;
    int j = number;
    car *cr = new car [number];
 
    for (int i = 0; i < number; i++)
    {
        cout << "Автомобиль #" << number - --j << endl << endl;
        cout << "Введите производителя: ";
        getline(cin, cr[i].name);
        cout << "Укажите год выпуска: ";
        cin >> cr[i].age;
    }
    cout << "Ваша коллекция:\n\n";
    for (int i  = 0; i < number; i++)
        cout << cr[i].age << " " << cr[i].name << endl;
    delete [] cr;
}

Пробовал в Xcode, Qt, g++. Результат один и тот же. Ошибок компилятора нет.

@Firework · 31.07.2014, 11:28

C++

int main()
{
    int number;
    cout << "Сколько автомобилей поместить в каталог? ";
    cin >> number;
    int j = number;
    car *cr = new car [number];
 
    for (int i = 0; i < number; i++)
    {
        cout << "Автомобиль #" << number - --j << endl << endl;
        cout << "Введите производителя: ";
        cin.sync();
        cin.clear();
        getline(cin, cr[i].name);
        cout << "Укажите год выпуска: ";
        cin >> cr[i].age;
    }
    cout << "Ваша коллекция:\n\n";
    for (int i  = 0; i < number; i++)
        cout << cr[i].age << " " << cr[i].name << endl;
    delete [] cr;
}

@proninyaroslav · 31.07.2014, 11:36 **[ТС]**

Не помогло. Не понимаю в чём причина.

Добавлено через 3 минуты
Факт в том, что если просто добавить в этот цикл сосвем другую стринг переменную, которая вообще не относится к структуре, происходит тоже самое (!). Или если вынести getline(cin, cr[i].name); за пределы цикла то тоже самое.

@Firework · 31.07.2014, 11:39

Как это не помогло? У меня всё работает как надо (компилятор MSVS 2010)

@proninyaroslav · 31.07.2014, 11:47 **[ТС]**

Мда, сколько чудес приносит нам C++. Просто добавил после cout << "Автомобиль #" << number - --j << endl << endl; стринг-переменную (самое главное она не относится к структуре) и getline(). После этого гетлайны, идущие после неё, стали записывать в структуру значения. Я не понимаю как это работает. Просто хотел проверить, пишет ли он вообще в другие стринг-переменные в самом цикле. Может подскажете, почему это помогло?

C++

#include <iostream>
#include <string>
 
using namespace std;
 
struct car
{
    string name;
    int age;
};
 
int main()
{
    int number;
    cout << "Сколько автомобилей поместить в каталог? ";
    cin >> number;
    int j = number;
    car *cr = new car [number];
 
    for (int i = 0; i < number; i++)
    {
        cout << "Автомобиль #" << number - --j << endl << endl;
        //решение проблемы
    string h;
    getline(cin, h);
        cout << "Введите производителя: ";
        getline(cin, cr[i].name);
        cout << "Укажите год выпуска: ";
        cin >> cr[i].age;
    }
    cout << "Ваша коллекция:\n\n";
    for (int i  = 0; i < number; i++)
        cout << cr[i].age << " " << cr[i].name << endl;
    delete [] cr;
}

@zss · 31.07.2014, 11:53

Весь сыр-бор из-за следующего

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

cin >> cr[i].age;

После этого чтения указатель входного потока cin стоит на символе конца строки ('\n')
Соответственно в цикле следующая операция чтения строки getline(cin, cr[i].name);
воспринимает этот символ за целую (пустую) строку.
Я этот нюанс решаю простым дополнительным чтением символа

C++

1 2	сin >> cr[i].age; cin.get();

@proninyaroslav · 31.07.2014, 11:57 **[ТС]**

Усовершенствовал код, просто добавив cin.get() вместо стринга и getline(). Получилось так:

C++

for (int i = 0; i < number; i++)
    {
        cout << "Автомобиль #" << number - --j << endl << endl;
    cin.get();
        cout << "Введите производителя: ";
        getline(cin, cr[i].name);
        cout << "Укажите год выпуска: ";
        cin >> cr[i].age;

Мне кажется, что проблема появляется, когда ввожу int значение перел циклом, которое потом определяет количество элементов в масиве. Видимо он добавляет конец строки во входную очередь и getline() завершается.

@zss · 31.07.2014, 12:10

Не туда вставили. Потеряете первую букву у первого производителя.

C++

for (int i = 0; i < number; i++)
    {
        cout << "Автомобиль #" << number - --j << endl << endl;
        cout << "Введите производителя: ";
        getline(cin, cr[i].name);
        cout << "Укажите год выпуска: ";
        cin >> cr[i].age;
        cin.get();
}

Кстати, это можно написать по пижонски
(cin >> cr[i].age).get();

@proninyaroslav · 31.07.2014, 12:21 **[ТС]**

zss, вообще то не потерял). Если внимательно посмотрите, то выше цикла у меня следующее:

C++

1
2
3

    int number;
    cout << "Сколько автомобилей поместить в каталог? ";
    cin >> number;

В этом то и проблема. После этого он символ новой строки оставляет во входной очереди. Во-вторых, когда цикл закончится первый раз, строка cin >> cr[i].age; ему не помешает, так как cin.get(), находящийся выше всех гетлайнов, примет конец строки из входной очереди и очистит её. Иначе тогда бы пришлось написать так:

C++

    int number;
    cout << "Сколько автомобилей поместить в каталог? ";
    cin >> number
//принимает символ новой строки
cin.get();
    int j = number;
    car *cr = new car [number];
 
    for (int i = 0; i < number; i++)
    {
        cout << "Автомобиль #" << number - --j << endl << endl;
        cout << "Введите производителя: ";
        getline(cin, cr[i].name);
        cout << "Укажите год выпуска: ";
        cin >> cr[i].age;
//опять принимает символ новой строки. Проще добавить cin.get() в начале цикла
cin.get()
    }

@Voivoid · 31.07.2014, 12:38

Вместо cin.get() лучше тогда уж перед каждым getline делать следующее:

C++

1	std::cin.ignore( std::cin.rdbuf()->in_avail() );

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

@proninyaroslav 2 / 2 / 0 Регистрация: 29.01.2013 Сообщений: 42
	31.07.2014, 11:36 [ТС]	3
	Не помогло. Не понимаю в чём причина. Добавлено через 3 минуты Факт в том, что если просто добавить в этот цикл сосвем другую стринг переменную, которая вообще не относится к структуре, происходит тоже самое (!). Или если вынести getline(cin, cr[i].name); за пределы цикла то тоже самое. 0

@Firework 102 / 93 / 36 Регистрация: 01.04.2013 Сообщений: 654
	31.07.2014, 11:39	4
	Как это не помогло? У меня всё работает как надо (компилятор MSVS 2010) 0

Создать динамическую струтуру-массив с типом string и int

Решение