Arraylist i wyświetlanie jego elementów

Arraylist i wyświetlanie jego elementów
BA
  • Rejestracja:prawie 13 lat
  • Ostatnio:około 8 lat
  • Postów:128
0

Witam. Ćwiczę sobie Arraylisty i mam takie pytanie.

Kopiuj
package zad1;

import java.util.ArrayList;

public class Wykladowca {

	private String imie;
	private String nazwisko;
	private String przedmiot;

	public Wykladowca(String imie, String nazwisko, String przedmiot) {
		this.imie = imie;
		this.nazwisko = nazwisko;
		this.przedmiot = przedmiot;
	}

	ArrayList<Wykladowca> wykladowcy = new ArrayList<Wykladowca>();

	public void DodajWykladowce() {
		wykladowcy.add(new Wykladowca(imie, nazwisko, przedmiot));
	}

	public void UsunWykladowce() {
		wykladowcy.remove(wykladowcy);
	}

	public void WyswietlWykladowce() {

		System.out.println(wykladowcy.get(0));

	}

}

i main:

Kopiuj
package zad1;

public class Dziekanat {

	public static void main(String[] args) {
		
		Wykladowca wykladowca = new Wykladowca("Michal","Kichal","wf");
		wykladowca.DodajWykladowce();
		wykladowca.WyswietlWykladowce();
	}

}

Co robie źle że nie mogę wyświetlić wykładowcy? Tylko zwraca mi coś w stylu zad1.Wykladowca@19e7ce87

dodanie znacznika `` - furious programming


...
edytowany 1x, ostatnio: flowCRANE
BA
  • Rejestracja:prawie 13 lat
  • Ostatnio:około 8 lat
  • Postów:128
0

Próbowałem też tak

Kopiuj
public void WyswietlWykladowce() {

		//System.out.println(wykladowcy.get(0));
		Wykladowca imie = wykladowcy.get(0); 
		System.out.println("Imie: "+imie);

		
	}

...
n0name_l
  • Rejestracja:ponad 12 lat
  • Ostatnio:ponad 4 lata
  • Postów:2412
1

Co robie źle

#Nie myslisz. Dlaczego wykladowca ma liste wykladowcow? O.o
#Nie masz wlasnej implementacji toString, wiec jest uzyta implementacja z klasy Object, ktora wyglada jak wyglada.

BA
  • Rejestracja:prawie 13 lat
  • Ostatnio:około 8 lat
  • Postów:128
0

Ok więc jak zmienie:

Kopiuj
ArrayList<String> wykladowcy = new ArrayList<String>();

a w metodzie

Kopiuj
public void DodajWykladowce() {
		wykladowcy.add(imie,nazwisko,przedmiot);
	}

mi to nie zadzaiala bo bede mogl dodac tylko samo imie. I właśnei się zastanawiam jak to ominąć abym mógł dodać kilka pól z konstruktora.


...
n0name_l
  • Rejestracja:ponad 12 lat
  • Ostatnio:ponad 4 lata
  • Postów:2412
2

Hm. Programowanie nie polega na losowym dobieraniu ciagu znakow z mysla "moze sie tym razem uda.", tylko na mysleniu. Ale zeby myslec, trzeba miec tez kapke wiedzy, wiec do dziela -> http://docs.oracle.com/javase/tutorial/java/concepts/index.html

edytowany 2x, ostatnio: n0name_l
BA
  • Rejestracja:prawie 13 lat
  • Ostatnio:około 8 lat
  • Postów:128
0

Dobra poradziłem sobie wstępnie.

Zmieniłem przykład żeby zacząć od nowa.

Zrobiłem klasę osoba z której dziedcziczy mi pracownik i klienta. Oprocz tego beda klasy faktura o raz towar.

Arrayliste tworze w nastepujacy sposob

Kopiuj
ArrayList<Osoba> osoby = new ArrayList<Osoba>();
		osoby.add(new Pracownik("Michal", "Michalski"));
		osoby.add(new Klient("Andrzej", "Michalski"));

no i na razie proste wyświetlanie:

Kopiuj
 for(int i=0; i< osoby.size(); i++) 
         { 
                 System.out.println(""+osoby.get(i).getImie()+" "+osoby.get(i).getNazwisko()); 
         } 

O to chodziło czy jesz\ze coś nie obiektowo ?


...
n0name_l
  • Rejestracja:ponad 12 lat
  • Ostatnio:ponad 4 lata
  • Postów:2412
1

Juz lepiej. Poczytaj jeszcze o toString zamiast tego wyswietlania z getterami. A i jeszcze zamiast tego for, poczytaj o alternatywnej skladni do iterowania po calej kolekcji for(T elem : collection) ...

edytowany 1x, ostatnio: n0name_l
BA
  • Rejestracja:prawie 13 lat
  • Ostatnio:około 8 lat
  • Postów:128
0

foreach znam :D Na razie próbowałem czy będzie coś z tego co napisałem. Mam jeszcze pytanie. Czy w miarę poprawnie jest takie przesłonięcie

metoda w klasie osoba:

Kopiuj
public int getNumer() {

		return numer;
	}

a potem coś takiego mam np. w pracowniku:

Kopiuj
@Ovveride
	public int getNumer(){
		
		return numer_pracownika;
	}

Oczywiście na razie pomijając to że mam zmienić na

Kopiuj
toString

Zanim odpowiedziałeś ogarnąłem sobie toString()

Kopiuj
@Override
	public String toString() {

		return "Imie: " + imie + " \nNazwisko: " + nazwisko + "\nNumer pracownika: " + numer_pracownika ;

	}

