Bruchklasse erstellen

[JaVaN0oB]

Hallo an alle 🙂
Ich habe da leider ein Problem...
Ich soll in der Uni eine Bruchklasse schreiben mit folgenden Aufgaben:

• Einen Konstruktor, der als Parameter den Zähler und Nenner als ganze Zahl erhält.
• Einen Konstruktor, welcher eine Zeichenkette erhält. Diese Zeichenkette repräsentiert einen Bruch, wobei garantiert ist, dass die übergeben Zeichenkette immer im Format 123/456 ist.
• Eine (überschreibende) toString-Methode, welche einen Bruch 3434 im Format 3/4 als String zurückgibt (ohne Leerzeichen und Zeilenumbruch!).
• Eine Instanzmethode addiere, welche als Parameter den zweiten Bruch erhält und als Rückgabewert den Ergebnisbruch zurückgibt (weder der aktuelle noch der übergebene Bruch werden verändert).
• Eine von außen nicht zugreifbare Methode kuerze, welche den aktuellen Bruch kürzt und zudem sicherstellt, dass der Bruch immer im gekürzten Zustand vorliegt.

Leider stelle ich mich etwas blöd an mit bzw. durch die vorgegebene Klasse (siehe unten).

Könnt ihr mir helfen?

Das habe ich bereits:
[CODE lang="java" title="Bruch.java"]public class Bruch {
private int zaehler;
private int nenner;

public Bruch(int a, int b){
this.zaehler = a;
this.nenner = b;
kuerze();
}

public String toString() {
String buffer = zaehler + "/" + nenner;
return buffer;
}

public void kuerze() {

int limit = Math.min(zaehler, nenner);
for (int i = 2; i <= limit; i++) {
if (zaehler % i == 0 && nenner % i == 0) {
zaehler /= i;
nenner /= i;
}
}
}

public int getZaehler(){
return zaehler;
}
public int getNenner(){
return nenner;
}

public Bruch addiere(Bruch br){

int z1 = zaehler * br.getNenner();
int z2 = nenner * br.getZaehler();
int n= nenner*br.getNenner();
return new Bruch(z1+z2, n);
}

}[/CODE]

Wichtig ist allerdings die vorgegebene Klasse, die ist nicht veränderbar....:

[CODE lang="java" title="TestBruch.java [READONLY]"]import java.io.*;
import java.lang.reflect.*;

public class TestBruch {
public static void main(String args[]) throws IOException {
Bruch a, b;
BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
int zaehler, nenner;
if(args.length == 2) {
a = new Bruch(args[0]);System.out.println(a);
b = new Bruch(args[1]);System.out.println(b);
}
else if(args.length == 0) {
zaehler = Integer.parseInt(in.readLine());
nenner = Integer.parseInt(in.readLine());
a = new Bruch(zaehler, nenner);
System.out.println(a);
zaehler = Integer.parseInt(in.readLine());
nenner = Integer.parseInt(in.readLine());
b = new Bruch(zaehler, nenner);
System.out.println(b);
}
else {
try { codeVerification(Bruch.class); }
catch(SecurityException e) {
e.printStackTrace(System.out);
}
return;
}
Bruch c;
switch(in.readLine().charAt(0)) {
// case '*':
// c = a.multipliziere(b);
// System.out.println(a+"\n"+b+"\n"+c);
// break;
case '+':
c = a.addiere(b);
System.out.println(a+"\n"+b+"\n"+c);
break;
default:
System.out.println("not supported");
}
}
private static void codeVerification(Class<?> cls) throws
SecurityException {
for(Field f : cls.getDeclaredFields()) {
if(Modifier.isStatic(f.getModifiers()))
System.out.println("Es werde keine Klassenvariablen benötigt, aber die folgende gefunden: " + f.getName());
if(!Modifier.isPrivate(f.getModifiers())) {
System.out.println("Alle Attribute dürfen von außerhalb der Klasse nicht zugreifbar sein, das folgende Attribut ist aber von außen zugreifbar: " + f.getName());
}
}
Method kuerze = null;
for(Method m : cls.getDeclaredMethods()) {
switch(m.getName()) {
case "toString":
case "addiere":
case "multipliziere":
if(!Modifier.isPublic(m.getModifiers()))
System.out.println(m.getName() + " ist nicht public!");
break;
case "kuerze":
kuerze = m;
default:
if(!Modifier.isPrivate(m.getModifiers()))
System.out.println(m.getName() + " ist nicht private!");
}
if(Modifier.isStatic(m.getModifiers()))
System.out.println("Es werden keine static-Methoden benötigt aber die folgende gefunden: " + m.getName());
}
if(kuerze == null) {
System.out.println("Es fehlt eine von außen nicht zugreifbare Methode kuerze!");
}
else {
if(!kuerze.getReturnType().equals(Void.TYPE))
System.out.println("Für die Verwendung der kuerze-Methode wird kein Rückgabetyp benötigt!");
if(kuerze.getParameterCount()!=0) {
System.out.println("Die kuerze-Methode benötigt keine Parameter, da sie direkt auf die Attribute zugreifen soll.");
}
}

for(Constructor<?> constr : cls.getDeclaredConstructors()) {

if(!Modifier.isPublic(constr.getModifiers()))
System.out.println("In dieser Aufgabe sollten alle Konstruktoren von außen zugreifbar sein, was auf " + constr.toGenericString() + " nicht zutrifft.");
}
}
}[/CODE]

