メソッドが期待値を返さない

Question

以下のコードで何が間違っているかわかりません。私のリターンは機能していません。 私はコードでいくつかの詳細をコメントしました。

itemは、テキストファイルの行です。 「Port; 2302;」

portTextBox.Text = Config_check(item, "Port", 1); 
    private string Config_check(string item, string contains, int i) 
    { 
     string part = "defualt"; 
     if (item.Contains(contains)) 
     { 
      MatchCollection Parts = Regex.Matches(item, @"(?i); *(.+?);(?-i)", RegexOptions.Singleline); 
      foreach (Match m in Parts) 
      { 
       part = m.Groups[i].Value; // sets part to '2302' 
       MessageBox.Show(part); // Shows '2302', Does open message box 
       return part; // Dosnt work 
      } 
      MessageBox.Show(part); // Shows 'Defualt' why?!?! 
      return part; // Dosnt work 
     } 
     MessageBox.Show(part); // Shows 'Defualt' why?!?! 
     return part; // returns 'Defualt' 
    } 

Answer 1

ている例では、おそらくあなたの問題は、ループ内でreturn文です。あなたがあなたの方法から戻るならば、残りの方法はもう実行されません。だから、おそらくあなたがしなければならないすべては、内側リターンを削除することです。しかし

portTextBox.Text = Config_check(item, "Port", 1); 
private string Config_check(string item, string contains, int i) 
{ 
    string part = "defualt"; 
    if (item.Contains(contains)) 
    { 
     MatchCollection Parts = Regex.Matches(item, @"(?i); *(.+?);(?-i)", RegexOptions.Singleline); 
     foreach (Match m in Parts) 
     { 
      part = m.Groups[i].Value; 
      MessageBox.Show(part); 
     } 
     MessageBox.Show(part); 
    } 
    MessageBox.Show(part); 
    return part; 
} 

、私はあなたのコードの残りの部分を知っていないとして、それはまだ動作しない場合があります。

partにある唯一の値が、最後に保存された値になるように、foreachループ内にpartを上書きするので、これは実際には非常に良い試みではありません一致するのはPartsです。

---

EDIT

あなたは、foreachループをしたくない場合は、単に最後の項目を取ることができる：

portTextBox.Text = Config_check(item, "Port", 1); 
private string Config_check(string item, string contains, int i) 
{ 
    string part = "defualt"; 
    if (item.Contains(contains)) 
    { 
     MatchCollection Parts = Regex.Matches(item, @"(?i); *(.+?);(?-i)", RegexOptions.Singleline); 
     part = Parts.OfType<Match>().LastOrDefault() ?? "default"; 
     MessageBox.Show(part); 
    } 
    MessageBox.Show(part); 
    return part; 
} 

あなたは（明らかに）内部のメッセージボックスを削除することができますforeachループ。