Flächeninhalt eines Kreises berechnen

[neila_1982]

Hallo,

ich soll den Flächeninhaltes eines Kreises programmieren. Dies habe ich gemacht. Aber in der Main -Methode gibt er mir keinen Flächeninhalt aus.

Was habe ich falsch gemacht?

package Vertex;

import java.lang.*;

public class Circle extends GemetricObject {
	double radius;
	

	public Circle(Vertex position, double radius){
		super(position, radius*2, radius*2);		
	}
	
	@Override
	public String toString(){
		return super.toString() + " " + radius/2;
	}
	
	@Override 
	public double area(){
		return Math.PI * (radius*radius)/4;
	}
	public static void main(String[] args) {
		Vertex v = new Vertex(2, 7);
		Circle c = new Circle(v, 4);
		System.out.println(c.area());
	}
}

Ausgabe:
0.0

 

[eRaaaa]

poste mal die superklasse

p.s.: ist deine flächenberechnung nicht sowieso falsch?

 

[SlaterB]

der radius der Subklasse ist 0, die Superklasse könnte da auch nix richten

 

[Schandro]

Es existiert in GeometricObject bereits die Membervariable "radius". Deswegen gibts die doppelt und die von Circle ist 0. Poste mal den ganzen Code.

 

[neila_1982]

Nein, nein die Flächenberechnung ist richtig.
Denn im Konstruktor setze ich ja den Radius * 2 und im Flächeninhalt muss ich den wieder teilen.

zur Hilfe noch die Superklasse

package Vertex;

public class GemetricObject {

	double weight;
	double height;
	Vertex position;
	
	public GemetricObject(Vertex position, double weight, double height){
		this.position= position;
		this.weight = weight;
		this.height = height;
	}
	
	@Override
	public String toString(){
		return position + " " + weight + " " + height;
		
	}
	
	public void moveTo(Vertex p){
		position = p;
	}
	
	public void move(Vertex v){
		position.x = position.x + v.x;
		position.y = position.y + v.y;
	}
	
	public double area(){
		return weight*height;
	}
	
	public boolean hasWithIn(Vertex p){
		if(p.x >= position.x && p.y >= position.y &&  p.x <= position.x + p.x 
				&& p.y <= position.y + p.y)
		{return true;}
		return false;
	}
	
	public boolean touches(GemetricObject that){
		//this liegt rechts von that
		
		if (that.position.x > this.position.x + this.weight)
			{return false;}
		
		//this liegt links von that
		
		if(this.position.x > that.position.x + this.height)
			{return false;}
		
		//this liegt unterhalb von that
		
		if (that.position.y < this.position.y + this.height)
			{return false;}
		
		//this liegt oberhalb von that
		
		if (this.position.y > that.position.y + that.height)
			{return false;}
		
		return true;
	}
	public static void main(String[] args) {
		Vertex b = new Vertex(3, 5);
		Vertex b1 = new Vertex(3, 5);
		GemetricObject a = new GemetricObject(b, 7, 8);
		GemetricObject a1 = new GemetricObject(b, 7, 8);
		System.out.println(a.touches(a1));
		System.out.println(a1.area());
		

	}

}

Hier funktioniert die Flächenberechnung für das Rechteck einwandfrei.

 

[SlaterB]

der radius der Subklasse ist 0, die Superklasse könnte da auch nix richten

^
|

im Konstruktor könntest du den radius setzen, falls gewünscht

 

[neila_1982]

Aber wenn ich im Konstruktor den Radius setze, dann kann ich ihn später doch nicht mehr ändern, oder?

 

[faetzminator]

Wieso nicht? wenn die Variable [c]final[/c] wäre, dann könntest du sie später nicht mehr ändern.

 

[0x7F800000]

Wieso nicht? wenn die Variable [c]final[/c] wäre, dann könntest du sie später nicht mehr ändern.

öhhm, ne, wenn die variable final wäre, dürfte man die nicht mal im konstruktor setzen :noe:

Um member-variablen vom zugriff von außen zu schützen sind ganz andere mechanismen da (stichwort [c]private[/c]) so wie es jetzt ist, kann jede andere Klasse im ganzen package hingehen und jederzeit am radius rumfummeln wie sie will. Der grund warum da 0 rauskommt: der radius wird implizit mit 0 initialisiert, und niemals wieder beschrieben. Die im Konstruktor übergebene Information wird weggeworfen, und der radius bleibt 0.