Ich danke euch im voraus für die Hilfe!

 

[mrBrown]

Was für Fehler gibts denn bzw. welche Probleme hast du?

 

[fhoffmann]

Es fehlt noch:

Einen Konstruktor, welcher eine Zeichenkette erhält.

 

[JaVaN0oB]

Es fehlt noch:

@mrBrown Was @fhoffmann sagt
Allerdings tu ich mich dabei ein bisschen schwer und stehe völlig auf dem Schlauch🤦‍♂️

Ansonsten sind das die auftretenden Fehler:
TestBruch.java:10: error: constructor Bruch in class Bruch cannot be applied to given types;
a = new Bruch(args[0]);System.out.println(a);
^
required: int,int
found: String
reason: actual and formal argument lists differ in length
TestBruch.java:11: error: constructor Bruch in class Bruch cannot be applied to given types;
b = new Bruch(args[1]);System.out.println(b);
^
required: int,int
found: String
reason: actual and formal argument lists differ in length
2 errors

 

[LimDul]

Welche String-Funktionen dürft/sollt ihr den verwenden?

Einen Konstruktor mit einem String zu definieren sollte ja möglich sein, wenn du den Konstruktor mit den zwei Ints hinbekommen hast.

Jetzt musst du den String auseinandernehmen. Es ist garantiert laut Aufgabe, dass der String immer das Format "ziffern/ziffern" hat.
Jetzt gibt es mehrere Möglichkeiten, die einfachste ist vermutlich mittels String.split: https://docs.oracle.com/en/java/jav...java/lang/String.html#split(java.lang.String)

Alternativ kann man sich mittels indexOf die position des / holen und dann mittels substring den Teil vor dem / und den Teil hinter dem / holen.
Egal wie man es man macht - dann hat man zwei Strings, den Teil vor dem / und den Teil dahinter.
Und die kann man jetzt mittels Integer.parseInt in zwei Ints umwandeln, die du dann nur noch in nenner & zaehler speichern musst https://docs.oracle.com/en/java/jav.../lang/Integer.html#parseInt(java.lang.String)

Zwei Anmerkungen zu dem Code:
* Deine kuerze Methode hat einen Fehler, sie betrachtet jeden Divisor nur 1x. Erzeug mal den Bruch 4/4 - da kommt nicht das richtige raus.
* Bzgl. kuerze steht noch dabei, dass sie nicht von außen zugreifbar sein soll

 

[JaVaN0oB]

Welche String-Funktionen dürft/sollt ihr den verwenden?

Einen Konstruktor mit einem String zu definieren sollte ja möglich sein, wenn du den Konstruktor mit den zwei Ints hinbekommen hast.

Jetzt musst du den String auseinandernehmen. Es ist garantiert laut Aufgabe, dass der String immer das Format "ziffern/ziffern" hat.
Jetzt gibt es mehrere Möglichkeiten, die einfachste ist vermutlich mittels String.split: https://docs.oracle.com/en/java/jav...java/lang/String.html#split(java.lang.String)

Alternativ kann man sich mittels indexOf die position des / holen und dann mittels substring den Teil vor dem / und den Teil hinter dem / holen.
Egal wie man es man macht - dann hat man zwei Strings, den Teil vor dem / und den Teil dahinter.
Und die kann man jetzt mittels Integer.parseInt in zwei Ints umwandeln, die du dann nur noch in nenner & zaehler speichern musst https://docs.oracle.com/en/java/jav.../lang/Integer.html#parseInt(java.lang.String)

Zwei Anmerkungen zu dem Code:
* Deine kuerze Methode hat einen Fehler, sie betrachtet jeden Divisor nur 1x. Erzeug mal den Bruch 4/4 - da kommt nicht das richtige raus.
* Bzgl. kuerze steht noch dabei, dass sie nicht von außen zugreifbar sein soll

Wäre das zum Kürzen besser? (Abgesehen von dem Zugreifen von außen)

