C# obiektowość zadanie

Question

Czesc, mam zadanie do wykonania, wklejam odrazu jego tresc:

1. Zdefiniować klasę Books. Utworzyć 10-elementową tablicę obiektów klasy do przechowywania in- formacji o książkach (author, title, year). Klasa musi zawierać metody Set i Get dla wszystkich pól

   danych oraz funkcje wczytującą i wyświetlającą dane. Program powinien w pętli wpisywać dane do

   tablicy obiektów i w kolejnej pętli wypisać informacje dla wszystkich pozycji na ekranie.

Dopiero zaczynam z nauka obiektowosci, dlatego zadanie wykonalem, jednak nie jestem pewien czy dobrze, na pewno da sie lepiej. Nie do konca rozumiem tresc zadania. Gdyby ktos mogl rzucic okiem na moj kod i powiedziec czy jest ok i pokrywa sie z trescia czy moze cos poprawic, moze ktos doradzi jak ulepszyc zadanie. Z gory dzieki.

using System;

namespace cwiczenie
{
    class Program
    {
        static void Main(string[] args)
        {
            Books zad = new Books("Noname","No title",0);

            zad.Read();
            zad.Print();


            Console.ReadKey();
        }
    }

    class Books
    {
        private object[] tab = new object[10];
        private string author;
        private string title;
        private int year;
        public string GetAuthor() { return author; }
        public string GetTitle() { return title; }
        public int GetYear() { return year; }
        public void SetAuthor(string a) { author = a; }
        public void SetTitile(string t) { title = t; }
        public void SetYear(int y) { year = y; }

        public Books(string a, string t, int y)
        {

            author = a;
            title = t;
            year = y;
        }

        public void Read()
        {
            for (int i = 0; i < 1; i++)
            {
                Console.Write("Podaj autora ksiazki: ");
                tab[0] = Console.ReadLine();
                Console.Write("Podaj tytul ksiazki: ");
                tab[1] = Console.ReadLine();
                Console.Write("Podaj rok wydania ksiazki: ");
                tab[2] = Console.ReadLine();
                Console.Write("Informacje: ");
                tab[3] = Console.ReadLine();
                Console.Write("Informacje: ");
                tab[4] = Console.ReadLine();
                Console.Write("Informacje: ");
                tab[5] = Console.ReadLine();
                Console.Write("Informacje: ");
                tab[6] = Console.ReadLine();
                Console.Write("Informacje: ");
                tab[7] = Console.ReadLine();
                Console.Write("Informacje: ");
                tab[8] = Console.ReadLine();
                Console.Write("Informacje: ");
                tab[9] = Console.ReadLine();
            }

            
        }

        public void Print()
        { 
            for(int i=0; i<1;i++)
            {
                Console.WriteLine("Autor: " + tab[0]);
                Console.WriteLine("Tytul: " + tab[1]);
                Console.WriteLine("Rok wydania: " + tab[2]);
                Console.WriteLine("Dodatkowe informacje: " + tab[3]);
                Console.WriteLine("Dodatkowe informacje: " + tab[4]);
                Console.WriteLine("Dodatkowe informacje: " + tab[5]);
                Console.WriteLine("Dodatkowe informacje: " + tab[6]);
                Console.WriteLine("Dodatkowe informacje: " + tab[7]);
                Console.WriteLine("Dodatkowe informacje: " + tab[8]);
                Console.WriteLine("Dodatkowe informacje: " + tab[9]);

            }

        }
       

        
       
    }
    
}

 

Comments

Udalo mi sie zmienic na cos takiego:

using System;

namespace cwiczenie
{
    class Program
    {
        static void Main(string[] args)
        {
            Books zad1 = new Books();
            zad1.Author = "Noname";
            zad1.Title = "Notitle";
            zad1.Year = "0";
            zad1.Read();
            zad1.Print();

            
            Console.ReadKey();
        }
    }

    class Books
    {
        private string _author;
        private string _title;
        private string _year;
        public string Author { get { return _author; } set { _author = value; } }
        public string Title { get { return _title; } set { _title = value; } }
        public string Year { get { return _year; } set { _year = value; } }
        Books[] tab1 = new Books[3];

        public void Read()
        {
            Console.WriteLine("Podaj Autora,Tytul oraz rok wydania ksiazki: ");
            for(int i=0; i<tab1.Length; i++)
            {
                tab1[i] = new Books();
                Console.Write("Podaj autora "+(i+1)+" ksiazki: ");
                tab1[i]._author = Console.ReadLine();
                Console.Write("Podaj tytul " + (i + 1) + " ksiazki: ");
                tab1[i]._title = Console.ReadLine();
                Console.Write("Podaj rok produkcji " + (i + 1) + " ksiazki: ");
                tab1[i]._year = Console.ReadLine();
            }
        }

        public void Print()
        {
            Console.WriteLine("Wyswietlam: ");
            for(int i=0; i<tab1.Length; i++)
            {
                Console.WriteLine("Ksiazka nr."+(i+1)+" Autor: "+tab1[i]._author+" Tytul: "+tab1[i]._title+" Rok wydania: "+tab1[i]._year);
            }
        }
       
    }
    
}

Byc moze troche lepiej, nie do konca jeszcze rozumiem zastosowanie tych properties, moglbym prosic o objasnienie, gdy nie wpisujac nic do tablicy na koncu wyswietlalo mi te podstawowe wartosci z set'erow np."Noname" itd... Z gory dziekuje za odpowiedz !

 

---

Wciąż trochę nie tak.

Books[] tab1 = new Books[3];

Książka nie ma trzech książek. Książka ma informacje o swoim autorze, tytule i roku - i tyle. Podobnie, jak samochód nie ma w sobie panelu sterującego dziesięcioma samochodami. Wszystkie pętle po książkach powinny znajdować się poza klasą Książka - po prostu w main()ie.

A znowu na boku.

private string _author;
public string Author { get { return _author; } set { _author = value; } }

To można zapisać dosłownie

public string Author { get; set; }

To automatycznie jest jednocześnie polem, jak i getterem/setterem.

Answer 1

Trochę inaczej :)

Książka nie ma 10 obiektów. To książek ma być 10. Książka ma tylko autora, tytuł i rok. W main()ie zrób tablicę 10 książek, po czym w pętli wczytaj dane do każdej książki i w osobnej pętli wypisz informacje każdej książki osobno.

(EDIT: a przynajmniej tak to rozumiem. Ale trochę dziwi mnie czemu klasa ma się nazywać Books a nie Book, więc może się mylę.)

A swoją drogą:

        public string GetAuthor() { return author; }
        public void SetAuthor(string a) { author = a; }

To nie Java, tak się nie pisze get/set :) Poczytaj o properties.

class Book
{
    public string Author { get; set; }