Probleme mit Arraylist

[adlfjdlaöfjdaös]

Hallo

ich könnte einen Tipp gebrauchen, wie ich die Daten bei AddressBook.addAddress(); speichern kann. Ich hole mir die Daten welche in den Variablen gespeichert sind. Wie kann ich aber die ganzen Daten in die Arraylist übergeben.

MfG

    private void oKBtn_actionPerformed(ActionEvent e) {
      String firstName;
      String lastName;
      String address;
      String email;
      String phoneNumber;
      
      firstName = firstNameTx.getText("");
      lastName = lastNameTx.getText("");
      address = addressTx.getText("");
      phoneNumber = phoneNumberTx.getText("");
      email = eMailTx.getText("");
      
      AddressBook.addAddress(a);
      
      firstNameTx.setText("");
      lastNameTx.setText("");
      addressTx.setText("");
      phoneNumberTx.setText("");
      eMailTx.setText("");
    }

public class AddressEntry {
    
    private String firstName;
    private String lastName;
    private String address;
    private String email;
    private String phoneNumber;
    
    public AddressEntry() {
        super();
    }
    
    
  public void setFirstName(String firstName) {
    this.firstName = firstName;
  }

  public String getFirstName() {
    return firstName;
  }
    
  public void setLastName(String lastName) {
    this.lastName = lastName;
  }

  public String getLastName() {
    return lastName;
  }
    
  public void setAddress(String address) {
    this.address = address;
  }

  public String getAddress() {
    return address;
  }
    
  public void setEmail(String email) {
    this.email = email;
  }

  public String getEmail() {
    return email;
  }
    
  public void setPhoneNumber(String phoneNumber) {
    this.phoneNumber = phoneNumber;
  }

  public String getPhoneNumber() {
    return phoneNumber;
  }
  
  
}

public class AddressBook {
    public AddressBook() {
        super();
    }
    
    private ArrayList<AddressEntry> address = new ArrayList<AddressEntry>();

    public void addAddress(AddressEntry a) {
        address.add(a);
    }
}

 

[Gast2]

AddressBook.addAddress(a);

Damit fügst du doch schon ne Adresse hinzu. Die Variable a hast du vermutlich vorher schonmal irgendwo angelegt. Du musst aber aus den Daten die du aus den Textfeldern gelesen hast jeweils immer einen AddressEntry erstellen, dass du dann hinzufügen kannst.

 

[dsfdfdsfds]

Meinst du so??
Wie kann ich aber die ganze Eigenschaften von AddressEntry in Addressbook.addAddress() einfügen

    private void oKBtn_actionPerformed(ActionEvent e) {
      String firstName;
      String lastName;
      String address;
      String email;
      String phoneNumber;
      
      firstName = firstNameTx.getText("");
      lastName = lastNameTx.getText("");
      address = addressTx.getText("");
      phoneNumber = phoneNumberTx.getText("");
      email = eMailTx.getText("");
      
      AddressEntry.setFirstName(firstName);
      AddressEntry.setLastName(lastName);
      AddressEntry.setAddress(address);
      AddressEntry.setPhoneNumber(phoneNumber);
      AddressEntry.setEmail(email);
      
      AddressBook.addAddress(a);
      
      firstNameTx.setText("");
      lastNameTx.setText("");
      addressTx.setText("");
      phoneNumberTx.setText("");
      eMailTx.setText("");
    }
}

 

[Camill]

Was soll

a

denn sein?

So müsste das aussehen:

		AddressEntry entry = new AddressEntry();
		entry.setFirstName(firstName);
		entry.setLastName(lastName);
		entry.setAddress(address);
		entry.setPhoneNumber(phoneNumber);
		entry.setEmail(email);
		addressBook.addAddress(entry);

Ich würde die ganzen Eigenschaften direkt beim erstellen eines neuen Eintrags als Parameter übergeben, dann spart man sich das ganze

set...

.

 

[Gast2]

Nein, so sicherlich nicht. Du fügst a hinzu, aber änderst die Eigenschaften in irgend nem anderen Objekt.

Du musst aber aus den Daten die du aus den Textfeldern gelesen hast jeweils immer einen AddressEntry erstellen, dass du dann hinzufügen kannst.

      AddressEntry a = new AddressEntry();
      a.setFirstName(firstName);
      a.setLastName(lastName);
      a.setAddress(address);
      a.setPhoneNumber(phoneNumber);
      a.setEmail(email);
      
      AddressBook.addAddress(a);

 

[dfsdfsdf]

Danke!
Jetzt taucht wieder diese Fehlermeldung auf. Was genau ist damit gemeint?

