String aus Klasse X in Klasse Y befördern

[original1909]

Mahlzeit zusammen!

ich hatte mir als kleine Freizeitaufgabe ein JFrame mit einem JDialog erstellt, womit ich für ein Busunternehmen die Touren verwalten kann. Bin Anfänger in Sachen Java und möchte möglichst flott und möglichst sicher mit Java umgehen können.

Es existieren 3 Klassen:

1. "Bustouren" JFrame mit Main-Methode
2. "Dialogneu" beeinhaltet den JDialog um neue Touren hinzufügen zu können
3. "Busunternehmen" beeinhaltet sämtliche Getter- und Settermethoden, sowie die toString()-Methode um die Variablen passend in einen String für die JList (basiert auf einem DefaultListModel) zu bringen.

Mein Problem ist nun, dass ich in der Klasse "Dialogneu" die Daten für eine neue Bustour eingebe um Sie anschließend im JFrame in eine Liste zu werfen. Der String wird in "Busunternehmen" erzeugt und an "Dialogneu" zurückgegeben, jedoch muss ich diesen String nun in die Klasse "Bustouren" bekommen, da ich sie von dort aus in die Liste einbauen möchte.

Ich hoffe mir kann jemand helfen. Hier sind zur Sicherheit und besseren Übersicht nocheinmal die Quelltexte:

Bustouren:

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.lang.*;

public class Bustouren extends JFrame {
  // Anfang Variablen
  
  private DefaultListModel dlm = new DefaultListModel();
  private JList jList1 = new JList(dlm);
  private JScrollPane jScrollPane1 = new JScrollPane();
  private JMenuBar jmb = new JMenuBar();
  private JMenuBar jmbBus = new JMenuBar();
  private JButton jbtNeu = new JButton();
  private JButton jbtaendern = new JButton();
  private JButton jbtschliessen = new JButton();
  JFrame bustouren = this;
  
  // Ende Variablen

  public Bustouren(String title) {
    // Frame-Initialisierung
    super(title);
    addWindowListener(new WindowAdapter() {
      public void windowClosing(WindowEvent evt) { System.exit(0); }
    });
    int frameWidth = 727;
    int frameHeight = 549;
    setSize(frameWidth, frameHeight);
    Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
    int x = (d.width - getSize().width) / 2;
    int y = (d.height - getSize().height) / 2 ;
    setLocation(x, y);
    Container cp = getContentPane();
    cp.setLayout(null);
    // Anfang Komponenten

    jList1.setBounds(32, 96, 633, 393);
    cp.add(jList1);
    jScrollPane1.setBounds(664, 96, 17, 393);
    cp.add(jScrollPane1);
    setJMenuBar(jmb);
    setJMenuBar(jmbBus);
    jbtNeu.setBounds(40, 40, 115, 33);
    jbtNeu.setText("Neue Tour");
    cp.add(jbtNeu);
    jbtNeu.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jbtNeuActionPerformed(evt);

      }
    });

    jbtaendern.setBounds(176, 40, 113, 33);
    jbtaendern.setText("Daten ändern");
    cp.add(jbtaendern);
    jbtaendern.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jbtaendernActionPerformed(evt);
      }
    });

    jbtschliessen.setBounds(528, 40, 129, 33);
    jbtschliessen.setText("Schließen");
    cp.add(jbtschliessen);
    jbtschliessen.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jbtschliessenActionPerformed(evt);
      }
    });

    // Ende Komponenten

    setResizable(false);
    setVisible(true);
  }

  // Anfang Ereignisprozeduren

  public void jbtNeuActionPerformed(ActionEvent evt) {
    Dialogneu dlr = new Dialogneu(bustouren,"Moeller", true);
    
    //Bustouren tour1 = new Bustouren("Tour 1");
    


  }

  public void jbtaendernActionPerformed(ActionEvent evt) {
  //new Aenderndialog();
  }


  public void jbtschliessenActionPerformed(ActionEvent evt) {
         bustouren.dispose();
  }


  // Ende Ereignisprozeduren

  public static void main(String[] args) {

  new Bustouren("Bustouren");
  
 }
}

