Tablice struktur oraz klasy

Question

Dobry Wieczór 

Mam za zadanie napisać zawierający funkcje: WCZYTAJ_dane,
WYPISZ_dane, umożliwiający:
- Stworzenie tablicy struktur w której będą przechowywane dane
książek zgromadzonych w bibliotece: imię autora, nazwisko autora,
tytuł, rok wydania, nr wydania
- Wpisanie danych do tablicy
- Wyświetlenie danych

A dodatkowo, później przerobić program na klasy. 
Po kompilacji wyświetla mi się w 12 wersie błąd :"ksiazka/ksiegozbior was not declared in this scope"
I nie wiem co powinna poprawić. 
Dołączam mój kod :

#include <iostream>

using namespace std;
int n=1;
struct dane
{char imie_autora[15];
char nazwisko_autora[50];
char tytul[30];
int rok_wydania;
int nr_wydania;
};
void wpisz(ksiazka * ksiegozbior)
{   cout<<endl;
    cout<<endl;
    cout << "---WPISYWANIE DANYCH KSIAZKI---"<< endl;
	for (int i = 0; i < n; i++){

        cout<<endl<<endl<<" Podaj dane ksiazki nr: "<<i+1<<" : "<<endl;
        cout<<"_______________________________________"<< endl<<endl;
		cout<<endl<< " Imie autora: ";
		cin >> ksiegozbior[i].imie_autora;
		cout<<endl;
		cout<< " Nazwisko autora: ";
		cin >> ksiegozbior[i].nazwisko_autora;
		cout<<endl;
		cout<<"Tytul ksiazki: ";
		cin>>ksiegozbior[i].tytul;
		cout<<endl;
		cout << " Rok wydania: ";
		cin >> ksiegozbior[i].rok_wydania;
		cout << " Nr wydania: ";
		cin >> ksiegozbior[i].nr_wydania;

	}
}

void drukuj(ksiazka * ksiegozbior, int nr) {
     int l,k;
     int nr;
    cout<<endl;
        cout << "---DRUKOWANIE DANYCH PRACOWNIKA---"<< endl;
        cout << "__________________________________________________________________________ " << endl;
     for (int l = 0; l < k; l++)
    {
		cout << " Podaj nr ksiazki: ";
		cin >> nr;
   if(x<=n)
		{nr=x-1;

    cout<<endl <<endl;
	cout << " Imie autora: " << ksiegozbior[nr].imie_autora << endl;
	cout << " Nazwisko autora: " <<ksiegozbior[nr].nazwisko_autora<< endl;
	cout << " Tytul ksiaki: " << ksiegozbior[nr].tytul << endl;
	cout << " Rok wydania: " << (ksiegozbior[nr].rok_wydania) << endl;
	cout << " Nr wydania: " << ksiegozbior[nr].nr_wydania << endl;
	cout<<endl<<endl;
		}
    }
}
int main()
{
    int a;
    cout<<"                                  ---WITAJ!---                                       "<<endl;
	cout << "                   Ilu ksiazek chcesz wpisac ? "<<endl;
	cin >> a;
	ksiazka * ksiegozbior = new ksiazka[a];
	return 0;
}

Będę wdzięczna za jakąkolwiek pomoc :) 

Pozdrawiam 

Answer 1

Spójrz, jak masz nazwaną strukturę, przechowującą dane o książce... ;)

Comments

Podaj treść komunikatów.

---

W obu przypadkach jest taki sam :error: base operand of '->' has non-pointer type 'Ksiazka' "

---

Poproszę w takim razie Twój cały aktualny kod, zrobię potem w nim małe sprzątanie i od razu wszystkie niejasności powinny się rozwiać.

---

#include <iostream>

using namespace std;

class Ksiazka
{
    char imie_autora[15];
    char nazwisko_autora[50];
    char tytul[30];
    int rok_wydania;
    int nr_wydania;
public:
    void wpisz(Ksiazka * ksiegozbior, int n)
{   int i;
    cout<<endl;
    cout<<endl;
    cout << "---WPISYWANIE DANYCH KSIAZKI---"<< endl;

        cout<<endl<<endl<<" Podaj dane ksiazki nr: "<<i+1<<" : "<<endl;
        cout<<"_______________________________________"<< endl<<endl;
        cout<<endl<< " Imie autora: ";
        cin >> imie_autora;
        cout<<endl;
        cout<< " Nazwisko autora: ";
        cin >> nazwisko_autora;
        cout<<endl;
        cout<<"Tytul ksiazki: ";
        cin>> tytul;
        cout<<endl;
        cout << " Rok wydania: ";
        cin >> rok_wydania;
        cout << " Nr wydania: ";
        cin >> nr_wydania;
}
void drukuj(Ksiazka * ksiegozbior, int nr) {

    cout<<endl;
        cout << "---DRUKOWANIE DANYCH KSIAZKI---"<< endl;
        cout << "__________________________________________________________________________ " << endl;

        cout << " Podaj nr ksiazki: ";
        cin >> nr;


    cout<<endl <<endl;
    cout << " Imie autora: " << imie_autora << endl;
    cout << " Nazwisko autora: " <<nazwisko_autora<< endl;
    cout << " Tytul ksiaki: " << tytul << endl;
    cout << " Rok wydania: " << rok_wydania << endl;
    cout << " Nr wydania: " << nr_wydania << endl;
    cout<<endl<<endl;

    //}
}
};

int main()
{
    int a;
    int b = 0;
    cout<<"                                  ---WITAJ!---                                       "<<endl;
    cout << "                   Ilu ksiazek chcesz wpisac ? "<<endl;
    cin >> a;

    Ksiazka * ksiegozbior = new Ksiazka[a];

    for(int i = 0; i < a; ++i)
          ksiegozbior[i]->wpisz();

    ksiegozbior[b]->drukuj();

    return 0;
}

 

---

Dobra, mam chwilę czasu. Nie do końca zmieniłaś tak, jak Cię o to prosiłem. ;) Ale też i ja się pomyliłem, zapomniałem że operator indeksowania nie zwraca wskaźnika, tylko referencję - wybacz mi, zmęczony byłem.

Twój kod powinien wyglądać jakoś tak:

#include <iostream>
using namespace std;

class Ksiazka
{
    char imie_autora[15];
    char nazwisko_autora[50];
    char tytul[30];
    int rok_wydania;
    int nr_wydania;
public:

    void wpisz() /* pozbywamy się pierwszego argumentu, wskaźnika do Ksiazka - metody klas posiadają wskaźnik
                    this, który jest takim domyślnym argumentem - jest to wygodne. Dzięki temu nie trzeba
                    pisać za każdym razem "ksiegozbior -> imie_autora", wystarczy napisać "imie_autora" i metoda
                    będzie "wiedzieć", którego obiektu się to tyczy.
                    Tak samo wyrzucam drugi argument - to, ile książek wczytamy, teraz nie zależy od obiektu
                    klasy Ksiazka, tylko będzie "zależne zewnętrznie" od tego, ile sobie
                    takich obiektów zdefiniujemy (np. w mainie).
                */
    {
        cout<<endl;
        cout<<endl;
        cout << "---WPISYWANIE DANYCH KSIAZKI---"<< endl;
        cout<<endl<<endl<<" Podaj dane ksiazki "<<endl;
        cout<<"_______________________________________"<< endl<<endl;
        cout<<endl<< " Imie autora: ";
        cin >> imie_autora;
        cout<<endl;
        cout<< " Nazwisko autora: ";
        cin >> nazwisko_autora;
        cout<<endl;
        cout<<"Tytul ksiazki: ";
        cin>> tytul;
        cout<<endl;
        cout << " Rok wydania: ";
        cin >> rok_wydania;
        cout << " Nr wydania: ";
        cin >> nr_wydania;
    }

    void drukuj() // w zasadzie komentarz do metody wyżej wyczerpuje sprawę
    {
        cout<<endl;
        cout << "---DRUKOWANIE DANYCH KSIAZKI---"<< endl;
        cout << "__________________________________________________________________________ " << endl;
        cout<<endl <<endl;
        cout << " Imie autora: " << imie_autora << endl;
        cout << " Nazwisko autora: " <<nazwisko_autora<< endl;
        cout << " Tytul ksiaki: " << tytul << endl;
        cout << " Rok wydania: " << rok_wydania << endl;
        cout << " Nr wydania: " << nr_wydania << endl;
        cout<<endl<<endl;
    }
};

int main()
{
    unsigned int a; // niech a i b nie będą mogły być ujemne, skoro oznaczają liczbę książek (nie da się mieć -3 książki - o ile się komuś ich nie pożyczyło - , prawda?)
    unsigned int b = 0; /*  Ta zmienna jest wykorzystana do wypisania jednej, wybranej z książek.
                            Zainicjalizuję tę zmienną zerem - niech nie zawiera w sobie śmieci,
                            tylko pokazuje na konkretny egzemplarz.
                        */
    cout<<"                                  ---WITAJ!---                                       "<<endl;
    cout << "                   Ilu ksiazek chcesz wpisac ? "<<endl;
    cin >> a;
    Ksiazka * ksiegozbior = new Ksiazka[a];
    for(int i = 0; i < a; ++i)
    {
        cout<<"Ksiazka numer "<<i+1;
        ksiegozbior[i].wpisz();
    }
    ksiegozbior[b].drukuj();

    delete[] ksiegozbior; // zwalniamy przydzieloną nam pamięć
    return 0;
}

Dodałem stosowne komentarze i jakieś sensowne formatowanie kodu - prawda że o wiele przyjemniej się to czyta?

Na Twoim miejscu popracowałbym jeszcze nad komunikatami, bo obecnie są rozwlekłe i źle się je czyta. Ale to tylko kosmetyka.

Czy rozumiesz, dlaczego usunąłem argumenty z Twoich metod? Czy rozumiesz, dlaczego dostęp do wybranego elementu księgozbioru nie jest już częścią klasy Ksiazka?

Answer 2

Czym jest zmienna k w pętli w void drukuj?

Comments

zmienna l,k,x są nie potrzebne, zapomniałam wcześniej usunąć je