Error(87,18): non-static method addAddress(tw.AddressEntry) cannot be referenced from a static context

 private void oKBtn_actionPerformed(ActionEvent e) {
      
      AddressEntry data = new AddressEntry();
            
      data.setFirstName(firstNameTx.getText());
      data.setLastName(lastNameTx.getText());
      data.setAddress(addressTx.getText());
      data.setPhoneNumber(phoneNumberTx.getText());
      data.setEmail(eMailTx.getText());
      
      AddressBook.addAddress(data);
      
      firstNameTx.setText("");
      lastNameTx.setText("");
      addressTx.setText("");
      phoneNumberTx.setText("");
      eMailTx.setText("");
    }
}

 

[Gast2]

Äh ja... du musst dir natürlich vorher nen AddressBook Objekt anlegen.

 

[ddfdfadf]

Ich blicke bei diesen ArrayListen überhaupt nicht mehr durch. Es heißt ja die werden dynamisch erweitert. Wenn ich aber getAddressCount aufrufe wird immer 1 angezeigt.

Wenn ich den Button drücke ist ja data nur während dieser Funktion gültig. Somit wird das neu angelegt. Bei addAddress werden ja die Daten hinzugefügt.

Wieso werden die Daten immer überschrieben, brauche ich doch ein Array[].

   private void oKBtn_actionPerformed(ActionEvent e) {

        AddressEntry data = new AddressEntry();
        AddressBook addressBook = new AddressBook();

        data.setFirstName(firstNameTx.getText());
        data.setLastName(lastNameTx.getText());
        data.setAddress(addressTx.getText());
        data.setPhoneNumber(phoneNumberTx.getText());
        data.setEmail(eMailTx.getText());

        addressBook.addAddress(data);

        firstNameTx.setText("");
        lastNameTx.setText("");
        addressTx.setText("");
        phoneNumberTx.setText("");
        eMailTx.setText("");

        System.out.println(addressBook.getAddressCount());
    }

    public int getAddressCount() {
            return address.size();
    }

 

[Schandro]

du darfst natürlich nicht bei jedem klick ein neues AddressBook Object erstellen, du musst immer wieder auf dasselbe Object zugreifen was du dir irgendwo als Membervariable merkst.

 

[dfgfdgdfsg]

Meinst du das auf diese Art.

public class Adressbuch{

public Adressbuch(){
super();
static Adressbuch addressBook = new AddressBook();
}
}

In der class MainFrame wollte ich die Membervariable nicht definieren. Auf diese Art wird aber ein Error angezeigt.

 

[Schandro]

In der class MainFrame wollte ich die Membervariable nicht definieren

Und warum nicht? Klingt doch nach ner guten Idee.


Eine statische Instanz von Adressbuch innerhalb von Adressbuch klingt dagegen nach einer grauenvollen Idee. Nicht nur wegen dem Syntaxfehler

 

[sfsfsdfdsf]

Ich dachte MainFrame will ich für die Grafik lassen und die anderen Variablen in die jeweilige Klasse.

Ich weiß es klingt nach einer grauenvollen Idee und es ist falsch. Aber ich dachte ich bräuchte sowas wie eine globale Variable die in class Adressbuch ist.

Es müßte ja dann public und static sein oder ?

 

[Schandro]

Ich dachte MainFrame will ich für die Grafik lassen und die anderen Variablen in die jeweilige Klasse.

Dann nimm eine passendere Klasse und tu da die Membervariable rein. Aber machs auf keinen Fall static, static ist böse.

 

[sdfdsfsdf]

Danke für die Hilfe. Irgendwie verstehe ich nicht ganz diese Klassen

Wieso will er jetzt die Daten nicht speicher.

 private AddressBook addressBook = new AddressBook();

    private void oKBtn_actionPerformed(ActionEvent e) {

        AddressEntry data = new AddressEntry();
        data.setFirstName(firstNameTx.getText());
        data.setLastName(lastNameTx.getText());
        data.setAddress(addressTx.getText());
        data.setPhoneNumber(phoneNumberTx.getText());
        data.setEmail(eMailTx.getText());

        addressBook.addAddress(data);
        addressBook.storeAddress(data);

public class AddressBook {

    private ArrayList<AddressEntry> address = new ArrayList<AddressEntry>();

    public void addAddress(AddressEntry a) {
        address.add(a);
    }
    
  public void storeAddress(AddressEntry add) {
    // now write objects to file
    try {
      FileOutputStream fs = new FileOutputStream("addressbook.dat");
      ObjectOutputStream os = new ObjectOutputStream(fs);
      os.writeObject(add);
    } catch (IOException e) {
      System.err.println("Objects cannot be save");
    }
  }
}

 

[adsfdsfdsf]

Glaube habe den Fehler gefunden

os.writeObject("addressbook.dat");