Methode aufrufen

[scratchy1]

Aber das soll sie doch, sie soll 100-mal aufgerufen werden, wie soll ich denn sonst das Maximum herauskriegen?
Jetzt habe ich glaube ich auch die Zahl gefunden: Die Zahl, bei der am meisten Schritte ausgeführt werden bis wir eine sich wiederholende Sequenz erhalten, ist 97 mit 118 Schritten. Ich vergleich jetzt nicht mehr immer mit dem Nachfolger, sondern mit dem bisherigen Maximum:

public class wundersam {

    public static void main(String[] args) {
        // TODO Auto-generated method stub

        int max = 0;
        for (int s = 1; s <= 100; s = s + 1) {

            if (wundersamTest.wunder(s) > max) {
                max = wundersamTest.wunder(s);
                System.out.println(max + "maximale Schritte bei" + s);
            }

        }
    }
}

Die Ausgabe ist 118 maximale Schritte bei 97 und (komischerweise) 115 maximale Schritte bei 73,
obwohl 115<118. hmm

 

[Robat]

Wir kürzen das ganze einfach mal ab. Was wir meinen ist sowas hier. So vermeidest du den doppelten Aufruf in *einer* Iteration

public class wundersam {

    public static void main(String[] args) {
        // TODO Auto-generated method stub

        int max = 0;
        for (int s = 1; s <= 100; s = s + 1) {
            int current =wundersamTest.wunder(s);
            if (current > max) {
                max = current;
                System.out.println(max + "maximale Schritte bei" + s);
            }

        }
    }
}

 

[mihe7]

Aber das soll sie doch, sie soll 100-mal aufgerufen werden, wie soll ich denn sonst das Maximum herauskriegen?

Genau, sie soll 100-mal aufgerufen werden aber nicht z. B. 200-mal. Aktuell rufst Du wundersam(s) einmal in der if-Bedingung und einmal im then-Zweig auf.

EDIT: @Robat, hast Du hier irgendwie eine Kamera bei mir? Kaum schreib ich ihm, antwortest Du auch

@scratchy1 verstehst Du den Unterschied zwischen Deiner Lösung und der von @Robat?

 

[Robat]

Verdammt .. aufgeflogen

 

[scratchy1]

Hallo Leute,
ja ich hab das so verstanden, wie Dus(Robat) im Code geschrieben hast. Also der Unterschied ist, dass ich es dann nur einmal aufrufen würde und in meinem Fall öfter. Da das Ergebnis aber das gleiche ist, frag ich mich aus welchem Grunde ich es möglichst wenig aufrufen soll (aus Gründen der Komplexität/Aufwand/Ästhetik etwa) ?

 

[Robat]

Naja die Berechnung der Zahl kann theoretisch recht lange dauern .. du sparst damit also Zeit.

 

[mihe7]

Abgesehen davon wiederholst Du den gleichen Quellcode, was die Zahl der Fehlerquellen erhöht.

 

[scratchy1]

Ach ok, deswegen, vielen Dank, ich werd das dann nachholen:

public class wundersam {

    public static void main(String[] args) {
        // TODO Auto-generated method stub

        int max = 0;
        for (int s = 1; s <= 100; s = s + 1) {
            int schritte = wundersamTest.wunder(s);
            if (schritte > max) {
                max = schritte;
                System.out.println(max + "maximale Schritte bei" + s);
            }

        }
    }
}

 

[mihe7]

Genau, wenn Du Deinen Quellcode jetzt liest, dann hat der Code wesentlich größere Aussagekraft. Die Intention des Codes ist wesentlich klarer, weil man relativ schnell erkennt, dass es darum geht, das Maximum einer Schrittzahl zu ermitteln.

Wenn Du Deine alte Variante zum Vergleich ansiehst:

          if (wundersamTest.wunder(s) <= wundersamTest.wunder(s + 1)) {
                max = wundersamTest.wunder(s + 1);
            } else {
                max = wundersamTest.wunder(s);
            }

Da musst Du erstmal überlegen, was hier passiert. Problem 1: Du siehst in dem Stück Code nicht, was wundersamTest.wunder(s) zurückgibt. Da könnte genauso gut f(s) stehen. Problem 2: im Code steht "wenn f(s) <= f(s+1) ist, dann setze max auf f(s+1) sonst auf f(s)". Da musst Du erstmal das Überlegen anfangen, was das eigentlich zu bedeuten hat:

f(0) = 1
f(1) = 100
f(2) = 2
f(3) = 3

D. h. bei s=2 gilt f(2) <= f(3), dann wird max auf f(3) = 3 gesetzt. Das ist aber irgendwie kein Maximum (bzw. ein äußerst lokales )

Kurz: so etwas schlechter Code; nicht, weil er falsch ist, sondern weil die Intention nicht ersichtlich ist.

Zum Vergleich:

            int schritte = wundersamTest.wunder(s);
            if (schritte > max) {
                max = schritte;
            }

Aha, es werden schritte gerechnet und das Maximum ermittelt. Damit ist sofort alles klar.

 

[scratchy1]

Vielen Dank,
ja verglichen zur allerersten Version, ist es besonders ersichtlich.