Removed deprecated `Position`-Interface

src/de/tud/kom/p2psim/impl/network/modular/st/positioning/AppJobberPositioning.java

@@ -9,12 +9,12 @@ import java.io.UnsupportedEncodingException;
 import java.util.ArrayList;
 import java.util.Random;
 
-import de.tud.kom.p2psim.api.common.Position;
 import de.tud.kom.p2psim.api.common.SimHost;
 import de.tud.kom.p2psim.impl.network.modular.db.NetMeasurementDB;
 import de.tud.kom.p2psim.impl.network.modular.st.PositioningStrategy;
 import de.tud.kom.p2psim.impl.util.positioning.GeoSpherePosition;
 import de.tudarmstadt.maki.simonstrator.api.Randoms;
+import de.tudarmstadt.maki.simonstrator.api.component.sensor.location.Location;
 
 public class AppJobberPositioning implements PositioningStrategy {
 
@@ -25,7 +25,7 @@ public class AppJobberPositioning implements PositioningStrategy {
 	private Random rnd = Randoms.getRandom(AppJobberPositioning.class);
 
 	@Override
-	public Position getPosition(
+	public Location getPosition(
 			SimHost host,
 			NetMeasurementDB db,
 			NetMeasurementDB.Host hostMeta) {

src/de/tud/kom/p2psim/impl/network/modular/st/positioning/GNPPositioning.java

@@ -24,12 +24,12 @@ package de.tud.kom.p2psim.impl.network.modular.st.positioning;
 import java.util.List;
 import java.util.Vector;
 
-import de.tud.kom.p2psim.api.common.Position;
 import de.tud.kom.p2psim.api.common.SimHost;
 import de.tud.kom.p2psim.impl.network.modular.common.GNPToolkit;
 import de.tud.kom.p2psim.impl.network.modular.db.NetMeasurementDB;
 import de.tud.kom.p2psim.impl.network.modular.st.PositioningStrategy;
 import de.tud.kom.p2psim.impl.topology.PositionVector;
+import de.tudarmstadt.maki.simonstrator.api.component.sensor.location.Location;
 
 /**
  * Applies the (virtual) GNP position as the host's position
@@ -40,7 +40,7 @@ import de.tud.kom.p2psim.impl.topology.PositionVector;
 public class GNPPositioning implements PositioningStrategy {
 
 	@Override
-	public Position getPosition(SimHost host, NetMeasurementDB db,
+	public Location getPosition(SimHost host, NetMeasurementDB db,
 			NetMeasurementDB.Host hostMeta) {
 
 		if (hostMeta == null)
@@ -51,7 +51,7 @@ public class GNPPositioning implements PositioningStrategy {
 
 	}
 
-	public static class GNPPosition extends PositionVector implements Position {
+	public static class GNPPosition extends PositionVector {
 
 		private List<Double> coords;
 
@@ -63,7 +63,7 @@ public class GNPPositioning implements PositioningStrategy {
 		}
 
 		@Override
-		public double getDistance(Position netPosition) {
+		public double distanceTo(Location netPosition) {
 			if (!(netPosition instanceof PositionVector))
 				throw new AssertionError(
 						"Can not calculate distances between different position classes: "

src/de/tud/kom/p2psim/impl/network/modular/st/positioning/GeographicalPositioning.java

@@ -21,12 +21,12 @@
 
 package de.tud.kom.p2psim.impl.network.modular.st.positioning;
 
-import de.tud.kom.p2psim.api.common.Position;
 import de.tud.kom.p2psim.api.common.SimHost;
 import de.tud.kom.p2psim.impl.network.modular.common.GeoToolkit;
 import de.tud.kom.p2psim.impl.network.modular.db.NetMeasurementDB;
 import de.tud.kom.p2psim.impl.network.modular.st.PositioningStrategy;
 import de.tud.kom.p2psim.impl.topology.PositionVector;
+import de.tudarmstadt.maki.simonstrator.api.component.sensor.location.Location;
 
 /**
  * Applies a geographical position as defined by the GeoIP project.
@@ -49,8 +49,7 @@ public class GeographicalPositioning implements PositioningStrategy {
 		
 	}
 	
-	public class GeographicalPosition extends PositionVector implements
-			Position {
+	public class GeographicalPosition extends PositionVector {
 
 		private double latitude;
 		private double longitude;
@@ -70,7 +69,7 @@ public class GeographicalPositioning implements PositioningStrategy {
 		 * 
 		 */
 		@Override
-		public double getDistance(Position netPosition) {
+		public double distanceTo(Location netPosition) {
 			if (!(netPosition instanceof PositionVector))
 				throw new AssertionError(
 						"Can not calculate the distance between two different position classes: "
@@ -80,12 +79,6 @@ public class GeographicalPositioning implements PositioningStrategy {
 			return GeoToolkit.getDistance(this.latitude, this.longitude,
 					other.getEntry(1), other.getEntry(0));
 		}
-		
-		@Override
-		public double getAngle(Position target) {
-			throw new AssertionError(
-					"getAngle is not defined for this Position-Type");
-		}
 
 		@Override
 		public int getTransmissionSize() {

src/de/tud/kom/p2psim/impl/network/modular/st/positioning/PlanePositioning.java

@@ -20,11 +20,11 @@
 
 package de.tud.kom.p2psim.impl.network.modular.st.positioning;
 
-import de.tud.kom.p2psim.api.common.Position;
 import de.tud.kom.p2psim.api.common.SimHost;
 import de.tud.kom.p2psim.impl.network.modular.db.NetMeasurementDB;
 import de.tud.kom.p2psim.impl.network.modular.st.PositioningStrategy;
 import de.tud.kom.p2psim.impl.topology.placement.PositionDistribution;
+import de.tudarmstadt.maki.simonstrator.api.component.sensor.location.Location;
 
 /**
  * Positioning on a 2D-Plane, used for all Movement-Enabled Routing-Scenarios
@@ -42,7 +42,7 @@ public class PlanePositioning implements PositioningStrategy {
 	}
 
 	@Override
-	public Position getPosition(
+	public Location getPosition(
 			SimHost host,
 			NetMeasurementDB db,
 			de.tud.kom.p2psim.impl.network.modular.db.NetMeasurementDB.Host hostMeta) {

src/de/tud/kom/p2psim/impl/network/modular/st/positioning/SimpleEuclidianPositioning.java

@@ -22,13 +22,13 @@ package de.tud.kom.p2psim.impl.network.modular.st.positioning;
 
 import java.util.Random;
 
-import de.tud.kom.p2psim.api.common.Position;
 import de.tud.kom.p2psim.api.common.SimHost;
 import de.tud.kom.p2psim.impl.network.modular.db.NetMeasurementDB;
 import de.tud.kom.p2psim.impl.network.modular.st.PositioningStrategy;
 import de.tud.kom.p2psim.impl.network.simple.SimpleSubnet;
 import de.tud.kom.p2psim.impl.topology.PositionVector;
 import de.tudarmstadt.maki.simonstrator.api.Randoms;
+import de.tudarmstadt.maki.simonstrator.api.component.sensor.location.Location;
 
 /**
  * Implementation of the <code>NetPosition</code> interface representing a two
@@ -48,7 +48,7 @@ public class SimpleEuclidianPositioning implements PositioningStrategy {
 	 * @see de.tud.kom.p2psim.impl.network.modular.st.PositioningStrategy#getPosition(de.tud.kom.p2psim.api.common.Host, de.tud.kom.p2psim.impl.network.modular.db.NetMeasurementDB, de.tud.kom.p2psim.impl.network.modular.db.NetMeasurementDB.Host)
 	 */
 	@Override
-	public Position getPosition(
+	public Location getPosition(
 			SimHost host, 
 			NetMeasurementDB db,
 			NetMeasurementDB.Host hostMeta) {
@@ -59,7 +59,7 @@ public class SimpleEuclidianPositioning implements PositioningStrategy {
 
 	}
 
-	public class SimpleEuclidianPosition extends PositionVector implements Position {
+	public class SimpleEuclidianPosition extends PositionVector {
 
 		private double xPos;
 
@@ -93,7 +93,8 @@ public class SimpleEuclidianPositioning implements PositioningStrategy {
 		 * de.tud.kom.p2psim.api.network.NetPosition#getDistance(de.tud.kom.
 		 * p2psim .api.network.NetPosition)
 		 */
-		public double getDistance(Position ep) {
+		@Override
+		public double distanceTo(Location ep) {
 			double xDiff = 0;
 			double yDiff = 0;
 			SimpleEuclidianPosition point = (SimpleEuclidianPosition) ep;
@@ -118,11 +119,6 @@ public class SimpleEuclidianPositioning implements PositioningStrategy {
 			return Math.pow(Math.pow(xDiff, 2) + Math.pow(yDiff, 2), 0.5);
 		}
 
-		@Override
-		public double getAngle(Position target) {
-			throw new AssertionError("getAngle is not defined for this Position-Type");
-		}
-
 		@Override
 		public int getTransmissionSize() {
 			return 16; // 2 * double

src/de/tud/kom/p2psim/impl/network/modular/st/positioning/TorusPositioning.java

@@ -23,12 +23,12 @@ package de.tud.kom.p2psim.impl.network.modular.st.positioning;
 
 import java.util.Random;
 
-import de.tud.kom.p2psim.api.common.Position;
 import de.tud.kom.p2psim.api.common.SimHost;
 import de.tud.kom.p2psim.impl.network.modular.db.NetMeasurementDB;
 import de.tud.kom.p2psim.impl.network.modular.st.PositioningStrategy;
 import de.tud.kom.p2psim.impl.topology.PositionVector;
 import de.tudarmstadt.maki.simonstrator.api.Randoms;
+import de.tudarmstadt.maki.simonstrator.api.component.sensor.location.Location;
 
 /**
  * Applies a uniformly-distributed random position on a 2-or multi-dimensional
@@ -48,7 +48,7 @@ public class TorusPositioning implements PositioningStrategy {
 	Random rand = Randoms.getRandom(TorusPosition.class);
 
 	@Override
-	public Position getPosition(SimHost host, NetMeasurementDB db,
+	public Location getPosition(SimHost host, NetMeasurementDB db,
 			NetMeasurementDB.Host hostMeta) {
 
 		double[] rawPos = getRawTorusPositionFor(host);
@@ -93,7 +93,7 @@ public class TorusPositioning implements PositioningStrategy {
 		this.halfTorusDimensionSize = torusDimensionSize * 0.5d;
 	}
 
-	public class TorusPosition extends PositionVector implements Position {
+	public class TorusPosition extends PositionVector {
 
 
 		TorusPosition(double[] rawPos) {
@@ -101,7 +101,7 @@ public class TorusPositioning implements PositioningStrategy {
 		}
 
 		@Override
-		public double getDistance(Position netPosition) {
+		public double distanceTo(Location netPosition) {
 			if (!(netPosition instanceof PositionVector))
 				throw new AssertionError(
 						"Can not calculate distances between different position classes: "

src/de/tud/kom/p2psim/impl/network/routed/RoutedNetLayer.java

@@ -28,7 +28,6 @@ import java.util.Map;
 import de.tud.kom.p2psim.api.analyzer.ConnectivityAnalyzer;
 import de.tud.kom.p2psim.api.analyzer.MessageAnalyzer.Reason;
 import de.tud.kom.p2psim.api.analyzer.NetlayerAnalyzer;
-import de.tud.kom.p2psim.api.common.Position;
 import de.tud.kom.p2psim.api.common.SimHost;
 import de.tud.kom.p2psim.api.common.SimHostComponent;
 import de.tud.kom.p2psim.api.linklayer.LinkLayer;
@@ -57,6 +56,7 @@ import de.tudarmstadt.maki.simonstrator.api.component.network.Bandwidth;
 import de.tudarmstadt.maki.simonstrator.api.component.network.NetID;
 import de.tudarmstadt.maki.simonstrator.api.component.network.NetInterface;
 import de.tudarmstadt.maki.simonstrator.api.component.network.NetworkComponent;
+import de.tudarmstadt.maki.simonstrator.api.component.sensor.location.Location;
 import de.tudarmstadt.maki.simonstrator.api.component.transport.ConnectivityListener;
 
 /**
@@ -548,7 +548,7 @@ public class RoutedNetLayer implements SimNetworkComponent, NetworkComponent,
 		}
 
 		@Override
-		public Position getNetPosition() {
+		public Location getNetPosition() {
 			return host.getTopologyComponent().getRealPosition();
 		}
 

src/de/tud/kom/p2psim/impl/network/simple/SimpleNetLayer.java

@@ -19,11 +19,10 @@
  */
 
 
-
-package de.tud.kom.p2psim.impl.network.simple;
-
+
+package de.tud.kom.p2psim.impl.network.simple;
+
 import de.tud.kom.p2psim.api.analyzer.MessageAnalyzer.Reason;
-import de.tud.kom.p2psim.api.common.Position;
 import de.tud.kom.p2psim.api.common.SimHost;
 import de.tud.kom.p2psim.api.network.BandwidthImpl;
 import de.tud.kom.p2psim.api.network.NetMessage;
@@ -34,105 +33,106 @@ import de.tud.kom.p2psim.impl.transport.AbstractTransMessage;
 import de.tudarmstadt.maki.simonstrator.api.Message;
 import de.tudarmstadt.maki.simonstrator.api.Monitor;
 import de.tudarmstadt.maki.simonstrator.api.component.network.NetID;
-
-public class SimpleNetLayer extends AbstractNetLayer {
-
-	private double currentDownBandwidth;
-
-	private double currentUpBandwidth;
-
-	private final SimpleSubnet subNet;
-
-	public SimpleNetLayer(SimHost host, SimpleSubnet subNet, SimpleNetID netID,
-			Position netPosition, BandwidthImpl bandwidth) {
-		super(host, netID, bandwidth, netPosition, null);
-		this.subNet = subNet;
-		subNet.registerNetLayer(this);
-	}
-
-	public boolean isSupported(TransProtocol transProtocol) {
-		if (transProtocol.equals(TransProtocol.UDP))
-			return true;
-		else
-			return false;
-	}
-
-	public void send(Message msg, NetID receiver, NetProtocol netProtocol) {
-		// outer if-else-block is used to avoid sending although the host is
-		// offline
-		if (this.isOnline()) {
-			TransProtocol usedTransProtocol = ((AbstractTransMessage) msg)
-					.getProtocol();
-			if (this.isSupported(usedTransProtocol)) {
+import de.tudarmstadt.maki.simonstrator.api.component.sensor.location.Location;
+
+public class SimpleNetLayer extends AbstractNetLayer {
+
+	private double currentDownBandwidth;
+
+	private double currentUpBandwidth;
+
+	private final SimpleSubnet subNet;
+
+	public SimpleNetLayer(SimHost host, SimpleSubnet subNet, SimpleNetID netID,
+			Location netPosition, BandwidthImpl bandwidth) {
+		super(host, netID, bandwidth, netPosition, null);
+		this.subNet = subNet;
+		subNet.registerNetLayer(this);
+	}
+
+	public boolean isSupported(TransProtocol transProtocol) {
+		if (transProtocol.equals(TransProtocol.UDP))
+			return true;
+		else
+			return false;
+	}
+
+	public void send(Message msg, NetID receiver, NetProtocol netProtocol) {
+		// outer if-else-block is used to avoid sending although the host is
+		// offline
+		if (this.isOnline()) {
+			TransProtocol usedTransProtocol = ((AbstractTransMessage) msg)
+					.getProtocol();
+			if (this.isSupported(usedTransProtocol)) {
 				NetMessage netMsg = new SimpleNetMessage(msg, receiver,
-						getLocalInetAddress(),
+						getLocalInetAddress(),
 						netProtocol);
 				if (hasAnalyzer) {
 					netAnalyzerProxy
 							.netMsgEvent(netMsg, getHost(), Reason.SEND);
-				}
-				subNet.send(netMsg);
-			} else {
-				throw new IllegalArgumentException("Transport protocol "
-						+ usedTransProtocol
-						+ " not supported by this NetLayer implementation.");
-			}
-		} else {
-			int assignedMsgId = subNet.determineTransMsgNumber(msg);
+				}
+				subNet.send(netMsg);
+			} else {
+				throw new IllegalArgumentException("Transport protocol "
+						+ usedTransProtocol
+						+ " not supported by this NetLayer implementation.");
+			}
+		} else {
+			int assignedMsgId = subNet.determineTransMsgNumber(msg);
 			Monitor.log(SimpleNetLayer.class, Monitor.Level.DEBUG,
-					"During send: Assigning MsgId " + assignedMsgId
-					+ " to dropped message");
-			((AbstractTransMessage) msg).setCommId(assignedMsgId);
+					"During send: Assigning MsgId " + assignedMsgId
+					+ " to dropped message");
+			((AbstractTransMessage) msg).setCommId(assignedMsgId);
 			NetMessage netMsg = new SimpleNetMessage(msg, receiver,
-					getLocalInetAddress(),
+					getLocalInetAddress(),
 					netProtocol);
 			if (hasAnalyzer) {
 				netAnalyzerProxy.netMsgEvent(netMsg, getHost(), Reason.DROP);
-			}
-		}
-
-	}
-
-	@Override
-	public String toString() {
+			}
+		}
+
+	}
+
+	@Override
+	public String toString() {
 		return "NetLayer(netID=" + getLocalInetAddress() + ", "
-				+ (isOffline() ? "online" : "offline") + ")";
-	}
-
-	@Override
-	public int hashCode() {
-		final int PRIME = 31;
-		int result = 1;
-		long temp;
-		temp = Double.doubleToLongBits(currentDownBandwidth);
-		result = PRIME * result + (int) (temp ^ (temp >>> 32));
-		temp = Double.doubleToLongBits(currentUpBandwidth);
-		result = PRIME * result + (int) (temp ^ (temp >>> 32));
-		result = PRIME * result + ((subNet == null) ? 0 : subNet.hashCode());
-		return result;
-	}
-
-	@Override
-	public boolean equals(Object obj) {
-		if (this == obj)
-			return true;
-		if (obj == null)
-			return false;
-		if (getClass() != obj.getClass())
-			return false;
-		final SimpleNetLayer other = (SimpleNetLayer) obj;
-		if (Double.doubleToLongBits(currentDownBandwidth) != Double
-				.doubleToLongBits(other.currentDownBandwidth))
-			return false;
-		if (Double.doubleToLongBits(currentUpBandwidth) != Double
-				.doubleToLongBits(other.currentUpBandwidth))
-			return false;
-		if (subNet == null) {
-			if (other.subNet != null)
-				return false;
-		} else if (!subNet.equals(other.subNet))
-			return false;
-		return true;
-	}
-
-}
+				+ (isOffline() ? "online" : "offline") + ")";
+	}
+
+	@Override
+	public int hashCode() {
+		final int PRIME = 31;
+		int result = 1;
+		long temp;
+		temp = Double.doubleToLongBits(currentDownBandwidth);
+		result = PRIME * result + (int) (temp ^ (temp >>> 32));
+		temp = Double.doubleToLongBits(currentUpBandwidth);
+		result = PRIME * result + (int) (temp ^ (temp >>> 32));
+		result = PRIME * result + ((subNet == null) ? 0 : subNet.hashCode());
+		return result;
+	}
+
+	@Override
+	public boolean equals(Object obj) {
+		if (this == obj)
+			return true;
+		if (obj == null)
+			return false;
+		if (getClass() != obj.getClass())
+			return false;
+		final SimpleNetLayer other = (SimpleNetLayer) obj;
+		if (Double.doubleToLongBits(currentDownBandwidth) != Double
+				.doubleToLongBits(other.currentDownBandwidth))
+			return false;
+		if (Double.doubleToLongBits(currentUpBandwidth) != Double
+				.doubleToLongBits(other.currentUpBandwidth))
+			return false;
+		if (subNet == null) {
+			if (other.subNet != null)
+				return false;
+		} else if (!subNet.equals(other.subNet))
+			return false;
+		return true;
+	}
+
+}

src/de/tud/kom/p2psim/impl/topology/DefaultTopologyComponent.java

@@ -28,7 +28,6 @@ import java.util.Random;
 import java.util.Set;
 
 import de.tud.kom.p2psim.api.common.HostProperties;
-import de.tud.kom.p2psim.api.common.Position;
 import de.tud.kom.p2psim.api.common.SimHost;
 import de.tud.kom.p2psim.api.linklayer.mac.MacAddress;
 import de.tud.kom.p2psim.api.linklayer.mac.MacLayer;
@@ -178,7 +177,7 @@ public class DefaultTopologyComponent implements TopologyComponent {
 	@Override
 	public void positionChanged() {
 		for (PositionListener listener : positionListener) {
-			if (listener.getPosition().getDistance(position) < listener
+			if (listener.getPosition().distanceTo(position) < listener
 					.getRadius()) {
 				listener.reachedPosition();
 			}
@@ -212,14 +211,6 @@ public class DefaultTopologyComponent implements TopologyComponent {
 		return position;
 	}
 
-	@Override
-	public Position getPosition(PositionAccuracy accuracy) {
-		/*
-		 * TODO :)
-		 */
-		return position;
-	}
-
 	@Override
 	public Topology getTopology() {
 		return topology;

src/de/tud/kom/p2psim/impl/topology/PositionVector.java

@@ -25,7 +25,6 @@ import java.util.Arrays;
 
 import com.vividsolutions.jts.geom.Coordinate;
 
-import de.tud.kom.p2psim.api.common.Position;
 import de.tudarmstadt.maki.simonstrator.api.component.sensor.location.Location;
 
 /**
@@ -50,7 +49,7 @@ import de.tudarmstadt.maki.simonstrator.api.component.sensor.location.Location;
  * @author Bjoern Richerzhagen
  * @version 1.0, 04/25/2011
  */
-public class PositionVector implements Position, Location {
+public class PositionVector implements Location {
 
 	private int dimensions;
 
@@ -255,55 +254,6 @@ public class PositionVector implements Position, Location {
 		return result;
 	}
 
-	@Override
-	public double getDistance(Position position) {
-		if (position instanceof PositionVector) {
-			PositionVector pv = (PositionVector) position;
-			if (pv.getDimensions() == getDimensions()) {
-				double dist = 0;
-				for (int i = 0; i < dimensions; i++) {
-					// faster as Math.pow
-					dist += (pv.getEntry(i) - getEntry(i))
-							* (pv.getEntry(i) - getEntry(i));
-				}
-				return Math.sqrt(dist);
-			} else {
-				throw new AssertionError(
-						"Can not compute distance between Vectors of different length!");
-			}
-		} else {
-			throw new AssertionError("Incompatible Types!");
-		}
-	}
-
-	@Override
-	public double getAngle(Position target) {
-		if (target instanceof PositionVector) {
-			PositionVector t = (PositionVector) target;
-			/*
-			 * Calculates the angle using atan2 - this implies that the first
-			 * two dimensions in your vector are the plane you are interested
-			 * in.
-			 */
-			return Math.atan2(t.getEntry(1) - this.getEntry(1), t.getEntry(0)
-					- this.getEntry(0));
-		} else {
-			throw new AssertionError(
-					"Can only calculate an Angle on elements of type position vector");
-		}
-	}
-
-	@Override
-	public Position getTarget(double distance, double angle) {
-		/*
-		 * The call to "clone" ensures correct typing
-		 */
-		PositionVector newPos = clone();
-		newPos.setEntry(0, newPos.getEntry(0) + distance * Math.cos(angle));
-		newPos.setEntry(1, newPos.getEntry(1) + distance * Math.sin(angle));
-		return newPos;
-	}
-
 	@Override
 	public int getTransmissionSize() {
 		return getDimensions() * 8;
@@ -496,13 +446,39 @@ public class PositionVector implements Position, Location {
 
 	@Override
 	public double distanceTo(Location dest) {
-		assert (dest instanceof PositionVector);
-		return this.getDistance((PositionVector) dest);
+		if (dest instanceof PositionVector) {
+			PositionVector pv = (PositionVector) dest;
+			if (pv.getDimensions() == getDimensions()) {
+				double dist = 0;
+				for (int i = 0; i < dimensions; i++) {
+					// faster as Math.pow
+					dist += (pv.getEntry(i) - getEntry(i))
+							* (pv.getEntry(i) - getEntry(i));
+				}
+				return Math.sqrt(dist);
+			} else {
+				throw new AssertionError(
+						"Can not compute distance between Vectors of different length!");
+			}
+		} else {
+			throw new AssertionError("Incompatible Types!");
+		}
 	}
 
 	@Override
 	public float bearingTo(Location dest) {
-		assert (dest instanceof PositionVector);
-		return (float) ((this.getAngle((PositionVector) dest) - (Math.PI / 2)) * 180 / Math.PI);
+		if (dest instanceof PositionVector) {
+			PositionVector t = (PositionVector) dest;
+			/*
+			 * Calculates the angle using atan2 - this implies that the first
+			 * two dimensions in your vector are the plane you are interested
+			 * in.
+			 */
+			return (float) Math.atan2(t.getEntry(1) - this.getEntry(1),
+					t.getEntry(0) - this.getEntry(0));
+		} else {
+			throw new AssertionError(
+					"Can only calculate an Angle on elements of type position vector");
+		}
 	}
 }

src/de/tud/kom/p2psim/impl/topology/TopologyFactory.java

@@ -39,6 +39,7 @@ import de.tud.kom.p2psim.impl.network.modular.db.NetMeasurementDB;
 import de.tud.kom.p2psim.impl.topology.movement.AbstractWaypointMovementModel;
 import de.tud.kom.p2psim.impl.topology.placement.GNPPlacement;
 import de.tud.kom.p2psim.impl.topology.views.latency.GNPLatency;
+import de.tudarmstadt.maki.simonstrator.api.Binder;
 import de.tudarmstadt.maki.simonstrator.api.Host;
 import de.tudarmstadt.maki.simonstrator.api.Monitor;
 import de.tudarmstadt.maki.simonstrator.api.Monitor.Level;
@@ -88,7 +89,8 @@ public class TopologyFactory implements HostComponentFactory {
 	@XMLConfigurableConstructor({ "worldX", "worldY" })
 	public TopologyFactory(double worldX, double worldY) {
 		topo = new DefaultTopology(new PositionVector(worldX, worldY));
-		de.tud.kom.p2psim.impl.topology.Topology.setTopology(topo);
+		// Make the topology component available globally
+		Binder.registerComponent(topo);
 	}
 
 	@Override

src/de/tud/kom/p2psim/impl/topology/movement/AbstractMovementModel.java

@@ -26,11 +26,12 @@ import java.util.List;
 import java.util.Random;
 import java.util.Set;
 
+import de.tud.kom.p2psim.api.topology.Topology;
 import de.tud.kom.p2psim.api.topology.movement.MovementListener;
 import de.tud.kom.p2psim.api.topology.movement.MovementModel;
 import de.tud.kom.p2psim.api.topology.movement.MovementSupported;
 import de.tud.kom.p2psim.impl.topology.PositionVector;
-import de.tud.kom.p2psim.impl.topology.Topology;
+import de.tudarmstadt.maki.simonstrator.api.Binder;
 import de.tudarmstadt.maki.simonstrator.api.Event;
 import de.tudarmstadt.maki.simonstrator.api.EventHandler;
 import de.tudarmstadt.maki.simonstrator.api.Randoms;
@@ -87,7 +88,8 @@ public abstract class AbstractMovementModel implements MovementModel {
 	@Override
 	public void addComponent(MovementSupported component) {
 		if (worldDimensions == null) {
-			worldDimensions = Topology.getWorldDimension();
+			worldDimensions = Binder.getComponentOrNull(Topology.class)
+					.getWorldDimensions();
 		}
 		components.add(component);
 	}

src/de/tud/kom/p2psim/impl/topology/movement/CsvMovement.java

@@ -27,26 +27,19 @@ import java.util.LinkedHashMap;
 import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
-import java.util.Random;
 import java.util.Set;
 
-import org.jfree.data.io.CSV;
-
 import de.tud.kom.p2psim.api.common.SimHost;
-import de.tud.kom.p2psim.api.common.SimHostComponent;
 import de.tud.kom.p2psim.api.network.SimNetInterface;
+import de.tud.kom.p2psim.api.topology.Topology;
 import de.tud.kom.p2psim.api.topology.movement.MovementInformation;
-import de.tud.kom.p2psim.api.topology.movement.MovementModel;
 import de.tud.kom.p2psim.api.topology.movement.MovementSupported;
-import de.tud.kom.p2psim.impl.network.gnp.topology.Host;
 import de.tud.kom.p2psim.impl.simengine.Simulator;
 import de.tud.kom.p2psim.impl.topology.PositionVector;
-import de.tud.kom.p2psim.impl.topology.Topology;
 import de.tud.kom.p2psim.impl.topology.movement.modular.ModularMovementModel;
 import de.tud.kom.p2psim.impl.topology.movement.modular.attraction.AttractionPoint;
 import de.tud.kom.p2psim.impl.topology.movement.modular.transition.FixedAssignmentStrategy;
-import de.tud.kom.p2psim.impl.topology.movement.modular.transition.TransitionStrategy;
-import de.tudarmstadt.maki.simonstrator.api.Randoms;
+import de.tudarmstadt.maki.simonstrator.api.Binder;
 import de.tudarmstadt.maki.simonstrator.api.Time;
 import de.tudarmstadt.maki.simonstrator.api.util.XMLConfigurableConstructor;
 
@@ -82,7 +75,8 @@ public class CsvMovement extends AbstractMovementModel {
 	@XMLConfigurableConstructor({ "movementPointsFile" , "minMovementSpeed", "maxMovementSpeed" })
 	public CsvMovement(String movementPointsFile, double minMovementSpeed, double maxMovementSpeed) {
 		super();
-		this.worldDimensions = Topology.getWorldDimension();
+		this.worldDimensions = Binder.getComponentOrNull(Topology.class)
+				.getWorldDimensions();
 		this.file = movementPointsFile;
 		this.readPathInfos = new LinkedList<LinkedList<CsvPathInfo>>();
 		this.stateInfo = new LinkedHashMap<MovementSupported, CsvMovementInfo>();
@@ -144,7 +138,7 @@ public class CsvMovement extends AbstractMovementModel {
 
 		double distancePerMoveOperation = pathInfo.getSpeed()
 				* getTimeBetweenMoveOperations() / Time.SECOND;
-		double distance = actPos.getDistance(targetPos);
+		double distance = actPos.distanceTo(targetPos);
 
 		steps = (int) Math.round(distance / distancePerMoveOperation);
 

src/de/tud/kom/p2psim/impl/topology/movement/TargetMovement.java

@@ -59,7 +59,7 @@ public class TargetMovement extends AbstractMovementModel {
 			PositionVector rVec = new PositionVector(x, y);
 			rVec = rVec.minus(pos);
 
-			double length = rVec.getDistance(new PositionVector(0, 0));
+			double length = rVec.distanceTo(new PositionVector(0, 0));
 
 			PositionVector vec;
 			if (length > distance) {

src/de/tud/kom/p2psim/impl/topology/movement/local/OnlineMapQuestMovement.java

@@ -25,6 +25,7 @@ import java.io.InputStream;
 import java.net.MalformedURLException;
 import java.net.URL;
 import java.util.HashMap;
+
 import org.apache.commons.io.IOUtils;
 import org.json.JSONArray;
 import org.json.JSONException;
@@ -34,12 +35,13 @@ import com.graphhopper.util.PointList;
 import com.graphhopper.util.shapes.GHPoint;
 import com.graphhopper.util.shapes.GHPoint3D;
 
+import de.tud.kom.p2psim.api.topology.Topology;
 import de.tud.kom.p2psim.api.topology.movement.MovementSupported;
 import de.tud.kom.p2psim.impl.topology.PositionVector;
-import de.tud.kom.p2psim.impl.topology.Topology;
 import de.tud.kom.p2psim.impl.topology.movement.modularosm.GPSCalculation;
 import de.tud.kom.p2psim.impl.util.Either;
 import de.tud.kom.p2psim.impl.util.Left;
+import de.tudarmstadt.maki.simonstrator.api.Binder;
 
 /**
  * This movement strategy uses the data from MapQuest
@@ -63,7 +65,8 @@ public class OnlineMapQuestMovement extends AbstractLocalMovementStrategy {
 										//if the distance is smaller than the given one, the node will choose the next point in the list
 	
 	public OnlineMapQuestMovement() {
-		this.worldDimensions = Topology.getWorldDimension();
+		this.worldDimensions = Binder.getComponentOrNull(Topology.class)
+				.getWorldDimensions();
 		latLeft = GPSCalculation.getLatLower();
 		latRight = GPSCalculation.getLatUpper();
 		lonLeft = GPSCalculation.getLonLeft();
@@ -74,7 +77,8 @@ public class OnlineMapQuestMovement extends AbstractLocalMovementStrategy {
 			PositionVector destination) {
 		PositionVector newPosition = null;
 		
-		if (destination.getDistance(comp.getRealPosition()) < getMovementSpeed(comp)) {
+		if (destination
+				.distanceTo(comp.getRealPosition()) < getMovementSpeed(comp)) {
 			newPosition = destination.clone();
 		} else {
 			//if not set already for this node or new destination is different than last one