Język Programowania C++ Stephen Prata ćwiczenie 7 rozdział 5

Question

Program powinien wczytywać markę i rok produkcji określonej liczby samochodów i zapisywać je w dynamicznej tablicy struktur car, aby na koniec wyświetlić wczytane dane. Niestety poprawnie wyświetla jedynie ostatni zestaw danych (informacje o ostatnim samochodzie).

#include <iostream>
struct car{
    std::string marka;
    int rok;
};
int main()
{
    std::cout<<"Ile samochodow chcesz skatalogowac?";
    int n;
    std::cin>>n;
    for(int i = 0; i < n; i++)
    {
        car * ps = new car[n];
        std::cout<<"Samochod nr #"<<(i+1)<<std::endl;
        std::cout<<"Podaj marke samochodu: ";
        std::cin>>(*(ps+i)).marka;
        std::cout<<"Podaj rok produkcji: ";
        std::cin>>(*(ps+i)).rok;
        while(i == (n-1))
        {
            for(int j = 0; j < n; j++)
            {
                std::cout<<(*(ps+j)).marka<<" ";
                std::cout<<(*(ps+j)).rok<<"\n";
            }
        delete ps;
        break;
        }
    }
    return 0;
}

 

Comments

A gdybym w string'u chciał przechować 2-członową nazwę, w jaki sposób miałbym to zrobić?

Answer 1

car * ps = new car[n];

powinno być przed pętlą for, bo w tej chwili za każdym razem tworzysz nową tablicę

while(i == (n-1))
        {
            for(int j = 0; j < n; j++)
            {
                std::cout<<(*(ps+j)).marka<<" ";
                std::cout<<(*(ps+j)).rok<<"\n";
            }
        delete ps;

takie coś powinno być po pętli for, bo sprawdzanie za każdym razem warunku skończenia pętli skoro wystarczy poczekać aż się ta pętla skończy jest bez sensu 

i powinno być:

delete [] ps;

bo usuwasz tablicę

Answer 2

#include <iostream>
struct car{
    std::string marka;
    int rok;
};
int main()
{
    std::cout<<"Ile samochodow chcesz skatalogowac?";
    int n;
    std::cin>>n;
    
    car * ps = new car[n];
    
    for(int i = 0; i < n; i++)
    {
        std::cout<<"Samochod nr #"<<(i+1)<<std::endl;
        std::cout<<"Podaj marke samochodu: ";
        std::cin>>(*(ps+i)).marka;
        std::cout<<"Podaj rok produkcji: ";
        std::cin>>(*(ps+i)).rok;
        //while(i == (n-1))
        //{
    }
    
    for(int j = 0; j < n; j++)
            {
                std::cout<<(*(ps+j)).marka<<" ";
                std::cout<<(*(ps+j)).rok<<"\n";
            }
        delete [] ps;
        
        //}
    
    return 0;
}

Przeanalizuj sobie co robiłeś źle, ten kod który wkleiłem działa. Niepotrzebny był while, a drugi for źle umieszczony.