Problem bei "Quersumme-Programm"

[S4ntaClauZ]

Hallo erstmal,

ich bin sowohl hier neu, wie auch allgemein in Sachen Java. Aufgrund eines Informatik-Studiums lerne ich jetzt Java, und wir haben da letztens eine Aufgabe bekommen die hieß:

Quersumme der letzten 7 Ziffern einer 10 stelligen Zahl bilden, das Ergebnis durch 7 teilen und den Rest ausgeben.

Ich habe das Programm rein logisch gesehen glaub ich richtig, allerdings gibt es noch irgendein Problem, das ich nicht ausfindig machen kann, und zwar stimmt das Ergebnis bei höheren Zahlen nicht bzw. geht gar nicht. Ich schätze mal, dass es ein Problem mit dem Datentyp(int) gibt, leider weiß ich noch nichtmal welchen Datentyp ich hernehmen muss. Wäre super wenn ihr mit helfen könntet.

Hier das Programm:

import java.io.*;

public class Quersumme {

public static void main(String[] arg) throws IOException {

InputStreamReader reader = new InputStreamReader(System.in);
      BufferedReader input = new BufferedReader(reader);

System.out.print("Bitte geben Sie eine 10-stellige Zahl ein. "); 
String Zahl10 = input.readLine(); 

int Z10 = Integer.parseInt(Zahl10); 

int a = Z10 % 10;
int a2 = Z10 / 10;

int b = a2 % 10;
int b2 = a2 / 10;

int c = b2 % 10;
int c2 = b2 / 10;

int d = c2 % 10;
int d2 = c2 / 10;

int e = d2 % 10;
int e2 = d2 / 10;

int f = e2 % 10;
int f2 = e2 / 10;

int g = f2 % 10;

int Q = a + b + c + d + e + f + g; 

int ERGEBNIS = Q % 7; 

System.out.println("Das Ergebnis ist "+ERGEBNIS+"."); 

	}
}

 

[Final_Striker]

Quersumme der letzten 7 Ziffern einer 10 stelligen Zahl bilden, das Ergebnis durch 7 teilen und den Rest ausgeben.

Du teilst nicht, du rechnest modulo.

Ich habe das Programm rein logisch gesehen glaub ich richtig, allerdings gibt es noch irgendein Problem, das ich nicht ausfindig machen kann, und zwar stimmt das Ergebnis bei höheren Zahlen nicht bzw. geht gar nicht.

Geht das vielleicht auch etwas genauer.

 

[XHelp]

1. Hinweis: schau dir Schleifen an.
Zu deiner Frage: int ist nur 32 bit lang, d.h. du kannst nicht alle 10-stellige Zahlen darstellen, bei knapp über 2 Milliarden ist Schluss. Verwende einfach

long

 

[Toffel2]

und sollt ihr das wirklich mit den ganzen Variablen machen oder doch nicht eher durch eine Rekursion?

 

[S4ntaClauZ]

Du teilst nicht, du rechnest modulo.




Geht das vielleicht auch etwas genauer.

Das mein ich doch, modulo ist doch eigentlich teilen und nur den rest ausgeben.
Mehr kann ich dazu leider auch nicht sagen, bei kleinen Zahlen fuunktioniert es, bei größeren irgendwann nichtmehr, lass das Programm einfach mal laufen.

1. Hinweis: schau dir Schleifen an.
Zu deiner Frage: int ist nur 32 bit lang, d.h. du kannst nicht alle 10-stellige Zahlen darstellen, bei knapp über 2 Milliarden ist Schluss. Verwende einfach long

ok danke ich werd das mal ausprobieren!

und sollt ihr das wirklich mit den ganzen Variablen machen oder doch nicht eher durch eine Rekursion?

mehr haben wir bis jetzt nicht gelernt, ich hab mir auch schon gedacht das kann man doch sicher viel einfacher machen, aber ich denke wir sollten die Aufgabe schon mit unserem bisherigen Wissen lösen.

 

[S4ntaClauZ]

Habe es jetzt mit double probiert aber jetzt kommen nur falsche Ergebnisse raus...???:L

 

[XHelp]

