Fragen zur Implementierung eines Adressbuches

[SmittyJoe]

Ich habe aus einem Klassendiagramm eine Java-Implementierung "Adressbuch" angefertigt!

Zunächst die beiden Klassen Adressbuch und Adresse:

Adressbuch:

import java.util.ArrayList;

public class Adressbuch {
	
	 private ArrayList<Adresse> buch = null;
	    
		public Adressbuch(){
	        buch = new ArrayList<Adresse>();
	    }
	    
		public void einfuegen(Adresse adresse){
	        buch.add(new Adresse("Marvin","Wildman","Blastreet",9,12345,"Blatown"));
	        buch.add(new Adresse("Anne","Wildman","Woodstreet",10,6789,"Exampletown"));
	        buch.add(new Adresse("William","Wildman","Eaglestreet",11,73975,"Blubvalley"));
	    }
		
		
		public void sucheNachname(String nachname){
			for(Adresse s: buch){
				if("Green".equals(nachname)){
				System.out.println(s);
				}
			}
				
		}

	    public void sucheVorname(String vorname){
	    	for(Adresse s: buch){
				if("Anne".equals(vorname)){
				System.out.println(s);
				}
			}
	    }

		public String toString(){
			return buch.toString();
			
		}

}

Adresse:

public class Adresse {
	
	public static String vorname;
    public static String nachname;
    public static String strasse;
    public static int hausnr;
    public static int plz;
    public static String ort;
   
    

	public Adresse(String vorname, String nachname, String strasse, int hausnr, int plz,String ort){
		this.vorname = vorname;
		this.nachname = nachname;
		this.strasse = strasse;
		this.hausnr = hausnr;
		this.plz = plz;
		this.ort = ort;
	}



	public String toString(){
    	return ""+vorname +nachname+strasse+hausnr+plz+ort;
    }

}

Fragen:

1.Kann man das Klassendiagramm wie oben aufgezeigt "besser" implementieren? Enthält es grobe Fehler?
2.Wie kann ich die Methoden "sucheVorname" und "sucheNachname" so implementieren das ich den Suchbegriff irgendwie einfuegen kann und dieser gesucht wird anstelle der Suche des Begriffes innerhalb der if-Anweisung?
3.Sind Konstruktor und toString-Methode richtig implementiert?

 

[Ruzmanz]

1.Kann man das Klassendiagramm wie oben aufgezeigt "besser" implementieren? Enthält es grobe Fehler?

- Die Liste mit den Adressen sollte so lauten, wie im UML-Diagramm angeben wurde. "adressen"
- Auf keinen Fall sind die Variablen in Adresse static, da das ein Logikfehler ist. Zudem wären dann die Variablen im UML-Diagramm unterstrichen.
- Die Konstruktoren sind ok. Die Methoden sind alle falsch implementiert (Logikfehler), aber entsprechen dem UML-Diagramm.

Vielleicht solltest du dir BlueJ zulegen und die Methoden manuell nacheinander austesten. Prinzipell gehst, du schon den richtigen Weg, aber es ist trotzdem nicht richtig ... Bsp:

return ""+vorname +nachname+strasse+hausnr+plz+ort;

Liefert die Ausgabe: "MarvinWildmanBlastreet912345Blatown". Das will doch kein Mensch lesen

 

[geqoo]

Oje :autsch:

Hier erst einmal die richtige einfügen-Methode:

public void einfuegen(Adresse adresse) {
  buch.add(adresse);
}

Natürlich wird das Konstrukt dann so aufgerufen, aber nicht direkt implementiert ...

Adresse adresse = new Adresse("Marvin", "Wildman", "Blastreet", 9, 12345, "Blatown");
adressbuch.einfuegen(adresse);

Die korrekte sucheNachname Methode:

public Adresse sucheNachname(String nachname) {
  for(Adresse a: buch) {
    if (a.nachname.equals(nachname)) {
      return a.nachname();
    }
  }
}

Aus Gründen der Kapselung sollte zwar in der Klasse Adresse noch eine Getter-Methode für Vor- und Nachname sein, aber das will das Klassendiagramm anscheinend nicht haben.

[edit]Kleinigkeit am Diagramm übersehen und geändert[/edit]

 

[SmittyJoe]

- Die Liste mit den Adressen sollte so lauten, wie im UML-Diagramm angeben wurde. "adressen"
- Auf keinen Fall sind die Variablen in Adresse static, da das ein Logikfehler ist. Zudem wären dann die Variablen im UML-Diagramm unterstrichen.
- Die Konstruktoren sind ok. Die Methoden sind alle falsch implementiert (Logikfehler), aber entsprechen dem UML-Diagramm.

Vielleicht solltest du dir BlueJ zulegen und die Methoden manuell nacheinander austesten. Prinzipell gehst, du schon den richtigen Weg, aber es ist trotzdem nicht richtig ... Bsp:

