In CWPoint gibt es 4 verschiedene Methoden zur Abstandsberechnung:
Der Javadoc (und den Funktionsnamen) zufolge berechnen (1) und (2) die Entfernung zwischen 2 Punkten in km, (3) und (4) die Entfernung in RAD (was soll das sein? Bogenmaß? Das wäre dann aber keine Entfernung, sondern ein Winkel...)
Schaut man sich jetzt die Implementierungen an, dann ist
(1) ein Aufruf einer Library-Funktion,
(2) ein Aufruf von (1)
(3) eine Berechnung
(4) ein Aufruf von (2)(!)
Letzteres hat mich stutzig gemacht. Ist das richtig so? Sollte nicht lieber (3) aufgerufen werden (damit die Funktion das macht, was sie dokumentiert?) Oder sollten alle Aufrufer von (4) auf (1) geändert werden, und (3) und (4) rausfliegen ((3) wird nirgendwo anders aufgerufen)?
Code:
(1)public double getDistance (CWPoint dest)
(2)public double getDistance (double latDecD, double lonDecD)
(3)public double getDistanceRad (double latDecD, double lonDecD)
(4)public double getDistanceRad (CWPoint ll)
Der Javadoc (und den Funktionsnamen) zufolge berechnen (1) und (2) die Entfernung zwischen 2 Punkten in km, (3) und (4) die Entfernung in RAD (was soll das sein? Bogenmaß? Das wäre dann aber keine Entfernung, sondern ein Winkel...)
Schaut man sich jetzt die Implementierungen an, dann ist
(1) ein Aufruf einer Library-Funktion,
(2) ein Aufruf von (1)
(3) eine Berechnung
(4) ein Aufruf von (2)(!)
Letzteres hat mich stutzig gemacht. Ist das richtig so? Sollte nicht lieber (3) aufgerufen werden (damit die Funktion das macht, was sie dokumentiert?) Oder sollten alle Aufrufer von (4) auf (1) geändert werden, und (3) und (4) rausfliegen ((3) wird nirgendwo anders aufgerufen)?