Diverse Exceptions - Troubleshooting

[huckleberry]

Hallo Forum,

Adjazenzliste von hier: http://www.java-forum.org/java-basics-anfaenger-themen/95061-adjazenzliste-datenstruktur.html

TreeMap<Double, SpecialObject>  myLocalVIR; // als Parameter in Methode übergeben
Iterator<SpecialObject> itLocalVIR = myLocalVIR.values().iterator(); //Iterator
SpecialObject vpData = null;
//Mehr als 2 Knoten?		
if (myLocalVIR.size() > 1){
	log.info("Nodecount: 2 or more nodes.");
	MeineAdjazList tmpNode = new MeineAdjazList();
	vpData = myLocalVIR.firstEntry().getValue();
	if (vpData == null) vpData=itLocalVIR.next();
	while(!myLocalVIR.isEmpty()){
		// Put last/first (vpData) as FromNode
		tmpNode.setFromNode(vpData);
		// calc nearest next Node and their distances
		tmpNode.setToNode(calcRest(tmpNode.fromNode, myLocalVIR));
		tmpNode.setDistanceBetween(tmpNode.calcDistanceBetween());
		// remove that node from myLocalVIR
		myLocalVIR.remove(this.getKey(vpData, myLocalVIR));
		// add toNode and save for next round
		myReturnVIR.add(tmpNode);
		vpData = tmpNode.toNode; // tmpNode.fromNode = tmpNode.toNode

Ich habe hier noch kleinere Problemchen, welche ich auch noch nicht wirklich verstehe:

1. -- NullPointerException (Zeilen 8&9 oben). Mit if (myLocalVIR.size() > 1) führe ich Nachfolgenden Code doch nur aus, wenn ich mehr als 2 Knoten habe? Und wenn dem so ist, nöcht eich zB die nächste Value aus myLocalVIR holen. Meine Lösung ist doppelt gemoppelt, wobei ich anders eine NullPointerException bekomme ???:L
2. -- NullPointerException 2 (Zeile 15 oben). Wie man unten sieht, haben meine Adjazenzlisten die Struktur MeineAdjazList(SpecialObject fromNode, double distance, SpecialObject toNode). Anfangs ist meine tmpNode leer und dann wird fromNode gesetzt und dann toNode berechnet. Aus diesen beiden wird dann deren Abstand zueinander berechnet, womit dann alle 3 Werte komplett wären.. Wird die Exceptoin geworfen, weil ich den mittleren Wert aus den beiden anderen berechne?
3. -- NullPointerException 3 (Zeile 7 unten). Er setzt den mittleren Wert und gibt returnt es gleichzeitig zurück, da ich es beispielsweise für weitere Berechnungen brauchen könnte, und mir dann so nochmaliges auslesen aus tmpNode.getDistance() erspare..

public class MeineAdjazList {
	protected SpecialObject fromNode;
	protected double distanceBetween;
	protected SpecialObject toNode;
//... 
	public double calcDistanceBetween() {
		return this.distanceBetween = calcDistance(this.fromNode.getPosition(), this.toNode.getPosition());
	}

Vielen Dank für jegliche Hinweise und mfg Huck

 

[eRaaaa]

In Zeile 1 deklarierst du doch myLocalVIR nur

TreeMap<Double, SpecialObject>  myLocalVIR;

...erstellst aber garkeine TreeMap ? (new TreeMap()....) daher ist myLocalVIR halt null..und somit krachts auch direkt in Zeile2, da du eine Methode versuchst aufzurufen...

 

[huckleberry]

In Zeile 1 deklarierst du doch myLocalVIR nur

Sorry habe es etwas unglücklich kommentiert. Die TreeMap wird in dieser Methode als Parameter übergeben. Das wollte ich damit ausdrücken, deswegen habe ich so nochmal stehen gehabt..
Man kann davon ausgehen, dass diese existiert und entsprechend gefüllt ist.

Hier noch mal die gesamt Klasse:

public class MeineAdjazList {

	protected SpecialObject fromNode;
	protected double distanceBetween;
	protected SpecialObject toNode;
	
	public DHGroupCalc() {
		super();
	}

	public DHGroupCalc(SpecialObject fromNode, SpecialObject toNode) {
		super();
		this.fromNode = fromNode;
		this.toNode = toNode;
	}

	public DHGroupCalc(SpecialObject fromNode, double distanceBetween, SpecialObject toNode) {
		super();
		this.fromNode = fromNode;
		this.distanceBetween = distanceBetween;
		this.toNode = toNode;
	}


	public double calcDistanceBetween() {
		this.setDistanceBetween(calculateDistance(this.fromNode.getPosition(), this.toNode.getPosition()));
		return this.getDistanceBetween();
	}

	/**
	 * @return the distanceBetween
	 */
	public double getDistanceBetween() {
		return this.distanceBetween;
	}
	
	/**
	 * @param distanceBetween the distanceBetween to set
	 */
	public void setDistanceBetween(double distanceBetween) {
		this.distanceBetween = distanceBetween;
	}
	
	/**
	 * @return the fromNode
	 */
	public SpecialObject getFromNode() {
		return this.fromNode;
	}
	
	/**
	 * @param fromNode the fromNode to set
	 */
	public void setFromNode(SpecialObject fromNode) {
		this.fromNode = fromNode;
	}
	
	/**
	 * @return the toNode
	 */
	public SpecialObject getToNode() {
		return this.toNode;
	}
	
	/**
	 * @param toNode the toNode to set
	 */
	public void setToNode(SpecialObject toNode) {
		this.toNode = toNode;
	}
}

Ausserdem habe ich mit Eclipse alle getter und setter machen lassen, wobei

    public SpecialObject getToNode() {
        return toNode;
    }

ich alle mit "this." erweitert habe.. !?!?!? Denkfehler bei mir?

    public SpecialObject getToNode() {
        return this.toNode;
    }

 

[huckleberry]

/**
 * @param myLocalVIR
 */
public void printRing(ArrayList<MeineAdjazList> myLocalVIR) {
	MeineAdjazList adjl;
	for(int i=0; !myLocalVIR.isEmpty(); i++){
		adjl = myLocalVIR.get(i);
		log.info("From: "+adjl.fromNode.getId()+" is "+adjl.distanceBetween+" m away to "+adjl.toNode.getId());
		myLocalVIR.remove(i);
	}
} // end printKeyRing

NullPointerException für die for-Schleife in Zeile 6
Ich nehme mal an, dass es an !myLocalVIR.isEmpty() liegT?