Hamming Code

[Belij]

Guten Tag
Ich habe hier ein Programm geschrieben, wo man eine vierstellige Bitfolge eingibt und ein dazugehöriger Hamming code erstellt wird.
Also ich bin Neuling in Sachen Programmierung, deshalb habe ich noch einige Fragen.
Und zwar habe ich einige Probleme mit der Eingabe, ich möchte das man nur 0 und 1 eingeben kann und es sollen genau vier Stellen sein. Mit anderen Worten, wüsste ich gern wie ich hier eine Eingabekontrolle am besten machen kann.
Desweiteren würde ich gern Verbesserungsvorschläge hören, ich bin mir sicher das ich das eine oder andere vielleicht zu umständlich gemacht habe oder besser machen könnte.

import java.io.*;

public final class Hamming {
	private Hamming() {}

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

		int[] Daten = { 1, 1, 0, 1, 0, 0, 0 };
		int a = 0, b = 0, c = 0, d = 0;
		InputStreamReader isr = new InputStreamReader(System.in);
		BufferedReader br = new BufferedReader(isr);
		System.out.print("Gib die vierstellige Bitfolge an : ");
		String eingabe = br.readLine();
		String teilen[] = eingabe.split(" ");
		
		a = Integer.parseInt(teilen[0]);
		b = Integer.parseInt(teilen[1]);
		c = Integer.parseInt(teilen[2]);
		d = Integer.parseInt(teilen[3]);

		Daten[2] = a;
		Daten[4] = b;
		Daten[5] = c;
		Daten[6] = d;
		
		if ((Daten[2] + Daten[4] + Daten[6]) % 2 == 0) {
			Daten[0] = 0;
		} else {
			Daten[0] = 1;
		}
		if ((Daten[2] + Daten[5] + Daten[6]) % 2 == 0) {
			Daten[1] = 0;
		} else {
			Daten[1] = 1;
		}
		if ((Daten[4] + Daten[5] + Daten[6]) % 2 == 0) {
			Daten[3] = 0;
		} else {
			Daten[3] = 1;
		}
		System.out.println("Die Hamming Codierung ist: " + Daten[0] + " "
				+ Daten[1] + " " + Daten[2] + " " + Daten[3] + " " + Daten[4]
				+ " " + Daten[5] + " " + Daten[6]);
	}

}

 

[stulleman]

Also ich bin auch kein Profi, aber zur Laufzeit kannst du in der CMD den user nicht beschränken was er eintippt.
Du könntest nachdem der User enter gedrückt hat testen ob etwas anderes als eine 0 oder eine 1 drinn ist, und dann den user bitten es erneut zu versuchen (;

Und übrigens kannst du einfach die Scanner-Klasse benutzen um Eingaben zu realisieren, ist sehr viel einfacher!

Max

 

[Gast2]

Und zwar habe ich einige Probleme mit der Eingabe, ich möchte das man nur 0 und 1 eingeben kann und es sollen genau vier Stellen sein. Mit anderen Worten, wüsste ich gern wie ich hier eine Eingabekontrolle am besten machen kann.

Das ginge am einfachste mit ner Regex:

		String eingabe = br.readLine();
		if (!eingabe.matches("[01]( [01]){3}")) {
			System.out.println("falsche eingabe");
			return;
		}

EDIT:
Grad das mit den 4 Stellen gelesen, ist editiert.

 

[Shulyn]

Ein einfaches beispiel:

Lösung von Eike ist viel schöner *Shulyn geht in die Ecke und schmollt*

[Java]
// InputStreamReader isr = new InputStreamReader(System.in);
// BufferedReader br = new BufferedReader(isr);
Scanner tastatur = new Scanner(System.in);
// hier z.B. Schleife beginnen...

System.out.print("Gib die vierstellige Bitfolge an : ");
String eingabe = tastatur.next("0-9"); // bin mir nicht sicher ob das Pattern so passt. Seit jahren keine Scanner Class mehr benutzt
char[] zeichen = eingabe.toCharArray();

for (char input : zeichen) {
try {
int testInt = Integer.valueOf(input);
if (testInt == 0 || testInt == 1) {
// zeichen ok
} else {
// Zeichen keine 0 oder 1... Fehler...
}

} catch (NumberFormatException e) {
// NumberFormatException, also war es keine Zahl-- Fehler weiter
// behandeln
}
}

// wenn Fehler aufgetreten nochmal durch die Schleiche, sonst weiter in der methode...

[/code]

Oder so :

1. Methose einlesen
2. Methose validieren
3. wieder zu 1. wenn validierung nicht ok war, oder weiter mit der zuweisung...

 

[thewulf00]

Du könntest noch Deine eigene Eingabe erstellen. D.h. Du fängst alle Tastendrücke auf und schreibst das Zeichen dann hin, wenn es ne 1 oder 0 ist. Die Funktion verlässt Du erst, wenn Du vier Zeichen hast. So mach ich das immer. Dabei musst Du aber darauf achten, dass Du auch Backspace (#8) und evtl. Pfeiltasten implementierst.

 

[Belij]

Hey danke für die schnellen antworten

Ich habe den Teil von EikeB reingebastelt, ist echt prima.

nur eine kleine Frage hätte ich noch dazu

!eingabe.matches("[01]( [01]){3}"))

diesen Teil verstehe ich nicht ganz "("[01]( [01]){3}"))" was haben diese Zahlen zu bedeuten ?

 

[Gast2]

Das ist nen Regex:
Regulärer Ausdruck ? Wikipedia

 

[thewulf00]

!eingabe.matches("[01]( [01]){3}"))

