Array-Vector Problem

[tempomat]

Hi Leute,

ich habe ein Problem mit den Vectoren. Es geht um das Einlesen
von Werten aus einer Text-Datei und deren Weiterverarbeitung.

Hier die Text-Datei (werte.txt) mit den Werten:

4;2
32;1
78;4
34;2
18;0

Hier der Code:

public class Test {
  // habe hier ein Vector-Array erstellt, global
  private Vector werte[] = new Vector[5];

  public void dateiEinlesn() {

    for (int i = 0; i < werte.length; i++) {
      werte[i] = new Vector();
    }

    try {
      FileReader filename = new FileReader("werte.txt");
      BufferedReader file = new BufferedReader(filename);

      String zeile;
      String buffer[] = new String[2];

      int counter = 0;

      while ((zeile = file.readLine()) != null) {


        buffer = zeile.split(";");

        for (int k = 0; k < 2; k++) {
          werte[counter].addElement(Integer.parseInt((String) buffer[k]));
        }

        counter++;
      }
    } catch (Exception e) {
      System.out.println(e);
    }
  }

  // in dieser Methode habe ich auch
  // das Problem
  public void bearbeiten() {

    int zahl = 0;

    for (int i = 0; i < werte.length; i++) {
      [b]// es funktioniert nicht so[/b]
      zahl = werte[i].firstElement();

      [b]// auch nicht so[/b]
      zahl = werte[i].elementAt(1);
    }
  }
}

Fehler: siehe bearbeiten-Methode
incompatible types
found: java.lang.Object
required: int

Was mache ich falsch? Das sind doch alle int-Werte. Habe die doch
mit "Integer.parseInt((String) buffer[k])" als int-Werte eingefügt.

Gruß
tempomat

 

[Schandro]

Der Rückgabewert eines Vectors ohne Generics ist vom Typ Object. Benutzt Generics:
Vector <Integer> myVector = new Vector <Integer> ();
Dadurch kannst du nur Integer und int's (durch Autoboxing) in den Vector einfügen, Der Rückgabewert von elementAt ist ebenfalls Integer..

Oder du castest einfach

Übrigens ist ArrayList neuerer (und schneller, dafür nicht Threadsafe) als Vector

 

[tempomat]

Danke für die schnelle Antwort.

Werde mir diese Generics mal anschauen.

Aber das mit dem casten funktioniert nicht:

zahl = (int) werte.firstElement();
zahl = (int) werte.elementAt(1);

Was mache ich falsch?

Entweder ist es schon zu spät für mich, oder es liegt nur daran, dass ich ein Anfänger-Programmierer bin.

 

[Schandro]

Was soll an den cast nicht funktionieren?

 

[tempomat]

Bei...

int zahl = 0;

zahl = (int) werte.firstElement();

zahl = (int) werte.elementAt(1);


bekomme ich immer noch folgende Fehlermeldung:
Fehler:
incompatible types
found: java.lang.Object
required: int

Also, in der Java-Api steht:
E elementAt(int index)

Was ist "E" für ein Rückgabetyp? Was ist das für eine Klasse?

 

[Marco13]

Du solltest den Vector auf keinen Fall als Vector kennen. Es kann ruhig ein Vector SEIN, aber er sollte nur als "List" bekannt sein. Aber noch allgemein: Warum verwendest du einen Array von Vectoren? ???:L

 

[ARadauer]

Du solltest den Vector auf keinen Fall als Vector kennen. Es kann ruhig ein Vector SEIN, aber er sollte nur als "List" bekannt sein.

auf keinen Fall? übertreiben wir mal nicht...
die vorteile überhaupt für einen anfänger sind maginal...

 

[tfa]

auf keinen Fall? übertreiben wir mal nicht...
die vorteile überhaupt für einen anfänger sind maginal...

Warum sollte man es nicht von Anfang an gleich richtig machen?

 

[SlaterB]

wieso mit einem Dreirad fahren statt einem Rennrad?

 

[Marco13]

Ein IMHO nicht ganz passender Vergleich - dort "List" statt "Vector" und "get" statt "getElementAt" hinzuschreiben erfordert keine Übung und keine ... "höhere Qualifikation" (aber umgekehrt, vielleicht )

 

[ARadauer]

Warum sollte man es nicht von Anfang an gleich richtig machen?

ja eh...aber auf keinen Fall ist schon einwenig hart.

 

[Schandro]

Was ist "E" für ein Rückgabetyp? Was ist das für eine Klasse?

E ist ein Typ, den du per Generics festgelegt hast:
List <Integer> myList = new ArrayList <Integer> ();
bei myList steht "E" für "Integer"

 

[sparrow]

