Hilfe bei der Aufgabe

[jonathanpizza]

Wir haben eine Aufgabe bekommen, wo ich seit Stunden nicht weiter komme. Wir sollten zunächst einen einfachen Rechner (nach seinen Vorgaben) programmieren, was mit problemlos gelungen ist. Den Code dazu werde ich euch einfügen.

public class Rechner
{
private double ergeb, a, b;


public Rechner()
{
ergeb = 0;
}

/** a und b können jeweils ein Wert des Types double vergeben oder ein neuer Wert gegeben werden.
* Eigentlich war diese Methode in zwei geteilt, ich habe diese aber in eine zusammengefasst.
* Trotzdem macht diese Methode in meinen Augen nicht viel Sinn,
* da bei jeder neuen Rechenoperation sowieso neue Werte vergeben werden.
*/
public void setA(double neuerAWert){
a = neuerAWert;

}

public void SetB(double neuerBWert){
b = neuerBWert;

}

//Mit den folgenden Methoden kann man einen Wert für jeweils a und b vergeben.
//Mit diesen wird dann die ausgewählte Rechenoperation durchgeführt und das Ergebnis wird ausgegeben.

public double addiere(double a, double b){ // Methode zur Addition der Werte

ergeb = a + b;
return ergeb;

}




public double summe(){

int ergeb;


for (int a = 1; a <= ergeb; a++){
ergeb = (int)addiere(a, b);
System.out.println(ergeb);
}

return ergeb;
}


public double berechneFakultaet(int f) {

int fakultaet = 1; //außerhalb der Schleife anlegen und Wert zuweisen
int faktor = 1;

while (faktor <= f) {
fakultaet = fakultaet * faktor;
faktor++;
}

System.out.println("Die Fakultät von " + f + " ist: " + fakultaet);

return fakultaet;
}






public double subtrahiere(double a, double b){ // Methode zur Subtraktion der Werte

ergeb = a - b;
return ergeb;

}

public double multipliziere(double a, double b){ // Methode zur Multiplikation der Werte

ergeb = a*b;
return ergeb;

}

public double dividiere(double a, double b){ // Methode zur Division der Werte



if (a == 0 || b == 0) { //if-Anweisung: falls einer der Variablen den Wert 0 hat, wird eine Fehlermeldung ausgegeben.

System.err.println();
System.err.println ("Fehler! Sie haben versucht mit 0 zu teilen."); // Alternative print Anweisung, Error wird ausgegeben
return ergeb = 0;
}
else //Ansonsten wird das Ergebnis einer "normalen" Rechnung (ohne "0" als Wert) ausgegeben.
ergeb = a/b;

return ergeb;
}

public double getErgebnis(){ // Mit dieser Methode kann das Ergebnis nochmal aufgerufen werden.

System.out.println("Das vorherige Ergebnis lautet: " + ergeb + "."); // Das Ergebnis der letzten Rechenoperation wird angezeigt.

return ergeb;

}
}

Dazu sollen wir folgendes machen:

a.
Die Methode double summe(int, int) soll die Summe aller Ganzzahlen von der ersten bis zur zweiten Ganzzahl ausgeben. Dabei soll sie auf die bereits bestehende Methode addiere(double, double) zugreifen! Es soll egal sein, in welcher der beiden Parameter größer ist. Sie soll also nicht „selber rechnen

b.
Die Methode double fakultaet(int) soll die Fakultät des Übergabeparameters zurückgeben. Dabei soll sie auf die bereits bestehende Methode multipliziere(double, double) zugreifen! Sie soll also nicht „selber rechnen

c.
Die Methode double fakultDifferenz(int, int) soll die Differenz zweier Fakultäten zurückgeben und ebenfalls nicht selber rechne

d. Schreiben Sie eine Klasse Teste. Diese Klasse soll eine Methode boolean testen() beinhalten, die mit geeigneten Parametern testet, ob die drei oben genannten Funktionen funktionieren. Es dürfen hier noch keine Unit-Tests genutzt werden.


a und b habe ich versucht, die anderen beiden noch nicht angefangen.

Ich verstehe nicht ganz wie ich auf die Variablen der Method addiere(double, double) und fakultaet(int) zugreifen soll..



Noch dazu hat er gesagt dass die Codes ungefähr so aussehen sollen:

