Hilfe zum einem Problem mit Vector

[Batschi88]

Hallo Leute, ich hoff ihr könnt mir weiterhelfen.

public class Vector{

private double [] vec; // internes Feld
private int dim; // Dimension des Vektorfeldes


Der Konstruktor erzeugt einen Vektor mit dim Elementen und füllt den Vector mit den feld-Elementen auf: Beachten Sie dabei folgendes:
Ist die Dimension nicht positiv, so wird ein Vektor der Dimension 3 erzeugt. Ist das double - Feld kleiner als die Dimension des Vektors, so werden restliche Komponenten des Vektors mit 0 aufgestellt. Meine Implementierung ist aber beim compilieren fehlerhaft, ich hoff Ihr könnt mir weiterhelfen.

public Vector(int dim, double []feld){

if(dim<0){
this.dim=3;
vec=new double[dim];}
else{
this.dim=dim
vec=new double[dim];
for(int i=0;i<dim;i++){
vec[i]=feld[i];
if(i>feld.length){
vec[i]=0;}
}
}
}

Wo ist mein Fehler???

 

[hiti]

Hallo Batschi,

- ich würde vec=new double[dim]; erst machen, wenn dim sicher gesetzt ist.
die Zeile ist ja nicht Abhängig von der Bedingung dim<0...
- Mit : vec=new double[dim]; machst du ein double-Array.
Das brauchst du vermutlich nicht, denn feld ist ja schon eines.
Eher musst du ein Vector als Ergebnis erzeugen und füllen.
- Du gibst keinen Vector zurück.

Etwa so etwas:

  public Vector getFeldVector(int dim, double[] feld)
  {
    if(dim<0){
      dim=3;
    }
    // ergebnisvector erzeugen
    Vector vec = new Vector(dim);
    
    for(int i=0;i<dim;i++) {
      if (i>feld.length) {
        vec.add(null);
      }
      vec.add(new Double(feld[i]));
    }
    return vec;
  }

Gruss, hiti.

 

[SlaterB]

und noch zum ursprünglichen Problem:
Semikolon fehlte bei this.dim = dim
und eine Klammer zu wenig,
sowas solltest du unbedingt selber korrigieren können, was waren denn die Fehlermeldungen?

class Vector
{

    private double[] vec;
    private int dim;

    public Vector(int dim, double[] feld)
    {

        if (dim < 0)
        {
            this.dim = 3;
            vec = new double[dim];
        }
        else
        {
            this.dim = dim;
            vec = new double[dim];
            for (int i = 0; i < dim; i++)
            {
                vec[i] = feld[i];
                if (i > feld.length)
                {
                    vec[i] = 0;
                }
            }
        }
    }
}

 

[Batschi88]

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException:5

 

[SlaterB]

dein Code hat gar keine main-Methode,
soll man sich per Glaskugel dazudenken, dass du irgendwo mit Index 5 auf ein kleineres Array, z.B. vec eines Vectors, zugreifst?

 

[Batschi88]

public static void main(String [] args){

double [] feld={4.0,2.1,3.5,4.6,3.0};

Vector v=new Vector(10,feld);
System.out.println(v.vec[8]);

Dann bringt er mir die Exception:
xception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 5


Aber theoretisch müsste er mir doch, 0 ausgeben, da ich ja das rechtliche double feld, dann mit 0 auffülle oder???

 

[SlaterB]

oder auch nicht

.
                vec[i] = feld[i];
                if (i > feld.length)
                {
                    vec[i] = 0;
                }

->

.
                if (i >= feld.length)
                {
                    vec[i] = 0;
                } else {
                    vec[i] = feld[i];
                }

 

[Leroy42]

Wenn dein geposteter main-Code stimmt kann die Exception
ja nur in der Zeile

System.out.println(v.vec[8]);

geworfen werden.

Aber was für ein Instanzattribut soll den vec sein? :shock:

Und wieso soll der Index 5 falsch sein, wenn du nirgendwo mit
Index 5 referenzierst? :noe:

Ich schätze mal, du hast einen fallig völschen main-Code gepostet!

 

[Leroy42]

Ach du hast dir ja eine eigene Klasse Vector deklariert! :lol: