Prüfen, ob ein String 2x das selbe Zeichen hat

[NicoRyl]

Hallo zusammen.

Im Rahmen meines E-Mail Validators möcte ich prüfen, ob die E-Mail 2x das @-Zeichen enthalten hat.
Weiß jemand, wie ich das machen könnte?

Ich hatte an etwas gedacht wie " if (email.contains("@") >= 2) {return "Email nicht gestattet."} "

Vielen Dank schonmal im Voraus!

 

[Flown]

contains sagt nur aus, ob ein Zeichen enthalten ist. Was du möchtest ist zählen.
Du ziehst eine Schleife über deinen String und sieht char für char nach, ob es ein '@' ist.
Alternativ: regex.

 

[Gelöschtes Mitglied 65838]

falls es einmal enthält -> substring machen mit dem rest
rest durchgehen -> falls es nochmal eins hat fehler

 

[Flown]

falls es einmal enthält -> substring machen mit dem rest

Nein man kann das mit indexOf machen (da kann man einen Startpunkt festlegen). Substring legt einen neuen String an.

 

[NicoRyl]

falls es einmal enthält -> substring machen mit dem rest
rest durchgehen -> falls es nochmal eins hat fehler

Vielen Dank, hat funktioniert.

if (email.substring(findAt + 1).contains("@")) {
check = false;
}

Ich hab das erste @ was er findet in "findAt" gespeichert und dann ab dort nach einem weiteren gesucht. Funktioniert einwandfrei!

 

[Flown]

Wenn du das schon so machst:

int firstIndex = email.indexOf('@');
if (firstIndex == -1  || email.indexOf('@', firstIndex + 1) != -1) {
  // Keine Email
}

 

[MoxxiManagarm]

Offtopic: Ich hatte zufällig gerade eine Anforderung wo ich bei JavaScript den Text zwischen 2 #-Zeichen brauchte. Die Lösung sah dem hier sehr ähnlich, indexOf mit startIndex ftw.

 

[Wurstkopp]

Oder via lastIndexOf:

email.indexOf('@') == email.lastIndexOf('@')

 

[Flown]

Oder via lastIndexOf:

email.indexOf('@') == email.lastIndexOf('@')

Der Fall das es gar nicht vorkommt wird hier nicht abgedeckt.

 

[Wurstkopp]

Der Fall das es gar nicht vorkommt wird hier nicht abgedeckt.

1. War das nicht die Fragestellung: " Ich hatte an etwas gedacht wie if (email.contains("@") >= 2) {return "Email nicht gestattet."} "
2. Ging es mir um die Verwendung von lastIndexOf. Die Abfrage entsprechend umzustellen ist jetzt ja nicht die große Kunst:

        int firstAppearance = email.indexOf('@');
        if (firstAppearance == -1 || firstAppearance != email.lastIndexOf('@')) {
            return "Es muss exakt ein @ Zeichen vorkommen";
        }

 

[mrBrown]

BTW: Mail-Adressen dürften mehr als ein '@' enthalten, allein die Anzahl ist kein Zeichen für invalide Mail-Adressen.