[Java] Array - Klassenmember(?)

[The_Dentist]

Hey,

Ich habe folgendes Problem und zwar soll ich zwei Klassen schreiben.

Einmal eine "Singer" und einmal eine "Coach" Klasse. Beide Klassen haben ihre Attribute und Methoden.

Nun soll ich der Coach Klasse eine Methode hinzufügen, mit der es möglich ist, Sänger (also Objekte von Singer) einem Array vom Typ Singer (Singer[] s = new Singer[x - Länge]) hinzuzufügen, sodass man das Array durchgehen kann und alle Sänger findet, welche dem gleichen Coach zugeordnet sind.

Dieses Array soll sich in der Coach Klasse befinden.

Mein Ansatz sieht derzeit, wie folgt aus:

 Spoiler

[COACH - Klasse]

    public void select(Singer s) throws Exception{
        this.s = s;
        s.setCoach(this);
        NumberOfObjekts++;
    }
    
    
    
    public void ausgabe()
    {
        System.out.println("Der Coach: " + Name + " hat " + NumberOfObjekts + " Sänger!" + " \nMit den Namen: ");
        //for(Singer t: singer) System.out.print(t.getName());
        
    }

Mit der Select Methode soll es möglich sein, neue Sänger in das Team
(Array von Singer) aufzunehmen.

[Singer - Klasse]

  public void setCoach(Coach c) throws Exception{
        if(this.c == null) this.c = c;
        else
            throw new Exception("Der Coach darf nicht überschrieben werden!");
    }

Bei der Select Klasse hänge ich jetzt, soweit ich weiß kann ich nicht einfach was wie:

 public void select(Singer s) throws Exception{
        this.s = s;
        s.setCoach(this);
        NumberOfObjekts++;
        singer = new Singer[NumberOfObjekts];
        singer[NumberOfObjekts - 1] = s;
    }

machen. Aber wie gehe ich alternativ an das Problem heran?

In der Klasse Coach habe ich bereits ein Singer Array singer angelegt.

Ich wusste nicht genau, wie man diese Beziehungen nennt.

Kann mir Jemand bei dem Problem helfen?

Wäre demjenigen sehr dankbar!

LG
The_Dentist!

€DIT: Hat sich erledigt, Lösung des Problemes wurde gefunden!

[NotEnoughForYou]

Wieso heißt deine Methode select? Wäre ein Name wie insert nicht passender?

Angenommen du hast deine Klasse Coach:

class Coach {
	private Singer[] singer = new Singer[20];

	public void insert(Singer s) {
		int pos = getEmptyPos();

		if(pos != -1) {
			singer[pos] = s;
		}
	}

	private int getEmptyPos() {
		for (int i = 0; i < singer.length; i++) {
			if(singer[i] == null) {
				return i;
			}
		}

		return -1; // array voll
	}
}

So in etwa könnte das Einfügen aussehen. Ist allerdings ungetestet.

Dann könntest du sowas wie folgt machen:

Coach c = new Coach();

c.insert(new Singer());

Und dann für die Ausgabe eben das Array durchgehen.

Schöner wäre es natürlich mit einer Arraylist, aber ich nehme an, ihr müsst Arrays verwenden.

[The_Dentist]

Originally Posted by NotEnoughForYou

Wieso heißt deine Methode select? Wäre ein Name wie insert nicht passender?

Aufgabenstellung...
Danke schon mal, ich versuche es gerade mal!


€dit:

 Spoiler

    public void ausgabe()
    {

        System.out.println("Der Coach: " + Name + " hat " + NumberOfObjekts + " Sänger!" + " \nMit den Namen: ");
        for(Singer t: singer) if(t != null) System.out.print(t.getName());
        
        
    }

[Main Klasse]

   static Coach steffen;
    static Singer marc;
    
    public static void main(String[] args) throws Exception {
        
        steffen = new Coach("Steffen");
        marc = new Singer("Marc", 25, true, 100);
        steffen.select(marc);
        
        marc.ausgabe();
        steffen.ausgabe();
        
    }

Leider funktioniert die Ausgabe immer noch nicht, bekomme keine Ausgabe mit Namen. Das wundert mich allerdings, da der Wert singer[0] ungleich null ist. Eigentlich müsste er diesen doch ausgeben, oder?

So:

LG

[NotEnoughForYou]

Zeig mal deinen ganzen Code.

Überschreiben doch lieber die toString Methode und gebe es über ein System.out.print aus, ist eleganter

[The_Dentist]

[Coach - Klasse]

 Spoiler

public class Coach {
    
    private String Name; 
    private Singer[] singer = new Singer[20];
    private Singer s;
    private int NumberOfObjekts = 0;
    
public String getName(){
        return Name;
    }

    public Coach(String Name){
       this.Name = Name;
    }
    
    
    
    public void select(Singer s) throws Exception{
      int pos = getEmptyPos(); 
      if(pos!= -1) {
            singer[pos] = s;
            s.setCoach(this);
            NumberOfObjekts++;
        }
    }
    
      private int getEmptyPos() {
      for(int i = 0; i< singer.length; i++) {
        if(singer[i] == null) {
            return i;
      }
      
    }
      return -1;        // array voll

    }
    
    public void ausgabe()
    {
        System.out.println("######################################################################################################################################################################");
        System.out.println("Der Coach: " + Name + " hat " + NumberOfObjekts + " Sänger!" + " \nMit den Namen: ");
        for(Singer t: singer) if(t != null) System.out.print(t.getName());
        System.out.println("######################################################################################################################################################################");
        
    }
 
}