i podobnie jest w kliencie

natomiast w main tak wyświetlam:

Kopiuj
		ArrayList<Osoba> osoby = new ArrayList<Osoba>();
		osoby.add(new Pracownik("Michal", "Michalski", 1));
		osoby.add(new Klient("Andrzej", "Michalski", 2));

		for(Osoba o : osoby){
			System.out.println(o);
			
		}

...
BA
oczywiście numery pracownika jak i klienta zwraca poprawne
n0name_l
Chyba jest okej. Z tym numerem jeszcze nie wiem zbytnio o co chodzi. Jesli masz go we wszystkich klasach, to wrzuc pole do bazowej i zrob jednego gettera, szkoda kod przepisywac w kolko. + Prawdopodobnie jakbys wrzucil calosc to ktos madrzejszy ode mnie by sie wypowiedzial, bo tak to nie ma raczej odnosnie czego.
BA
Numer będzie potrzebny do faktury gdzie bedzie sie znajdowac takie id klienta i id sprzedawcy
BA
Głownie chodziło mi o implementacje z tym toString() faktycznie ułatwia pracę zamiast się bawić get() i set(). Reszty spokojnie jestem pewien i sobie doczytam.
0

Jeżeli chcesz "wydrukować" jakiś obiekt, a nie chce Ci się/ nie masz czasu/możliwości tworzyć metody toString możesz użyć ToStringBuilder.reflectionToString(Object object) z pakietu commons lang.

edytowany 1x, ostatnio: flowCRANE
BA
  • Rejestracja:prawie 13 lat
  • Ostatnio:około 8 lat
  • Postów:128
0

Panowie a jak pobrać do klasy Faktura w której sobie utworze takie pola:

Kopiuj
int numer_faktury;
	String imie_sprzedawcy;
	String nazwisko_sprzedawcy;
	String imie_klienta;
	String nazwisko_klienta;
	String data_wystawienia;

dane z klas Sprzedawca i Klient które dziedziczą po klasie Osoba?

Jeśli dałbym w ten sposób

Kopiuj
String imie_sprzedawcy = Sprzedawca.imie;

to wtedy pole imie musi być statyczne co spowoduje że nie będzie działać poprawnie dodawanie do HashSet.

Proszę o naprowadzenie

//edit zmienilem arraylist na hashset ponieważ chciałem pobawić się z eliminacją duplikatami.


...
edytowany 1x, ostatnio: balu
bogdans
Moderator
  • Rejestracja:prawie 17 lat
  • Ostatnio:prawie 5 lat
2

Najprościej to zmienić definicje klasy Faktura:

Kopiuj
    int numer_faktury;
    Sprzedawca sprzedawca;
    Klient klient;
    GregorianCalendar data_wystawienia;

To smutne, że głupcy są tak pewni siebie, a ludzie mądrzy - tak pełni wątpliwości. Bertrand Russell
edytowany 1x, ostatnio: bogdans
BA
  • Rejestracja:prawie 13 lat
  • Ostatnio:około 8 lat
  • Postów:128
0
bogdans napisał(a):

Najprościej to zmienić definicje klasy Faktura:

Kopiuj
    int numer_faktury;
    Sprzedawca sprzedawca;
    Klient klient;
    GregorianCalendar data_wystawienia;

Tak tylko wtedy wykonanie tego:

Kopiuj
@Override
	public String toString() {

		return "Imie sprzedawcy: " + pracownik.imie
				+ "\n Nazwisko sprzedawcy: " + pracownik.nazwisko
				+ " \nImie klienta: " + klient.imie + " \nNazwisko klienta: "
				+ klient.nazwisko+ "\n";
	}

da NullPointerException


...
edytowany 1x, ostatnio: balu
bogdans
Moderator
  • Rejestracja:prawie 17 lat
  • Ostatnio:prawie 5 lat
0

Bo jeszcze musisz napisać odpowiedni konstruktor w klasie Faktura.


To smutne, że głupcy są tak pewni siebie, a ludzie mądrzy - tak pełni wątpliwości. Bertrand Russell
BA
Klasa wewnetrzna no tak ;D Dzieki wielkie
bogdans
O jakiej klasie wewnętrznej Ty piszesz?
bogdans
Moderator
  • Rejestracja:prawie 17 lat
  • Ostatnio:prawie 5 lat
1

Ja nie widzę żadnej klasy wewnętrznej. Konstruktor może wyglądać np. tak (jeżeli numeracja faktur jest automatyczna):

Kopiuj
public class Faktura
{
    private int numer_faktury;
    private Sprzedawca pracownik;
    private Klient klient;
    private GregorianCalendar data_wystawienia;
    private static int ostatniNumer = 0;
    public Faktura(Sprzedawca pracownik,Klient klient,GregorianCalendar data_wystawienia)
    {
         this.pracownik = pracownik;
         ...
         ostatniNumer++;
         this.numer_faktury = ostatniNumer;
    }
    ...
}

To smutne, że głupcy są tak pewni siebie, a ludzie mądrzy - tak pełni wątpliwości. Bertrand Russell
BA
  • Rejestracja:prawie 13 lat
  • Ostatnio:około 8 lat
  • Postów:128
0

No właśnie bo inaczej pomyślałem niż ty napisałeś. Ok tak by to załatwiło sprawę, numer faktury na razie dałem pobierany z konstruktora , ale faktycznie lepszym będzie "automatyczna" numeracja.


...

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.