for-schleife

[Spin]

Hi

  /**
     * Einkauflsite erstellen
     */
    public void partyOrganize() {
        Cocktail co;
        int array[];
        /* Aktuellen Cocktail aus der Combobox holen*/
        String c = (String) jC_party0.getSelectedItem();
        int count;
        /* alle CocktailObjekte aus der Liste holen*/
        for (int i = 0; i < cl.sizeCocktails(); i++) { // erste for schleife
             co = cl.getCocktail(i); // co mit Cocktail füllen
            /* Wenn der Objektname des Cocktail gleich dem namen des Strings
             * in der ComboBox ist*/
            if (co.getName().equals(c)) { // Cocktailname mit dem Namen in der ComboBox vergleichen
                array = new int[co.getBestandteile().size()]; // array mit der Cocktailgröße initialisieren
                System.out.println(co.getName() + " " + 1);

                for (int j = 0; j < co.getBestandteile().size(); j++) { // zweite for Schleife
                    Bestandteil bestand = co.getBestandteil(i);
                    System.out.println(bestand.getMenge() + " " + 2);
                    // von jedem Cocktailbestandteil die Menge * dem Wert im Spinner erhöhen
                    array[j] = bestand.getMenge() * (Integer) jSpinner1.getModel().getValue();
                    // Von dem Bestandteil neuen Bestand setzen
                    System.out.println(array[j] + " " + 3);
                    
//                    bestand.setMenge(count);

                }
                partyShoppingList(co, array);
                break;
            } // if vorbei

        }

    }

Kann mir jemand sagen , warum meine zweite for schleife nicht bis zum letzten element der Bestandteilliste läuft?