und wie kommst du auf double? oO

 

[Marcinek]

Bei welcher eingabe erhälst due welches falsche Ergebnis?

 

[S4ntaClauZ]

OH sry, hab aus versehen statt long, double benutzt. Jetzt funktionierts einwandfrei, vielen Dank!:toll:

 

[njans]

Wenn du es mit einer Schleife machst, gehts "einfacher"

int summe = 0;
for(int postion = 0; position < Zahl10.length;position++)
{
  summe += Integer.parseInt(Zahl10.charAt(position));
}
}

 

[Marcinek]

OH sry, hab aus versehen statt long, double benutzt. Jetzt funktionierts einwandfrei, vielen Dank!:toll:

Was kommt raus, wenn du 123456789 eingibst?

 

[S4ntaClauZ]

Was kommt raus, wenn du 123456789 eingibst?

Da kommt 0 raus.

 

[Marcinek]

--- ok ich dachte es wäre die Quersumme, aber das stimmt ja nicht so ganz ^^

<- hier stand müll ->

 

[Andi_CH]

Beim Anfangsansatz finde ich schräg, dass man zuerst mit Parsint eine Zahl zusammenbaut und dann mühsam wieder auseinandernimmt.
Loop doch lieber gleich über den String und parse die einzelnen Ziffern einzeln.

Unser Azubi hatte wieder mal zu wenig Arbeit 🙂

Sein Vorschlag:

public class Quersumme {

	private static int toInt(char c) {
		return Integer.parseInt(""+c);
	}

	private static int[] split (long pValue) {
		String s = ""+pValue;
		int[] res = new int[s.length()];
		for (int i=0; i<s.length(); i++) {
			res[i] = toInt(s.charAt(i));
		}
		System.out.println(Arrays.toString(res));
		return res;
	}
	
	private static int quersumme(int[] intArr) {
		int res = 0;
		for (int i=0; i<7; i++) {
			res += intArr[intArr.length-1-i];
		}
		return res;
	}

	public static void main(String[] args) {
		long wert = 1234567890;
		int[] intArr = split(wert);
		int quersumme = quersumme(intArr);
		System.out.println("Eingabewert = " + wert);
		System.out.println("Array von Ziffern = " + Arrays.toString(intArr));
		System.out.println("Quersumme der letzten 7 Stellen = " + quersumme);
		System.out.println("Quersumme / 7.0 = " + quersumme / 7.0);
	}
}

 

[Andi_CH]

Das Hauptprogramm mit einlesen:

	public static void main(String[] args) {
		int[] intArr;
		InputStreamReader reader = new InputStreamReader(System.in);
		BufferedReader input = new BufferedReader(reader);
		do {
			System.out.print("Bitte geben Sie eine 10-stellige Zahl oder exit ein. "); 
			try {
				String eingabe = input.readLine();
				if ( "exit".equalsIgnoreCase(eingabe)) {
					break;
				} 
				if (eingabe.length() < 7) {
					System.out.println("Eingabe ist zu kurz");
				} else {
					intArr = new int[eingabe.length()];
					for (int i=0; i<intArr.length; i++) {
						intArr[i] = Integer.parseInt("" + eingabe.charAt(i));
					}
					int quersumme = quersumme(intArr);
					System.out.println("Eingabe = " + eingabe);
					System.out.println("inArr = " + Arrays.toString(intArr));
					System.out.println("Quersumme = " + quersumme);
					System.out.println("Quersumme / 7.0 = " + quersumme / 7.0);
				}
			} catch (NumberFormatException e) {
				System.out.println("Bitte nur Ziffern eingeben");
			} catch (IOException e) {
				e.printStackTrace();
			}
		} while (true);
		System.out.println("Auf Wiedersehen");
	}

 

[S4ntaClauZ]

Ok vielen Dank! Ich versteh jetzt zwar noch nicht allzu viel von dem Code aber ich werde mich da mal durcharbeiten 😉

Meine Professorin hat sich mein Programm schon angeschaut und ist sehr zufrieden, was anderes war gar nicht verlangt, man sollte die Aufgabe schon mit unseren bisherigen Kenntnissen lösen.