Prob mit java.util.regex.matches-String wird nicht erkannt?

[KarlKralle]

Hallo!
Nach stundenlangem rumprobieren komme ich einfach nicht weiter.
und zwar bin ich dabei für eine Praktikumsaufgabe für die FH ein Programm zu schreiben:
Man soll einen 12stelligen String eingeben.
Ist der String nicht zwölf stellen lang soll das Programm abbrechen.
An erster Position soll im String ein P ein V ein K oder ein M stehen-wenn nicht erfolgt wieder ein abbruch
Die restlichen stellen sollen nur mit zahlen gefüllt sein -wenn nicht ... Abbruch.
Dann erfolgt eine Ausgabe die den String "entschlüsselt"

So weit so gut...
Ich habe es soweit hingekriegt, dass wenn man was anderes als ein P,V,K oder M eingibt das programm abbicht und ausgibt "keine korrekte Ersatzteilbezeichnung"
Ebenso klappt es wenn der String keine 12 zeichen lang ist.
Aber wenn ich vom String die zahlen in einen nueen kopiere und prüfe ob der neue zahlenstring was anderes als Zahlen enthält und das zutrifft will er das Programm einfach nicht abbrechen... sondern gibt das was ich nicht will aus "Ersatzteilbezeichnung korrekt."
Ich hab schon probiert das nur der zahlenstring abgefragt wird oder das er schon abbrechen soll sobald im zahlenstring ein p auftaucht aber das macht er auch nicht.
Irgendwas stimmt da wohl mit dem string nicht aber ich weis nicht was!?
hab auch schon den zahlenstring durch System.out.println ausgeben lassen um zu gucken ob da wirklich nur zahlen drinstehen was auch stimmt.

Das ist eines meiner ersten JAVA programme daher tippe ich auf irgendeinen "dummen" anfängerfehler.
Vielleicht liegt es auch an den if-Anweisungen?
Hoffe ich hab mich verständlich ausgedrückt und ihr könnt mir weiterhelfen.
Vielen Dank schonmal im vorraus...

import java.util.regex.*;
import javax.swing.*;


class lagerverwaltung

{
  static boolean checkString(String s)    //Ersatzteilbezeichnung korrekt?  Part Number correct?
  {
    String zahlen = s.substring(1);       // string wird aufgeteilt. string partition
    String gtmp = s.substring(0, 1);
    String geraetestring = gtmp.toUpperCase(); // geraetestring in Uppercase
    if (s.length() != 12)                 // if stringlength != 12 return false . incorrect String
    {
      return false;
    }
    if ((zahlen.matches("[^0-9]")) || (geraetestring.matches("[^PVMK]"))) // if zahlen contains no! numbers OR geraetestring contains others than PVM or K return false
    {
      return false;
    }
    return true;
  }

  static void findTool(String s) //output
  {
    String gtmp = s.substring(0, 1);
    String geraetestring = gtmp.toUpperCase();
    String ersatzteil = s.substring(1, 4);
    String regalreihe = s.substring(4, 6);
    String regalnr = s.substring(6, 8);
    String regalebene = s.substring(8, 10);
    String regalfach = s.substring(10, 12);
    if (geraetestring.matches("[P]"))
    {
      System.out.println("Pumpe");
    }
    if (geraetestring.matches("[V]"))
    {
      System.out.println("Ventil");
    }
    if (geraetestring.matches("[M]"))
    {
      System.out.println("Messeinrichtung");
    }
    if (geraetestring.matches("[K]"))
    {
      System.out.println("Kleinmaterial");
    }
    System.out.println("Ersatzteilnummer:"+ersatzteil);
    System.out.println("Regalreihe:"+regalreihe);
    System.out.println("Regal-Nr:"+regalnr);
    System.out.println("Regalebene:"+regalebene);
    System.out.println("Regalfach:"+regalfach);
  }



  public static void main(String[] args)
  {
    String s = JOptionPane.showInputDialog("Bitte korrekte Ersatzteilbezeichnung eingeben:" );
    if (checkString(s)==false)
    {
      System.out.println("Keine korrekte Ersatzteilbezeichnung!");
      System.exit(0);                         // Beendet das Programm
    }
//    if (checkString(s)==true)
//    {
      System.out.println("Ersatzteilbezeichnung korrekt!");
      findTool(s);
//    }
    System.exit( 0 );                           // Beendet das Programm
  }
}

 

[Wildcard]

hmm

if(!string.matches([PVKM]\\d{11}))
  System.out.println("passt nicht");

Hätte ich gesagt.

 

[André Uhres]

Oder so:

       if ((zahlen.matches("[0-9]*")) && (geraetestring.matches("[PVMK]"))) {
            return true;
        }
        return false;

Allgemein gilt: Negationen möglichst vermeiden.

 

[Guest]

hmm

if(!string.matches([PVKM]\\d{11}))
  System.out.println("passt nicht");

Hätte ich gesagt.