[Singer - Klasse]

 Spoiler

public class Singer {
    
    private final String Name;
    private final int Alter;
    private final boolean Geschlecht; // 1 = Mann, 0 = Frau
    private final int Follower; 
    private Coach c;

    /*
     *
     * @param Name
     * @param Alter
     * @param Geschlecht
     * @param Follower
     */
    public Singer(String Name, int Alter, boolean Geschlecht, int Follower){
        this.Name = Name;
        this.Alter = Alter;
        this.Geschlecht = Geschlecht;
        this.Follower = Follower;
    }

    public void setCoach(Coach c) throws Exception{
        if(this.c == null) this.c = c;
        else
            throw new Exception("Der Coach darf nicht überschrieben werden!");
    }
    public Coach getCoach(){
        return c;
    }
    
    public String getName(){
        return Name;
    }
    
    public void ausgabe(){              
        
        if(getCoach() != null){
        
        if(Geschlecht){
            System.out.print("----------------------------------------------------------------------------------------------------------------------------------------------------------------------\n");
            System.out.print("Ein Neuer männlicher Sänger mit folgenden Attributen wurde angelegt: " + Name + " " + Alter + " Follower: " + Follower + "\n\nSein Coach heißt: " + getCoach().getName() + "\n");
            System.out.print("----------------------------------------------------------------------------------------------------------------------------------------------------------------------\n");
        }
        else{
            System.out.print("----------------------------------------------------------------------------------------------------------------------------------------------------------------------\n");
            System.out.print("Ein Neuer weiblicher Sänger mit folgenden Attributen wurde angelegt: " + Name + " " + Alter + " Follower: " + Follower  + "\n\nIhr Coach heißt: " + getCoach().getName() + "\n");
            System.out.print("----------------------------------------------------------------------------------------------------------------------------------------------------------------------\n");
        }
        
       }
        
       else{
            if(Geschlecht){
            System.out.print("----------------------------------------------------------------------------------------------------------------------------------------------------------------------\n");
            System.out.print("Ein Neuer männlicher Sänger mit folgenden Attributen wurde angelegt: " + Name + " " + Alter + " Follower: " + Follower + "\n\nEr hat noch keinen Coach " + "\n");
            System.out.print("----------------------------------------------------------------------------------------------------------------------------------------------------------------------\n");
        }
        else{
            System.out.print("----------------------------------------------------------------------------------------------------------------------------------------------------------------------\n");
            System.out.print("Ein Neuer weiblicher Sänger mit folgenden Attributen wurde angelegt: " + Name + " " + Alter + " Follower: " + Follower +  "\n\nSie hat noch keinen Coach " + "\n");
            System.out.print("----------------------------------------------------------------------------------------------------------------------------------------------------------------------\n");
        }
        }
    }
}

€dit: Leider immer noch keine Lösung.

[RecK]

Ein normales Objektarray zu verwenden, wenn dynamisch viele Objekte hinzugefügt werden können ist unsinnig. Die Länge muss vorher festgelegt werden.
Jeder der dann einem mit ReDim-Funktionen kommt, "scheißt" ja ganz schön auf Performance...

0815-Lösung fürs Listing wäre:

private ArrayList<Singer> customers = new ArrayList<>();

public List<Singer> getAllSingers() {
    return customers;
}

public void addSinger(Singer s) {
   customers.add(s);
}

Was bereits angesprochen wurde ist die Überschreibung der toString Methode. Jede Klasse erbt standardmäßig in Java von der Klasse Objekt, mein ich. Dort gibt es eine toString Methode. Default mäßig gibt die Klasse über toString einen String der das Objekt serialisiert (meist nutzlos) darstellt.

Also zum Überschreiben der Methode ist @Override wichtig.
Implementiere also nun folgendes in deine Coach-Klasse:

@Override
public String toString() {
   StringBuilder sb = new StringBuilder(coachName); // stringbuilder ist deutlich schneller (besseres stringhandling)
   sb.append(" hat folgende Kunden: ");
   for(Singer s : customers) {
     sb.append("\n");
     sb.append(s.getName()); // oder toString Methode von Singer
   }
   sb.append("Weiterer Text...");
   return sb.toString();
}

Das wäre der vollständige Workaround für diese Problematik.
Hoffe es hilft dir.

#edit:
Deine Main-Methode ist ja soweit immer das Gleiche.
Die Ausabe wäre nun jedoch nur noch:
Coach c = ...
System.out.println(c); // .toString wird automatisch aufgerufen (kann aber auch geschrieben werden, c.toString())


lg, Tim

[dowhile]

Das sind Hausaufgaben, und dafür darf er natürlich nicht den vollen Sprachumfang nutzen. Das mit einem Array zu lösen ist in der Praxis natürlich nicht gut, aber hier geht es darum, dass er es trotzdem auch damit lösen kann.
Analog darf er natürlich auch nicht toString() nutzen (bzw. muss wenn dann toString() in ausgabe() aufrufen), sondern muss sich an die Aufgabenstellung halten.
StringBuilder ist nicht schneller, bzw. die manuelle Konkatenation wird vom Compiler optimiert (und ggf. durch einen StringBuilder ersetzt).
"select" ist ein sinnvoller Name, weil ein Coach einen Singer beim Context für sein Team "auswählt".