C#「値を返さないコード パスがあります。」のエラー

Question

###前提・実現したいこと

C#にて、対象文字列とワイルドカードを引数として渡したら
対象文字列がワイルドカードの条件に当てはまるかどうかをTrue or Falseを返すようなメソッドを
作りたいと考えています。

なお、ワイルドカードは「ABC*,DEF??,*GH」といったように複数の条件をカンマ区切りで
一度に指定できるようにしたいです。

例えば、ObjText="TestText"である場合
Wildcard="ABC,Te*"ならTrueを返し、Wildcard="Te*,ABC,"でもTrueを返すが
Wildcard="ABC,DE*"ならFalseを返すといった形です。

###発生している問題・エラーメッセージ

CS0161	'Form1.test(string, string)': 値を返さないコード パスがあります。

下記のように記述したところ、上記のようなエラーが出てコンパイルできない状況です。

###該当のソースコード

    public bool test(string ObjText, string Wildcard)
    {

        string[] stArrayData = Wildcard.Split(',');

            foreach (string stData in stArrayData)
            {
               //ワイルドカードの文字列を正規表現の文字列に変換する
                var regexPattern = Regex.Replace(stData, ".",
                  m =>
                  {
                      string s = m.Value;
                      if (s.Equals("?"))
                      {
                          //?は任意の1文字を示す正規表現(.)に変換
                          return ".";
                      }
                      else if (s.Equals("*"))
                      {
                          //*は0文字以上の任意の文字列を示す正規表現(.*)に変換
                          return ".*";
                      }
                      else
                      {
                          //上記以外はエスケープする
                          return Regex.Escape(s);
                      }
                  }
                  );

                bool rg = new Regex(regexPattern).IsMatch(ObjText);
                if (rg)
                {
                    return true;
                }
                else
                {
                    return false;
                }
            } 
    }

###試したこと
例外が起きた場合に if(rg)の中のreturn分を通らない場合があるのかもしれないと考え、
Try-Catchをいろいろな場所に記述してCatch内にreturnを入れてみたりしたのですが、エラーに変化はありませんでした。ワイルドカードを正規表現に変換するところの処理は正直あまり理解できておらず、見当違いなことをしているかもしれず恐縮なのですが、お知恵をお貸しいただければ幸いです。

###補足情報(言語/FW/ツール等のバージョンなど)
C#を使ったワイルドカードの使用については、下記を参考にしました。
http://qiita.com/kazuhirox/items/5e314d5e7732041a3fe7

Answer 1

public int Test(int[] xs){
    foreach(var x in xs){
        return 1;
    }
}

これは一見、すべてのパスで値を返しそうですが、
xsが空の場合はforeach内に入らないのでパス無しになります。

Comments

[Lark7]

なるほど。。
当方のコードの問題はforeach内に入らないという抜け道があったのですね。
どんな場合でも一度はforeach内に入る前提でいろいろとトライしてしまっていました。
takasima20さんのご回答と合わせて、foreach外にreturn falseを記述する意味を改めて納得いたしました。
ありがとうございました。

Answer 2

エラーとは関係ないですが、Wildcardの最初のやつしか判定しないスね。
自分だったら、ループ内では true のみ return し
ループの外(全部ハズレだった場合) false を return する
って形にするかなあ。

Comments

[Lark7]

おっしゃる通りですね。。。
ご指摘通り修正したらうまくいきました。
煮詰まっていたので大変助かりました！ありがとうございました。

Answer 3

メソッドの分割に慣れていないようですね。
ワイルドカードを正規表現クラスに返還するメソッドを加えてみました。

public bool test(string text, string wildcards){
    foreach(string wildcard in wildcards.Split(',')){
        var reg = ConvertWildcardToReges(wildcard);
        if(reg.IsMatch(ObjText)){
            return True;
        } else {
            return False;
        }
    }
}

private Regrex ConvertWildcardToReges(string wildcard){
    //ワイルドカードの文字列を正規表現の文字列に変換する 
    var regexPattern = Regex.Replace(wildcard, ".", 
        m => { 
            string s = m.Value; 
            if (s.Equals("?")) { 
               //?は任意の1文字を示す正規表現(.)に変換 
               return "."; 
            } else if (s.Equals("*")) { 
               //*は0文字以上の任意の文字列を示す正規表現(.*)に変換 
               return ".*"; 
            } else { 
               //上記以外はエスケープする 
               return Regex.Escape(s); 
            } 
       });

       retrun new Regex(regexPattern).IsMatch(ObjText); 
}

ちなみにこのコードはエラーが残ったままです。（バグを新しく作りこんでしまったかもしれません。）

testメソッドがずいぶんスッキリしていることに同意してもらえるはずです。
このコードを見て、エラーの原因に気づいてもらえたらよいのですが・・・

さらにヒントを続けます。

var ret = test("TestTest", "");

このretの値はTrueでしょうかFalseでしょうか。
おそらく、変更前のコードではわかりづらかったかもしれませんが、このコードで考えるとすぐわかると思います。

短いメソッドのほうがエラーは見つけやすいので、上手にメソッドの分割をしたほうが良いと思います。

Comments

[Lark7]

コメントありがとうございます。
＞メソッドの分割に慣れていないようですね。
おっしゃる通り、我流でVBAを触ってきて、C#はここ最近始めたばかりでして、なんとかコンパイル通るようにするのが精いっぱいという状況でお恥ずかしい限りです。
（メソッドの分割流儀などはVBAでもC#でもそれほど違いがある話もないと思うので言い訳にもなりませんが。。。）
前にコメントいただいたozwkさんのご指摘からすると
wildcardsがnullの場合のパスなしで、ご提案いただいたコードでもtestメソッドはコンパイル通らないかなぁと考えました。
下記はまさにwildcardsがnullの例、ということですかね？
var ret = test("TestTest", "");

[iwamoto_takaaki]

nullではないです。

長さ０の文字です。Nullと0の差は説明しずらいですが、test("TestTest", null)は呼び出し側でエラーになります。

wildcardをSplitしたときに長さ０の配列が返ってくるので、foreachの中の処理は実行されず。値が帰らないまま処理が抜けます。

foreach構文のあとにreturn False;を入れるというのが答えになります。

ループや条件式が複雑になるとこの手モレが発生しやすくなります。一目でわかる大きさにメソッドを分割するように心がけてください。