String überprüfen

[WayWay]

Hallo erstmal .. ich bin ein anfänger ^^
Ich wollt ein String überprüfen undzwar so dass wenn andere zeichen außer (A-Z) (a-z) und (0-9) kommen , soll es mir ein fehler übergeben ... nur funktioniert das ganze nicht so richtig
hier mein Java Code

	public static boolean fehler(String name) {
		boolean fehler = false;
		for (int i = 0; i < name.length() || fehler; i++) {
			if ((65 > (int) name.charAt(i) && (int) name.charAt(i) > 90)
					|| (48 > (int) name.charAt(i) && (int) name.charAt(i) > 59)
					|| (97 > (int) name.charAt(i) && (int) name.charAt(i) > 127)) {
				fehler = true;
			}

		}
		return fehler;

	}

	public static void main(String[] args) {

		if (!fehler("4544")) {
			System.out.print("fehler");
		} else
			System.out.print("kein fehler");
	}

 

[Final_Striker]

[c]65 > (int) name.charAt(i) && (int) name.charAt(i) > 90[/c]


"wenn das Zeichen kleiner als 65 und größer als 90 ist"

Kann ja nur immer falsch sein, da ein Zeichen nicht gleichzeitig kleiner 65 und größer 90 sein kann.

 

[XHelp]

Ich denke da wäre RegEx angebracht. Sowas wie:

public static boolean checkString(String s) {
  return s.matches("[\\w&&[^_]]+");
}

Außerdem ist deine Auswertung ziemlich komisch. Du überprüfst, ob es kein Fehler ist und gibst dann "Fehler" aus...

 

[WayWay]

[c]65 > (int) name.charAt(i) && (int) name.charAt(i) > 90[/c]


"wenn das Zeichen kleiner als 65 und größer als 90 ist"

Kann ja nur immer falsch sein, da ein Zeichen nicht gleichzeitig kleiner 65 und größer 90 sein kann.

Ach du scheiße bin ich blääd danke ^^
hat trotzdem nicht funktioniert

Ich denke da wäre RegEx angebracht. Sowas wie:

public static boolean checkString(String s) {
  return s.matches("[\\w&&[^_]]+");
}

Außerdem ist deine Auswertung ziemlich komisch. Du überprüfst, ob es kein Fehler ist und gibst dann "Fehler" aus...

Ja ich darf RegEx nicht benutzen trotzdem danke

 

[xehpuk]

Du hast es bei allen drei Vergleichen so gemacht.

Außerdem siehts so aus, als wären deine Operatoren vertauscht.

public static boolean fehler(String name) {
	for (int i = 0; i < name.length(); i++) {
		char c = name.charAt(i);
		if (('0' > c || c > '9')
			&& ('A' > c || c > 'Z')
			&& ('a' > c || c > 'z'))
			return true;
	}
	return false;
}

 

[WayWay]

naja habs raus trotzdem danke an alle !!
mich erfreuts dass man hier schnell antwort bekommt
hier mein Lösung

	public static boolean fehler(String name) {
		boolean fehler = false;
		for (int i = 0; i < name.length(); i++) {
			if (65 < (int) name.charAt(i) && (int) name.charAt(i) > 90) {

			} else {
				if (48 < (int) name.charAt(i) && (int) name.charAt(i) < 59) {
					
				} else {
					if (97 < (int) name.charAt(i) && (int) name.charAt(i) < 127) {
						
					} else {
						fehler = true;
					}
				}
			}

		}
		return fehler;

	}

	public static void main(String[] args) {

		if (fehler("45?4")) {
			System.out.print("fehler");
		} else
			System.out.print("kein fehler");
	}

 

[XHelp]

Ersetze die Zahlen lieber durch chars, dann kann man es auch besser lesen.
Außerdem wenn dir den ursprüngliche Inhalt egal ist, dann kannst du den String auch einfach in Kleinbuchstaben umwandeln. So hast du weniger Abfragen:

public static boolean checkError(String s) {
	s = s.toLowerCase();
	for (Character c:s.toCharArray()) {
		if ((c<'0'||c>'9') && (c<'a'||c>'z')) {
			return true;
		}
	}
	return false;
}