fehler beim incrementellen schreiben in einen vector

[roberlin]

hallo leute! bin ersti und meine aufgabe ist es, ein programm zu entwerfen, das wachstum simuliert:

aus "O" wird "X" und aus "X" wird "OX", also sowas sollte dabei herauskommen:

O
X
OX
XOX
OXXOX
XOXOXXOX...

habe es auf der mit folgendem code versucht:

[code]import java.util.*;

public class grow {
    
//    at first, we declare constants for X and O:
    
    static final char o = 'O';
    static final char x = 'X';

    public static void main(String[] args) {
        
/*        to figure out a working loop (or loops) for the growing the input into the vector will be "O" in the beginning
we declare one vector "vec" for the current line and one vector "newVec" for the new line of growth. for now, we simply set the number of iterations to 5*/
        
        Vector<Character> vec = new Vector();
        Vector<Character> newVec = new Vector();
        
        int ind = 0; //index of vec
        int ind1 = 0; //index of newVec
        int it = 5; //number of iterations
        
        vec.insertElementAt(o, ind);
        
        for (int i=0; i<it; i++){         // loop iterations
            
            for (int i1 = 0; i1<vec.size(); i1++){  /*loop through vec*/
                
                }                
                if (vec.get(i1)==o){        
                    newVec.insertElementAt(x, ind1);
                    ind1++;
                }
                if (vec.get(i1)==x){
                    newVec.insertElementAt(o, ind1);
                    ind1 = ind1 + 1;
                    newVec.insertElementAt(x, ind1);
                    ind1 = ind1 + 1;
                }
            }
            for(int i2 = 0; i2 < newVec.size(); i2++) {
                   System.out.println(newVec.get(i2));
                   vec = newVec;
                   ind1 = 0;
                }
        }
    }
}

[code/]

ich bekomme aber immer eine speicherbezogene fehlermeldung in eclipse und weiss grad echt nich mehr weiter, hängt es viell mit der anfangsgröße oder mit der vergrößerung (incrementCapacity) zusammen? habe auch schon probiert die vektoren mit Vector<Character> vec = new Vector(1, 1); und ähnlichem zu deklarieren, bringt aber auch nix, please help, thx!!

 

[SlaterB]

deine Klasse war gar nicht kompilierbar, da die beiden ifs außerhalb der inneren Schleife standen,

so

public class grow
{

    // at first, we declare constants for X and O:

    static final char o = 'O';
    static final char x = 'X';

    public static void main(String[] args)
    {
        Vector<Character> vec = new Vector();
        Vector<Character> newVec = new Vector();

        int ind = 0; // index of vec
        int ind1 = 0; // index of newVec
        int it = 5; // number of iterations

        vec.insertElementAt(o, ind);

        for (int i = 0; i < it; i++)
        { // loop iterations

            for (int i1 = 0; i1 < vec.size(); i1++)
            {
                if (vec.get(i1) == o)
                {
                    newVec.insertElementAt(x, ind1);
                    ind1++;
                }
                if (vec.get(i1) == x)
                {
                    newVec.insertElementAt(o, ind1);
                    ind1 = ind1 + 1;
                    newVec.insertElementAt(x, ind1);
                    ind1 = ind1 + 1;
                }
            }
        }
        for (int i2 = 0; i2 < newVec.size(); i2++)
        {
            System.out.println(newVec.get(i2));
            vec = newVec;
            ind1 = 0;
        }
    }
}

gibts keine Fehlermeldung, die Ausgabe ist
X
X
X
X
X

was ist nun die Frage?
und was ist 'eine speicherbezogene fehlermeldung'?
hat die keinen Namen/ Text?

 

[roberlin]

ups, hatte da noch einen andren block am anfang der for-schleife, da hab ich die schliessende klammer zu löschen vergessen...

die ausgabe von

x
x
x...

hatte ich auch schon, aber wie das wachstum/die ausgabe aussehen sollte habe ich ja eingangs beschrieben, das krieg ich irgendwie nicht hin, ist mein ansatz vollkommen daneben, müsste doch eigentlich mit den schleifen so klappen. aus der aus gabe

