Verschlüsselung von String

[Raistlin]

Moin Gemeinde,
ich habe folgendes Problem:
für die aktuelle Übung in Java muss ich einen Codierer schreiben, der folgendermaßen funktioniert. Er soll einen String einlesen, die einzelnen Buchstaben um drei Stellen im Alphabet nach vorn verschieben und dann wieder ausgeben. Ich bin schonmal soweit, dass er den String einließt, er gibt ihn aber wieder unverändert aus, wenn er mehr als ein Zeichen enthält. Hier ist meine Methode:

public String encode() throws IOException {
    String d = in.readLine();
    char[] e = d.toCharArray();
    for (int i = 0; i == (e.length - 1); i++){
      System.out.println (e[i]);
      if (e[i]!= ' '){
        f = (int)e[i];
        System.out.println (f);
        f += offset;
        System.out.println (f);
        if (f > 90 && f < 97 || f > 122)
          f -=26;       
      }  
      else 
        c = e[i];
      c = (char)f;
      e[i] = c;
    }
    in.close();
    res = new String (e);
    return res;   
  }

ich habe wohl irgendwo in meiner for-Schleife nen Fehler, den ich nicht finde.
Kann mir da jemand helfen?

 

[Andi_CH]

Zeile 4

i == xxx

ist keine gute Idee denke ich mal. Wie wärs mit

i <= xxx

for (int i = 0; i < e.length; i++){
...
}

 

[AmunRa]

for (int i = 0; i < (e.length ); i++){

edit :

so ist das auch schöner

if (f > 'Z' && f < 'a' || f > 'z')

 

[Volvagia]

Schau dir mal die for an.

btw. "Character.getType(f) == Character.UPPERCASE_LETTER || Character.getType(f) == Character.LOWERCASE_LETTER"
Du solltest vielleicht auch bessere Varnamen vergeben, bei späteren Modifikationen weißt du sicher nicht mehr, was welche macht.

Edit: Lahm.

 

[Raistlin]

Ihr seid genial. Und so schnell!! Vielen Dank. Problem gelöst. Da hätt ich echt selbst drauf kommen können.