Binär -> Dezimal

[Mr. Jack Herer]

Guten Abend,

Ich soll für die Uni ein Programm schreiben, das eine Binärzahl einliest, diese in eine Dezimalzahl umrechnet und ausgibt.
Mir ist klar dass ich das auch über einen einzigen einfachen Befehl erreichen kann, aber Ziel der Übung ist den Vorgang "per Hand" durchzuführen. Mein Ansatz sieht so aus:
[JAVA=42]import java.util.Scanner;

public class Binaer {

public static void main(String[] args) {
Scanner scanner = new Scanner(System.in);

System.out.println("Geben sie eine Binaerzahl ein, um diese in eine Dezimalzahl umrechnen zu lassen: \n");



String bin = scanner.next();
int laenge = bin.length();
int binInt = Integer.parseInt( bin );
double sum = 0;


switch (binInt){
case 1:
case 0: for (int n = 0 ; +laenge <= n ; n++){

if ((int)bin.charAt(0+n) == '0')
sum = sum + 0;
else
sum = sum + Math.pow(2, +laenge+ - +n); }

System.out.println("Binaerzahl: " +bin+ "\nAls Dezimalzahl: " + sum);


default:
System.out.println("Eine Binaerzahl besteht nur aus 0 und 1!");
}
}
}
[/code]


Allerdings gibt er mir hier nur die Binärzahl aus und für die Dezimalzahl immer den Wert 0.
Was mache ich falsch? Ich habe schon gegoogelt und das Forum durchsucht, wollte aber auch keine einfach Copy&Paste Antwort, sondern wissen wieso mein Code nicht funktioniert.-
Vielen Dank schonmal für alle hilfreichen Antworten!

Grüße
Michael

 

[eRaaaa]

Allerdings gibt er mir hier nur die Binärzahl aus und für die Dezimalzahl immer den Wert 0.

Ich hätte eher gesagt es kommt wohl eher immer " System.out.println("Eine Binaerzahl besteht nur aus 0 und 1!");"
Denn wenn ich eine Binärzahl eingebe(die nicht 0 oder 1 ist) trifft natürlich nie das switch bei case 1 oder 0 zu
String bin = 1001;
int binInt = 1001 --> != 0 oder 1

Überdenke das vllt. erstmal!

 

[Mr. Jack Herer]

Ah verdammt das Switch habe ich noch nachträglich kurz vor dem Erstellen des Threads eingefügt.
Bitte lass das einfach außen vor!
Der eigentliche Code besteht nur aus:
[JAVA=42]import java.util.Scanner;

public class Binaer {

public static void main(String[] args) {
Scanner scanner = new Scanner(System.in);

System.out.println("Geben sie eine Binaerzahl ein, um diese in eine Dezimalzahl umrechnen zu lassen: \n");



String bin = scanner.next();
int laenge = bin.length();
int binInt = Integer.parseInt( bin );
double sum = 0;


for (int n = 0 ; +laenge <= n ; n++){

if ((int)bin.charAt(0+n) == '0')
sum = sum + 0;
else
sum = sum + Math.pow(2, +laenge+ - +n); }

System.out.println("Binaerzahl: " +bin+ "\nAls Dezimalzahl: " + sum);

}
}
}
[/code]

 

[eRaaaa]