diesen Teil verstehe ich nicht ganz "("[01]( [01]){3}"))" was haben diese Zahlen zu bedeuten ?

In eckigen Klammern [01] stehen die erlaubten Zeichen, und die {3} sagt, dass die Klammer davor genau 3 mal gefunden werden muss. (Es sind hier zwei Abschnitte: einmal vor der Klammer, einmal in der Klammer - zusammen 4 Zeichen)

 

[Andi_CH]

Lösung von Eike ist viel schöner *Shulyn geht in die Ecke und schmollt*

... hat aber den Nachteil dass eine Eingabe wie 1010 nicht als gültig erkannt wird ;-)

eingabe.matches("[01]( [01]){3}|([01]){4}")

 

[thewulf00]

... hat aber den Nachteil dass eine Eingabe wie 1010 nicht als gültig erkannt wird ;-)

Oh man. Die unendlichen Varianten der RegExen. Kannst Du mir kurz erklären, wieso er es nicht erkennt - ich verstehs nicht so ganz.

 

[faetzminator]

Oh man. Die unendlichen Varianten der RegExen. Kannst Du mir kurz erklären, wieso er es nicht erkennt - ich verstehs nicht so ganz.

[c][01]( [01]){3}[/c] erwartet ein Space zwischen jeder Zahl (darum auch zuerst ein 0 oder 1, danach 3 Mal ein " " und 0 oder 1.
[c][01]{4}[/c] erwartet vier 0 und/oder 1.
[c][01]( [01]){3}|([01]){4}[/c] kombiniert mit dem Oder ([c]|[/c]) beide Validierungen.

 

[Belij]

Also ich denke das die Lösung [01]( [01]){3} in meinem Fall die bessere Lösung ist, da ich ja den String aufteilen und Variablen zuweisen möchte.
Deshalb auch String teilen[] = eingabe.split(" "); überall wo ein Leerzeichen ist soll der String zerteilt werden.

Wenn ich [01]{4} einsetze sagt er mir java.lang.ArrayIndexOutOfBoundsException.
Wäre noch interessant zu erfahren, wie ich das in dem Fall zerteile und Variablen zuweise.

Aber weningsten habe ich jetzt verstanden, wie das mit Regex läuft.

 

[thewulf00]

Eine Aufteilung ohne Leerzeichen ist viel einfacher und geht viel schneller (wenn jedes Zeichen einzeln extrahiert werden soll) - daher ist es auch zu bevorzugen. (... denn die User wollen es lieber ohne Leerzeichen eingeben.)

Aufteilung wäre dann so:

String xyz = "abc";
char[] einzelneZeichen = xyz.toCharArray();

RegExp wäre dann laut Andi_CH diese:

([01]){4}

 

[Andi_CH]

Da stand ja "[01]( [01]){3}" und da beim "x" ist ein Abstand "[01](x[01]){3}"

Also steht da:

[01] Es muss ein 0 oder 1 kommen

( [01]) Es muss ein Abstand gefolgt von einem 0 oder 1 kommen (die (-Klammern wären dazu alleine noch nicht nötig)

{3} Alles vorhergenhende in den ( Klammern wird 3 mal widerholt

Der könnte auch so geschrieben werden "[01] [01] [01] [01]" wird aber sicher niemand so machen ;-)

Aber das alles nur zur Erklärung - ich würde auf jeden Fall eine Eingabe ohne Abstände vorsehen.

Den Regexp kannst du benutzen um zu prüfen ob der user alles richtig eingegeben hat, und im Fall nicht, den auffordern das zu tun ;-) (Bitte an ein Abbruchkriterium denken - z.B. wenn der user x eingibt, sonst wird es mühsam)