、単純な文字列配列反復は、これは、セッション変数を持つ文字列配列の各要素を比較した後、trueまたはfalseを返す必要があり、単純なコードである

Question

を動作しません。これは、 'plans'という文字列配列が正しい属性を取得してもforeach内で最初の要素のみを繰り返し処理するため、配列変数の最初のものとは異なる他の要素と一致する場合はtrueを返しません...あなたは問題がのforeach cicleに権利があると言うことができるが、私はそれを参照してくださいカント...私は百倍のようにこれをやったと私は私が間違っているの何...ありがとう理解することはできません

私は

EDIT ...あなたはのstring.Split経由で値を追加する前に、文字列配列「計画」の長さを設定する明示的にする必要があります信じて

protected bool ValidatePlans() 
{ 
    bool authorized = false; 

    if (RequiredPlans.Length > 0) 
    { 
     string[] plans = RequiredPlans.Split(','); 
     foreach (string plan in plans) 
     { 
      if (MySessionInfo.Plan == plan) 
       authorized = true; 
     } 
    } 
    return authorized; 
} 

Answer 1

私は私はあなたの問題を理解100％わからないんだけど、あなたのような文字列を使用している場合、私は疑問に思います：

"Blue, Green" 

...とMySessionInfo.Planは"Green"（スペースなし）であるときtrueを戻すには、このメソッドを期待。 これが問題の場合は、Split結果に余分なスペースがあるという単純な問題です。

if (MySessionInfo.Plan == plan) 

...これに：これは、これを変更することにより、固定することができ

if (MySessionInfo.Plan == plan.Trim()) 

それはそれではない場合は、多分私はあなたの問題を誤解しました。

編集：your comment to HABJAN's answerから、私は聞いている：コードあなたが掲載されて正確にあなたがを使用している実際のコードと同じ？もし私が頭を傷つけなければならないのは、彼が提案した修正がどのように提案されたのか（その方法がより効率的になるので、良い方法だが）はあなたの問題を解決できたはずである。

Answer 2

：いいえ、私は間違っていた...あなたの方法配列を取り込むのはうまくいくはずです。あなたは自分自身をArray.IndexOf()を使用しての代わりに、すべての作業を行うことを検討することをお勧めします私の答え

Answer 3

を無視：

protected bool ValidatePlans() 
{ 
    return (RequiredPlans.Length > 0 
     && Array.IndexOf(RequiredPlans.Split(','), MySessionInfo.Plan) >= 0); 
} 

Answer 4

protected bool ValidatePlans() 
{ 
    if (RequiredPlans.Length > 0) 
    { 
     string[] plans = RequiredPlans.Split(','); 
     foreach (string plan in plans) 
     { 
      if (MySessionInfo.Plan == plan) 
       return true; 
     } 
    } 

    return false; 
} 

Answer 5

いくつかの可能性：MySessionInfo.Planを想定し

A.が強く型付けされていません文字列として試してみてください。

if（MySessionInfo.Plan.Equals（plan）） { authorized = true; 休憩。 }

B.末尾にスペースがないことを確認し、比較の際に、.splitを行った後。あなたはMySessionInfo.Plan.Equals（plan.Trim（））

C.手段「それは最初の要素だけ、の繰り返し処理を続けてforeachの」わからないを行うことができます。ループが配列内の他の要素を通過しないことは確かですか？

Answer 6

あなたのコードは、私には罰金に見えますが、あなたが全体を置き換えることができます：

return RequiredPlans.Length > 0 && RequiredPlans.Split(',').Any(s => MySessionInfo.Plan == s); 

Answer 7

これはあなたの入力は次のように探しているものであればRequiredPlansが、私は問題はあなたがを反復処理しているということはないと思う、 =「青、緑」のような文字列

あり、確かに最初の値は複数回になりますが、値をトリミングしていないことを確認してください。したがって、最初の後のすべてに先頭のスペースがあり、一致が失敗します。

はすなわち、それはブルー、ブルー、ブルーを返していないことを確認してください、RequiredPlansが真にユニークな文字列のアイテムを持っていることを確認してください

if (MySessionInfo.Plan == plan.Trim()) 

Answer 8

を試してみてください。

他にも言及したように、.Trim（）には大文字と小文字の区別がないことを確認するために.ToLower（）をスローしないでください。

その中にあるものがすべて明確に機能している必要があります。それが個々のテストとして機能することを確認するために、一時的なRequiredPlans文字列を作成しない場合。

protected bool ValidatePlans() 
{ 
    if (RequiredPlans.Length > 0) 
    { 
     string[] plans = RequiredPlans.Split(','); 
     foreach (string plan in plans) 
     { 
      if (MySessionInfo.Plan.Trim().ToLower() == plan.Trim().ToLower()) 
       return true; 
     } 
    } 

    return false; 
}