Input/Output ArrayList Summe ausgeben

[vaekek]

Ich habe ein Projekt welches aus einer Tester Methode , einer Flugzeug Methode und einer Fluggesellschafts Methode besteht, in der Flugzeug Methode wird ein Flugzeug erzeugt, welches mit 3 Parameter in der Fluggesellschafts Klasse einer ArrayList hinzugefügt wird.
Nun habe ich 3 Parameter ("Name des Flugzeugs", aktiv/inaktiv(fliegt gerade), wert des Flugzeugs)

Nun möchte ich diese 2 Werte(aktiv/inaktiv) und Die Werte der Flugzeuge summieren.

die Ausgabe soll nun sein:

Flugzeuge die gerade Fliegen: 4
Flugzeuge insgesamt: 5
Flugzeug Wert summiert: 12931293

Nun habe mir überlegt diesen aktiv/inaktiv Parameter mit true/false mitzugeben, jedoch weiss ich nicht wie ich diese werte summieren kann.

 

[looparda]

Du kannst über deine Flugzeuge mit einer Schleife iterieren und einen Zähler benutzen. Wenn aktiv, dann erhöhst du um 1, sonst nicht.

 

[vaekek]

Du kannst über deine Flugzeuge mit einer Schleife iterieren und einen Zähler benutzen. Wenn aktiv, dann erhöhst du um 1, sonst nicht.

kannst du ein Beispiel machen? wenn ich 2 Variablen definiere und beide erhöhe mit foreach und eines nur wenn der wert true ist, erhalte ich dennoch einen falschen wert.

 

[tommysenf]

summe = 0
aktive = 0

für jedes Flugzeug f in alleFlugzeuge
   wenn f aktiv dann aktive = aktive + 1
   summe = summe + wert von f
ende für

 

[looparda]

Boolean[] zustaende = {true, true, false, false, true};
int anzahlAktiv = 0;
int anzahlAlle = 0;
for ( Boolean zustand : zustaende ) {
    if(zustand.equals(true)) {
        anzahlAktiv = anzahlAktiv + 1;
    }
    anzahlAlle = anzahlAlle + 1;
}
System.out.println(anzahlAktiv + " von " + anzahlAlle + " sind aktiv.");

Zur Vereinfachung und kleinen Transferaufgabe auf deine eigenen Datenstrukturen habe ich mir ein Array von Boolean angelegt, damit ich nicht deine Klassen nachbauen muss. Das Prinzip ist jedoch das gleiche, wenn du über boolean iterierst und Flugzeuge und deren Zustände (die ja ebenfalls boolean sind).

 

[vaekek]

Das hilft mir schon sehr, danke ! wie mache ich das nun mit einer ArrayList welche Beispielsweise 5 Flugzeuge beinhaltet?

 

[tommysenf]

Das hilft mir schon sehr, danke ! wie mache ich das nun mit einer ArrayList welche Beispielsweise 5 Flugzeuge beinhaltet?

Mach es am besten so wie mit einer ArrayList mit 4 Flugzeugen halt nur einmal mehr


Nein im Ernst, zeig mal deinen Ansatz als Code wie du es probieren würdest, ann verbessern wir gern...

 

[vaekek]

import java.util.ArrayList;

public class Fluggesellschaft {

    ArrayList<Flugzeug> fl = new ArrayList<Flugzeug>();
    int aktiv = 0;
    int allefluege = 0;

    public boolean add(Flugzeug flug) {

      
        for (Flugzeug flugzeug : fl) {
          
            if(flugzeug.equals(true)){
                aktiv = aktiv+1;
            }
            allefluege = allefluege+1;
          
        }
        return false;

    }

}

wäre das nun brauchbar?

 

[tommysenf]

