C＃の奇妙な振る舞い、またはそれは私がした間違いですか？

Question

「0」を「ページ」パラメータに渡されたときに、このアクションはZERO結果を返し、なぜ誰もが説明できます：私は一時変数がtryparseするために再使用していますので

[HttpPost] 
public ActionResult SearchProperties(string id, string offerTypeID, string propertyTypeID, string page) 
{ 
    int temp = 0; 

    var props = from s in db.Properties 
        where s.Approved && s.Available 
        select s; 

    if (!String.IsNullOrEmpty(id)) 
    { 
     Int32.TryParse(id, out temp); 
     props = from s in props 
        where s.PropertyType.PropertyTypeCategoryID == temp 
        select s; 

    } 

    if (!String.IsNullOrEmpty(offerTypeID)) 
    { 
     Int32.TryParse(offerTypeID, out temp); 
     props = from s in props 
        where s.OfferTypeID == temp 
        select s; 
    } 

    if (!String.IsNullOrEmpty(propertyTypeID)) 
    { 
     Int32.TryParse(propertyTypeID, out temp); 
     props = from s in props 
        where s.PropertyTypeID == temp 
        select s; 
    } 

    props = props.OrderBy(s => s.PropertyID); 

    int i = 0, skip = 0; 
    if (!String.IsNullOrEmpty(page)) 
    { 
      Int32.TryParse(page, out temp); 
      skip = temp * 10; 
    } 
    else 
    { 
     skip = 0; 
    } 

    props = props.Skip(skip).Take(10); 


    var marks = (from s in props.ToList() 
         select s); 
    return Json(new { markers = marks }); 
} 

はそれですか？
このコードは何の例外も注意を払わないので、ゼロレコードを返します。

Answer 1

あなたのコードはもっと複雑である必要があります。それを単純化すると正しい結果が得られ、少なくともデバッグが容易になるはずです。フレームワークでintに変換させてください。パラメーターが必要ない場合は、それらをヌル可能にします。

[HttpPost] 
public ActionResult SearchProperties(int? id, int? offerTypeID, int? propertyTypeID, int? page) 
{ 

    var props = from s in db.Properties 
        where s.Approved && s.Available 
        select s; 

    if (id.HasValue) 
    { 
     props = from s in props 
        where s.PropertyType.PropertyTypeCategoryID == id.Value 
        select s; 
    } 

    if (offerTypeID.HasValue) 
    { 
     props = from s in props 
        where s.OfferTypeID == offerTypeID.Value 
        select s; 
    } 

    if (propertyTypeID.HasValue) 
    { 
     props = from s in props 
        where s.PropertyTypeID == propertyTypeID.Value 
        select s; 
    } 

    props = props.OrderBy(s => s.PropertyID); 

    // use null coalescing operator to default to 0 
    page = page ?? 0; 

    int skip = page * 10; 

    props = props.Skip(skip).Take(10); 


    var marks = (from s in props.ToList() 
         select s); 
    return Json(new { markers = marks }); 
} 

Answer 2

実際にはTryParseを正しく利用せずに、TryParseで一時変数を再利用することになります。パラメータのいずれかが空でない場合は、TryParseが成功したかどうかをさらに制限するために何も問題はありません。

あなたがしなければならないことは、いつでも条件文にTryParseを呼び出すことです。

if(Int32.TryParse(propertyTypeID, out temp)) 
{ 
    props = ... 
} 

することもでき、短絡すぎ初期条件への評価、あなたが必要とするためBOTHそれは非nullまたは空の構文解析に成功することを：だから、それが失敗した場合

if(!String.IsNullOrEmpty(id) && Int32.TryParse(id, out temp)) 
{ 
    props = ... 
} 

文字列を整数に解析する際に、条件文を気にする必要はありません。