Wo ist der Fehler im Programmcode?

jürgensens

Mitglied
In der Vorlesung wurde der sog. Gray-Code vorgestellt. In dieser Aufgabe soll von uns Studenten ein Generator
für Gray-Codes implementiert werden.

Erstellen Sie eine neue Klasse GrayCode.
a) Implementieren Sie zunächst die statische Klassenmethode int prevLength(int len). Diese Methode soll die Länge des nächst kleineren Gray-Codes zurückgeben. Hat ein Gray-Code die Länge len, so hat der nächst kleinere Gray-Code die Länge len/2 , wenn len durch zwei teilbar ist. Ansonsten ist die Länge des nächst kleineren Gray-Codes durch die größte 2er-Potenz bestimmt, die kleiner len ist.

b) Ergänzen Sie danach String[] generate(GrayCodeControl gcc, int len). Diese Methode erzeugt mittels Rekursion zuerst den nächst kleineren Gray-Code, um diesen anschließend zu einem Gray-Code Glen der Länge len zu erweitern. Die Basisfälle stellen hierbei die Gray-Codes G1 = {"0"} sowie G2 = {"0"; "1"} dar. Für die Länge len = 4 soll demnach die Zeichenkettenreihung G4 = {"00"; "01"; "11"; "10"} zurückgegeben werden.

ACHTUNG – WICHTIG: Gleich zu Beginn Ihrer generate müssen Sie die Methode gcc.logGenerate(gcc, len) aufrufen und ihr die Argumente des aktuellen Aufrufs übergeben. Diese Methode dient der Überprüfung Ihrer Implementierung und ist für die automatische Bewertung im EST essentiell!

MEIN CODE:

Java:
public class GrayCode {
    
    public static int prevLength(int len){
        if(len %2 == 0) return len/2;
        int i=0;
        while((int) Math.pow(2, i) < len) {
            i++;
        }
        return (int) Math.pow(2, i-1);
    }
    
    public static String[] generate(GrayCodeControl gcc, int len) {
        gcc.logGenerate(gcc, len);
        if(len ==1) {
            return new String [] {"0"};
        }else if(len ==2) {
            return new String[] {"0","1"};
        }else {            
            String [] shortArray= generate(gcc,prevLength(len));
            String[] returnArray = new String[len];
            int j=0;
            for(int i =0; i<returnArray.length; i++) {
                if(shortArray.length>i) returnArray[i]=shortArray[i];
                else {
                    returnArray[i]=shortArray[shortArray.length-1 -j];
                    j++;
                }
            }
    
            for(int i=0; i<shortArray.length;i++) {
                returnArray[i] = "0"+shortArray[i];    
            }
            for(int i=returnArray.length-1; i>shortArray.length;i--) {
                returnArray[i]= "1"+shortArray[i]; 
            }
            
            return returnArray;
        } 
    }
}
Wo ist der Fehler?
 
Zuletzt bearbeitet von einem Moderator:
Erstens, verwende [code=java]...[/code] Tags für deinen Code, und zweitens: Statt einfach die Aufgabenstellung und deinen Code zu nennen und dann zu fragen "Wo ist der Fehler?", erzähle doch erstmal, warum du glaubst (bzw. worin es sich manifestiert), dass dein Code einen Fehler enthält.
 
Ich bin neu auf dem Forum deswegen kenne ich mich noch nicht so gut aus

Wo genau es am Code hapert weiß ich selber nicht, ich versuche rekursiv den GrayCode zu bilden. Die Basisfälle habe ich bereits getestet und die stimmen so. Mein Problem ist der Rekursionsfall, bei dem ich in einem Array den rekursiven GrayCode abspeichere und in dem zweiten Array diesen mit der 0 bzw 1 vorne erweitere.

Für das Beispiel len=2 bin ich meinen Code durchgegangen und erhalte als Ergebnis 00 01 10 11
es sollte aber 00 01 11 10 rauskommen
 
Wo genau es am Code hapert weiß ich selber nicht
Ja, das ist auch okay und war auch nicht meine Frage. Meine Frage bezog sich nur darauf, dass du keinen Hinweis darauf gegeben hast, dass überhaupt etwas falsch ist an deinem Code, bzw. warum du glaubst, dass dein Code überhaupt einen Fehler hat (Unterschied zwischen tatsächlichem und erwartetem Verhalten/Output) - und nicht, was die Ursache für diesen Fehler ist.
 

Neue Themen


Zurück
Oben