Dialogneu:

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import busunternehmen.*;

public class Dialogneu extends JDialog {
  // Anfang Variablen
  private JLabel jlbziel = new JLabel();
  private JLabel jlbfahrer = new JLabel();
  private JLabel jlbstrecke = new JLabel();
  private JLabel jlbzeit = new JLabel();
  private JTextField jTextField1 = new JTextField();
  private JTextField jTextField3 = new JTextField();
  private JTextField jTextField4 = new JTextField();
  private String[] jComboBox1Daten = {"Heinz Werner","Klaus Peter","Peter Werner","Susi Sorglos","Paul Panzer","Atze Schröder"};
  private JComboBox jComboBox1 = new JComboBox(jComboBox1Daten);
  private JButton jbtadd = new JButton();
  private JButton jbtesc = new JButton();
  JDialog Dialogneu = this;
  Bustouren tour;
  
    // Ende Variablen

    public Dialogneu(JFrame owner, String title, boolean modal) {
    // Dialog-Initialisierung
    super(owner, title, modal);
    
    addWindowListener(new WindowAdapter() {
      public void windowClosing(WindowEvent evt) { System.exit(0); }
    });
    int frameWidth = 345;
    int frameHeight = 275;
    setSize(frameWidth, frameHeight);
    Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
    int x = (d.width - getSize().width) / 2;
    int y = (d.height - getSize().height) / 2 ;
    setLocation(x, y);
    Container cp = getContentPane();
    cp.setLayout(null);
    // Anfang Komponenten

    jlbziel.setBounds(24, 32, 59, 24);
    jlbziel.setText("Reiseziel:");
    jlbziel.setFont(new Font("MS Sans Serif", Font.PLAIN, 13));
    cp.add(jlbziel);
    jlbfahrer.setBounds(24, 64, 42, 24);
    jlbfahrer.setText("Fahrer:");
    jlbfahrer.setFont(new Font("MS Sans Serif", Font.PLAIN, 13));
    cp.add(jlbfahrer);
    jlbstrecke.setBounds(24, 96, 83, 24);
    jlbstrecke.setText("Reisestrecke:");
    jlbstrecke.setFont(new Font("MS Sans Serif", Font.PLAIN, 13));
    cp.add(jlbstrecke);
    jlbzeit.setBounds(24, 128, 59, 24);
    jlbzeit.setText("Reisezeit:");
    jlbzeit.setFont(new Font("MS Sans Serif", Font.PLAIN, 13));
    cp.add(jlbzeit);
    jTextField1.setBounds(152, 32, 153, 24);
    jTextField1.setText("");
    cp.add(jTextField1);
    jTextField3.setBounds(152, 96, 153, 24);
    jTextField3.setText("");
    cp.add(jTextField3);
    jTextField4.setBounds(152, 128, 153, 24);
    jTextField4.setText("");
    cp.add(jTextField4);
    jComboBox1.setBounds(152, 64, 153, 24);
    cp.add(jComboBox1);
    jbtadd.setBounds(24, 176, 129, 33);
    jbtadd.setText("Hinzufügen");
    cp.add(jbtadd);
    jbtadd.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jbtaddActionPerformed(evt);
        Dialogneu.dispose();

      }
    });

    jbtesc.setBounds(176, 176, 129, 33);
    jbtesc.setText("Abrrechen");
    cp.add(jbtesc);
    jbtesc.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jbtescActionPerformed(evt);
      }
    });

    // Ende Komponenten

    setResizable(false);
    setVisible(true);
  }
  // Anfang Ereignisprozeduren

  //Anfang Ereignisprozeduren
  public String jbtaddActionPerformed(ActionEvent evt) {
  
  String a = jTextField1.getText();
  String b = jTextField3.getText();
  String c = jTextField4.getText();
  Object f = jComboBox1.getSelectedItem();
  
  String gh = Busunternehmen.setListDaten(a,b,c,f);
  return gh;
  }

  public void jbtescActionPerformed(ActionEvent evt) {
         this.dispose();
  }

  // Ende Ereignisprozeduren
}

