Addition von Arrays über die Parameterliste

[LucaToni]

Hallo, versuche mich gerade an folgender Aufgabe.

Ich erstelle ein Array-Objekt/Vektor.

Nun möchte ich mit einer Methode ein weiteres Objekt übergeben und die beiden Vektoren addieren.

Mein Code sieht momentan so aus:

public class Vek
{
    double []vektor;
    int dimension;
    
    public Vek(double []vektor)
    {
        vektor=new double[vektor.length];
        dimension=vektor.length;
    }
    
    public double componentAt(int i)
    {
        return vektor[i];
    }
    public Vek add(Vek b)
    {   
    Vek res = new Vek();
            for(int i=0;i<vektor.length;i++)
                res.vektor[i]=vektor[i]+b.vektor[i];
            return res;
        }
}

Jemand ne Idee??
Vor allem was muss ich denn bei Vek res= new Vek(??????); als Parameter übergeben. Ich denke mein Fehler ist irgendwo in der Methode add(Vek b)

 

[TM69]

Da du dein Objekt instanzieren willst durch:

res= new Vek(??????);

und deine Konstruktor:

Vek(double []vektor)

lautet.

müsste der Aufruf wie folgt erfolgen

double[] daten;
Vek vek = new Vek(daten);

 

[MoxxiManagarm]

Im Konstruktor erstellst du zwar ein neues Array mit der entsprechenden Länge, übernimmst die Werte aber nicht.

 

[TM69]

da muss ich @MoxxiManagarm recht geben.

 

[kneitzel]

Und dann auch noch ein paar kleine Anregungen:
- Du speicherst die Länge des Arrays auch doppelt. Das Array kennt seine Länge (vektor.length), daher ist es nicht notwendig, diese noch einmal zu speichern. Merkst Du teilweise ja auch daran, dass Du die in add noch nicht einmal nutzt! Wenn Du das von Außen anfragen können willst, dann geht so ein Getter natürlich auch:

public int getDimensions() {
  return vektor.length;
}

- in add solltest Du prüfen, ob die beiden Vektoren die gleiche Größe haben. Sonst bekommst Du ggf. eine ArrayIndexOutOfBoundsException.

 

[LucaToni]

Danke für den Hinweis.

Im Konstruktor erstellst du zwar ein neues Array mit der entsprechenden Länge, übernimmst die Werte aber nicht.

public class Vek
{
    double []vektor;
    int dimension;
    
    public Vek(double []vektor)
    {   this.vektor=vektor;
        vektor=new double[vektor.length];
        dimension=vektor.length;
    }
    
    public double componentAt(int i)
    {
        return vektor[i];
    }
    public Vek add(Vek b)
    {   
    Vek res = new Vek(vektor);
            for(int i=0;i<vektor.length;i++)
                res.vektor[i]=vektor[i]+b.vektor[i];
              System.out.println(res);
        return res;
    }
}

Jetzt habe ich das Problem, dass die Schleife zwar richtig summiert aber meine neues Array in den anderen beiden gepeichert wird und nicht in einem neuen array mit dem namen res... -.-

 

[LucaToni]

Und dann auch noch ein paar kleine Anregungen:
- Du speicherst die Länge des Arrays auch doppelt. Das Array kennt seine Länge (vektor.length), daher ist es nicht notwendig, diese noch einmal zu speichern. Merkst Du teilweise ja auch daran, dass Du die in add noch nicht einmal nutzt! Wenn Du das von Außen anfragen können willst, dann geht so ein Getter natürlich auch:

public int getDimensions() {
  return vektor.length;
}

- in add solltest Du prüfen, ob die beiden Vektoren die gleiche Größe haben. Sonst bekommst Du ggf. eine ArrayIndexOutOfBoundsException.

Ja die Variable habe ich nicht mehr benutzt, dachte vielleicht brauche ich die im Laufe der Aufgabe irgendwann und hab die deshalb schonmal deklariert und zugewiesen. Aber danke für den Hinweis.

 

[MoxxiManagarm]

this.vektor=vektor;
vektor=new double[vektor.length]

DU vermischt 2 mögliche Variante.
Entweder du weißt den vektor direkt zu Oder du erstellst ein neues Array mit der Gleichen Größe und kopierst die Werte einzeln rein.

 

[LucaToni]

DU vermischt 2 mögliche Variante.
Entweder du weißt den vektor direkt zu Oder du erstellst ein neues Array mit der Gleichen Größe und kopierst die Werte einzeln rein.

OKay. hab jetzt nur this.vektor=vektor gelassen und die zuweisung entfernt.
Aber warum wird mein Ergebnis-Array nicht in einem neuen Array gespeichert sondern im Array welches ich übergebe? Habe doch in der Methode mit res = new Vektor(vektor) eine neues Objekt res erzeugt oder?

 

[TM69]

Ersteinmal bitte poste deinen aktuellen Stand, damit wir dir konkret etwas dazu sagen können.
Zum anderen soweit ich dich verstehe, hast du:

this.vektor = vektor

benutzt. Dieses kopiert keine Daten in ein anderes Array, sondern ist nur eine Zuweisung das Array zu verwenden.

 

[kneitzel]

Was @TM69 wohl meinte noch einmal etwas ausführlicher:
Vek res = new Vek(vektor);

Dadurch teilen sich der aktuelle Verktor und der neue Vektor das eine Array. Ein Array ist ein Referenz-Typ und daher hast Du in beiden Vektoren die Referenz auf das selbe Array.
Eine Änderung an dem Array ist somit in beiden Vektoren sichtbar.

 

[LucaToni]

Ersteinmal bitte poste deinen aktuellen Stand, damit wir dir konkret etwas dazu sagen können.
Zum anderen soweit ich dich verstehe, hast du:

this.vektor = vektor

benutzt. Dieses kopiert keine Daten in ein anderes Array, sondern ist nur eine Zuweisung das Array zu verwenden.

public class Vektor
{
    double []vektor;
    int dimension;
    Vektor v2;
    public Vektor(double []vektor)
    {   this.vektor=vektor;
        //dimension=vektor.length;
        
    }
    
    public double componentAt(int i)
    {
        return vektor[i];
    }
    
    public Vektor add(Vektor xyz)
    {   
      Vektor res = new Vektor(vektor);
      
                for(int i=0;i<vektor.length;i++)
                {
                    res.vektor[i]=vektor[i]+xyz.vektor[i];
                }
                System.out.println(res.vektor[0]); //Testausgabe res an der Stelle 0
      return res;
    }
    
  
        
        
    }