Fehlermeldung bei for-Schleife

[Anfänger123]

Hallo!

Ich komm hier grad nich weiter. Das compilieren von diesem Programm geht gut, aber warum kommen im laufenden Programm dann Fehlermeldungen, liegt anscheinend der 73. Zeile (Beschränkung der for-Schleife a=a+1).

import java.awt.*;
import java.awt.event.*;

import java.io.CharArrayWriter.*;

import java.util.*;

import javax.swing.*;
import javax.swing.event.*;


/**
  *
  * Beschreibung
  *
  * @version 1.0 vom 28.04.2010
  * @author
  */
public class KennzeichenKontrolle extends JFrame {
  // Anfang Attribute
  private JTextField Eingabe = new JTextField();
  private JButton Button = new JButton();
  private JTextArea Ausgabe = new JTextArea("");

  // Ende Attribute
  public KennzeichenKontrolle(String title) {
    // Frame-Initialisierung
    super(title);
    setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);

    int frameWidth = 268;
    int frameHeight = 132;
    setSize(frameWidth, frameHeight);

    Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
    int x = (d.width - getSize().width) / 2;
    int y = (d.height - getSize().height) / 2;
    setLocation(x, y);

    Container cp = getContentPane();
    cp.setLayout(null);
    // Anfang Komponenten
    Eingabe.setBounds(8, 8, 145, 24);
    Eingabe.setText("Kennzeichen");
    cp.add(Eingabe);
    Button.setBounds(160, 8, 81, 25);
    Button.setText("Prüfen");
    Button.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent evt) {
          Button_ActionPerformed(evt);
        }
      });
    cp.add(Button);
    Ausgabe.setBounds(8, 40, 225, 33);
    Ausgabe.setText("(Strings)");
    cp.add(Ausgabe);
    // Ende Komponenten
    setResizable(false);
    setVisible(true);
  }

  // Anfang Methoden
  public void Button_ActionPerformed(ActionEvent evt) {
    String Ort = "";
    String Kennung = Eingabe.getText();

    int a = 1;

    for (int i = 0; i < a; i++) {
      if (Character.isLetter(Kennung.charAt(i))) {
        char c = Kennung.charAt(i);
        Ort = Ort + c;
        a=a+1;
      }
    }

    if ((Ort.length() != 0) && (Ort.length() < 4)) {
      Ausgabe.setText("Ort: " + Ort);
    }
    else {
      Ausgabe.setText("Kein gültiges Kennzeichen");
    }
  }

  // Ende Methoden
  public static void main(String[] args) {
    new KennzeichenKontrolle("Kennzeichen-Kontrolle");
  }
}

Guten Morgen alle

 

[eRaaaa]

Naja, liegt Allgemein an der Schleife bzw. konkret an [c]Character.isLetter(Kennung.charAt(i))[/c]
Wenn du nur Buchstaben, also z.B. HH eingibst, würdest du jetzt charAt(0), charAt(1) UND dann weiter auch charAt(2) prüfen und dann kracht es, weil es ja nur eben zwei Characters gibt

 

[Sonecc]

statt a mit 1 zu initialisiern würde ich Kennung.length() nehmen.
also

a = Kennung.length()
Dann muss auch a nicht erhöht werden, sondern stellt direkt das Ende der Schleife

 

[Anfänger123]

Danke, mein Ziel ist es ein einfaches Programm zu schreiben dass ein Kennzeichen auf Richtigkeit überprüft in der Form. M-PAX12 zum Beispiel, also das bei '-' Schluss ist deswegen das a =a+1. Gibt es da noch eine andere Möglichkeit außer mit Pattern?

 

[Anfänger123]

OK, danke alle! Ich habs jetzt. Ich zähle nur 1 dazu wenn noch ein Ziechen kommt.

    String Ort = "";
    String Kennung = Eingabe.getText();

    int a = 1;

    for (int i = 0; i < a; i++) {
      if (i < Kennung.length()) {
      char c = Kennung.charAt(i);
        if (Character.isLetter(c)) {

          Ort = Ort + c;
          a = a + 1;
        }
      }
    }

sieht jetzt so aus

 

[bygones]

und warum ueberhaupt a und nicht [c]Kennung.length()[/c] ?... wie sonecc schon sagte.
erspart dir das erste if

und generell: Variablen IMMER mit Kleinbuchstaben beginnen lassen