Nein, nein die Flächenberechnung ist richtig.

Nein, ist sie nicht.

PS: was zum henker ist ein "GemetricObject" ^^

 

[faetzminator]

öhhm, ne, wenn die variable final wäre, dürfte man die nicht mal im konstruktor setzen :noe:

Eine Klassenvariable (hat er) ohne Initialwert (hat er nicht) schon!?

 

[neila_1982]

GeometricObject ist eine Klasse zum Berechnen von Flächeninhalten. Die Klasse Circle soll nach Aufgabenstellung eine Unterklasse von GeometricObject sein.

 

[0x7F800000]

Eine Klassenvariable (hat er) ohne Initialwert (hat er nicht) schon!?

what? echt?... Moment mal...

edit: okay, das mit final nehm ich zurück :autsch: Hab sowas nie gewusst^^ Dann sind bei immutablen Objekten letztendlich alle getter und setter doch für den A***?!

Naja, wie auch immer, das ändert erstmal nichts an der Tatsache, dass der wert sinnvollerweise im Konstruktor abgespeichert werden sollte.

GeometricObject ist eine Klasse zum Berechnen von Flächeninhalten. Die Klasse Circle soll nach Aufgabenstellung eine Unterklasse von GeometricObject sein.

Jaja, das ist alles schön und gut, nur so steht es nicht da... Da steht irgendwas anderes, was sich sehr merkwürdig anhört und schmerzhaft auf's Auge drückt.

 

[Schandro]

edit: omfg warum kann man sinnlose Postings von sich selbst nicht einfach löschen? Das hätte euch erspart diese meine zwei Sätze zu lesen....

 

[neila_1982]

Also ich habe jetzt im Konstruktor den Radius = 0 gesetzt, und das auch wenn ich einen anderen Wert setze... Der Flächeninhalt gibt immer noch null... Daran kann es also nicht liegen.;(

 

[Schandro]

:autsch:^^

bevors hier noch mehr Missverständnise gibt:

package Vertex;
 
import java.lang.*;
 
public class Circle extends GemetricObject {
    double radius;
    
 
    public Circle(Vertex position, double radius){
        super(position, radius*2, radius*2);
        this.radius = radius;
    }
    
    @Override
    public String toString(){
        return super.toString() + " " + radius/2;
    }
    
    @Override 
    public double area(){
        return Math.PI * (radius*radius)/4;
    }
    public static void main(String[] args) {
        Vertex v = new Vertex(2, 7);
        Circle c = new Circle(v, 4);
        System.out.println(c.area());
    }
}

 

[eRaaaa]

:autsch:^^

bevors hier noch mehr Missverständnise gibt:

package Vertex;
 
import java.lang.*;
 
public class Circle extends GemetricObject {
    double radius;
    
 
    public Circle(Vertex position, double radius){
        super(position, radius*2, radius*2);
        this.radius = radius;
    }
    
    @Override
    public String toString(){
        return super.toString() + " " + radius/2;
    }
    
    @Override 
    public double area(){
        return Math.PI * (radius*radius)/4;
    }
    public static void main(String[] args) {
        Vertex v = new Vertex(2, 7);
        Circle c = new Circle(v, 4);
        System.out.println(c.area());
    }
}

dann ist die flächenberechnung aber immernoch falsch ;D

 

[0x7F800000]

dann ist die flächenberechnung aber immernoch falsch ;D

...Und die toString methode auch :autsch: Aber zumindest wird da im konstruktor was gemacht^^

 

[Schandro]

jep, ich hab nur die eine Zeile im Konstruktor hinzugefügt. Mehr wollte der TO auch net, warum sollte man ihn zur Wahrheit zwingen wollen?^^

 

[neila_1982]

Ich habe es kapiert.:toll::applaus:

Tut mir leid, dass ich nicht sofort verstanden habe

 

[Landei]

öhhm, ne, wenn die variable final wäre, dürfte man die nicht mal im konstruktor setzen

I beg to differ. Ich setze andauernd final-Variablen in Konstruktoren, und Initialisierungsblöcken (statisch oder nicht) und der Compiler findet das cool.