このC＃コードを最適化する方法はありますか？

Question

私は自分のコードを持っており、私はそれを最適化したいと思います。私にとってはすでに最適化されているようです。誰かがどのように私はそれをもう少し最適化することができるか提案することができますか？私はどうなる

if (target == "power") 
{ 
    return new JsonResult { Data = new { RC = new Data.AdminPower(datastoreValue).Refresh() } }; 
} 
if (target == "notes") 
{ 
    return new JsonResult { Data = new { RC = new Data.AdminNotes(datastoreValue).Refresh() } }; 
} 
if (target == "book") 
{ 
    return new JsonResult { Data = new { RC = new Data.AdminBook(datastoreValue).Refresh() } }; 
} 
return null; 

Answer 1

switch文は、これらの状況に最適です：

switch(target) 
{ 
case "power": 
    return new JsonResult { Data = new { RC = new Data.AdminPower(datastoreValue).Refresh() } }; 
case "notes": 
    return new JsonResult { Data = new { RC = new Data.AdminNotes(datastoreValue).Refresh() } }; 
case "book": 
    return new JsonResult { Data = new { RC = new Data.AdminBook(datastoreValue).Refresh() } }; 
default: 
    return null; 
} 

いけない本当に私が何か分からない

Answer 2

唯一のものはif-elseifチェーンにif秒に変更されます。あなたは他に何も実際にはありませんパフォーマンスを向上させるために行うことができます。

Answer 3

あなたのメソッドが"book"の値でより頻繁に呼び出されることがわかっている場合は、最初にそれを配置する必要があります。基本的に周波数の順番で並べ替えます。

Answer 4

...それは毎回返送されるので、カントー休憩を必要とするが、その良い習慣最適化についてのあなたの考えは だ場合、スピードの問題として は、単に」第二と第三の前にいくつかの「他」を入れていますが、少ないコード行を意味している場合、これはあなたを助けることができるようなものになることがあります。

return 
((target == "power") ? new JsonResult { Data = new { RC = new Data.AdminPower(datastoreValue).Refresh() } } : 
((target == "notes") ? new JsonResult { Data = new { RC = new Data.AdminNotes(datastoreValue).Refresh() } } : 
((target == "book") ? new JsonResult { Data = new { RC = new Data.AdminBook(datastoreValue).Refresh() } } : null)))) 

Answer 5

他の人が言及したように、間違った比較の総数を最小限に抑えるために、発生頻度が最も高い順に並べる必要があります。

「力」「ノート」と「ブック」を列挙型またはint型の表現に変更することは可能かもしれません。文字列の比較よりもわずかに速いかもしれません。

あなたができることはたくさんありますが、これは重要な最適化につながります。