Java Anfänger... Finde fehler nicht

[winZard]

Ich möchte folgendes Machen. Aber irgendwie komme ich nicht so recht weiter.

Ich währe euch sehr dankbar wenn ihr mir helfen würdet

Klasse Bruch
-Zähler : int
-Nenner : int
+ invertieren () // Vertauschen von Nenner und Zähler
+ quadrieren() // Quadriert den Bruch
+ showBruch() // Format Zähler / Nenner
+ showBruchNett() // in Form von Zähler, nächste Zeile ----- darunter Nenner


Klasse Rechenumgebung
+ multipliziere (Bruch, Bruch) //Rückgabewert ist Bruch
+ divideren (Bruch, Bruch) // Rückgabewert ist Bruch
+ main

[highlight=Java]public class bruch
{
private int z;
private int n;

public void setz(int z)
{
this.z = z;
}

public void setn(int n)
{
this.n = n;
}

public int getZ()
{
return this.z;
}

public int getN()
{
return this.n;
}

public void invertiren()
{
this.n = z;
this.z = n;
}

public void quadrieren()
{
double nenner = this.n*this.n;
double zaler = this.z*this.z;
}

public void showBruch()
{
System.out.println("Bruch: ("+n+"/"+z+")");
}

public void showBruchNett()
{
System.out.println("n \n ----- \n z");
}
}[/highlight]
[highlight=Java]public class Rechenumgebung
{
public static void main(String[] args)
{
BRUCH bruch = new BRUCH(10, 8);
bruch.bruch2(10, 6);
System.out.println(getnenner());
System.out.println(getzaler());
}

public multi();
{
this.multzaler = z*10;
this.multinenner = n*5;
}
public int getzaler()
{
return this.multzahler;
}

public int getnenner()
{
return this.multinenner;
}

public divi();
{
this.divizaler = z/10;
this.divinenner = n/5;
}
public int getzaler2()
{
return this.divizahler;
}

public int getnenner2()
{
return this.divinenner;
}


}
[/highlight]

 

[SlaterB]

solange du keine Fehler/ Probleme nennst, wie soll man da helfen?

generell der Tipp:
int/int wird in Java abgerundet,

3/5 = 1
77/78 = 0

edit:
aus der main:
System.out.println(getnenner());

getnenner() ist eine nicht-statische Methode, die kannst du doch nicht aus einer statischen Methode heraus aufrufen, du brauchst ein Objekt,

getnenner() enthät einen Verweis auf ein Klassenattribut, aber es ist gar keins definiert?
und warum ist diese Methode nicht in Bruch definiert?