Das array welches übergeben wird, hat immer 3 mal die gleiche zahl. ;(
danke

 

[javimka]

Die Schleife läuft durch. Aber das Ergebnis von [c]bestand.getMenge() * (Integer) jSpinner1.getModel().getValue();[/c] ist immer das gleiche.

 

[Spin]

Danke dir, und wie änder ich das jetzt?

Muss ich dazu irgendwie den wert vom jSpinner anders holen?
Nur im bestand.getMenge ist irgendwie immer das gleiche drin.

Er multiüliziert ja und gibt bei 2 auch 400 aus, anstatt 200.

 

[javimka]

Danke dir, und wie änder ich das jetzt?.

Was willst du denn, dass das Ding macht?

 

[Landei]

Ich hab zwar nur flüchtig rübergeguckt, aber ich denke, es muss heißen:
Bestandteil bestand = co.getBestandteil(j);

 

[Spin]

Ok , das mit dem j hat gestimmt.
Danke dir.


Nur noch die ausgabe und ich habe meinen Fehler erkannt.

 loop:
        for (int i = 0; i < c.getBestandteile().size(); i++) {
            bestand = c.getBestandteil(i);
            System.out.println(bestand.getZutat().getEinheit() + " " + 5);
            
            for (int j = 0; j < array.length; j++) {
                /* Für den jeweiligen Cocktail , Menge usw. dem Model hinzufügen*/
                model.addElement(array[j] + " " + bestand.getZutat().getEinheit() + " " +
                        bestand.getZutat().getName());
                
                System.out.println((array[j] + " " + bestand.getZutat().getEinheit() + " " +
                        bestand.getZutat().getName()));
                
                break;
                
            }

Ich hole mir den ersten Bestand und bekomme einheit und name
In der zweiten vorschleife hole ich den ersten wert im array[0].
Und alles wird ins modell gespeichert.

Nun hat aber mein cocktail noch mehr bestandteile und ich hole nun
das zweite bestandteil. Mein zweite forschleife fängt wieder bei array[o] an und holt sich somit wieder den ersten wert :/

Mit break muss ich das unterbrechen, aber ich will dass mein j noch erhöht wird, einfach j++ nach break funktioniert nicht :/

danke

 

[SlaterB]

der Sinn deines Codes ist nicht klar und erklärt wird er auch nicht gerade,
durch das break; wird die j-Schleife immer bei j=0 abgebrochen, wieso dort überhaupt eine Schleife?

j++ nach break; macht keinen Sinn, j ist nach der Schleife gar nicht mehr definiert und ein neuer Schleifendurchlauf wird immer wieder bei j=0 anfangen, oder möchtest du gerade das ändern?

 

[Spin]

Hey Slater ich finde es toll, dass du gerade ein wenig Zeit für mich hast.

Was versteht ihr denn an meinen Schleifen nicht? Sie sind doch kommentiert.
Nungut ich versuche nochmal zu erklären.

Ich habe in diesen Code ein Array in dem x zahlen gespeichert werden. Diese x Zahlen sind das doppelte oder dreifache usw. (je nachdem was für ein wert im jSpinner ist) der einzelnde Menge eines Cocktail Bestandteils.

Cuba Libre:

200 cl Cola -> 400cl Cola (JSpinner =2 ) ist nun im Array.
analog zu den anderen.


Das Array ist nun mit den multiplizierten Werten gefüllt.
Anschließend muss ich dieser Array ausgeben.

Ich hole mir wieder die einzelnden Bestandteile (1. for schleife) und nutze diesmal nur den Namen und die Einheit.
Die Menge ist ja diesmal anders, deswegen setze ich diese aus dem Array.

Die Verschachtelung meiner Schleifen stimmt aber nicht , weil man j sich nicht erhöht, sondern immer bei 0 anfängt.
Wie kann ich das ändern?

Das wä#re toll , wenn du mir das sagen könntest.

 loop:
        for (int i = 0; i < c.getBestandteile().size(); i++) {
            bestand = c.getBestandteil(i);



            
            for ( j = 0; j < array.length; j++) {
                System.out.println(j);
                /* Für den jeweiligen Cocktail , Menge usw. dem Model hinzufügen*/
                model.addElement(array[j] + " " + bestand.getZutat().getEinheit() + " " +
                        bestand.getZutat().getName());

                System.out.println((array[j] + " " + bestand.getZutat().getEinheit() + " " +
                        bestand.getZutat().getName()));

                break;

            }
            j++;
        }

So will es auch nicht : j bleibt immer 0.
Somit wird auch immer nur der erste Wert im Array ausgegeben .

Danke dir.



Aso ich brauche eine zweite schleife, weil mein Array ja immer mit anderen Zahlen gefüllt ist. Bei einen Mai Tai sind mehr Bestandteile und mehr Mengen

 

[faetzminator]

Du willst alle Elemente des Arrays bearbeiten? Einfach [c]break;[/c] entfernen (, was dir bereits SlaterB versuchte zu sagen).

 

[Spin]

Dann habe ich aber das Problem , das meine Bestandteile 10 mal oder mehr ausgegeben werden.

Ich habe denn nicht einmal Coba Libre 100 cl Colca.

Ich hätte denn :
100 cl Colca
100 cl Colca
100 cl Colca
100 cl Colca
100 cl Colca
50 cl Limettensaft
50 cl Limettensaft
50 cl Limettensaft


usw.

deswegen muss ich da schluss machen.


Ich will doch einfach nur .

Äußere Schleife (0) -> Innereschleife (0) --> Äußereschleife(1) -> Innere schleife (1) usw.



Ohje

 for (int i = 0; i < c.getBestandteile().size(); i++) {
            bestand = c.getBestandteil(i);



            
           
                System.out.println(j);
                /* Für den jeweiligen Cocktail , Menge usw. dem Model hinzufügen*/
                model.addElement(array[i] + " " + bestand.getZutat().getEinheit() + " " +
                        bestand.getZutat().getName());

                System.out.println((array[i] + " " + bestand.getZutat().getEinheit() + " " +
                        bestand.getZutat().getName()));

              

            }

Klar , denn einfach so und es geht


danke danke danke