Busunternehmen:

 package busunternehmen;
 
 public class Busunternehmen{

 String a,b,c;
 Object f;
 
        public Busunternehmen(String a1, String b1, String c1, Object f1) {

        a = a1;
        b = b1;
        c = c1;
        f = f1;

        }
        
  public void getZiel(String a1) {
    this.a = a1;
  }
  
  public void getFahrer(String f1) {
    this.f = f1;
  }

  public void getStrecke(String b1) {
    this.b = b1;
  }
  
  public void getZeit(String c1) {
    this.c = c1;
  }

  public String setZiel() {
    return a;
  }

  public Object setFahrer() {
    return f;
  }
  
  public String setStrecke() {
    return b;
  }
  
  public String setZeit() {
    return c;
  }
  
  public static String setListDaten(String a,String b,String c,Object f) {
  String abc = "Ziel: "+a+", Fahrer: "+f+", Strecke: "+b+" km, Reisezeit: "+c+ " Stunden";
  return abc;
  }


 }

[/quote]

Edit Illuvatar: Codetags repariert.

 

[0x7F800000]

boah, das nennst du übersicht?

statte deine Bustouren-Klasse halt mit entsprechnden gettern/settern/addern oder was auch immer für methoden, und übergebe der anderen klasse (irgendein dialog oder was war das?) im konstruktor die referenz auf eine Bustouren-instanz, dann kannst du da jederzeit alles verändern wie du willst...

 

[Orignal1909]

rofl.. anfänger.. kann ich auch nix für ... ^^

 

[Original1909]

erbitte Tipps wie man sowas umbauen kann, damit ich da alles jederzeit verändern kann wie ich will..

 

[0x7F800000]

Deinem dialog übergibst du im konstruktor nicht irgendein JFrame sondern gleich "Bustouren"

Dialogneu extends JDialog implements ActionListener{
       //referenz abspeichern
     private Bustouren bustouren;
    
    //constructor
    public Dialogneu(Bustouren bt, String title, boolean modal){
        ...
        bustouren=bt;
        ...
    }

    //funktionalität des actionlisteners
    public void actionPerformed(ActionEvent e){
        
    //variablen aus den feldern einlesen, zum string zusammenfügen, und irgendwie in die "jList1" packen
    //musst halt einen entsprechenden adder in Bustouren schreiben...
    bustouren.add((new Busunternehmen(a,b,c,d)).toString());
    }

    JButton hinzufügen=new JButton("Hinzufügen");
    ...
    hinzufügen.addActionListener(this);
}

"toString()" soll eine nicht-statische methode sein, die komplett diese dämliche "setListDaten" ersetzt (diese methode ist kein setter, greift auf keine liste zu und tut nix mit daten, was soll denn der name???)

Weitere anmerkungen:
1)Variablennamen wie "a,b,c,d,f1,,23,g637,jbn452,jList1" usw usw usw usw.... sind unbedingt zu vermeiden, die sagen einem einfach gar nix.

2)Object wird an der einen stelle hoffentlich noch durch einen echten "Fahrer" ersetzt?

3)Keine deutschen begriffe bitte. sieht einfach sehrmerkwürdig aus. Sowas darf man weder der deutschen, noch der englischen sprache antun...

4) zertrümmern einer einzigen methode "public void actionPerformed()" in zwei methoden, von den eine einen komplett unlesbaren namen hat trägt auch nicht zur übersicht bei

