Błąd w programie

Question

Witam pisałem program który ma wpisać ilość samochodów skatalogowania, następnie wypisać ich dane(nazwe, i rok produkcji). Szukam błędu w pętli.

#include <iostream>

int main()
{
    struct salon
    {
        char nazwa[25];
        int rok;
    };

    int ilosc;
    std::cout << "Ile samochodow do skatalogowania: ";
    std::cin >> ilosc;
    salon car;
    for(int i=0; i < ilosc; i++)
        {
            std::cin.get(car.nazwa, i);
            std::cin >> car.rok;
        }//for

    std::cout << "Marka: "<< car.nazwa[25] << "\nRok produkcji: " << car.rok << std::endl;
    std::cin.get();
    return 0;
}

 

Answer 1

Na początek tworzysz obiekt car, potem kilka razy coś do niego przypisujesz, a następnie wypisujesz 25 znak jego nazwy. Przemyśl ten kod. I używaj std::string zamiast tablicy charów

Answer 2

Deklaracje struktury powinieneś raczej wyrzucić poza maina. Popatrz jak działa u Cb funkcja cin::get(gdzie ma wstawić, ile znaków) w Twoim przypadku zaczynasz od 0. Poza tym jesli chcesz miec kilka samochodów to musisz utworzyć tablice struktur. W innym przypadku caly czas nadpisujesz to co miałes. Dalej idąc poza petla chcesz wyprintowac car.nazwa[25] co jest wyjściem poza tablice.

Answer 3

Chcesz wczytać ileś tam samochodów do bazy danych a tworzysz miejsce tylko na jeden rekord tej bazy:

salon car;

To po pierwsze, po drugie nie rozumiesz najwyraźniej jak działa ta linijka:

std::cin.get(car.nazwa, i);

pierwszy argument metody get to zmienna, do której wczytane zostaną dane wpisane z klawiatury przez użytkownika. Natomiast i w tym przypadku oznacza ilość wczytanych znaków.

Poza tym w tej pętli ciągle nadpisujesz jeden i ten sam rekord co delikatnie mówiąc nie ma sensu.

Powinieneś utworzyć albo dynamicznie tablicę struktur salon albo wykorzystać kontener vector.

Answer 4

To spróbuj tak:

#include <iostream>
#include <string>
 
struct salon
    {
        std::string nazwa;
        int rok;
    };
 
int main()
{
    int ilosc;
    std::cout << "Ile samochodow do skatalogowania: ";
    std::cin >> ilosc;
    salon * car = new salon[ilosc];
    for(int i=0; i < ilosc; i++)
        {
            std::cout << "\nPodaj marke: ";
			std::cin.ignore(1);
            //std::cin >> car[i].nazwa;    // dziala poprawnie
            std::getline(std::cin, car[i].nazwa); //nie mozna wpisac danych
            std::cout << "Podaj rok produkcji: ";
			std::cin >> car[i].rok;
        }//for
    std::cout << std::endl;
    for(int i=0; i < ilosc; i++)
        std::cout << "Marka: "<< car[i].nazwa << "\nRok produkcji: " << car[i].rok << std::endl;
	std::cin.ignore(1);
    std::cin.get();
    return 0;
}

Bo łapie ci ten znak nowej linii, więc go z bufora usuwasz za pomocą cin.ignore(1)

Zapomniałem dodać, że na koniec ładnie by było pamiętać o zwolnieniu dynamicznie przydzielonej pamięci.

Comments

Dziękuję nie znałem tej metody.

Tak zrobiłem zupełnie o tym zapomniałem.

Answer 5

Poprawiłem program według wytycznych. Dziękuję bardzo. Wszystko działa poprawnie, jednak przy wpisywaniu danych za pomocą std::cin, próbowałem zrobić to za pomocą std::cin.get() lub std::getline(), program nie pozwala na wpisanie tekstu. Dlaczego ?.

#include <iostream>
#include <string>

struct salon
    {
        std::string nazwa;
        int rok;
    };

int main()
{
    int ilosc;
    std::cout << "Ile samochodow do skatalogowania: ";
    std::cin >> ilosc;
    salon * car = new salon[ilosc];
    for(int i=0; i < ilosc; i++)
        {
            std::cout << "\nPodaj marke: ";
            std::cin >> car[i].nazwa;    // dziala poprawnie
            //std::getline(std::cin, car[i].nazwa); //nie mozna wpisac danych
            std::cout << "Podaj rok produkcji: ";
            std::cin >> car[i].rok;
        }//for
    std::cout << std::endl;
    for(int i=0; i < ilosc; i++)
        std::cout << "Marka: "<< car[i].nazwa << "\nRok produkcji: " << car[i].rok << std::endl;
    std::cin.get();
    return 0;
}

Comments

Ponieważ kod 

std::cin >> car[i].nazwa;

zostawia po sobie znak nowego wiersza, a linijka 

std::getline(std::cin, car[i].nazwa);

go wczytuje. Powinieneś wczytać ten znak np. za pomocą cin.get ();

---

próbowałem już tym sposobem:

std::cin.get(car[i].nazwa,25);

jednak cały czas pokazuje mi błąd sprawdzałem co on oznacza i dotyczy to parametrów.

---

std::cin.get(car[i].nazwa,25);

cin.get() pobiera zmienne typu char, a ty masz zdefiniowaną string. Musiałbyś zamienić zmienną nazwa na tablicę znaków. Ewentualnie możesz zapisać to tak:

std::cin>>nazwa;

lub jeżeli chcesz pobrać całą linię:

std::getline(cin,nazwa);