Dreieck Kreis

[lwqyt]

Hey ich bin's mal wieder, habe die Aufgabe das ich 2 klassen und 1ne Hauptklasse erstellen soll, 1ne Klasse für umfang und fläche eines Dreiecks und eine für das gleiche mit nem Kreis, dies soll natürlich in ner methode stehen. Ich habe immo nur probleme mit dem hauptprogramm da wir neu mit der OOP angefangen haben.

import java.io.*;
public class Dreieck2 
{
	public double seiteA;
	public double seiteB;
	public double seiteC;
	
	public static void main (String args[])
	throws IOException
		{
			BufferedReader input=new BufferedReader (new InputStreamReader(System.in));
			
		
			double a,b,c;
			String sa,sb,sc;
			
			System.out.println ("Seite A ?");
			sa=input.readLine();
			a = Integer.parseInt(sa);
			
			System.out.println ("Seite B ?");
			sb=input.readLine();
			b = Integer.parseInt(sb);
			
			System.out.println ("Seite C ?");
			sc=input.readLine();
			c = Integer.parseInt(sc);
			
			

		}
			public static void showumfang (double a, double b, double c)
			{
				double erg=0;
				erg=a+b+c;
				System.out.println("Der Umfang von den Seiten "+a+ +b+ +c+"ist"+erg);
			}
			
			public static void showflaeche (double a, double b, double c)
			{
				double A=0;
				double s=0;
				s=(a+b+c)/2;
				A = Math.sqrt(s*(s-a)*(s-b)*(s-c));
				System.out.println("Die Fläche von den Seiten "+a+ +b+ +c+"mit der Seite s"+s+"ist"+A);
			}
		
}

import java.io.*;
public class Kreis2
{
	public double radius;
	
	public static void main (String args[])
	throws IOException
		{
			BufferedReader input=new BufferedReader (new InputStreamReader(System.in));
			
		
			double r;
			String sr;
			
			System.out.println ("Der Radius ?");
			sr=input.readLine();
			r = Integer.parseInt(sr);
			
			
		}
			public static void showumfang (double r)
			{
				double U=0;
				U=2 * Math.PI * r;
				System.out.println("Der Umfang des Kreises mit dem Radius"+r+"beträgt"+U);
			}
			
			public static void showflaeche (double r)
			{
				double A=0;
				A=Math.PI * r * r;
				System.out.println("Die Flaeche des Kreises mit dem Radius"+r+"beträgt"+A);
			}
		
}

mein Anfang zur hauptklasse

public class Haupt2
{
	public static void main(String args[])
	{
		Dreieck2 b1, b2, b3;
		Kreis2			 r1;			//Referenz erzeugen
		b1 = new Dreieck2 ();
		b2 = new Dreieck2 ();
		b3 = new Dreieck2 ();
		r1 = new Kreis2	  ();
		
		b1.seiteA = ;
		b2.seiteB = ;
		b3.seiteC = ;
		r1.radius = ;

		

	}
}

wäre nett wenn mir mal wieder einer helfen würde

mfg. Jojo

 

[Marcinek]

Also ich habe mir die KLassen nicht im Detail angeschaut. Aber ich meine Code gesehen zu haben, der Flächen berechnet.

Um das Objektorjentiert richtig zu machen, brauchst du zunächst eine Klasse oder Interface "Figur".

Davon abgeleitet, implementiert "Kreis" und/oder "Dreieck".

Nun musst du im Interface entsprechende Methoden zur Berechnung der Fläche vorgeben, die die beiden Anderen implementieren.

Fertig

Gruß,

Martin

 

[lwqyt]

das was du meinst habe ich, habe eig nur probleme bei der hauptklassee bei OOP muss man ja nur die beiden klassen mit zb. der methode im gleichen projekt haben wie die hauptklasse und die dann in der hauptklasse aufrufen

 

[Marcinek]

Keine Ahnung was du meinst.

Fakt ist: Beim OOP ist genau so wie bei prozeduralen Programmieren.

Fakt ist: Die Klassen oben in deinem Posting entsprechen nicht den vorgaben, wie ich sie genannt habe.

 

[lwqyt]

Unsere Lehrerin hat uns das mit den vorklassen so beigebracht, nur das mit der Hauptklasse hat se uns net erklärt

 

[Marcinek]

Schaffst du schon.

Bitte lese dir den Link in meiner Signatur durch.

 

[tschero]

Hi,

mache es Dir ERSTMAL so einfach wie möglich (wobei Marcinek natürlich völlig Recht hatte).

Mache Dir je eine Klasse Kreis und eine Klasse Dreieck mit den entsprechenden Berechnungen.
Dazu nimm überall die Methode

public static void main (String args[])

raus (eine main Klasse brauchst Du nur einmal).

Mache Dir eine Hauptklasse, DORT packst Du die

public static void main (String args[])

rein und erzeugst darin die benötigten Objekte von Kreis und Dreieck.

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package zumtesten;

/**
 *
 * @author
 */
public class Kreis
{
    //Konstruktor
    public Kreis()
    {

    }

}

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package zumtesten;

/**
 *
 * @author
 */
public class Dreieck
{
    //Kontstruktor
    public Dreieck()
    {

    }

}

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package zumtesten;

/**
 *
 * @author
 */
public class ZumTesten
{
    public static Kreis myKreis;
    public static Dreieck myDreieck;

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args)
    {
        myKreis = new Kreis();
        myDreieck = new Dreieck();
    }
}

...über Sachen wie public static usw. muss Du natürlich noch nachdenken.
Ich hoffe ich konnte Dein Problem treffen.

Gruß
tschero

 

[lwqyt]

Ja danke habs jetz so gemacht und klappt super.

