Null Pointer Exeption

[GroovyChuc]

Ich bin echt am verzweifeln.
Wo ist hier bitte eine NullPointerExeption? Ich habe das Array ganz klar definiert. Hier beispielhaft mit

arrayX[2] = 15.4;

public class asd 
{
private Double [] arrayX;
private Double [] arrayY;


public void Distance() {
for (Integer i=0; i<6; i++) {
Double [] arrayX = new Double [i];
}
 
arrayX[2] = 15.4;


}  
}

Mit freundlichen Grüssen

 

[pl4gu33]

Double [] arrayX = new Double ;

du erzeugst damit nur Arrays im Block der For-Schleife außerhalb sind sie nicht mehr sichtbar und es wir dein Attribut "private Double [] arrayX;" genommen, welches immer noch Null ist

wenn dann:

public class asd 
{
private Double [] arrayX;
private Double [] arrayY;
 
public void Distance() {
for (Integer i=0; i<6; i++) {
 arrayX = new Double [i];
}
 
arrayX[2] = 15.4;
 
}  
}

aber so überschreibst du immer wieder das Array und es wird auf die Größe des "i" des letzten Durchlaufs initalisiert ... was genau hast du denn mit der Schleife vor?

 

[GroovyChuc]

na ja so sieht mein eigentlicher Quellcode aus:

public class Punkt extends Form
{
public final Integer id;
private Double x;
private Double y;
private Double [] arrayX;
private Double [] arrayY;

public Punkt(Double x, Double y) {
counter++;
id= counter;
this.x = x;
this.y = y;
}

public String getPunkt(Integer id, Double x, Double y) {
return "Punkt" + "[" + id + "," +x +"," +y +"]";
}
 
public void getId(Integer id){
System.out.println(id);
}

public Double Distance() {
for (Integer i=0; i<1; i++) {
arrayX = new Double [i];
}
for (Integer i=0; i<1; i++) {
arrayY = new Double [i];
}  
arrayX[id] = x;
arrayY[id] = y;

// methode zur distanzberechnung
return ..;
}  
}

Ich möchte also von zwei Punkten P(x1,y1) und Q(x2,y2) die Distanz berechnen.
Das ganze mache ich dann mit Phytagoras

 

[Pentalon]

Hi Groovy

Du erstellst zwei Felder mit der grösse von 0:

for (Integer i=0; i<1; i++) {
arrayX = new Double [i];
}
for (Integer i=0; i<1; i++) {
arrayY = new Double [i];

Welchen Zweck hat die Schleife, sie läuft von 0 bis kleiner 1, man könnte sie weglassen.
Du erstellst aber ein Array mit grösse 0, da wird eine Zuweisung wohl einen NullPointer Exception werfen.

Für die Zählschleife verwendet man besser INT und nicht die Wrapperklasse Integer.

Pentalon

 

[pl4gu33]

ich glaub du gehst an die Sache falsch ran...

also erst einmal brauchst du ja zur Berechnung deiner Distanz 2 Punkte.... daher ist die Methode SO in der Punkt Klasse falsch, da du dort nur einen Punkt hast.... entweder gibst du als Parameter nen 2ten Punkt mit oder lagerst die ganze Berechnung ganz aus....

vll. so... hab dir das jetzt mal schnell runtergeschrieben ohne die Form Klasse und Ids ka wozu die waren

public class Punkt {
	private Double x;
	private Double y;

	public Punkt(Double x, Double y) {
		this.setX(x);
		this.setY(y);
	}

	public Double getY() {
		return y;
	}

	public void setY(Double y) {
		this.y = y;
	}

	public Double getX() {
		return x;
	}

	public void setX(Double x) {
		this.x = x;
	}

}

public class Berechnung {
	public static void main(String[] args){
		Punkt pkt1 = new Punkt(1.0,2.0);
		Punkt pkt2 = new Punkt(2.0,4.0);
		berechneDistanz(pkt1,pkt2);
	}
	
	public static void berechneDistanz(Punkt pkt1, Punkt pkt2){
		//Berechnung + Ausgabe der Distanz

	}
}

 

[GroovyChuc]

ich glaub du gehst an die Sache falsch ran...

also erst einmal brauchst du ja zur Berechnung deiner Distanz 2 Punkte.... daher ist die Methode SO in der Punkt Klasse falsch, da du dort nur einen Punkt hast.... entweder gibst du als Parameter nen 2ten Punkt mit oder lagerst die ganze Berechnung ganz aus....

vll. so... hab dir das jetzt mal schnell runtergeschrieben ohne die Form Klasse und Ids ka wozu die waren

public class Punkt {
	private Double x;
	private Double y;

	public Punkt(Double x, Double y) {
		this.setX(x);
		this.setY(y);
	}

	public Double getY() {
		return y;
	}

	public void setY(Double y) {
		this.y = y;
	}

	public Double getX() {
		return x;
	}

	public void setX(Double x) {
		this.x = x;
	}

}

public class Berechnung {
	public static void main(String[] args){
		Punkt pkt1 = new Punkt(1.0,2.0);
		Punkt pkt2 = new Punkt(2.0,4.0);
		berechneDistanz(pkt1,pkt2);
	}
	
	public static void berechneDistanz(Punkt pkt1, Punkt pkt2){
		//Berechnung + Ausgabe der Distanz

	}
}

ok nur zwei sachen verstehe ich nicht.
1. Wieso setzt du bei "new Punkt" Werte rein, die sollen im Konstruktor der Klasse Punkt eingelesen werden.
2. Wie weiss die Methode welches x oder von welchem Punkt Objekt es nehmen soll

 

[pl4gu33]

ok nur zwei sachen verstehe ich nicht.
1. Wieso setzt du bei "new Punkt" Werte rein, die sollen im Konstruktor der Klasse Punkt eingelesen werden.
2. Wie weiss die Methode welches x oder von welchem Punkt Objekt es nehmen soll

1. naja die Werte gibst du ja so dem Konstruktor. (wenn ich das jetzt richtig verstehe, was du meinst)

2. Du hast ja dann 2 Objekte auf denen du die .getY() .getX() Methode aufrufen kannst und so bekommst du von den verschiedenen Objekten die verschiedenen Werte.

 

[GroovyChuc]

Ok vielen Dank
Hab das ganze jetzt noch etwas abgeändert, da ich selber die Parameter im Konstruktor eingeben will.
Ich hab da echt viel zu umständlich gedacht

public class Punkt {
    private Double x;
    private Double y;
 
    public Punkt(Double x, Double y) {
        this.x = x;
        this.y = y;
    }
 
    public Double getY() {
        return y;
    }
 
    public void setY(Double y) {
        this.y = y;
    }
 
    public Double getX() {
        return x;
    }
 
    public void setX(Double x) {
        this.x = x;
    }
       
    public Double berechneDistanz(Punkt pkt1, Punkt pkt2){
      Double a = Math.abs(pkt1.getX()-pkt2.getX());
      Double b = Math.abs(pkt1.getY()-pkt2.getY());
      Double c = Math.sqrt(a*a+b*b); 
      return c;
 
    }
 
}

 

[Pentalon]

Bei Deiner Methode berechneDistanz brauchst Du eigendlich nur einen Parameter übergeben.
Du kannst dann die Distanz aus dem Punkt des Parameters und dem Punkt der eigenen Instanz berechnen.

    public Double berechneDistanz(Punkt p){
      Double a = Math.abs(p.getX()-this.getX());
      Double b = Math.abs(p.getY()-this.getY());
      Double c = Math.sqrt(a*a+b*b); 
      return c;
 
    }

Jetzt werden 2 Punkte erstellt und die Distanz berechnet:

Punkt p1 = new Punkt(10,10);
Punkt p2 = new Punkt(20,20);
System.out.println("Die Distanz ist= "+p1.berechneDistanz(p2);

Pentalon