Problem z wyświetleniem elementów listy z użyciem klasy abstrakcyjnej

Question

Witam, mam pewien problem. Napisałem taką klasę 

 class Katalog : Pozycja
    {
        private string dzialTematyczny;
        List<Pozycja> pozycje = new List<Pozycja>();

        public Katalog()
        {
            dzialTematyczny = "pusty";
            tytul = "pusty";
            id = 0;
            wydawnictwo = "pusty";
            rokWydania = 0;
        }
        public Katalog(string dzialTematyczny_, string tytul_, int id_, string wydawnictwo_, int rokWydania_)
        {
            dzialTematyczny_ = dzialTematyczny;
            tytul_ = tytul;
            id_ = id;
            wydawnictwo_ = wydawnictwo;
            rokWydania_ = rokWydania;

        }

        public Pozycja dodajPozycje(string dzialTematyczny_, string tytul_, int id_, string wydawnictwo_, int rokWydania_)
        {
            Pozycja pozycja = new Katalog(dzialTematyczny_, tytul_, id_, wydawnictwo_, rokWydania_);
            pozycje.Add(pozycja);
            return pozycja;
        }
         public override void wypiszInfo()
        {
            foreach (Pozycja poz in pozycje)
            {
                Console.WriteLine(dzialTematyczny + " " + tytul + " " + id + " " + wydawnictwo + " " + rokWydania);
            }
        }

    }
}

Gdzie Pozycja jest klasą abstrakcyjną. metoda wypiszInfo wyświetla mi tylko puste, 0, itp. elementy które zapisałem w konstruktorze i za żadne skarby nie mogę przypisać nic innego. Mogł by ktoś uświadomić mi w czym popełniam błąd? 

Z góry dziękuję

Answer 1

Tak, bo po pierwsze:

class Katalog : Pozycja
    {
        List<Pozycja> pozycje = new List<Pozycja>();

Samo to dziedziczenie nie ma sensu. Tak, katalog jest listą pozycji. Ale katalog nie jest pozycją (chyba że to jakoś umiesz uzasadnić), więc tego dziedziczenia w ogóle nie powinno być.

Po drugie:

            foreach (Pozycja poz in pozycje)
            {
                Console.WriteLine(dzialTematyczny + " " + tytul + " " + id + " " + wydawnictwo + " " + rokWydania);
            }

Wypisujesz tutaj pola instancji klasy na której wywołałeś metodę, a nie obiektów po których się iterujesz.

Comments

Katalog dziedziczy po Pozycja pola tytul, rokWydania, itp., oraz ma w sobie virtualną funkcję wypiszInfo, którą muszę nadpisać. Zmieniłem pętlę na taką:

            foreach (Pozycja pozycja in pozycje)
            {
                Console.WriteLine(pozycja);
            }

Jednak nic to nie dało, wyświetla się PO3.Katalog

---

Katalog dziedziczy po Pozycja pola tytul, rokWydania, itp.

I próbuję przekazać że to nie ma sensu, bo katalog nie jest pozycją; katalog nie ma przecież "roku wydania".

Tak samo to:

        public Pozycja dodajPozycje(string dzialTematyczny_, string tytul_, int id_, string wydawnictwo_, int rokWydania_)
        {
            Pozycja pozycja = new Katalog

Do katalogu masz dodać pozycję, ale... dodałeś do listy pozycji katalog, który w środku ma własną listę pozycji? Nie brzmi to dla Ciebie dziwnie? W katalogu książek nie masz listy kolejnych katalogów, masz listę książek.

Na pewno Pozycja jest klasą abstrakcyjną? Nie ma żadnej klasy Książka ani nic takiego?

W każdym razie...

Console.WriteLine(pozycja);

No tak, bo teraz po prostu wypisujesz obiekt (jeśli nie ma przeładowanego ToString, to dokładnie do się pokaże). Musisz wypisać zawartość tego obiektu, więc albo wypisując jego pola, albo wywołując na nim jakąś metodę.

---

Tak, jest też klasa książka, czy czasopismo, jednak według zadania abstrakcyjna ma być pozycja. i rzeczywiście nie powinno dziedziczyć, a korzystać z asocjacji, za chwilę to naprawię. Dodałem przeładowanie i teraz działa, bardzo dziękuję.

---

@adrian17, jeśli chodzi o to :

    Pozycja pozycja = new Katalog()

to też wydaje mi się to dziwne, jednak gdy zamiast katalog dodawałem Pozycja to wyświetlał się komunikat, że "nie można stworzyć obiektu klasy abstrakcyjnej lub interfejsu" i coś takiego znalazłem gdzieś w poradnikach.

---

jednak gdy zamiast katalog dodawałem Pozycja

I ma rację, bo o to właśnie chodzi z klasą abstrakcyjną, że nie możesz jej tworzyć.

Ale jak masz funkcję o nazwie "dodaj pozycję do katalogu" to znacznie więcej sensu jest dodać do katalogu książkę lub czasopismo niż katalog.