for (int n = 0 ; +[B]laenge <= n[/B] ; n++){

deine Schleife wird erst gar nicht ausgeführt !

 

[Mr. Jack Herer]

Ja das ist mir auch klar ^^ Als Wert für die summe kommt immer noch der Anfangswert raus (0.0) deswegen habe ich schon vermutet ,dass die Schleife nie gestartet wird, aber wieso ist das so?

 

[eRaaaa]

Der Rumpf wird nur ausgeführt wenn die Bedingung wahr ist, was sie bei dir wohl nicht ist
n = 0 und laenge ist ja vermutlich wohl > 0 , demnach ist die Bedingung laenge <= n = false !

 

[Mr. Jack Herer]

Merce für die Hilfe! Habe das mit der Schleife erst jetzt richtig verstanden... dachte er führt die solange aus, bis die Länge der eingegebenen Zahl den Wert von n erreicht hat...
Aber jetzt ergibt das ganze wesentlich mehr Sinn!
Der Code funktioniert jetzt nach ein paar kleineren Abänderungen auch entlich!

import java.util.Scanner;

public class Binaer {

	public static void main(String[] args) {
		Scanner scanner = new Scanner(System.in);
		
		System.out.println("Geben sie eine Binaerzahl ein, um diese in eine Dezimalzahl umrechnen zu lassen: \n");
		
		

		String bin = scanner.next();
		int laenge = bin.length();
		double sum = 0;
		char [] ziffer = bin.toCharArray();
		
		
      for (int n = 0 ; +laenge > n ; n++){
    	
				 if (ziffer[n] == '0')
					 sum = sum + 0;
				 else
				     sum = sum + Math.pow(2, laenge-1 -n); }

		  System.out.println("Binaerzahl: " +bin+ "\nAls Dezimalzahl: " + sum);
		  

	  
	 }

	}

Danke nochmal für den Hinweis
schönen Abend noch.

 

[Landei]

Du brauchst keine starken Geschütze wie Math.pow

int sum = 0;
char [] ziffer = bin.toCharArray();
for(char c : ziffer) {
   sum = 2 * sum + (c - '0'); 
}

 

[Andi_CH]

for (int n = 0 ; +laenge > n ; n++){
...
}

+laenge ???:L

Äh, das bewirkt doch nichts? (Ist eigentlich nur eine Frage ob ich es richtig verstanden habe )

spannend ist auch

sum = sum + 0;

aber das ist ja wohl selbsterklärend

und dass die Konversion der "Binär"zahl 123 als Resultat 7.0 gibt ist auch sehr unschön

 

[Mr. Jack Herer]

@Andi_CH:
naja das + vor laenge hätt ich mir sparen können, da hast du recht
Aber das [Java]sum = sum + 0[/Java] brauch ich doch um zu beschreiben was er für den Fall machen soll, dass an der Stelle der Binärzahl eine '0' steht. Oder hätte [Java]sum = sum [/Java] auch gereicht?
Für den Fall dass man etwas anderes als 1 und 0 eingibt, hab ich noch hinzugefügt:

[Java]for (int i=0;i < laenge; i++){
if (ziffer != '1'&& ziffer != '0'){
System.out.println("Eine Binaerzahl besteht nur aus 0 und 1!");
System.exit( 0 );}}[/Java]


[Java]import java.util.Scanner;

public class Binaer {

public static void main(String[] args) {
Scanner scanner = new Scanner(System.in);

System.out.println("Geben sie eine Binaerzahl ein, um diese in eine Dezimalzahl umrechnen zu lassen: \n");



String bin = scanner.next();
int laenge = bin.length();
double sum = 0;
char [] ziffer = bin.toCharArray();


for (int i=0;i < laenge; i++){
if (ziffer != '1'&& ziffer != '0'){
System.out.println("Eine Binaerzahl besteht nur aus 0 und 1!");
System.exit( 0 );}}


for (int n = 0 ; +laenge > n ; n++){

if (ziffer[n] == '0')
sum = sum + 0;
else
sum = sum + Math.pow(2, laenge-1 -n); }

System.out.println("Binaerzahl: " +bin+ "\nAls Dezimalzahl: " + sum);}



}[/Java]

 

[XHelp]

Weder

sum=sum;

noch

sum=sum+0;

bewirkt irgendwas. Also kannst du auch direkt auf =='1' prüfen und den else-Zweig weglassen.

Benutz lieber IMMER die geschweiften Klammer und achte auf die Codeeinrückung.

 

[Andi_CH]

@Andi_CH:
naja das + vor laenge hätt ich mir sparen können, da hast du recht
Aber das [Java]sum = sum + 0[/Java] brauch ich doch um zu beschreiben was er für den Fall machen soll, dass an der Stelle der Binärzahl eine '0' steht. Oder hätte [Java]sum = sum [/Java] auch gereicht?

Nein, das brauchst du echt nicht. Der Kompiler schafft es definitv ohne und für den Menschen ist Komentar besser geeignet als Code

if (ziffer[n] == '1')
    sum = sum + Math.pow(2, laenge-1 -n); }

reicht völlig!

In komplexeren Beispielen als diesem hier würde ich einen Kommentar hinschreiben, so im Sinne

// Im Fall von Ziffer[n]='0' bleib sum unverändert

Ich denke hier ist es auch ohne Text klar genug : Im Falle von 1 wird etwas addiert. Im Falle von 0 eben nicht.

Wir sind ja schon so weit, hier das Ganze:

import java.util.Scanner;

public class Binaer {

	public static void main(String[] args) {
		Scanner scanner = new Scanner(System.in);

		System.out.println("Geben sie eine Binaerzahl ein, um diese in eine Dezimalzahl umrechnen zu lassen: \n");

		String bin = scanner.next();
		int laenge = bin.length();
		double sum = 0;
		char [] ziffer = bin.toCharArray();

		for (int i=0;i < laenge; i++){
			if (ziffer[i]  != '1'&& ziffer[i] != '0'){
				System.out.println("Eine Binaerzahl besteht nur aus 0 und 1!");
				System.exit( 0 );
			}
		}

		for (int n = 0 ; laenge > n ; n++){
			if (ziffer[n] == '1') {
				sum += Math.pow(2, laenge-1 -n);
			}
		}
		System.out.println("Binaerzahl: " +bin+ "\nAls Dezimalzahl: " + sum);
	}
}