public double summe (int a, int b) {
int ergeb;

....
ergeb = addiere(a, b);

.....

}


Würde mich sehr um Hilfe freuen! Danke schonmal!

 

[BestGoalkeeper]

Wahrscheinlich ist die Aufgabe c) so gemeint:

public class Rechner {
    public double facDif(int f, int g) {
        double fac = 1;
        double fac2 = 1;
        int a = 1;
        int b = f < g ? f : g;
        int c = f < g ? g : f;
        for (int i = a; i <= c; i++) {
            fac *= i;
            if (i == b) {
                fac2 = fac;
            }
        }
        return fac - fac2;
    }

    public static void main(String[] args) {
        Rechner rech = new Rechner();
        System.out.println(rech.facDif(1, 1));
        System.out.println(rech.facDif(1, 2));
        System.out.println(rech.facDif(2, 1));
        System.out.println(rech.facDif(5, 4));
        System.out.println(rech.facDif(4, 5));
        System.out.println(rech.facDif(7, 3));
        System.out.println(rech.facDif(3, 7));
    }
}

 

[Blender3D]

[CODE lang="java" title="Test" highlight="6,7,10"]public class Test {
public static TestItem[] tests = { new TestItem(TestItem.FUNC_TYPE.SUM, 2, 5, 14),
new TestItem(TestItem.FUNC_TYPE.FACT, 7, 5040), new TestItem(TestItem.FUNC_TYPE.FACDIFF, 3, 5, -114) };

public static void main(String[] args) {
for (TestItem t : tests)
System.out.println(t.type + "\t" + testen(t) + "\t" + t);
}

public static boolean testen(TestItem test) {
switch (test.type) {
case FACT:
return Rechner.fakultaet(test.param[0]) == test.result;
case FACDIFF:
return Rechner.fakultDifferenz(test.param[0], test.param[1]) == test.result;
case SUM:
return Rechner.summe(test.param[0], test.param[1]) == test.result;
}
return false;
}
}[/CODE]
[CODE lang="java" title="TestItem"]public class TestItem {
public enum FUNC_TYPE {
SUM, FACT, FACDIFF
};

public final int[] param;
public final int result;
public final FUNC_TYPE type;

public TestItem(FUNC_TYPE type, int a, int result) {
this.type = type;
param = new int[1];
param[0] = a;
this.result = result;
}

public TestItem(FUNC_TYPE type, int a, int b, int result) {
this.type = type;
param = new int[2];
param[0] = a;
param[1] = b;
this.result = result;
}

@Override
public String toString() {
String str = "[";
for (Integer i : param)
str += i + " ";
return str + "-> " + result + "]";
}
}[/CODE]
[CODE lang="java" title="Rechner" highlight="2-7,9-11,13-23"]public class Rechner {
public static double fakultaet(int n) {
double result = 1;
for (int value = 2; value <= n; value++)
result = multiplizier(result, value);
return result;
}

public static double fakultDifferenz(int a, int b) {
return subtrahiere(fakultaet(a), fakultaet(b));
}

public static double summe(int a, int b) {
if (a > b) {
int tmp = a;
a = b;
b = tmp;
}
int sum = 0;
for (int value = a; value <= b; value++)
sum = (int) addiere(sum, value);
return sum;
}

public static double addiere(double a, double b) {
return a + b;
}

public static double dividiere(double a, double b) {
if (b == 0)
throw new IllegalArgumentException("divisen by zero!");
return a / b;
}

public static double subtrahiere(double a, double b) {
return a - b;
}

public static double multiplizier(double a, double b) {
return a * b;
}
}[/CODE]

 

[BestGoalkeeper]

@Blender3D du berechnest die Fakultät zweimal.

 

[MoxxiManagarm]

Wahrscheinlich ist die Aufgabe c) so gemeint:

Die Methode double fakultDifferenz(int, int) soll die Differenz zweier Fakultäten zurückgeben und ebenfalls nicht selber rechnen

Die Methode soll auf andere Methoden zugreifen und nicht selber rechnen.

 

[BestGoalkeeper]

Naja, a) hätte dort stehen müssen, Sie dürfen keine Elementarrechenoperationen direkt verwenden und b) ist es unsinnig die Fakultät zweimal zu berechnen.

 

