Problem mit Strings

[Sushi]

Hi, habe eine methode zum erfassen von Wörten die vorne wie rückwärts das selbe bedeuten geschrieben. mein problem die methode gibt immer true aus. Danke schonmal für hilfe.

public static boolean istSatz (String satz){
		String s ="";
		int o = 0;
		int u = s.length() - 1;
		
		
		for (int i = 0; i < satz.length(); i++){
			char c = satz.charAt (i);
			if (Character.isLetter (c)){
				if (Character.isUpperCase(c))
					s = s + Character.toLowerCase(c);
				else s = s + c;}}
		
		
		
		if (o >= u){
			return true;
		}
		if (s.charAt(u) != s.charAt (o)) {
			return false;
		}
		else{
			for (int i = 1; i < s.length()-i; i++){
			s = s.substring(i, s.length()-i);}
			return true;
		}
	}[\code]

 

[Gast]

hi, dein code macht keinen sinn....
wenn du mal einen debugger nimmst, wirst du schnell sehen, dass
o = 0 und u = -1 ist und bleibt....!!!
dann gibt er einfach true aus, weil 0>=-1 wahr ist

 

[Gast]

public class Palindrome {

public static void main(String [] args){

System.out.println(isPalindrome("maoam"));
}
public static boolean isPalindrome(String word) {
int left = 0;
int right = word.length() -1;

while (left < right) {
if (word.charAt(left) != word.charAt(right)) {
return false;
}
left++;
right--;
}

return true;
}
}

 

[Sushi]

hm aber u ist odch die satz.length also wenn mein bsp "Ottok" wäre ist satz.length 5

 

[Gast]

guck dir mal den unteren code an........ und versuch's damit mal

 

[SlaterB]

u ist nicht satz.length() sondern s.length()

edit:
wobei noch zig andere seltsame Dinge danach kommen,
die Schleife im letzten else-Fall ist z.B. lustig,
macht nix sinnvolles, am Ende kommt immer 'return true'

 

[Gast]

genau

 

[Sushi]

ok damit habe ich mich wieder einmal vollkommen blamiert anscheinend.^^
naja üben üben üben..

 

[Sushi]

Ok mal eine ganz andere frage.

Wenn ich in der methode

for ( o = 0; o < satz.length(); o++){
char c = satz.charAt (o);
if (Character.isLetter (c)){
if (Character.isUpperCase(c))
s = s + Character.toLowerCase(c);
else s = s + c;}}


weglasse funktioniert jetzt alles eigentlich.

wenn ich sie mit einfüge spuckt er mir immer true aus. eigentlich sollte dieser teil doch nur die eingabe so verändern, dass alle buchstaben klein sind und satzzeichen ignoriert werden. Wie kann sie also das ergebnis beeinflussen?

 

[Gast]

Schreib mal deine komplette Klasse die jetzt funktioniert hier rein... ich verstehe nämlich gerade kein Wort von dem was du schreibst

außerdem wurde die lösung für dein problem oben schon komplett
abgetippt

 

[Sushi]

public class worttest {


public static void main(String[] args) {

System.out.println(istPalindromSatz("ottok"));

}

public static boolean istPalindromSatz (String satz){
String s= "";
int o = 0;
int u = satz.length() - 1;


for ( o = 0; o < satz.length(); o++){
char c = satz.charAt (o);
if (Character.isLetter (c)){
if (Character.isUpperCase(c))
s = s + Character.toLowerCase(c);
else s = s + c;}}



while (o < u) {
if (s.charAt(o) != s.charAt(u)) {
return false;
}
o++;
u--;
}

return true;
}
}

 

[Gast]

das ist auf den ersten blick die lösung von oben
warscheinlich kopiert hehehe
und wenn du jetzt noch diese vollkommen sinnlose for-schleife wegnimmst und es dann auch KAPIERT hast was da passiert...
dann ... Herzlichen Glückwunsch!!

 

[Sushi]

aber wenn ich die for schleife raus nehme unterscheidet mir java doch zwisdchen groß und kleinschreibung und auch satzzeichen werden nicht mehr ignoriert

 

[Gast]

ja, du musst wissen was du willst.....
oben wolltest du noch einfach vorwärts, rückwärts gleiche worte...

schreib doch einfach:
word = word.toLowerCase();

dann wird dein wort klein geschrieben und maoAm ist dann z.B. ein Palindrom

 

[Demoncwue]

Dazu hab ich auch eine frage:
welche funktion gibt es um aus einen String satz (zb: "bla& bla%)
alle sonderzeichen und leerzeichen zu entfernen, damit man palindrome über wortgrenzen hinweg erkennt?
danke für die hilfe

 

[Gast]

ja das interressiert mich auch
hat jemand eine idee wie man das machen kann?

 

[SlaterB]

replaceAll()

 

[Gast]

hm damit kann man doch nur die werte ausklammern die man hineinschreibt. stelle ich mir etwas kompliziert vor alle satzzeichen usw einzufügen.

 

[SlaterB]

wie soll man sonst mehrere Sachen löschen, wenn man sie nicht alle einzeln nennt?

aber du hast Glück, bei Regulären Ausdrücken gibts auch vordefinierte Gruppen von Zeichen

http://java.sun.com/javase/6/docs/api/java/util/regex/Pattern.html

public class Test {

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

		String st = "hi d#a!sskl,jd.";
		st = st.replaceAll("\\p{Punct}", "------");
		System.out.println(st);
	}

}

->
hi d------a------sskl------jd------