GUI: ArrayList anzeigen funktioniert nicht

[PrivateVoidFinal]

Hallo, ich habe folgendes Problem:
Ich programmiere eine kleine Personalverwaltungsapplikation. Diese soll auf Klick die ArrayList mit Personal auf der GUI ausgeben. Leider klappt das irgendwie nicht, kann mir jemand sagen, wo der Fehler liegt/was ich verändern müsste?

http://pastebin.com/szRfWdX4

 

[Joose]

Leider klappt das irgendwie nicht, kann mir jemand sagen,

Was klappt nicht? Was hast du bisher versucht?
"klappt nicht" ist keine ausreichende Fehlerbeschreibung. Passieren Fehler: Compilefehler oder werden zur Laufzeit Exceptions geworfen? Wenn ja welche? (inkl StackTrace)

Du kannst Code auch im Forum posten mittel Code-Tags
[code=java] ... dein code ...[/code]

Zu deinem Code: Die Variable "xx" ist lokal in der main Methode deklariert, sprich auch nur dort sichtbar.
Dort wo du sie brauchst deklarieren und initialisieren.

 

[PrivateVoidFinal]

Ok, here we go:

import java.util.ArrayList;
import java.util.List;
import java.util.ListIterator;
import java.util.stream.*;
import java.lang.String;

public class HumanResources {  
    public  List<Employee> hr = new ArrayList<>();
     
    public void init()  {
        hr.add(new Employee("Roland", 39495, 60000));
        hr.add(new Employee("Beate", 49595, 30000));
        hr.add(new Employee("Hermann", 94959, 50000));
    };
  
    public void print1(){
        for (Employee p : hr){
            p.print();
        }
    }
  
    public void print2(){
        for (int i = 0; i < hr.size(); i++){
            hr.get(i).print();
        }
    }
  
    //using the ListIterator
    public void print3 () {
        for (ListIterator<Employee> li = hr.listIterator();              
                                      li.hasNext();    )
            li.next().print();
    }
  
    //using streams technique
    public void print4 () {
        hr.stream().forEach(p -> p.print());
        hr.stream().forEach (Employee :: print);
    }
     
    public void dispense() {
        print1();
        for (Employee c : hr){
            c.increase_salary(100);
            hr.stream().forEach((p) -> p.increase_salary(100));
            print1();
          
        }
    }
  
    public static void main (String [] args){
        HumanResources pa = new HumanResources();
        pa.init();
        pa.dispense();
        pa.toString();
  
        System.out.println(pa);
    }
}

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JTextArea;

public class MyJFrameButton extends JFrame {  
    private final JButton schalter = new JButton("Show Employees");
    private final JLabel anzeige = new JLabel(" ");
    private final JLabel anzeige2 = new JLabel(" ");  
  
public MyJFrameButton (String kopfZeile){  
    super(kopfZeile);
    add(schalter, java.awt.BorderLayout.NORTH);
    add(anzeige, java.awt.BorderLayout.CENTER);
    add(anzeige2, java.awt.BorderLayout.SOUTH);
    schalter.addActionListener(ActionEvent -> anzeige.setText("Our company has the following employees"));
    HumanResources xx = new HumanResources();
    xx.init();
    xx.dispense();
    xx.toString();
    anzeige2.setText(xx);
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    setSize(500, 200);
    setVisible(true);
}

public static void main (String [] args){
    javax.swing.SwingUtilities.invokeLater(
            () -> new MyJFrameButton("Siemens AG"));  
} 
}

Ich will auf der GUI mittels anzeige2 die ArrayList anzeigen lassen. Nun habe ich das Objekt nicht in der Main, sondern sogar im Konstruktor von MyJFrameButton erstellt, was erstmal keine Fehlermeldungen ausgibt. Jedoch kommt bei dem Aufruf mit setText (was ja scheinbar die Methode sein soll, mit der ich das auf der GUI ausgeben kann) die Fehlermeldung:
"The method setText(String) in the type JLabel is not applicable for the arguments (Human Resources)".

 

[krgewb]

xx.toString();

Die Zeile hat keine Wirkung. Du musst es schon irgendwo hinterlegen. Zum Beispiel so:

String s  = xx.toString();
anzeige2.setText(s);

 

[Joose]

Is auch logisch die Methode setText erwartet einen String als Parameter, du übergibst ein HumanResources Objekt.

Vielleicht solltest du uns mal sagen was du dir erwartest dass auf dem Label draufsteht. Die Daten aller Employees aus dem xx Objekt?

Wo baust du dir einen String mit diesen Daten zusammen?
Deine ganzen print Methode in HumanResources machen nichts anderes als von allen Employees die print Methode aufzurufen, nirgends werden diese Daten zusammen gehängt oder ähnliches.

Die Methode "dispense" schaut überhaupt komisch aus, da du in der Schleife noch mal einen Ausdruck stehen hast der das selbe macht wie die Schleife in der er steht.

 

[Joose]

Zum Beispiel so:

String s  = xx.toString();
anzeige2.setText(s);

Du hast Recht das er das Ergebnis in einer String Variable speichern muss wenn er damit arbeiten will.
Es führt ihn aber nicht zum Ziel wenn er das Ergebnis von "toString" eines HumanResources Objekt auf das JLabel schreibt. Da so nur die Speicheraddresse des Objektes angezeigt wird.