[CODE lang="java" title="Kürzen"] public void kuerze() {
int m = Math.abs(zaehler);
int n = Math.abs(nenner);
int r;
do {
r = m % n;
m = n;
n = r;
}
while(n != 0);
if(m > 1) {
zaehler /= m;
nenner /= m;
}
}[/CODE]

Zum Konstruktor, entweder denke ich da gerade zu kompliziert oder ich bin doch einfach zu "doof" dafür 🤦‍♂️

 

[kneitzel]

Ja, das ist viel besser - außer dass Du die Variablen sinnvoller benennen könntest.

Und an was scheitert es beim Konstruktor? Du musst erst einmal einen weiteren Konstruktor einfügen, der einen String nimmt:

    public Bruch(String bruch){
        this.zaehler = ...;
        this.nenner = ...;
        kuerze();
    }

Jetzt ist nur die Frage, wie Du die ... ersetzen könntest. Ihr habt doch bestimmt schon diverse Dinge mit Strings gemacht, oder? Das müsstest Du Dir einfach noch einmal ansehen ...

 

[JaVaN0oB]

Ja, das ist viel besser - außer dass Du die Variablen sinnvoller benennen könntest.

Und an was scheitert es beim Konstruktor? Du musst erst einmal einen weiteren Konstruktor einfügen, der einen String nimmt:

    public Bruch(String bruch){
        this.zaehler = ...;
        this.nenner = ...;
        kuerze();
    }

Jetzt ist nur die Frage, wie Du die ... ersetzen könntest. Ihr habt doch bestimmt schon diverse Dinge mit Strings gemacht, oder? Das müsstest Du Dir einfach noch einmal ansehen ...

Ach, ich verstehe. Ich hatte da echt einen kompletten Aussetzer.

Mein Code sieht jetzt so aus:

[CODE lang="java" title="Bruch.java"]public class Bruch {
private int zaehler;
private int nenner;

public Bruch(int a, int b){
this.zaehler = a;
this.nenner = b;
kuerze();
}

public Bruch(String bruch){
this.zaehler = zaehler;
this.nenner = nenner;
kuerze();
}

public String toString() {
String buffer = zaehler + "/" + nenner;
return buffer;
}

private void kuerze() {
int m = Math.abs(zaehler);
int n = Math.abs(nenner);
int r;
do {
r = m % n;
m = n;
n = r;
}
while(n != 0);
if(m > 1) {
zaehler /= m;
nenner /= m;
}
}

private int getZaehler(){
return zaehler;
}
private int getNenner(){
return nenner;
}

public Bruch addiere(Bruch br){

int z1 = zaehler * br.getNenner();
int z2 = nenner * br.getZaehler();
int n= nenner*br.getNenner();
return new Bruch(z1+z2, n);
}

}[/CODE]

 

[httpdigest]

Mein Code sieht jetzt so aus:

[CODE lang="java" title="Bruch.java"]public class Bruch {
public Bruch(String bruch){
this.zaehler = zaehler;
this.nenner = nenner;
kuerze();
}
}[/CODE]

Hier solltest du nochmal nachdenken. 🙂
Dieser Konstruktor ist gerade äquivalent zu:

public Bruch(String bruch){
  this.zaehler = this.zaehler;
  this.nenner = this.nenner;
  kuerze();
}

Die Variable `bruch` (der Parameter) wird ja letztlich überhaupt nicht verwendet. Hier soll ja (laut Aufgabenstellung) z.B. sowas wie "123/456" als String stehen. Damit musst du natürlich was tun.

 

[JaVaN0oB]

Ich dank euch vielmals für eure Hilfe, ich hab es jetzt endlich gerafft und geschafft. Das Testsystem gibt mir die grünen Häkchen.
So sieht mein Code nun aus:

[CODE lang="java" title="Bruch.java"]public class Bruch {
private int zaehler;
private int nenner;

public Bruch(int a, int b){
this.zaehler = a;
this.nenner = b;
kuerze();
}

public Bruch(String bruch){
String Cache[] = new String[2];
Cache=bruch.split("/");
this.zaehler = Integer.parseInt(Cache[0]);
this.nenner = Integer.parseInt(Cache[1]);
kuerze();
}

public String toString() {
String buffer = zaehler + "/" + nenner;
return buffer;
}

private void kuerze() {
int m = Math.abs(zaehler);
int n = Math.abs(nenner);
int r;
do {
r = m % n;
m = n;
n = r;
}
while(n != 0);
if(m > 1) {
zaehler /= m;
nenner /= m;
}
}

private int getZaehler(){
return zaehler;
}
private int getNenner(){
return nenner;
}

public Bruch addiere(Bruch br){

int z1 = zaehler * br.getNenner();
int z2 = nenner * br.getZaehler();
int n= nenner*br.getNenner();
return new Bruch(z1+z2, n);

}
}[/CODE]