Du verwendest einen veralteten Browser. Es ist möglich, dass diese oder andere Websites nicht korrekt angezeigt werden. Du solltest ein Upgrade durchführen oder ein alternativer Browser verwenden.
Ich habe folgenden Code geschrieben, um die Teiler einer Zahl in eine ArrayList zu packen und anschliessend herauszugeben:
Java:
import java.util.*;
public class Arithmetic {
private int number;
private ArrayList factors;
public Arithmetic (int i){
number = i;
this.split();
}
private void split(){
factors = new ArrayList();
for (int i = 2; i < number; i++)
if ((i%number)==0)
factors.add(i);
}
public String toString(){
String str[] = new String [factors.size ()];
factors.toArray (str);
return str[1];
}
}
Ich krieg aber eine IndexoutofBoundsexception...und hab keine Ahnung warum..?
Vielen Dank!
ein Programm ohne main, alles hängt von den an die Klasse übergebenen Parametern ab,
wie kann man das so posten, die genauen Testdaten weglassen?
in welcher Zeile ist die Exception?, noch eine Info, die man doch nicht weglassen kann..,
wenn in Zeile 19 dieses Codes, dann ist factors anscheinend leer,
prüfe doch nach, ob in Zeile 14 überhaupt ein i eingefügt wird:
if ((i%number)==0) {
factors.add(i);
System.out.println("füge nun Element i in Liste ein");
}
ist so ein Befehl wirklich schwer?
was soll das denn sein? Wo ist der Typ der Liste? Da müsste es doch warnungen regnen ohne Ende...
Java:
factors = new ArrayList();
wieso überhaupt ArrayList und nicht LinkedList? auf random access kommt es hier doch absolut nicht an...
Java:
String str[] = new String [factors.size ()];
factors.toArray (str);
toll, da oben hast du jetzt die generische typisierung weggelassen, und jetzt versuchst du einen haufen Integer in ein String-Array zu packen. Schuss in den Fuß wie aus einem Bilderbuch :autsch: da könnte man grad ein Plakat "SO NICHT, DESWEGEN GIBT'S JA GENERICS!" drüberhängen :noe:
Java:
return str[1];
IndexOutOfBounds kommt wohl hier, weil das Array insgesammt 0 oder 1 elemente enthält. Es müssten wohl 0 elemente sein, weil ansonsten ein ClassCastException der OutOfBoundsException zuvorkommen würde, weil du ja ein Integer in String[] reinpacken willst
Gibt es überhaupt irgendeinen sinn in dieser merkwürdigen Array-umformerei????:L
hey slaterb, nicht so harsch..ich bin noch völlig am anfang!
die vorgegebene aufgabe lautet, dass wir die teiler einer zahl in eine arraylist packen sollen.
Die vorgegebenen Instanzvariablen lauten eben private int number und private ArrayList factors.
Entsprechend sollen wir sowohl eine entsprechende split() methode wie auch einen konstruktor, der auf diesen zurückgreift schreiben. Auch muss man eine toString() Methode schreiben. Ok, für das Einfügen fehlt ein Parameter, das habe ich gesehn..aber sonst...?
Die Testklasse:
Java:
package arithmetic;
/**
*
*
*/
public class NewClass {
public static void main(String[] args){
Arithmetic ar = new Arithmetic(88);
System.out.println(ar);
weit darüber hinaus,
wie man im ersten Programm lernt: System.out.println("Hello World");
einen wichtigeren Befehl gibts nicht, daran sollst du nicht sparen,
genau anschauen was das Programm macht, jede Schleife und jede Berechnung ausgeben! nicht vermuten, rätsel/ andere rätseln lassen,
aber siehe auch die anderen Postings zwischendurch
welcher parameter, was einfügen? Wir haben hier 4 mehr oder minder garvierende semantische fehler aufgelistet, hast du sie alle behoben? Wenn du das getan hast, sollte es eigentlich funktionieren...
welcher parameter, was einfügen? Wir haben hier 4 mehr oder minder garvierende semantische fehler aufgelistet, hast du sie alle behoben? Wenn du das getan hast, sollte es eigentlich funktionieren...
Hier fehlte der Parameter, die entsprechende Zahl muss ja einem bestimmten Index zugeordnet werden:
Java:
if ((number%i)==0)
factors.add(i, j);
j++;}
die generische spezifikation habe ich weggelassen, weil ich mich an den vorgaben der übung halten muss! ich sehe nicht ganz, inwiefern diese hier ein muss ist. eine arraylist vom typ int ist ja sowieso nicht möglich..?
und rätseln lassen will ich niemanden, aber es sind viele details, und mir sind (noch) nicht alle zusammenhänge klar.
gröbster unsinn. Du willst doch einfach nur einen neuen faktor an die liste anhängen, fertig. So wie es jetzt ist, ist es falsch, das ist noch eine Stelle an der später ArrayOufOfBounds kommen. Mach die änderung wieder rückgängig, davor war's richtig.
die generische spezifikation habe ich weggelassen, weil ich mich an den vorgaben der übung halten muss!
Zum einen glaube ich dir nicht, dass dir jemand explizit einen gefährlichen Fehler aus dem jahre <2000 aufgezwungen hat (ansonsten bitte entsprechende stelle zitieren), und zum anderen muss man offensichtlich blödsinnige Vorgaben als tippfehler interpretieren und ignorieren: nobody is perfect, übungsgruppenleiter sind es auch nicht, und mit ein wenig gesundem menschenverstand bei der interpretation der aufgabenstellung dürfen die doch ja wohl rechnen...
ich sehe nicht ganz, inwiefern diese hier ein muss ist.
Du hast eine untypisierte Liste verwendet, diese mit Integers vollgestopft, und anschließend versucht diese in ein String[] rüberzukopieren, ohne mit der Augenbraue zu zucken. Jetzt sag mir wieso du nicht siehst, dass es ein Muss ist *mit500KiloBetonzaunpfahlWink* Wenn du selber nicht aufpasst, und den compiler nicht helfen lässt, und auf ratschläge nur bedingt hörst, wie willst du dann solche Probleme sonst vermeiden? :bahnhof:
eine arraylist vom typ int ist ja sowieso nicht möglich
das ist korrekt. Dafür gibt es in java eine (zwar nicht ganz schöne aber recht brauchbare) Lösung mit Wrapper-Klassen Integer. Das was du da hast ist eine ArrayList<Integer>, nur dass es verstümmelt aufgeschrieben ist, weswegen der compiler beim groben Fehler in der toString-methode nicht meckert.
und rätseln lassen will ich niemanden, aber es sind viele details, und mir sind (noch) nicht alle zusammenhänge klar.
Am Anfang rafft keiner etwas vollständig, das wäre nicht normal, das erwartet auch keiner.
Schreib die Typen korrekt dran.
Ändere den groben x%y <-> y%x fehler in der berechnung
Mach dieses "add(i,j)" wieder weg
Mach diese gesamte Array-geschichte aus der toString-methode weg. Wenn du collections benutzt, dann vergiss die arrays lieber komplett.
Dann ist alles wieder gut.