Ein IMHO nicht ganz passender Vergleich - dort "List" statt "Vector" und "get" statt "getElementAt" hinzuschreiben erfordert keine Übung und keine ... "höhere Qualifikation" (aber umgekehrt, vielleicht )

Ich befürchte leider, dass Posts dieser Form dem Threadstarter wenig weiterhelfen. Gut finde ich, dass mitgeteilt wird, dass es Verbesserungsmöglichkeiten gibt. Schade finde ich, dass ihm niemand sagt warum.

 

[Schandro]

Schade finde ich, dass ihm niemand sagt warum

Dann sags du ihm doch

Warum man List statt Vector als Typ der Referenz benutzen sollte:
Weil es immer besser ist, Objecte anhand ihres Interfaces anzusprechen. Dadurch kann man die tatsächliche Implemetierung später ändern, ohne die Benutzung der Variable verändern zu müssen. Beispiel:

Vector <String> myList = new Vector <String> ();
// irgendwo anders im code wird die Referenz benutzt:
myList.addElement("Hallo");
System.out.println(myList.elementAt(0));

Sieht eigentlich ganz ok aus.

Was ist aber, wenn du plötzlich von Vector auf ArrayList umsteigen willst, weil ArrayList performanceschonender arbeitet?
Dann müsstest du alle Zeilen in denen myList vorkommt ändern:

ArrayList <String> myList = new ArrayList <String> ();
// irgendwo anders im code wird die Referenz benutzt:
myList.add("Hallo");
System.out.println(myList.get(0));

Wenn du dagegen schreibst:

List <String> myList = new Vector <String> ();
// irgendwo anders im code wird die Referenz benutzt:
myList.add("Hallo");
System.out.println(myList.get(0));

Dann brauchst du, sobald du auf ArrayList umsteigen willst, nur die erste Zeile auf:

List <String> myList = new ArrayList <String> ();

ändern, es wird im restlichen Code keinen Compilerfehler geben da überall, wo die Referenz "myList" benutzt wurde, nur Methoden aufgerufen wurden die im Interface "List" existieren. Vector und ArrayList implementieren beide "List"

 

[tempomat]

Hi, ich bin es nochmal.

Habe mir diese Generics angeschaut. Eine sehr gute Idee, finde ich. Werde heute abend das mal umprogrammieren. Wird mir die Arbeit wesentlich leichter machen.

Mich interessiert nur noch eines und zwar wieso funktioniert das so nicht? Nur, damit ich das verstehe und weiß, worin der Fehler liegt.

Bei...

int zahl = 0;

zahl = (int) werte.firstElement();

zahl = (int) werte.elementAt(1);


bekomme ich immer noch folgende Fehlermeldung:
Fehler:
incompatible types
found: java.lang.Object
required: int

@Marco13: Warum verwendest du einen Array von Vectoren?
Ein Freund hat mir das empfohlen. Er meinte, dass dies besser sei als Vector in einem Vector. Was sind Alternativen?

 

[tempomat]

@Schandro: Danke für deine sehr ausführliche Erklärung mit dem "List".

 

[Schandro]

Man kann nicht auf primitive Datentypen casten, wenn du keine Generics benutzt müsstest du schreiben:

int i = (Integer) vec.elementAt(0);

Der Typ Integer wird dann automatisch nach int umgeboxt.

@Schandro: Danke für deine sehr ausführliche Erklärung mit dem "List".

bitte

@Marco13: Warum verwendest du einen Array von Vectoren?
Ein Freund hat mir das empfohlen. Er meinte, dass dies besser sei als Vector in einem Vector. Was sind Alternativen?

Es kommt drauf an was du machen willst, aber ein Vector voller Vectoren sollte eigentlich kein Problem sein.
(bzw. eine Vector mit lauter "List"en ):

List <List <Integer> > myPackage = new Vector <List <Integer> >();
myPackage.add(new Vector <Integer> () );

 

[Marco13]

int zahl = 0;

zahl = (int) werte.firstElement();

zahl = (int) werte.elementAt(1);


bekomme ich immer noch folgende Fehlermeldung:
Fehler:
incompatible types
found: java.lang.Object
required: int

Mit
zahl = ((Integer) werte.elementAt(1)).intValue();
würde das wohl gehen, aber mit Generics und Autoboxing sollte sich das erübrigen.

@Marco13: Warum verwendest du einen Array von Vectoren?
Ein Freund hat mir das empfohlen. Er meinte, dass dies besser sei als Vector in einem Vector. Was sind Alternativen?

Z.B. eine List in einer List... ;-) Arrays sind mit Generics halt ziemlich unhandlich, und ggf. dann wieder nicht Typsicher

 

[tempomat]

Okey, danke für eure Antworten.