Nie działa getline()

Question

Witam.

Mam pewien problem. Stworzyłem strukturę,  następnie dynamiczne tablicę struktur.

 

using namespace std;

struct car
{
    string marka;
    int rok;
};

int main()
{
    int l_car;


    cout << "Ile samochdowo chcesz skatologwac? ";
    cin >> l_car;

    car *ps = new car[l_car];

    for (int i = 0; i < l_car; i++)
    {
        cout << "Samochod #" << i + 1 << endl;
        cout << "Prosze podac marke: ";
        getline(cin,ps[i]->marka);
        cout << "Prosze podac rok produkcji: ";
        cin >> ps[i]->rok;

    };

    return 0;
}

Problem jest w miejscu gdzie próbuję wczytać całą linię do struktury(getline). Program mi się nie kompiluje , a jak się skompiluje to omija część ze wczytywaniem.

Co zrobiłem źle? Proszę o pomoc.

Comments

Pokaż błędy kompilatora.

---

error:base operand of  '->'  has non-pointer type 'car'

---

@604kamper4

zastosuj . zamiast ->

---

Czemu? To pointer

---

Stosowanie kropki nie rozwiązuje problemu.

Już wszytko działa. Pomógł mi ktoś inny, ale tobie też dziękuję :)

---

@Hiskiel

Możesz użyć strzałki -> aby odwołać się do danego element lecz wcześniej musisz pobrać rzutować na wskaźnik

np.

car *ps = new car[10];
ps[0].rok = 2000;

cout << " (&ps[0])->rok; " << (&ps[0])->rok << " ps[0].rok; " << ps[0].rok << endl;

 

Answer 1

Witam,

dodaj na początek plik nagłówkowy <iostream>

#include <iostream>

Wydaje mi się że problemem jest bufor. Przykładowo jak piszesz jakąś liczbę np. 12 to aby ją zatwierdzić to musisz jeszcze przesłać znak '\n' (enter), który później trafia do funkcji getline() i ją zapełnia. Można tu zastosować metody ignore()/sync() do wyczyszczenia zawartości strumienia.

...
    int l_car;
    cout << "Ile samochdowo chcesz skatologwac? ";
    cin >> l_car;
    car *ps = new car[l_car];

    cin.ignore();
    cin.sync();
    
for (int i = 0; i < l_car; i++)
    {
        cout << "Samochod #" << i + 1 << endl;
        cout << "Prosze podac marke: ";

        getline(cin,ps[i].marka);

        cout << "Prosze podac rok produkcji: ";
        cin >> ps[i].rok;

        cin.ignore();
        cin.sync();
    }
...

 

Comments

Jeśli chodzi o #include <iostream> to mam to. Po prostu przypadkowo ominąłem tę linię. :)

Dodanie tych metod nie działa.

---

Cały kod

#include <iostream>

using namespace std;

struct car
{
    string marka;
    int rok;
};

int main()
{
    int l_car;
    cout << "Ile samochdowo chcesz skatologwac? ";
    cin >> l_car;
    car *ps = new car[l_car];

    cin.ignore();
    cin.sync();

for (int i = 0; i < l_car; i++)
    {
        cout << "Samochod #" << i + 1 << endl;
        cout << "Prosze podac marke: ";

        getline(cin,ps[i].marka);

        cout << "Prosze podac rok produkcji: ";
        cin >> ps[i].rok;

        cin.ignore();
        cin.sync();
    }

    return 0;
}

może coś źle dodałeś. Jakby co podaj komunikat błędu

---

Działa. Najwyraźniej coś źle przepisałem.

Dziena ;D

---

@Bondrusiek,

Dokładnie, po pobraniu ilości samochodów do skatalogowania, potrzebny jest znak nowego wiersza, aby można było wprowadzić kolejne dane.

#include <iostream>
#include <string>

// tworzę strukturę która zawiera informacje o samochodzie
struct Car
{
    std::string make;
    int yearMade{};
};

int main()
{
    int howManyCars = 0;
    std::cout << "Ile samochodow chcesz skatalogowac? ";
    (std::cin >> howManyCars).get(); // .get() znak nowego wiersza aby wprowadzić kolejne dane

    // tworzę dynamiczną tablicę struktur Car (collection)
    auto *collection = new Car[howManyCars];

    // pętla zbierająca dane do tablicy struktur
    for (int i = 0; i < howManyCars; i++)
    {
        std::cout << "Samochod numer " << i + 1 << std::endl;
        std::cout << "Podaj marke samochodu: ";
        getline(std::cin, collection[i].make);
        std::cout << "Podaj rok produkcji: ";
        (std::cin >> collection[i].yearMade).get(); // .get() znak nowego wiersza aby wprowadzić kolejne dane
    }
    std::cout << "Twoja kolekcja samochodow: \n";

    // pętla która wyświetla zebrane dane
    for (int i = 0; i < howManyCars; i++)
    {
        std::cout << collection[i].yearMade << " " << collection[i].make << std::endl;
    }
    delete[] collection; // usuwam dynamiczną tablicę

    return 0;
}

 

Answer 2

Witam, spróbuj  cin.ignore() w pętli.

Answer 3

Proszę działający kod :

#include <iostream>
using namespace std;
 
struct car
{
    string marka;
    int rok;
};
 
int main()
{
    int l_car;
 
 
    cout << "Ile samochdowo chcesz skatologwac? ";
    cin >> l_car;
 
    car *ps = new car[l_car];
 
    for (int i = 0; i < l_car; i++)
    {
        cout << "Samochod #" << i + 1 << endl;
        cout << "Prosze podac marke: ";
        string tmp;
        int tmpr;
        getline(cin,tmp);
        cout << "Prosze podac rok produkcji: ";
        cin >> tmpr;
 
    }
 
 
    return 0;
}

 

Comments

Jak myślisz po co tworzyłem strukturę?  Dane powinny znajdować się w  strukturze a nie w zwykłej zmiennej.

Przypisywanie zawartości zmiennej do struktury też nie działa

---

Sorki zapomniałem o tym by to później przypisać.