Aufgabe zu lokalen Klassen

[Tenori]

Guten Tag,
ich soll als Übungsaufgabe eine Klasse "Email" implementieren, die mithilfe einer lokalen Klasse überprüft, ob die angegebenen Eingaben richtig sind(den Konventionen für Emails entsprechen).
Meine Datei Email.java:

import java.util.regex.*;

public class Email
{
	private String betreff;
	private String text;
	private String empfaenger;
	private String absender;
	
	public Email(String empfaenger, String absender, String betreff, String text)
	{
		this.empfaenger = empfaenger;
		this.absender = absender;
		this.betreff = betreff;
		this.text = text;
	}
	
	public void senden()
	{
		class InternetMailAddress
		{
			private boolean isValid(String address)
			{
				// Prüfung der einzelnen Kriterien: - genau ein @ 
				//									- nur a-z,A-Z,0-9,_,.
				//									- mindestens 1 Zeichen für Empfängername bzw. Server
				//									- beginnen mit einem Zeichen, das nicht @ oder * ist
				//									- @servername darf wegfallen, wenn das letze Zeichen ein * ist, dann wird automatisch @it-designers.de angehängt
				boolean a = Pattern.matches(address,"$\\*"); // Wenn letztes Zeichen ein *   
				if(a)                                                        // ist, dann ersetzen
				{
					StringBuffer sb = new StringBuffer(address);
					sb.deleteCharAt(sb.length());
					sb.append("@it-designers.de");
					address= sb.toString();
				}
				boolean b = Pattern.matches(address,"[a-zA-Z0-9\\._]+@[a-zA-Z0-9\\._]");
				if(b)
				{
					return true;
				}
				return false;
			}
		}
		InternetMailAddress refTest = new InternetMailAddress();
		if(refTest.isValid(empfaenger) && refTest.isValid(absender))
		{
			System.out.println("Absender: " + absender);
			System.out.println("Empfaenger: " + empfaenger +"\nBetreff: " + betreff +"\nText: " + text);
		}
	}
}

Und die Datei TestEmail.java

public class TestEmail
{
   public static void main (String [] args)
   {
      Email e1 = new Email ("Klaus.Gross*", "Lotte.Klein@gmz.de", "Hallo", "Hallo Welt");
      e1.senden();

      Email e2 = new Email ("Klaus.Gross*", "Lotte.Kleingmz.de", "Hallo", "Hallo Welt");
      e2.senden();
   }
}

Wo ist der Fehler, also ich bin mir net sicher, wie das mit den Regulären Ausdrücken funktioniert, das sollte man sich mithilfe der API anlesen, das wurde im Buch nicht erläutert, darum ist das hier so der erste Versuch, wie ich das verstanden hatte.
Aber eigentlich müsste doch bei der Methode "e1.senden()" eine Ausgabe erfolgen, oder?

Grüße

 

[BrauniDeluxe]

Ausgabe erfolgt nur wenn die isValid() true returned .. tut es aber nicht

Hier mal nen Schnellschuss mit google (ohne das ersetzen von Email adressen .. musst beim aufruf dann richtige email adressen nehmen)

import java.util.regex.Matcher;
import java.util.regex.Pattern;

public class Email {
	private String betreff;
	private String text;
	private String empfaenger;
	private String absender;

	public Email(String empfaenger, String absender, String betreff, String text) {
		this.empfaenger = empfaenger;
		this.absender = absender;
		this.betreff = betreff;
		this.text = text;
	}

	public boolean isValid(String address) {
		String expression = "^[\\w\\-]([\\.\\w])+[\\w]+@([\\w\\-]+\\.)+[A-Z]{2,4}$";
		CharSequence inputStr = address;
		Pattern pattern = Pattern.compile(expression, Pattern.CASE_INSENSITIVE);
		Matcher matcher = pattern.matcher(inputStr);
		return matcher.matches();
	}

	public void senden() {
		if (isValid(empfaenger) && isValid(absender)) {
			System.out.println("Absender: " + absender);
			System.out.println("Empfaenger: " + empfaenger + "\nBetreff: "
					+ betreff + "\nText: " + text);
		} else {
			System.out.println("Email not valid");
		}
	}
}

Gruss
Dennis

 

