Scanner auf 10 Stellen begrenzt?

[i++]

Hi!
Vorhin schrieb ich ein Programm, welches eine Binärzahl auf eine Dezimalzahl umrechnet.
Beim Ausprobieren fiel mir auf, dass bei einer Int länge von 10 Schluss ist.
Sobald ich 11 Stellen ausgebe, zeigt mir Java exception an.
Somit funktioniert mein Programm nur bis zur Dezimalzahl 1023

Im Internet fand ich leider keine Information auf wie viele Stellen der Scanner beschränkt ist, könnt ihr mir Abhilfe schaffen, damit mein Programm auch bis n Stellen funktioniert? Kann man den Scanner "erweitern"?

 

[kneitzel]

Wenn Du Dir die Scanner Klasse ansiehst (z.B. https://docs.oracle.com/javase/7/docs/api/java/util/Scanner.html), dann siehst Du, dass da deutlich mehr möglich ist.

Wenn Du einen int Wert auslesen lässt, dann ist dieser natürlich auf den Wertebereich eines int Wertes beschränkt. So geht der int von -2147483648 (-2^31) bis 2147483647 (2^31 -1).

Es gibt aber auch noch den Datentyp long, der von -9223372036854775808 (-2^63) bis 9223372036854775807 (2^63 -1) geht.

Sollte dies auch nicht ausreichen, dann kann man BigInteger oder BigDezimal nutzen.

 

[mihe7]

Vermutung: Du verwendest Scanner#nextInt() - ein int ist ein 32-Bit breiter, vorzeichenbehafteter Datentyp, der größte darstellbare positive Wert beträgt also ca. 2,1 Mrd., was 10 Stellen entspricht. EDIT: @kneitzel war schneller

Hier mal Code, der auch bei sehr großen Zahlen noch funktioniert:

import java.math.BigInteger;
import java.util.Scanner;

public class Test {
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);
        String line = sc.nextLine();
        BigInteger z = BigInteger.ZERO;
        BigInteger TWO = BigInteger.valueOf(2);

        for (char c : line.toCharArray()) {
            if (c == '0') {
                z = z.multiply(TWO);
            } else if (c == '1') {
                z = z.multiply(TWO).add(BigInteger.ONE);
            }
        }
        System.out.println(z.toString());
    }   
}

 

[i++]

Danke euch zwei für eure schnellen Antworten!
Ich hab die Integer auf Long umgeändert und kann nun bis 65k anstatt nur 1k!
Könnt ihr bitte meinen Code mal bewerten und Verbesserungen mir erläutern? Bitte bedenkt, dass ich Programmieranfänger bin. In erster Linie bin ich froh, dass ich das Programm funktionierend hingebracht habe. Subjektiv würde ich sagen, dass es mit Sicherheit noch kompakter zum Programmieren ginge, also weniger Zeilen sowie Variablen. Ich häng das Programm mal an, bitte schreibt mir eure Verbesserungen.

package schwachsinn;
import java.util.*;

/**
 * Das Programm rechnet eine Binärzahl in eine Dezimalzahl um.
 * @author xyz
 *
 */
public class BinearAufDezimal {

    //Hauptprogramm ausführen
    public static void main(String[] args) {
        Scanner sc= new Scanner(System.in);
        
        //Eingabe der Binärzahl
        System.out.println("Geben Sie die Binärzahl ein:");
        long binary=sc.nextLong();
        
        //Berechnung der Ziffernanzahl
        int anzahl = (int)Math.log10( binary ) + 1;
    
        //Variablen initalisieren
        int decimal=0; long teiler=1; int i=0;
        
        for(int x=1; x<=anzahl; x++) {
            
            //Gibt jeweils die einserstelle, zehnerstelle etc... weiter
            long y=(binary/teiler)%10;
            
            //Umrechnung von Binär auf Dezimal
            int z=0;
            if (y==1) z=(int)(Math.pow(2, i));
            i++;
            teiler*=10;
            decimal=decimal+z;
            }
        
        //Ausgabe
        System.out.println("Die Dezimalzahl lautet: "+decimal);

        sc.close();
    }
}

 

[mrBrown]

Spontan: lies die Binärzahl als String und nicht als int/long ein


Ansonsten: nutz für if's immer Klammern und rück vernünftig ein

 

[kneitzel]

Also was mir bei Deinem Code auffällt:
- i ist doch immer x-1, also braucht man doch eigentlich nur eine der Variablen, oder?
- Die Begrenzung ist natürlich noch relativ stark. Wieso gehst Du nicht hin und liest einen String ein? Dann hast Du die Limitierung nicht mehr.
- Scanner auf System.in war schon öfters ein Thema. Scanner implementiert AutoCloseable, d.h. dein close() Aufruf ist soweit korrekt. Aber das schließt dann auch System.in, was blöd ist. Daher war schon einmal die Überlegung hier im Forum dass man dies einfach als public static final nutzt ... dann gibt es keinen close Aufruf. Und wenn man das mit close() haben will, dann wäre try with ressources besser.

Also eine Lösung, wie ich sie schreiben würde, wäre in etwa so:

package schwachsinn;

import java.util.*;

/**
 * Das Programm rechnet eine Binärzahl in eine Dezimalzahl um.
 */
public class BinaerAufDezimal {

    /**
     * Scanner on System.in to use globaly.
     */
    public static final Scanner IN = new Scanner(System.in);

    /**
     * Hauptprogramm.
     * @param args Kommandozeilen Argumente, die nicht ausgewertet werden.
     */
    public static void main(String[] args) {
        // Eingabe der Binärzahl in einen String.
        System.out.println("Geben Sie die Binärzahl ein:");
        String binary=IN.next().trim();

        // Länge des Strings prüfen.
        int anzahl = binary.length();
        if (anzahl > 64) {
            System.err.println("Eingabe ist beschränkt auf 64 Ziffern.");
            return;
        }

        long decimal = 0;
        for(int index=0; index<anzahl; index++) {

            decimal = decimal<<1; // Verschiebe um eins

            switch (binary.charAt(index)) {
                case '0':
                    // Nothing to do.
                    break;

                case '1':
                    decimal++;
                    break;

                default:
                    System.err.println("Ungültges Zeichen, geben sie nur 0 und 1 für die Zahl ein!");
                    return;
            }
        }

        System.out.println("Die Dezimalzahl lautet: "+decimal);
    }
}