return ""+vorname +nachname+strasse+hausnr+plz+ort;

Liefert die Ausgabe: "MarvinWildmanBlastreet912345Blatown". Das will doch kein Mensch lesen

Danke für deine Hilfe! Ich habe es ohne groß nachzudenken einfach implementiert ohne genau auf kleine Details im Klassendiagramm zu achten.

Dann müsste es dem Diagramm nach alles wo "buch" steht "adressen" lauten,oder irre ich mich?
Und wenn ich die Variablen als nicht-static implementiere, beziehen sie sich ja auf Objekte, welche ich erzeuge. Doch daraus folgt ja eine Änderung im Konstruktor,richtig?

 

[geqoo]

Wie du dein Adressbuch intern nennst, ist im Grunde egal.
Das schreibt das Klassendiagramm nicht vor.

 

[SmittyJoe]

Ist es mir generell erlaubt, wenn in einem Klassendiagramm keine Setter-und Getter Methoden in einer Klasse angegeben sind, diese zu implementieren? Den der Gedanke ist mir auch aufgekommen, habe aber aus Unwissenheit weggelassen und alles direkt implementiert! Danke nochmal für die Aufklärung

 

[geqoo]

Ob es erlaubt ist, weiß ich aus dem Stehgreif gerade gar nicht. Ich denke aber schon, da man Getter und Setter weglassen kann. Diese Information kann man ja aus den Variablen beziehen.
Generell braucht man sie nicht zu implementieren, solange sie keine spezielle Funktion erfüllen.
Du brauchst das aber auch nicht machen.

Wenn die Variablen im Klassendiagramm mit einem Plus gekennzeichnet sind, dann macht man sie eben public. Dann kann einfach jeder darauf zugreifen, anstatt den Zugriff mittels private auf Getter und Setter zu beschränken.

Genau das mit dem nicht-static ist der Sinn der Sache. Ein Adressbuch soll mehrere Adresse haben können und nicht nur eine. Der Konstruktor braucht dafür nicht verändert werden.

 

[Ruzmanz]

Wie du dein Adressbuch intern nennst, ist im Grunde egal.
Das schreibt das Klassendiagramm nicht vor.

Sofern die Rolle angeben ist und es keinen vernünftigen Grund gibt diese umzubenennen, sollte sie genau so im Quellcode heißen. Ich kenne das so aus der Literatur (z.B. UML 2 von Christoph Kecher). Zudem macht es Sinn, da beim Reverse Engineering das gleiche Klassendiagramm generiert wird.

 

[SmittyJoe]

Ich habe jetzt die Klassen neu implementiert,doch spuckt es mir Fehlermeldungen ohne Ende.

Schritt für Schritt Setter und Getter für Vor-und Nachname gleichermaßen implementiert:

public void setVorname(String vorname) {
		this.vorname = vorname;
	}

public String getVorname() {
		return vorname;
	}

Ja und bei beiden Suchmethoden ist in den Zeilen:

if(a.nachname.equals(nachname)){
                    return a.nachname();
                    }

if(a.vorname.equals(vorname)){
                    return a.vorname();
                    }

Kommen bei den return-Anweisungen Fehlermeldungen?

Und zur eigenen Kontrolle nochmal die Variablen der Klasse Adresse:

public String vorname;
	public String nachname;
	public String strasse;
	public int hausnr;
	public int plz;
	public String ort;

 

[calousy]

entweder
a.getNachname();
oder
a.nachname; (OHNE Klammern; das kannst du aber nur machen, da die Variablen public sind)

ich würde die Variablen private statt public machen, dann machen auch die get/set-Methoden Sinn (Kapselung).


PS: seh gerade das Klassendiagramm... laut dem gibt's keine get-/set-Methoden, sondern nur public Variablen ("+" vor dem Variablenname)

 

[SmittyJoe]

entweder
a.getNachname();
oder
a.nachname; (OHNE Klammern; das kannst du aber nur machen, da die Variablen public sind)

ich würde die Variablen private statt public machen, dann machen auch die get/set-Methoden Sinn (Kapselung).

Das liegt daran dass mit void nix zurückgeliefert werden kann. Muss also entscheiden wie die Methoden implementiert werden sollen. An den Settern und Gettern liegts nicht.

Ich muss mich bei der Implementierung an das Klassendiagramm richten, d.h Variablen public,normal würde ich sie ebenfalls private setzen.

 

[calousy]

wie mit void?
deine returns sind falsch. entweder über getNachname() oder einfach über den .variablenname, aber dann ohne Klammern am Ende. Oder hast du eine Methode String nachname()?

Dein Klassendiagramm hat keine Getter und Setter, also lass die weg. Noch dazu sind die überflüssig, da deine Variablen eh public sind.

 

[SmittyJoe]

Beide Methoden hier nochmal,hab versehentlich void drinne gelassen,was an der Situation wenig ändert:

public Adresse sucheNachname(String nachname){
                for(Adresse a: buch){
                    if(a.nachname.equals(nachname)){
                    return a.getNachname();
                    }
                }
                   
            }
     
            public Adresse sucheVorname(String vorname){
                for(Adresse a: buch){
                    if(a.vorname.equals(vorname)){
                    return a.getVorname();
                    }
                }
            }

Irgendwas versäume ich hier...!

 

[calousy]

Fehlermeldungen sind immer hilfreich aber der Rückgabewert stimmt nicht. Du gibst einen String und nicht das Adresse-Objekt zurück. Laut Klassendiagramm sollst du auch die Adresse zurückgeben (genauer gesagt ein Array... also mehrere Adressen; kann ja sein, dass es mehrere Adressen für einen Namen gibt...).
Und du brauchst einen Rückgabewert, wenn du nichts findest!

 

[SmittyJoe]

Fehlermeldungen sind immer hilfreich aber der Rückgabewert stimmt nicht. Du gibst einen String und nicht das Adresse-Objekt zurück. Laut Klassendiagramm sollst du auch die Adresse zurückgeben (genauer gesagt ein Array... also mehrere Adressen; kann ja sein, dass es mehrere Adressen für einen Namen gibt...).
Und du brauchst einen Rückgabewert, wenn du nichts findest!

Wie meinst du das? Und wie sähe es aus? Etwas verwirrt

 

[calousy]

hier mal eine Adresse

public Adresse sucheNachname(String nachname){
    for(Adresse a : buch){
        if(a.nachname.equals(nachname)){
           return a; // gefunden -> abbrechen & a (Adresse) zurückgeben
        }
    }
    return null; // alles durch, nix gefunden, null zurückgeben
}
     
public Adresse sucheVorname(String vorname){
  for(Adresse a : buch){
    if(a.vorname.equals(vorname)){
      return a;
    }
  }
  return null;
}

Es kann ja aber sein, dass es mehrere gleiche Nachnamen gibt, aber mit unterschiedlicher Adresse. Du musst also ein Array erstellen, die Adressen reinpacken und dann das Array zurückgeben....

 

[SmittyJoe]

Also ich implementiere in Adresse ein Array

public String [] adressen = new String[100];

wobei ich nicht weiss, ob es hier nicht so gut ist direkt die Anzahl im Array zu nennen.
Über die Main würde ich nun die Daten eingeben,diese dann in die Methode einfuegen hingeben.
Dort wird es dann in die ArrayList buch hinzugefügt, wo es dann von den Methoden überprüft wird.
Den Rest hast du gerade geschildert. Doch wie funktioniert das, dass das Array irgendwie etwas zurückgibt?!

 

[calousy]

Nein nein... Deine Methoden sollen ein Array vom Typ Adresse zurückgeben. Schau dir mal die Methoden im Klassendiagramm an. Da steht z.B.:
+sucheVorname(vorname: string) : Adresse[*]
das bedeutet: die Methode

• ist public (+ am Anfang)
• hat einen Parameter vorname vom Typ String
• gibt ein Array vom Typ Adresse zurück, die Größe des Arrays ist beliebig (*)

Also so:

public Adresse[] sucheVorname(String Vorname);

Dasselbe mit sucheNachname
Die Implementierung fehlt halt jetzt noch.

 

[SmittyJoe]

Vielen Dank,hab das gar nicht mit bedacht! Nimmts auch so an,aber beim zurückliefern von a kommt nun eine Fehlermeldung. Bei "a" fehlt noch was...oder?

 

[geqoo]

Da habe ich wohl beim nachträglichen Abändern etwas mit den Gettern und Objekten gepennt

Dass beim return von a eine Meldung kommt, ist doch normal.
Wie calousy schon schrieb:

public Adresse[] sucheVorname(String Vorname)

muss ein Array von Adresse zurückliefern. Bei a handelt es sich nur um eine einzelne Adresse.
Da du vorher nicht weißt, wie viele Adressen zurück kommen (können), kann man den Weg gehen und das Array vorher auf eine bestimmte Größe setzen - in dem Glauben, mehr Rückgaben kommen nicht.
Die sicherste Variante wäre hier, eine ArrayList zu verwenden.

public ArrayList<Adresse> sucheNachname(String nachname){
  ArrayList<Adresse> list = new ArrayList<Adresse>();

  for(Adresse a : buch){
    if(a.nachname.equals(nachname)){
      list.add(a); // Eintrag gefunden: Hinzufügen
    }
  }

  return list; // alles Gefundene zurückgeben
}

 

[calousy]

Hi,

geqoo hat Recht, hab gerade nochmal gegoogelt. Für [*] kann man ein Set oder eine List verwenden, was das ganze einfacher macht (CS201: A Beginners Guide to The Unified Modeling Language (UML)).