Question

Kann jemand mir sagen warum funktioniert meine else if nicht? Ich bekomme keine Ausgabe.

Erste Frage Aufrufe: 559 Aktiv: 09.03.2023 um 17:48

0

public static string GetDefaultDatenbank(string input) { string ausgabe = ""; string name = ""; String pattern = @"(id)|(strer)|(bach)|(sm)|"; String path = @"(rel)[.][0-9][.][0-9]{1,4}|(dev)[.][0-9][.][0-9]{1,5}";

        foreach (Match match in Regex.Matches(input, pattern, RegexOptions.IgnoreCase))
        {
            if (match.Success)
            {
                name = match.Value.ToLower();
            }
            else if (match.Success == false)
            {
                return ausgabe = "Kontrollieren Sie den Pfad.";
            }

        }
        return ausgabe;

Teilen Diese Frage melden

gefragt 31.01.2023 um 15:40

user3e539a
Punkte: 10

Kommentar schreiben

1 Antwort

hansenausberlin · Answer 1 · 2023-03-09T17:46:34Z

Du hast drei Pfade:

nichts passt (weder if, noch else if) -> return leerer string
match.success (if) -> du setzt name auf nen value, else if wird übersprungen und wenn der Loop dann durch ist returnst du ... tadaaa: nen leeren string
match.success = false (else if): du returnst ausgabe (das zusätzliche assignment ist hier übrigens überflüssig)

Also, selbst wenn es nen Match gibt, würde das vom nächsten gleich wieder überschrieben werden. Das ist ein semantischer Fehler. Da du aber am Ende name nie zurückgibst ist selbst das egal.

Mal angenommen das du jetzt nicht mehrere Matches brauchst und die irgendwie in ne Liste schreiben willst oder zu einem string zusammenbaust, würde es so funktionieren:

static string GetDefaultDatenbank(string input)
{
    const string pattern = @"(id)|(strer)|(bach)|(sm)|";
    foreach (Match match in Regex.Matches(input, pattern, RegexOptions.IgnoreCase))
    {
        if (match.Success)
        {
            return match.Value.ToLower();
        }
    }
    return  "Kontrollieren Sie den Pfad.";
}