> public multi();
> {

Semikolon? Rückgabewert? soll das ein Konstruktor sein?
dann groß geschrieben und für welche Klasse?

 

[winZard]

Zeile 11: Public multi();

invalid method declaration; return type required

Welchen return type muss ich angeben?

 

[SlaterB]

na den den du zurückgeben willst, wenn nix, dann void

siehe auch edit in vorherigen post

 

[andre111]

Bei Klassennamen den ersten Buchstaben groß schreiben. Verwende statt einzelne Buchstaben konkrete Variablennamen (zaehler, nenner...)
lg André

 

[winZard]

\\Edit

Ach jetzt verstehe ich!

 

[SlaterB]

wie kannst du
> Wie mach ich ein Objekt?
fragen und gleichzeitig in deinem Code schon
> BRUCH bruch = new BRUCH(10, 8 );
stehen haben?
abgesehen von falscher Groß-/ Kleinschreibung kennst du doch anscheinend den new-Operator

ganz kurz:
Objekte erzeugen mit new,
nicht-statische Methoden am Objekt aufrufen, so wie du es mit
> bruch.bruch2(10, 6);
auch schon kannst (abgesehen davon, dass es keine Methode bruch2 gibt..)

statische Methoden, falls überhaupt benötigt, an der Klasse aufrufen:
Klassenname.methode(parameter);

alles weitere macht hier zu erklären keinen Sinn, schau dir das grundlegende Kapitel in einem Lehrbuch an,
jedes Thema wäre wahrscheinlich noch mal zu wiederholen,
z.B.
Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 3 Klassen und Objekte
+
Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 6 Eigene Klassen schreiben

wenn du da ein Beispiel nicht verstehst, oder es danach nicht in eine Klasse umsetzen kannst,
ja dann kann man hier diskutieren, aber dann nimm bitte auch auf die Grundlagen bezug nehmen

'ich weiß von nix + bitte erklären' macht weniger Spass

edit:
> Ach jetzt verstehe ich!

umso besser

 

[Spacerat]

In dem Code ist zwar 'ne Menge nicht so Toll. Aber ein richtig dicker unverzeihlicher Fehler befindet sich z.B. hier:[highlight=java]public void invertiren()
{
this.n = z;
this.z = n;
}[/highlight]Hier wird der Bruch nämlich nicht invertirt, sondern 1 (z = n). Denn n == this.n und z == this.z
willman werte tauschen, geht das nur über 'ne Hilfsvariable[highlight=java]public void invertiren()
{
int n = this.n;
this.n = this.z;
this.z = n;
}[/highlight]

 

[andre111]

oder bei ganzzahlen mit xor verknüpfungen:

public void invertieren()
{
    n ^= z;
    z ^= n;
    n ^= z;
}

 

[FatFire]

Oder:[highlight=Java]public void invertieren()
{
n = n + z
z = n - z
n = n - z
}
[/highlight]

 

[0x7F800000]

Oder hier zwei verschiedene einzeilermethoden:
[highlight=Java]
int x=0;
int y=1;
System.out.println("x: "+x+" y: "+y);
x=y+0*(y=x);
System.out.println("x: "+x+" y: "+y);
x+=y-(y=x);
System.out.println("x: "+x+" y: "+y);
[/highlight]
...aber das sind alles blödsinnige rechenintensive hacks, das was Spacerat geschrieben hat ist das einzig richtige, lediglich seine Aussage

geht das nur über 'ne Hilfsvariable

trifft nicht ganz zu, wie man sieht

 

[FatFire]

Gibt ja auch Anhänger der Methode, so Operationen generell nicht direkt an lebenden Objekten zu gestalten, sondern als Rückgabewert ein neues Objekt zu erhalten (ich finde sowas irgendwie schicker, aber kann da nichts zu den Wertungen der Profis sagen).

Dann müsste da natürlich (müsste eigentlich sowieso) noch ein vernünftiger Konstruktor hin:
[highlight=Java] public Bruch(int z, int n) {
this.z = z;
this.n = n;
}[/highlight]
Invertieren würde dann so laufen:
[highlight=Java] public Bruch getInvertiert() {
return new Bruch(n, z);
}[/highlight]
Also Du erhältst einen neuen Bruch zurück. Wenn Du mit dem weiterrechnen möchtest, musst Du natürlich den auch zuweisen:
[highlight=Java]bruch = bruch.getInvertiert();[/highlight]
Vereinfacht einige Operationen ungemein.
Achja, und falls Du mal kürzen willst:
[highlight=Java] private static int getGGT(int a, int b) {
if (b == 0)
return a;
else
return getGGT(b, a % b);
}

public int getGGT() {
return Bruch.getGGT(z, n);
}

public Bruch gekuerzt() {
int ggt = getGGT();
return new Bruch(z / ggt, n / ggt);
}[/highlight]
Größter gemeinsamer Teiler wird über rekursiv laufenden euklidschen Algorithmus berechnet und damit dann gekürzt. Als ergebnis erhält man natürlich wieder als neuen Bruch das Ergebnis.

Naja, und Darstellungsmethoden sollten nicht direkt in diese Klasse gepackt werden (meine da showBruch() und showBruchNett()), besser ist es da doch, toString() zu überschreiben und die Darstellung woanders zu machen.

Gruß
FatFire

PS: Oh man, mir ist langweilig, kann Developer_X nicht mal einen neuen Thread aufmachen? Der Mensch will doch unterhalten werden.