4 Bit in einer for-schleife funktioniert nicht

[stormyark]

Hey, ich möchte eine vier Bit Zahl hochzählen lassen und es dann als Binärzahl ausgeben.

Mein Plan war es eigentlich das eine for-Schleife hochzählt und dann abfrägt:

"Ist die angegebene Zahl größer als 8?"
wenn ja
ersteStelle = "1";

"Ist die angegebene Zahl größer als 4?"
wenn ja
zweiteStelle = "1";

So setzten sich also die vier Bit zahlen nach einander zusammen

Das sind die vier Bit Zahlen
[CODE lang="java" title="Bitzahlen"]//0001 - 1
//0010 - 2
//0011 - 3
//0100 - 4
//0101 - 5
//0110 - 6
//0111 - 7
//1000 - 8
//1001 - 9
//1010 - 10
//1011 - 11
//1100 - 12
//1101 - 13
//1110 - 14
//1111 - 15[/CODE]

Wenn ich das mit einer normalen if-Abfrage mache und die Zahl vorher definiere funktioniert alles perfekt

[CODE lang="java" title="if-Abfrage"]vierBit = 12
if (vierBit >= 8) {
ersteStelle = "1";
vierBit = vierBit - vierterBit;
}
if (vierBit >= 4) {
zweiteStelle = "1";
vierBit = vierBit - dritterBit;
}
if (vierBit >= 2) {
dritteStelle = "1";
vierBit = vierBit - zweiterBit;
}
if (vierBit >= 1) {
vierteStelle = "1";
}
}
int teilergebnis = vierBit;
ergebnis = ("Die Bitzahl von "+teilergebnis+" ist: "+ersteStelle + zweiteStelle + dritteStelle + vierteStelle);
System.out.println(ergebnis);[/CODE]

Doch wenn ich den Code in eine for-Schleife packe

[CODE lang="java" title="for-Schleife"]vierBit = 0;
for(int i = 0; i <= 15; i++) {
int teilergebnis = vierBit;
if (vierBit >= 8) {
ersteStelle = "1";
vierBit = vierBit - vierterBit;
}
if (vierBit >= 4) {
zweiteStelle = "1";
vierBit = vierBit - dritterBit;
}
if (vierBit >= 2) {
dritteStelle = "1";
vierBit = vierBit - zweiterBit;
}
if (vierBit >= 1) {
vierteStelle = "1";
}
}
ergebnis = ("Die Bitzahl von "+teilergebnis+" ist: "+ersteStelle + zweiteStelle + dritteStelle + vierteStelle);
System.out.println(ergebnis);[/CODE]

doch dann ist die Ausgabe:

"Die Bitzahl von 15 ist: 0000"

wie löse ich das Problem

 

[kneitzel]

Mehrere Punkte:

a) Du rechnest in der Schleife mit der Variablen vierbit. Diese ist aber immer 0 und daher nie > als irgend eine Zahl welche >0.

b) Dann musst du evtl. auch schauen: Du rechnest in der Schleife etwas, aber die Ausgabe ist außerhalb der Schleife. Das macht so natürlich wenig Sinn.

c) Du setzt die Ergbnisse auf den Stellenimmer nur auf 1 aber nie zurück. Also werden die Stellen (wenn du den Fehler a behoben hast) alle nach und nach auf 1 gesetzt.

Bei sowas sind meine Ratschläge:
a) Immer erst genau in Worten formulieren, was Du machen willst.
b) Teile und Herrsche! Baue Methoden, die das machen. Du hast Die Unterteilung einer Zahl gebaut und das funktioniert? Dann mach das in eine Methode und in der Schleife rufst Du dann diese Methode einfach wiederholt auf. ==> Code ist einfacher zu überblicken und Fehler wie b) und c) sind deutlich einfacher zu vermeiden und der Code wird auch deutlich besser lesbar.

 

[stormyark]

Mehrere Punkte:

a) Du rechnest in der Schleife mit der Variablen vierbit. Diese ist aber immer 0 und daher nie > als irgend eine Zahl welche >0.

b) Dann musst du evtl. auch schauen: Du rechnest in der Schleife etwas, aber die Ausgabe ist außerhalb der Schleife. Das macht so natürlich wenig Sinn.

c) Du setzt die Ergbnisse auf den Stellenimmer nur auf 1 aber nie zurück. Also werden die Stellen (wenn du den Fehler a behoben hast) alle nach und nach auf 1 gesetzt.

Bei sowas sind meine Ratschläge:
a) Immer erst genau in Worten formulieren, was Du machen willst.
b) Teile und Herrsche! Baue Methoden, die das machen. Du hast Die Unterteilung einer Zahl gebaut und das funktioniert? Dann mach das in eine Methode und in der Schleife rufst Du dann diese Methode einfach wiederholt auf. ==> Code ist einfacher zu überblicken und Fehler wie b) und c) sind deutlich einfacher zu vermeiden und der Code wird auch deutlich besser lesbar.

Danke für dein Antwort, du hast mir sehr weiter geholfen

Hab jetzt die Lösung meines Problems gefunden

for (int i = 0; i <= 15; i++) {
      teilergebnis = i;
      vierBit = i;
      if (vierBit >= 8) {
        ersteStelle = "1";
        vierBit = vierBit - vierterBit;
      } else {
        ersteStelle = "0";
      }
      if (vierBit >= 4) {
        zweiteStelle = "1";
        vierBit = vierBit - dritterBit;
      } else {
        zweiteStelle = "0";
      }
      if (vierBit >= 2) {
        dritteStelle = "1";
        vierBit = vierBit - zweiterBit;
      } else {
        dritteStelle = "0";
      }
      if (vierBit >= 1) {
        vierteStelle = "1";
      } else {
        vierteStelle = "0";
      }
      ergebnis = ("Die Bitzahl von "+teilergebnis+" ist: "+ersteStelle + zweiteStelle + dritteStelle + vierteStelle);
      System.out.println(ergebnis);
}

 

[kneitzel]

Das sieht schon sehr gut aus. Falls Du noch etwas weiter daran arbeiten willst, dann wäre nur noch die Aufteilung in mehrere Methoden ein Punkt, den man sich ansehen könnte / sollte.

Aber die angesprochenen Fehler hast Du korrekt behoben (was ich jetzt so gesehen habe).

Ein Punkt, der aber vermutlich erst später kommen wird: Man kann sich noch überlegen, Werte anders zu speichern, also z.B. in Arrays zu packen und so. Aber ich weiss nicht, was Du Dir bisher so angesehen hast, daher ist es evtl. unklug, dies jetzt schon anzusprechen.