1. Bitte benutze Codetags
2. Wieso eine Methode add wo du nichts addest sondern doch eigentlich die Flugzeuge zählen möchtest.
3. if(flugzeug.equals(true) Überlege mal wann ein Flugzeug aktiv ist...

 

[looparda]

Die Flugzeugklasse am besten auch mit posten, damit wir besser über das "aktiv sein" reden können.

 

[vaekek]

wie mache ich Code Tags? "schäm"

-hat sich erledigt

 

[vaekek]

public class Tester {

    public static void main(String[] args) {
       
        Fluggesellschaft fl = new Fluggesellschaft("Swiss");
        Flugzeug eins = new Flugzeug("a830", true, 450000);
        Flugzeug zwei = new Flugzeug("a840", true, 600000);
        Flugzeug drei = new Flugzeug("a850", false, 500000);
       
        fl.add(eins);
        fl.add(zwei);
        fl.add(drei);
       
        fl.print();
       

    }

}

public class Flugzeug {

    private String name;
    private boolean aktiv;
    private int wert;
    public Flugzeug(String name, boolean aktiv, int wert) {
        this.name = name;
        this.aktiv = aktiv;
        this.wert = wert;
    }
    public String getName() {
        return name;
    }
    public void setName(String name) {
        this.name = name;
    }
    public boolean isAktiv() {
        return aktiv;
    }
    public void setAktiv(boolean aktiv) {
        this.aktiv = aktiv;
    }
    public int getWert() {
        return wert;
    }
    public void setWert(int wert) {
        this.wert = wert;
    }
   
   
   
}

import java.util.ArrayList;

public class Fluggesellschaft {

    ArrayList<Flugzeug> fl = new ArrayList<Flugzeug>();
    int aktiv = 0;
    int allefluege = 0;
    String name;
   
    public Fluggesellschaft(String name){
        this.name = name;
       
    }

   
    public boolean add(Flugzeug flug) {
       
        fl.add(flug);

       
        for (Flugzeug flugzeug : fl) {
           
           
            if(flugzeug.equals(true)){
                aktiv = aktiv+1;
            }
            allefluege = allefluege+1;
           
        }
        return false;

    }

    public void print(){
       
        System.out.println(this.name);
       
        System.out.println(aktiv);
        System.out.println(allefluege);
    }
}

 

[tommysenf]

Entweder per Hand indem du
[code=Java]Hier kommt den Code hin...[/code]

oder du benutz den Einfügen Button in der Toolbar

 

[looparda]

Was macht denn die Funktion equals, die du benutzt?

 

[vaekek]

muss ich .contains machen?

 

[tommysenf]

Du addierst jedesmal wenn ein Flugzeug hinzugefügt wird alle schon eingefügten Flugzeuge auf. Das ist falsch.
Entweder du addierst nur das einzufügende Flugzeug auf ohne Schleife oder du gehst nur in deiner print Methode durch die Flugzeugliste und suchst die entsprechenden Flugzeuge heraus.

 

[vaekek]

sprich foreach erst in der print methode ansteuern?

 

[tommysenf]

Ist eine gute Lösung

 

[vaekek]

import java.util.ArrayList;

public class Fluggesellschaft {
    ArrayList<Flugzeug> fl = new ArrayList<Flugzeug>();
    int aktiv = 0;
    int allefluege = 0;
    String name;

    public Fluggesellschaft(String name) {
        this.name = name;
    }

    public void add(Flugzeug flug) {
        fl.add(flug);
    }

    public void print() {
        for (Flugzeug flugzeug : fl) {
            if (flugzeug.equals(true)) {
                aktiv = aktiv + 1;
            }
            allefluege = allefluege + 1;
        }

        System.out.println(this.name);

        System.out.println(aktiv);
        System.out.println(allefluege);
    }
}

nun ist die Ausgabe Aktiv = 0;
und allefluege =3;
- Wie bekomme ich das hin, dass die aktiven auch mitgezählt werden?

 

[looparda]

equals Prüft ob zwei Objekte gleich sind. Im einfachsten Fall sind es zwei Objekte vom gleichen Typ. Also z.B. wie vorhin boolean und boolean. Du vergleichst gerade true (boolean) mit einem Flugzeug. Das hört sich sogar sprachlich ziemlich gefährlich und falsch an. Versuche auch hier wieder booleans zu vergleichen.

 

[vaekek]

    if (flugzeug.isAktiv()==(true)) {
                aktiv = aktiv + 1;
            }

-Fehler gefunden, danke !!!

 

[tommysenf]

Supi.

Noch ein paar Hinweise: Die Anzahl aller Flugzeuge entspricht der Größe der Flugzeugliste. Diese hat die Methode #size() .
Diese solltest du benutzen.

in einer if Abfrage sollte man sich den Vergleich mit true oder false sparen:

if(flugzeug.isActive()) {
   ....
}

Benutze immer vernünftige Namen:

List<Flugzeug> flugzeuge = new ArrayList<>();

 

[vaekek]

habe ich gearde noch ergänzt, wie kann ich nun noch den Gesamtwert der Flugzeuge ausgeben?

 

[looparda]

habe ich gearde noch ergänzt, wie kann ich nun noch den Gesamtwert der Flugzeuge ausgeben?

Das ist analog zum zählen der aktiven Flugzeuge, außer das du nicht um 1 inkrementierst, sondern um den Wert den jeweiligen Flugzeugs.

 

[vaekek]

import java.util.ArrayList;

public class Fluggesellschaft {

    ArrayList<Flugzeug> fl = new ArrayList<Flugzeug>();
    int aktiv = 0;
    String name;
    int gesamtwert;

    public Fluggesellschaft(String name) {
        this.name = name;

    }

    public void add(Flugzeug flug) {

        fl.add(flug);

    }

    public void print() {

        for (Flugzeug flugzeug : fl) {

            if (flugzeug.isAktiv()==(true)) {
                aktiv = aktiv + 1;
            }
            gesamtwert = gesamtwert + flugzeug.getWert();
        }

        System.out.println(this.name);

        System.out.println("Aktive Flüge:" + aktiv);
        System.out.println("Gesamte Flüge:" + fl.size());
        System.out.println("Gesamtwert der Flugzeuge:" + gesamtwert);
    }
}

- So erhalte ich nun auch noch den Gesamtwert Danke Euch für die Hilfe!

 

[tommysenf]

if (flugzeug.isAktiv()==(true))

Das solltest du dir ganz schnell abgewöhnen!


Wenn du etwas lernen möchtest kannst du ja noch versuchen die andere Variante mit der add Methode ohne Schleife zu implementieren...

 

[vaekek]

Das solltest du dir ganz schnell abgewöhnen!

Was ist daran verkehrt? Welche Lösung ist optimal?

 

[tommysenf]

Es ist nichts verkehrt. Optimaler währe die Lösung ohne Schleife.

 

[vaekek]

Wie überprüfe ich dann die einzelnen Werte der Liste?

 

[tommysenf]

Du fügst deiner Fluggesellschaft die Attribute wert und aktiveFlugzeuge hinzu. Diese aktualisierst du immer wenn ein Flugzeug hinzugefügt wird.
In der print Methode kannst du dann auf diese Attribute zugreifen.

 

[tommysenf]

Was ist daran verkehrt? Welche Lösung ist optimal?

in einer if Abfrage sollte man sich den Vergleich mit true oder false sparen:

Code (Java):
if(flugzeug.isActive()) {
....
}

 

[vaekek]

Kannst du die optimale Lösung posten?
Ist erst das dritte Projekt, welches ich OOP versuche zu lösen!

Danke für die kompetente Hilfe!

 

[tommysenf]

Nein, aber ich kann die gerne helfen.
Was verstehst du denn an

Du fügst deiner Fluggesellschaft die Attribute wert und aktiveFlugzeuge hinzu. Diese aktualisierst du immer wenn ein Flugzeug hinzugefügt wird.
In der print Methode kannst du dann auf diese Attribute zugreifen.

nicht?

 

[vaekek]

wie ich diese Werte übergebe,zwischenspeichere und danach immer wieder überprüfe

 

[tommysenf]

Als Attribute in der Klasse Fluggesellschaft wie du es in deinem erstem Beispiel schon getan hast. Von der OOP Modellierung her entspricht es "Eine Fluggesellschaft hat einen Namen, eine Anzahl von aktiven Flugzeugen und einen Wert, welcher sich aus der Summe der der Werte aller Flugzeuge ergibt."

 

[vaekek]

dann benötige ich die foreach in der print methode garnichtmehr?
- ich verstehe nicht wo ich nun ansetzten soll. sorry!

 

[tommysenf]

Schreib doch erst mal die Fluggesellschaft Klasse mit den entsprechenden Attributen ...

 

[vaekek]

public class Fluggesellschaft {
   
    int wert=0;
    boolean aktiv;

meinst du so?

 

[tommysenf]

Fast, eine Fluggesellschaft ist ja nicht aktiv oder inaktiv sondern hat eine Anzahl von aktiven Flugzeugen.
dann noch die Flugzeugliste, und die add Methode.

 

[vaekek]

import java.util.ArrayList;

public class Fluggesellschaft {
   
    int wert=0;
    int aktiveFlugzeuge;
    ArrayList<Flugzeug> fl = new ArrayList<Flugzeug>();
   
   
    public void add(Flugzeug flugzeug){
        fl.add(flugzeug);
       
       
    }

}

- Ist der Aufbau nun korrekt?

 

[tommysenf]

Genau.
Die Attribute wert und aktive Flugzeuge ändern sich ja jedes mal wenn wenn ein Flugzeug hinzugefügt wird. Also musst du sie dann auch aktualisieren.

 

[vaekek]

import java.util.ArrayList;

public class Fluggesellschaft {
   
    int wert=0;
    int aktiveFlugzeuge=0;
    ArrayList<Flugzeug> fl = new ArrayList<Flugzeug>();
   
   
    public void add(Flugzeug flugzeug){
        fl.add(flugzeug);
        wert = wert + flugzeug.getWert();
       
        if(flugzeug.isAktiv()){
            aktiveFlugzeuge = aktiveFlugzeuge + 1;
        }
       
    }

}

- Meinst du so?

 

[tommysenf]

Ja.
Weiterhin sollte man die Sichtbarkeit von Attributen immer möglichst einschränken und entsprechende getter Methoden bereitstellen um Sicherzustellen dass der wert und die Anzahl der aktiven Flugzeuge von Außen nicht geändert werden kann.

 

[vaekek]

Ja.
Weiterhin sollte man die Sichtbarkeit von Attributen immer möglichst einschränken und entsprechende getter Methoden bereitstellen um Sicherzustellen dass der wert und die Anzahl der aktiven Flugzeuge von Außen nicht geändert werden kann.

kannst du dazu ein Beispiel machen?

 

[tommysenf]

Sagt dir "private" was?
Als Beispiel kannst du die Klasse Flugzeug nehmen.

 

[vaekek]

import java.util.ArrayList;

public class Fluggesellschaft {

    private int wert = 0;
    private int aktiveFlugzeuge = 0;
    ArrayList<Flugzeug> fl = new ArrayList<Flugzeug>();

    public void add(Flugzeug flugzeug) {
        fl.add(flugzeug);
        wert = wert + flugzeug.getWert();

        if (flugzeug.isAktiv()) {
            getAktiveFlugzeuge();
        }

    }

    public int getAktiveFlugzeuge() {
        aktiveFlugzeuge = aktiveFlugzeuge + 1;
        return aktiveFlugzeuge;
    }

}

in der Art?

 

[tommysenf]

public int getAktiveFlugzeuge() {
aktiveFlugzeuge = aktiveFlugzeuge + 1;
return aktiveFlugzeuge;
}

An sich schon, nur wenn jemand die Anzahl der aktiven Flugzeuge wissen will wird ja der Wert dabei nicht verändert.

 

[vaekek]

import java.util.ArrayList;

public class Fluggesellschaft {

    private int wert = 0;
    private int aktiveFlugzeuge = 0;
    private ArrayList<Flugzeug> fl = new ArrayList<Flugzeug>();

    public void add(Flugzeug flugzeug) {
        fl.add(flugzeug);
        wert = wert + flugzeug.getWert();

        if (flugzeug.isAktiv()) {
            getAktiveFlugzeuge();
        }

    }

    private int getAktiveFlugzeuge() {
        aktiveFlugzeuge = aktiveFlugzeuge + 1;
        return aktiveFlugzeuge;
    }

}

oder eher so?

 

[vaekek]

macht sinn... ^.^

import java.util.ArrayList;

public class Fluggesellschaft {

    private int wert = 0;
    private int aktiveFlugzeuge = 0;
    private ArrayList<Flugzeug> fl = new ArrayList<Flugzeug>();

    public void add(Flugzeug flugzeug) {
        fl.add(flugzeug);
        wert = wert + flugzeug.getWert();
        aktiveFlugzeuge = aktiveFlugzeuge + 1;

        if (flugzeug.isAktiv()) {
            getAktiveFlugzeuge();
        }

    }

    private int getAktiveFlugzeuge() {
        return aktiveFlugzeuge;
    }

}

 

[tommysenf]

Nein, das ist "falscher" (Post 48)

Der Wert ändert sich in der add Methode und nicht in der get Methode.

Jetzt ist es Korrekt! (Post 49)