was genau heisst das ! vor dem string.matches und was die {11} hinter dem \\d , weis jetzt nicht ob ich das so machen kann weil ich den eigentlichen String ja in 2 neue kopiert hab... einmal den geraetestring der nur aus der ersten Stelle des eingegebenen Strings besteht und einmal den zahlen - string der aus den stellen vom eingegebenen String besteht die anch der ersten stelle folgen.


@André Uhres: oke habs mal so umgeändert das er ein true zurückgibt falls der String wie gewünscht eingegeben ist aber e funktioniert trotzdem nicht. Der will einfach nicht erkennen was mit dem zahlen - string ist.
was mich total verwirrt ist, das er bei dem geraetestring mit dem [PVMK] gar keine zicken macht aber der check bei dem zahlen-string nicht funktioniert.

Danke schonmal soweit an euch beide :toll:

 

[Wildcard]

was genau heisst das ! vor dem string.matches und was die {11} hinter dem \\d ,

! negiert den boolschen Ausdruck
{11} bedeutet genau 11 Zeichen von \\d (digit)

weis jetzt nicht ob ich das so machen kann weil ich den eigentlichen String ja in 2 neue kopiert hab... einmal den geraetestring der nur aus der ersten Stelle des eingegebenen Strings besteht und einmal den zahlen - string der aus den stellen vom eingegebenen String besteht die anch der ersten stelle folgen.

Brauchst du doch gar nicht. Wenn du den String zerlegen willst kannst du das hinterher immer noch machen.

 

[Guest]

Hm irgendwie funktionierts immer noch nicht so recht...
Dabei bin ich mir fast sicher das es funktioniert hat...
ich hab nur mal ein else auskommentiert um zu gucken obs dann immernoch klappt und den kommentar dann wieder rausgenommen und jetzt gibt er trotzdem immer ein true zurück grrr

hier der code wie ich ihn jetzt hab

import java.util.regex.*;
import javax.swing.*;


class lagerverwaltung

{
  static boolean checkString(String s)    //Ersatzteilbezeichnung korrekt?  Part Number correct?
  {
    if (s.length() == 12)                 // if stringlength != 12 return false . incorrect String
    {
      return true;
    }
    if (s.matches("[PKVM]\\d{11}"))
    {
      return true;
    }
    else
    {
      return false;
    }
  }

  static void findTool(String s) //output
  {
    String gtmp = s.substring(0, 1);
    String geraetestring = gtmp.toUpperCase();
    String ersatzteil = s.substring(1, 4);
    String regalreihe = s.substring(4, 6);
    String regalnr = s.substring(6, 8);
    String regalebene = s.substring(8, 10);
    String regalfach = s.substring(10, 12);
    if (geraetestring.matches("[P]"))
    {
      System.out.println("Pumpe");
    }
    if (geraetestring.matches("[V]"))
    {
      System.out.println("Ventil");
    }
    if (geraetestring.matches("[M]"))
    {
      System.out.println("Messeinrichtung");
    }
    if (geraetestring.matches("[K]"))
    {
      System.out.println("Kleinmaterial");
    }
    System.out.println("Ersatzteilnummer:"+ersatzteil);
    System.out.println("Regalreihe:"+regalreihe);
    System.out.println("Regal-Nr:"+regalnr);
    System.out.println("Regalebene:"+regalebene);
    System.out.println("Regalfach:"+regalfach);
  }



  public static void main(String[] args)
  {
    String s = JOptionPane.showInputDialog("Bitte korrekte Ersatzteilbezeichnung eingeben:" );
    if (checkString(s)==false)
    {
      System.out.println("Keine korrekte Ersatzteilbezeichnung!");
      System.exit(0);                         // Beendet das Programm
    }
//    if (checkString(s)==true)
//    {
      System.out.println("Ersatzteilbezeichnung korrekt!");
      findTool(s);
//    }
    System.exit( 0 );                           // Beendet das Programm
  }
}

 

[Wildcard]

    if (s.length() == 12)                 // if stringlength != 12 return false . incorrect String
    {
      return true;
    }

Das muss schonmal weg.

 

[Guest]

Ohman dummer Fehler
danke fürs drauf aufmerksam machen!

Und danke für die schnelle Hilfe!
Denke so sollte es jetzt funktionieren.:toll:

Eine Frage noch, dass

if (s.matches("[PKVM]\\d{11}"))

könnte nicht heissen, dass wenn ich einen String eingebe der zwar 11 digits hat aber irgendwo nicht als erste stelle ein P,M,K oder V drin ist dass er dann trotzdem ein true zurückgibt, richtig?

 

[Verjigorm]

könnte nicht heissen, dass wenn ich einen String eingebe der zwar 11 digits hat aber irgendwo nicht als erste stelle ein P,M,K oder V drin ist dass er dann trotzdem ein true zurückgibt, richtig?

kannst ja noch nen

if (s.startsWith("PKVM"))

vorher als Abfrage einfügen

 

[Wildcard]

Das matcht nur wenn der erste Buchstabe PKV oder M ist und danach 11 Ziffern kommen