x
x
x
x...

schliesse ich, dass die innere for-schleife nach einem durchgan endet, weil der vector sich nicht vergrößert und ich weiss nicht genau, wie ich es hinkriege, dass er wächst, sodass obige ausgabe erreicht wird...

die fehlermeldung reiche ich nach, bin grade auf arbeit und kann hier (keine adminrechte) kein eclipse installieren...

sorry für meine anfängerüberlegungen...

vielen dank!!

grüße. r

 

[SlaterB]

eine der wichtigsten Aufgaben wäre, die Listen nach jedem Durchgang zu tauschen,
du arbeitest 5x nur mit der O-Liste

sieht dann z.B. so aus:

public class grow
{

    // at first, we declare constants for X and O:

    static final char o = 'O';
    static final char x = 'X';

    public static void main(String[] args)
    {
        Vector<Character> vec = new Vector();
        Vector<Character> newVec = new Vector();

        int ind = 0; // index of vec
        int ind1 = 0; // index of newVec
        int it = 10; // number of iterations

        newVec.insertElementAt(o, ind);

        for (int i = 0; i < it; i++)
        { // loop iterations

            ind1 = 0;
            vec = newVec;
            newVec = new Vector<Character>();
            System.out.print("\ni: "+i+" ");
            for (int i1 = 0; i1 < vec.size(); i1++)
            {
                char c = vec.get(i1).charValue();
                System.out.print(c);
                if (c == o)
                {
                    newVec.insertElementAt(x, ind1);
                    ind1++;
                }
                if (c == x)
                {
                    newVec.insertElementAt(o, ind1);
                    ind1 = ind1 + 1;
                    newVec.insertElementAt(x, ind1);
                    ind1 = ind1 + 1;
                }
            }            
        }
    }
}

--------

edit: kürzer:

public class grow
{
    static final char o = 'O';
    static final char x = 'X';

    public static void main(String[] args)
    {
        Vector<Character> vec;
        Vector<Character> newVec = new Vector();
        newVec.add(o);

        int it = 10;
        for (int i = 0; i < it; i++)
        {
            vec = newVec;
            newVec = new Vector<Character>();
            System.out.print("\ni: " + i + " ");
            for (int i1 = 0; i1 < vec.size(); i1++)
            {
                char c = vec.get(i1).charValue();
                System.out.print(c);
                if (c == x)
                {
                    newVec.add(o);
                }
                newVec.add(x);
            }
        }
    }
}

 

[roberlin]

danke! werd das nachher gleich mal testen! die abkürzung gefällt mir gut, da hätte ich auch drauf kommen können.

wirklich verstanden hab ich es noch nich ganz: was genau vertauscht sich bei deiner variante? liegt es daran, dass du newVec erst innerhalb der schleife deklarierst?

warum bleibe ich bei o? nach meinem ersten durchgang müsste doch in newVec auch x stehen (und damit in vec für die zweite iteration der äußeren schleife) oder?

 

[SlaterB]

du liest 5x aus vec und schreibst in newVec,
in vec steht aber jedesmal nur der Anfangswert 'o' drin, vec ändert sich nicht
(wie kommst du zu der Aussage "müsste doch in newVec auch x stehen (und damit in vec für die zweite iteration"?)

erst duch einen Befehl wie
vec = newVec;
kommt derartige Dynamik ins Spiel

 

[roberlin]

genau hab ich doch aus diesem grund nach beenden der inneren for-schleife, wenn vec ausgelesen und verändert in newVec geschrieben ist, den inhalt von newVec geschrieben und dann an vec übergeben, bevor die äußere iterationsschleife erneut startet.

obwohl ich grad sehe dass mein
vec = newVec;
in der for-schleife steht, die ja nur zum auslesen und schreiben von newVec dient, also wäre mir geholfen wenn in meinem code statt

 for(int i2 = 0; i2 < newVec.size(); i2++) {
System.out.println(newVec.get(i2));
vec = newVec;
ind1 = 0;
}

