Meine Funktion für das beinhalten eines Punktes in einem Kreis funktioniert nicht

[Tobero]

public boolean contains(float x, float y, float cX, float cY)
    {
        double distance = Math.sqrt(Math.pow(x - cX, 2) + Math.pow(y- cY, 2));

        return distance <= radius;
    }

Diese Funktion liefer mir alle punkte rechts oben. Die rot markierten bäume werden von der Funktion dazu gezählt. Der graue Kreis zeigt den Radius.

 

[kneitzel]

Was für ein radius wird da genommen? Du übergibst alles als Parameter aber nicht den Radius.

Ansonsten kannst Du ja einmal eine Ausgabe einbauen, damit Du nachvollziehen kannst, was er rechnet (bzw. nutz den Debugger um zu schauen, mit was für Werten es aufgerufen wird und was da dann raus kommt.

 

[Tobero]

Mein Radius ist 700. Den habe ich in der Klasse gleich bestimmt.

 

[kneitzel]

Ich kann mich nur wiederholen: Prüf den Aufruf und schau mit was für Werten du da rechnest.

Die Methode selbst scheint korrekt zu sein. Daher muss der Fehler woanders liegen.

Nur da wir den anderen Code nicht kennen, können wir dazu nichts sagen. Möglichkeiten sind:
- radius ist falsch
- Parameter sind falsch (z.B. vertauscht).

 

[White_Fox]

Oder mal mit dem Debugger Zeile für Zeile anschauen und sich die Variablen ansehen.

 

[Tobero]

Ich habe jetzt alles überprüft, und es hat jetzt funktioniert. Ich habe mal wieder den Fehler gemacht das eine als world Position und das andere als screen Position gerechnet würde.