ansonsten viel spaß noch, pass auf dass da überprüft wird wie oft die leute reifen wechseln und bremsen testen k?^^

greetz

 

[Original1909]

werde mir mühe geben ^^

ich werde mal mein Glück versuchen, ich glaube sonst kann ich mich einliefern lassen, wenn dass da nich funzt xD

gruß

 

[original1909]

Danke nochmal für die Hilfe. Wenn ich das nun so mache, meckert er, dass die Klasse nicht Abstrakt ist, daher kann ich die einfach so ändern?

public abstract class Dialoghinzufuegen extends JDialog implements ActionListener{

 

[0x7F800000]

meckert er

wer ist genau "er" und was meckert er denn genau?
was soll der kompiler an nicht abstrakten klassen denn auszusetzten haben?? ???:L

public abstract class Dialoghinzufuegen extends JDialog implements ActionListener{

und ich nehme mal an dass "Dialoghinzufügen" einfach die englische übersetzung von "Dialogneu" ist, nicht wahr :roll: ?
Die klasse sollte auf keinen fall abstrakt gemacht werden, bzw was willst du denn davon ableiten?

 

[original1909]

genau, "Dialoghinzufuegen" ist die neue "Dialogneu". Habe nun alles gebaut, jedoch wenn ich "implements ActionsListener" dahinter schreiben, meckert DER COMPILER daran rum, dass die klasse nicht abstrakt ist, und wie schon angemerkt wäre das ja blödsinn. daher habe ich nun erstmal alles zusammengebaut:

public class Dialoghinzufuegen extends JDialog{
  // Anfang Variablen
  private JLabel jlbZiel = new JLabel();
  private JLabel jlbFahrer = new JLabel();
  private JLabel jlbStrecke = new JLabel();
  private JLabel jlbZeit = new JLabel();
  private JTextField jtfZiel = new JTextField();
  private JTextField jtfFahrer = new JTextField();
  private JTextField jtfEntfernung = new JTextField();
  private JTextField jtfZeit = new JTextField();
  private JButton jbtHinzufuegen = new JButton();
  private JButton jbtAbbrechen = new JButton();
  private Bustouren bustouren;
  JDialog hinzufuegen = this;

// ....
// ....

jbtHinzufuegen.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {

      String ziel = jtfZiel.getText();
      String fahrer = jtfFahrer.getText();
      String entfernung = jtfEntfernung.getText();
      String zeit = jtfZeit.getText();
      
      //String erzeugen
      
      bustouren.add((new bus(ziel,fahrer,entfernung,zeit)).toString());

      }
    });

nun regt er sich auf, weil er die Klasse "bus", was meine neue Fachklasse ist, nicht finden kann.

package bus;

public class bus{

  // Anfang Variablen
    String Reiseziel, Fahrer;
    int Strecke;
    double Zeit;
  // Ende Variablen


    public bus (String Reiseziel,String Fahrer,int Strecke,double Zeit) {
      this.Reiseziel = Reiseziel;
      this.Fahrer = Fahrer;
      this.Strecke = Strecke;
      this.Zeit = Zeit;
    }
  // Anfang Ereignisprozeduren

    public String getReiseziel() {
      return Reiseziel;
    }
    
    public String getFahrer() {
      return Fahrer;
    }
    
    public int getStrecke() {
      return Strecke;
    }
    
    public double getZeit() {
      return Zeit;
    }
    
    public void setReiseziel(String Reiseziel) {
      this.Reiseziel = Reiseziel;
    }
    
    public void setFahrer(String Fahrer) {
      this.Fahrer = Fahrer;
    }
    
    public void setStrecke(int Strecke) {
      this.Strecke = Strecke;
    }
    
    public void setZeit(double Zeit) {
      this.Zeit = Zeit;
    }
    
    public String toString(String ziel, String fahrer, int entfernung,double zeit) {
      return "Reiseziel: "+ziel+ ", Fahrer: " +fahrer+ ", Entfernung: "+entfernung+", Reisezeit: "+zeit;
    }
    