eher

 for(int i2 = 0; i2 < newVec.size(); i2++) {
System.out.println(newVec.get(i2));
vec = newVec;
ind1 = 0;
} 
vec = newVec;

nach der inneren for-schleife stünde (konzeptuell war es so gedacht...)? oder gehts dann auch noch nicht?

 

[roberlin]

shit, sorry, in der schleife muss das vec = newVec; dann ja weg, also so:

for(int i2 = 0; i2 < newVec.size(); i2++) {
System.out.println(newVec.get(i2));
ind1 = 0;
}
vec = newVec;

 

[roberlin]

das würde nämlich bedeuten, dass mein code in meinem ersten posting, bis auf die eine fehlerhafte } klammer nach der öffnung der inneren for-schleife und dem vec = newVec;, welches mir in die letzte for-schleife gerutscht ist statt dahinter, vom prinzip her richtig war, was meinem pessimismus bezgl meinem verständnis für java ein wenig abhilfe verschaffen würde...

in jedem fall bin ich dir sehr dankbar, mir sind ein paar weitere lichter aufgegangen

 

[SlaterB]

> und dann an vec übergeben, bevor die äußere iterationsschleife erneut startet.

nein dieser Befehl stand bei dir erst ganz am Ende nach den 5 Durchläufen

-----------

> oder gehts dann auch noch nicht?


so hab ichs ja auch mehr oder weniger, also: ja klar ist das mit dem Befehlt dann besser als vorher,
aber vec und newVec dürfen natürlich nicht die gleiche Liste sein,
sonst wird ein Durchlauf ewig dauern

newVec muss also noch ein neuer Vector werden, alles wie in meinem Code zu sehen

 

[roberlin]

sorry, ein letzter versuch, es zu raffen. im folgendem code

        import java.util.*;

        public class grow {



        static final char o = 'O';
        static final char x = 'X';

        public static void main(String[] args) {

            Vector<Character> vec = new Vector();
            Vector<Character> newVec = new Vector();

            int ind = 0; //index of vec
            int ind1 = 0; //index of newVec
            int it = 5; //number of iterations

            vec.insertElementAt(o, ind);

            for (int i=0; i<it; i++){

                  for (int i1 = 0; i1<vec.size(); i1++){ 

                        if (vec.get(i1)==o){
                        newVec.insertElementAt(x, ind1);
                        ind1++;
                        }
                        if (vec.get(i1)==x){
                        newVec.insertElementAt(o, ind1);
                        ind1 = ind1 + 1;
                        newVec.insertElementAt(x, ind1);
                        ind1 = ind1 + 1;
                        }
                  }
                  for(int i2 = 0; i2 < newVec.size(); i2++) {
                  System.out.println(newVec.get(i2));
                  ind1 = 0;
                  vec = newVec;
            }
       }
}
}

habe ich (verglichen mit dem ersten posting) jetzt mal nur die falsche klammer und die kommentare entfernt, das vec = newVec; aus der letzten schleife genommen und die zeilen so eingerückt, wie ich es gedacht habe. ich habe entdeckt, dass ganz unten eine weitere überflüssige } klammer steht. diese weggedacht, wäre es dann richtig? das vec = newVec steht ja ausserhalb der inneren for-schlleife oder muss es oben vor beginn der inneren for-schleife stehen, wie in deinem code? macht ja eignetlich keinen unterschied...

 

[SlaterB]

die Zeilen 37 bis 42 müssten alle einen weiter eingerückt werden,

Zeile 38 + 39 stehen in der for-i2-Schleife, das ist unnötig,
es reich, wenn die einmal am Ende der for-i-Schleife ausgeführt werden

immernoch fehlt wie gesagt: newVec muss ein neuer Vector sein,
wenn du nur vec = newVec; ausführst dann zeigen beide Variable auf denselben Vector,
das ist ungünstig, gelinde gesagt