[Tenori]

Danke erstmal,
aber wieso gibt denn meine Methode isValid() kein true zurück?
Wenn die Email in der richtigen Form eingegeben wird, sollte sie doch true zurückgeben, und das wird sie doch bei der Test-Datei bei dem Objekt e1.
Wo liegt bei mir der genaue Fehler? Ist nur ne Übungsaufgabe, darum würde ich das gerne wissen.
Ich glaube irgendwie das es bei mir in der Zeile "boolean a = Pattern.matches(address,"$\\*");" ist,
ich habe nur gefunden, dass $ für das letzte Zeichen steht, aber die Anwendung war mir nicht genau klar, weil ich irgendwie überprüfen muss , ob das letzte Zeichen ein "*" ist.
Kann natürlich auch sein, dass der Fehler woanders liegt, war jetzt nur ne Vermutung meinerseits.

Grüße

Edit: Ok, die Vermutung scheint falsch zu sein, auch wennich mit @aol.com oda so ausprobiere, und nicht mit * am Ende, die Überprüfung also egal ist, kriege ich keine Ausgabe, also wo liegt mein Fehler?

 

[BrauniDeluxe]

Naja ne .. die Absenderadresse bei e1 ist auch falsch .. das mit dem vervollständigen klappt ja nicht.

Kannst das letzte Zeichen auch überprüfen mit : myString.endsWith("*);

Bin da auch kein Experte und müsste da erst nach schauen, gibts aber bestimmt im JAVA-InselBuch wie du genau die Abfrage machst .. die auf Stern asl letztes Zeichen funktioniert so anscheinend nicht.
Am besten mal bisschen rum probieren, da lernt man am meisten.

Gruss
Dennis

 

[Ein Keks]

Pattern (Java Platform SE 6))
guck dir mal den matches befehl genau an was fällt dir auf? Genau erst regex dann input und jetzt guck ma bei dir : Pattern.matches(address,"$\\*") ...
war mal so nett und habs verbessert:

import java.util.regex.*;

public class Email {

	private String betreff;
	private String text;
	private String empfaenger;
	private String absender;

	public static void main(String[] args) {
		Email e1 = new Email("Klaus.Gross*", "Lotte.Klein@gmz.de", "Hallo",
				"Hallo Welt");
		System.out.println("Email 1:");
		e1.senden();

		Email e2 = new Email("Klaus.Gross*", "Lotte.Kleingmz.de", "Hallo",
				"Hallo Welt");
		System.out.println("Email 2:");
		e2.senden();
	}

	public Email(String empfaenger, String absender, String betreff, String text) {
		this.empfaenger = empfaenger;
		this.absender = absender;
		this.betreff = betreff;
		this.text = text;
	}

	private boolean isValid(String address) {
		if (Pattern.matches(".+\\*", address)){  //hättest du in die API gesehen wär dir das auch aufgefallen: erst regex dann inhalt!!!!
			StringBuffer sb = new StringBuffer(address);
			sb.deleteCharAt(sb.length()-1); //deleteCharAt fängt bei 0 an zu zählen...stichwort array
			sb.append("@it-designers.de");
			address = sb.toString();
		}
		return Pattern.matches("[a-zA-Z\\._\\-]+@[a-zA-Z\\._\\-]+", address); //also das war echt unnötig...
	}

	public void senden() {
		if (isValid(empfaenger) && isValid(absender)) {
			System.out.println("Absender: " + absender);
			System.out.println("Empfaenger: " + empfaenger + "\nBetreff: "
					+ betreff + "\nText: " + text);
		} else {
			System.out.println("Ein Fehler ist aufgetreten! Bitte überprüfen Sie ihre Eingabe!"); //der vollständdigkeit wegen
		}
	}
}

aber regex ist auch was fieses...

edit: ich sollte wohl mehr auf die überschrift achten --> dann eben doch mit innerer klasse ändert ja nicht viel

 

[Tenori]

Super,
danke dir. Darauf, dass zuerst die regex kommt und dann input, darauf hatte ich garnicht geachtet..
Ähm, die Aufgabe ist aus dem Kapitel über geschachtelte Klassen, und man sollte es mit ner inneren Klasse lösen, darum die innere Klasse.

Grüße