[temi]

hätte dort stehen müssen

Steht ja auch dort:

soll die Differenz zweier Fakultäten zurückgeben und ebenfalls nicht selber rechne

Das schließt auch Elementarrechenoperationen mit ein

 

[MoxxiManagarm]

Dabei soll sie auf die bereits bestehende Methode addiere(double, double) zugreifen! Es soll egal sein, in welcher der beiden Parameter größer ist. Sie soll also nicht „selber rechnen

a) hätte dort stehen müssen, Sie dürfen keine Elementarrechenoperationen direkt

Was "nicht selber rechnen" hier bedeutet steht unter Aufgabe a ganz gut erklärt. Man beachte das "also"

 

[Blender3D]

du berechnest die Fakultät zweimal.

Wie meinst Du das ?

 

[MoxxiManagarm]

Wie meinst Du das ?

Ich denke er meint, dass es rein performancetechnisch besser wäre die Fakultät der kleineren Zahl zuerst zu berechnen und für die größere Zahl mit dem Ergebnis der kleineren Zahl weiterzuarbeiten. Aus Perspektive der Performance wäre das natürlich auch richtiger, aber nicht Ziel dieser Übungsaufgabe.

 

[Blender3D]

Aus Perspektive der Performance wäre das natürlich auch richtiger, aber nicht Ziel dieser Übungsaufgabe.

Das siehst Du völlig richtig.

 

[Blender3D]

Wahrscheinlich ist die Aufgabe c) so gemeint:

Die Aufgabe ist sehr wahrscheinlich nicht so gemeint.

Deine Lösung ist selbstverständlich effizienter. Sie liefert aber nicht immer die korrekten Werte.

 

[BestGoalkeeper]

Sie liefert aber nicht immer die korrekten Werte.

Es kommt darauf an wie die "Fakultätsdifferenz" definiert ist... Wenn es um die absolute Differenz geht dann ja liefert immer korrekte Werte.

Worum es in seiner Aufgabe genau geht oder was das Lernziel sein soll kann ich aus der Aufgabenstellung nicht herauslesen.

 

[Blender3D]

Es kommt darauf an wie die "Fakultätsdifferenz" definiert ist... Wenn es um die absolute Differenz geht dann ja liefert immer korrekte Werte.

Mir scheint du nimmst es mit den lesen von Vorgaben nicht so ganz genau.
In der Angabe steht:
c.
Die Methode double fakultDifferenz(int, int) soll die Differenz zweier Fakultäten zurückgeben und ebenfalls nicht selber rechnen ..
Differenz = Minuend - Subtrahend

 

[BestGoalkeeper]

Also nicht absolute Differenz... Dann lass das "f < g ? f : g;" Geraffel halt weg.

Und nein, wenn die Aufgabenstellung nicht >gut< und eindeutig ist dann lese ich solche nicht mehr. Und hier mangelte es ja schon an der korrekten Wiedergabe der Aufgabenstellung....

 

[Meniskusschaden]

Sehr schöne Fallstudie zu "Premature optimization is the root of all evil.".

 

[BestGoalkeeper]

Sehr schöne Fallstudie zu "Premature optimization is the root of all evil.".

Nö, ich habe nur einen sinnvollen Anwendungsfall hergenommen. Wenn die Absicht der Aufgabenstellung ist, nichts dabei zu lernen dann ist das Ziel ja errreicht.

 

[Blender3D]

Wenn die Absicht der Aufgabenstellung ist, nichts dabei zu lernen dann ist das Ziel ja errreicht

Ich denke das Ziel der Aufgabenstellung ist es, ein Gefühl für die Verwendung von Funktionen zu entwickeln und eine Einführung für die Notwendigkeit von Testverfahren ( JUnit) zu vermitteln.
Ich denke nicht, dass es um die Optimierung von Algorithmen dabei geht.

 

[temi]

Und nein, wenn die Aufgabenstellung nicht >gut< und eindeutig ist dann lese ich solche nicht mehr. Und hier mangelte es ja schon an der korrekten Wiedergabe der Aufgabenstellung....

Sehe ich gar nicht so. Steht doch alles da. Wenn du es nicht lesen möchtest, dann solltest du auch nicht antworten.

 

[jonathanpizza]

Danke euch allen! Habs fast geschafft