Erste Schritte Rekursion?

[n3oth3on3]

public static boolean enthaelt(long zahl, int ziffer){
       
        return  (zahl <= 9)
            ? zahl == ziffer
            : enthaelt((zahl % 10), (ziffer));
               
    }
    
public static int gibAnzahl(long zahl, int ziffer){
       
       return (zahl <= ziffer)
                   ? 1
                   : (enthaelt(zahl,ziffer) == true) 
                   ? 1 + gibAnzahl((zahl % 10), (ziffer))
                   : 0;
       
    }

Das sind bisher meine beiden Methoden. Die Methode "enthaelt" funktioniert bereits nur die Methode "gibAnzahl" nicht, und zwar soll diese zb. von der Zahl 556662 die Ziffern 5 zählen, und dann Ausgeben wie oft diese Ziffer in der Zahl drinne vorkommt. Ich hab irgendwo einen logischen Fehler, weil z.b. für die Zahl 5711,1 = 2 funktioniert der Code, aber nicht für 871,1 = 1, dort gibt er mir auch 2 aus!

Was mach ich falsch?

Mit freundlichen Grüßen

 

[nillehammer]

Was mach ich falsch?

Du schreibst so komplizierten Code, dass Du ihn nicht selbst überblickst und deswegen in einem Forum nachfragen musst. Ternäre Operatoren sind zwar äußerst cool, aber verschachtelt genauso wie mehrfach verschachtelte if-else oder Schleifen ein absolutes No-Go. Korrigiere das. Dann findest Du den Fehler wahrscheinlich ohne Hilfe.

 

[n3oth3on3]

     public static int gibAnzahl(long zahl, int basis){
       
       return (enthaelt(zahl,basis) == true) 
                   ? 1 + gibAnzahl(zahl / 10 , basis)
                   : 0;

so klappt es nun, aber nicht für 0,0 kann mir einer sagen wieso? Es ist mir schon klar das er die 0 / 10 nicht teilen kann, aber normal rechnet er dann doch mit 0 weiter oder? weil es ein int ist oder?

 

[nillehammer]

zahl / 10

zahl ist ein int, 10 ist ein int. Das resultiert in einer sog. Ganzzahldivision sprich, wenn zahl 9 wäre, stünde da

9/10

. Das ergibt als Ganzzahldivision 0. Bei

0/10

natürlich auch.

War es das, was Du wissen wolltest? Falls nicht, schreib mal, was "geht nicht" heißt. In etwa so: "Ich übergebe a und b und erwarte, dass Methode foo x zurück gibt, es kommt aber y" oder Ich erwarte x, es fliegt aber folgende Exception (hier dann die vollständige Exception inkl. Stacktrace).

 

[hüteüberhüte]

Mach doch so:

    public static int zähle(long zahl, int ziffer, int zähler) {
        return zahl == 0 ? zähler : zahl % 10 == ziffer ? zähle(zahl / 10, ziffer, zähler + 1) : zähle(zahl / 10, ziffer, zähler);
    }

    public static void main(String[] args) {
        System.out.println(zähle(22122345622L, 2, 0));
    }

Du siehst, die Zeile ist lang und unübersichtlich. Ohne Klammerung funktionierts auch nur, weil der ternäre Operator "rechtsassoziativ" ist.

 

[hüteüberhüte]

Wenn du das ohne Akkumulator machen willst (für ein Edit ist es schon zu spät):

    public static int zähle(long zahl, int ziffer) {
        return zahl == 0 ? 0 : zahl % 10 == ziffer ? zähle(zahl / 10, ziffer, zähler) + 1 : zähle(zahl / 10, ziffer, zähler);
    }

(ungetestet)
Einmal wird die Arbeit vor dem Betreten gemacht, einmal hinterher. Bei bestimmten JVMs spielt das eine Rolle und auch in anderen Bereichen wie Funktionale Programmierung.