  // Ende Ereignisprozeduren

 

[0x7F800000]

1) zum action listener:
ich seh in dem code gar kein "implements ActionListener" wie soll ich denn sagen was daran falsch ist??
was zum henker soll heißen "JDialog hinzufügen=this" wo steht das überhaupt?
warum kann der an jbtHinzufügen gehängte Listener überhaupt auf "bustouren" zugreifen, die muss doch in einer anderen klasse liegen, das darf doch so gar nicht gehen? wird da etwa nicht gemeckert? seltsam...

2) zum bus

Klassennamen schreibt man GROß
was weiss ich warum die klasse bus irgendwoe nicht gefunden wird? das liegt aber bestimmt nicht an der "getFahrer()" methode, also weiß ich nicht warum du die überhaupt alle postest, das interessiert doch keinen...

höchstwahrscheinlich findet er die bus klasse nicht, weil die im anderen, nicht importierten package "bus" liegt, es sollte also heißen

add(new bus.Bus(...));

joah... soviel soweit... :bae:

 

[original1909]

Mahlzeit,

das "JDialog hinzufügen=this" gibt "this" einfach nur einen namen, daher kann ich dann schreiben:

jbtAbbrechen.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        hinzufuegen.dispose();
      }
    });

wenn ich einfach nur this.dispose(); schreibe, dann nörgelt der Compiler.

das implements ActionListener hatte ich im Eifer des Gefechts weggelassen, daher hier der Code:

public class Dialoghinzufuegen extends JDialog implements ActionListener{

//...

}

Folgende Fehlermeldung:

"Dialoghinzufuegen.java:5:8: Dialoghinzufuegen is not abstract and does not override abstract method actionPerformed(java.awt.event.ActionEvent) in java.awt.event.ActionListener"

wenn ich nun unten den String erzeugen will:

bustouren.add((new bus.Bus(ziel,fahrer,entfernung,zeit)).toString());
]

folgende Fehlermeldung:

Dialoghinzufuegen.java:80:29: cannot find symbol
symbol  : class Bus
location: package bus
      bustouren.add((new bus.Bus(ziel,fahrer,entfernung,zeit)).toString());


Hoffe das hilft weiter, diese Fehler dort zu lösen bereitet mir noch Schwierigkeiten.

Gruß, Pascal

 

[0x7F800000]

"Dialoghinzufuegen.java:5:8: Dialoghinzufuegen is not abstract and does not override abstract method actionPerformed(java.awt.event.ActionEvent) in java.awt.event.ActionListener"

Lol? Wozu hast du denn "implements ActionListener" an den klassenkopf geschrieben? etwa damit es schöner aussieht? oder war die idee vielleicht doch aus der klasse einen ActionListener zu machen? Ich gehe doch fest von der zweiten variante aus (rein optisch macht "implements" an sich keinen allzugroßen eindruck).
Damit er nicht meckert musst du halt die actionPerformed() methode implementieren. Also aus deinem namenlosen

jbtHinzufügen.addActionListener(new ActionListener...
...

einfach die actionPerformed() methode rausreissen direkt in Dialoghinzufügen-Klasse einfügen. Dann kannst du auch ganz normal "this" dadrin benutzen statt irgendwelchen finsteren hacks mit this=that=theShitFromOverThere etc...

und zum Bus...

nun ja bist du dir sicher dass du die klasse ordnungsgemäß umbenannt hast?...

 

[Original1909]

this=that=theShitFromOverThere etc...

ROFL!

nein nein, es sollte ein ActionListener aufgerufen werden, wenn Button Hinzufügen gedrückt wird. Hat sich nun aber erledigt..

Ich werde mir das mit der umbenannten Klasse heute abend zu gemühte führen, anschließend berichten.

Danke schonmal für die Hilfe

Gruß, Pascal