Hilfe bei Implementierung einer boolean Methode

[v1c0]

Wieso funktioniert die Implementierung nicht? sitze jetzt seit Stunden und verstehe es einfach nicht! Bitte um Hilfe! Danke im Voraus

 

[v1c0]

Wenn ich debugge springt er nach

if (counter == n)
    return true;

wieder in Zeile 12

 

[LimDul]

PS: Bitte Code nicht als Screenshot.

Die Methode liefert (fast) immer false.

Dein Problem ist, den rekursiver Aufruf von hasNOrderedPairs kannst du auch entfernen. Der macht genau gar nichts. Dein Code sieht im Prinzip wie folgt aus:

int counter=0;
if (text.length(()>1) {
if (....) {
n--;
}
if (counter == n) {
return true;
} else {
return false;
}

Das wird in den seltestenten Fälle was sinnvolles zurückgeben.

Du machst mit der Rückgabe des rekursiven Aufrufes nichts. Mit der musst du sinnvollerweise was tun. Kleiner Tipp: Wenn da true kommt, was willst du dann zurückliefern? Und was bei false?

 

[v1c0]

PS: Bitte Code nicht als Screenshot.

Die Methode liefert (fast) immer false.

Dein Problem ist, den rekursiver Aufruf von hasNOrderedPairs kannst du auch entfernen. Der macht genau gar nichts. Dein Code sieht im Prinzip wie folgt aus:

int counter=0;
if (text.length(()>1) {
if (....) {
n--;
}
if (counter == n) {
return true;
} else {
return false;
}

Das wird in den seltestenten Fälle was sinnvolles zurückgeben.

Du machst mit der Rückgabe des rekursiven Aufrufes nichts. Mit der musst du sinnvollerweise was tun. Kleiner Tipp: Wenn da true kommt, was willst du dann zurückliefern? Und was bei false?

OK! Werde ich nicht mehr machen!
Laut angäbe müssen wir die Methode allerdings rekursiv gestalten. Und naja, bei true, soll bei der Konsole true ausgegeben werden und bei false false haha sorry ich musss einen fetten Hänger in meinem denken haben

 

[v1c0]

 private static boolean hasNOrderedPairs(String text, int n, int index) {
        // TODO: Implementieren Sie hier Ihre Lösung für die Methode
        int counter = 0;
        if (index < text.length() - 1) {
            if (text.charAt(index) <= text.charAt(index + 1)) {
                n--;
            }  hasNOrderedPairs(text, n, index+1);
        }

        if (counter == n)
            return true;


        else
            return false;
    }

Habe es nun so versucht. jetzt gibt er mir zwar auf der Konsole was aus, aber nicht das richtige 😅

 

[v1c0]

problem gelöst, code sieht nun so aus:

 private static boolean hasNOrderedPairs(String text, int n, int index) {
        // TODO: Implementieren Sie hier Ihre Lösung für die Methode
        int counter = 0;
        if (index < text.length() - 1) {
            if (text.charAt(index) <= text.charAt(index + 1)) {
                n--;
            }  return hasNOrderedPairs(text, n, index+1);
        }

        if (counter == n)
            return true;


        else
            return false;
    }

Das return vor dem rekursiven Aufruf hat gefehlt. Nun stellt sich mir jedoch die Frage: Wieso genau brauche ich das??

 

[Oneixee5]

if (counter == n)
            return true;


        else
            return false;


// besser
return counter == n;