public class Dreieck2 
{
	public double seiteA;
	public double seiteB;
	public double seiteC;
	
	
			public static void showumfang (double a, double b, double c)
			{
				double erg=0;
				erg=a+b+c;
				System.out.println("Der Umfang von den Seiten A "+a+", B "+b+", C " +c+" ist "+erg);
			}
			
			public static void showflaeche (double a, double b, double c)
			{
				double A=0;
				double s=0;
				s=(a+b+c)/2;
				A = Math.sqrt(s*(s-a)*(s-b)*(s-c));
				System.out.println("Die Fläche von den Seiten A "+a+", B " +b+", C " +c+" mit der Seite S "+s+" ist "+A);
			}
		
}

public class Kreis2
{
	public double radius;
	
		
			public static void showumfang (double r)
			{
				double U=0;
				U=2 * Math.PI * r;
				System.out.println("Der Umfang des Kreises mit dem Radius "+r+" beträgt "+U);
			}
			
			public static void showflaeche (double r)
			{
				double A=0;
				A=Math.PI * r * r;
				System.out.println("Die Flaeche des Kreises mit dem Radius "+r+" beträgt "+A);
			}
		
}

import java.io.*;
public class Haupt2 {

	public static void main(String args [])
			throws IOException
			  {	  
			  BufferedReader input=new BufferedReader (new InputStreamReader(System.in));
			  
			  double a,b,c,r;
			  
			  System.out.println("Dreieck");
			  System.out.println("Geben sie die Seite a ein: ");
			  a = Double.parseDouble(input.readLine());
			  System.out.println("Geben sie die Seite b ein: ");
		      b = Double.parseDouble(input.readLine());
			  System.out.println("Geben sie die Seite c ein: ");
			  c = Double.parseDouble(input.readLine());
			  System.out.println("Kreis");
			  System.out.println("Wenn kein Kreis dann r=0 ");
			  System.out.println("Radius des Kreises: ");
			  r = Double.parseDouble(input.readLine());
			  

			  Dreieck2 s1;
			  Kreis2 r1;
			  r1 = new Kreis2();
			  s1 = new Dreieck2();

			  s1.seiteA=a;
			  s1.seiteB=b;
			  s1.seiteC=c;
			  r1.radius=r;



	System.out.println("");
	System.out.println("DREIECK");
	s1.showumfang(s1.seiteA,s1.seiteB,s1.seiteC);
	
	s1.showflaeche(s1.seiteA,s1.seiteB,s1.seiteC);
	if(r>0)
		{
		System.out.println("Kreis");
		r1.showumfang(r1.radius);
	
		r1.showflaeche(r1.radius);
		}
	else
		{
	
		}
	}

		

}

 

[Marcinek]

Aber dir sollte klar sein, dass das NIX mit OOP zu tun hat.

 

[lwqyt]

LoL aber das is doch Object orientierte programmieren. Also so stehts auf jedefall auf meinem AB

 

[Marcinek]

Die Aufgabe vielleicht.

Die Lösung nicht.

Wenn du diese so abgibst bekommst du höchstens einen Trostpunkt.

--
Du rufst statische Methoden an einem Objekt auf.

Du schreibst einfach die Werte in die Membervariablen, ohne dass diese genutzt werden.

Kreis und Dreieck haben nix gemeinsam, außer dass sie falsch implementiert sind.

Diese sollten im Sinne einer korrekten OOP von einer Klasse oder einem Interface "Figur" erben.

Dann sollte man sowas machen können:

Figur neueFigur = new Kreis(5.0);

Dann soll die Figur ihren Umfang und Fläche mitteilen.

Vielleicht ja beim nächsten Versuch.

 

[Daassan]

du solltest dir vllt doch nochmal die bedeutung von

static

anschauen

und viel mit oop hat es wirklich nicht zu tun

das sieht aus wie der versuch mit deinem C-kentnissen OOP zu erzwingen ^^
du erstellst ein leeres strukt
und dann füllst es im nachhinein
fehlt eigentlich nur noch das mallec() irgendwo

du könntest zb erstmal die Attribute privat machen das nich jeder die verhunebibeln kann
dazu gehören dann aber getter/setter um sie auszulesen
und ein netten konstruktor um sie bei der erstellung auch gleich zu füllen.

die static sachen sollten weg die sind für klassen funktionen
da deine aber auf bestimmte attribute zugreifen is das fehl am platze

und ein schickes interface wie es der marcinek meinte wäre sozusagen die saubere lösung letztlich sehe es so aus

public interface GeoFiguren
{
    public double showUmfang();
    public double showFlaeche();
}

public class Kreis implements GeoFiguren
{
    private double r;
    
    public Kreis(double radius)
    {
         r = radius;
    }

    public double showUmfang()
    {
        double erg = 0;
        //blabvla berechnung
        return erg;
    }
    
     public double showFlaeche()
     {
        double erg = 0;
        //blabvla berechnung
        return erg;
     }
}

(dreieck analog zu kreis)

public class haupt 
{
    public static void main(String[] args)
    {
        //werte abfragen
        
        GeoFigur myGeoFigur = new Kreis(inputR);
        sysout("umfang kreis:" + myGeoFigur .showUmfang() + "\nFläche Kreis:" + myGeoFigur.showflaeche());

        GeoFigur myGeoFigur = new Dreieck(inputA, inputB, inputC);
        sysout("umfang Dreieck:" + myGeoFigur.showUmfang() + "\nFläche Dreieck:" + myGeoFigur.showflaeche());
    }
}

das wäre zb ne saubere lösung
mit allen vorzügen der oop ausgestattet ^^

 

[lwqyt]

ok danke für eure ganze antworten ich werde sie mir zu herzen legen und merken.

mfg. jojo