3

私はこれを少し片付けることを考えていますが、私はC#に少し慣れていません

ResponseList responsesList = new ResponseList();
PagedResponseList pagedResponsesList = new PagedResponseList();
responsesList = responseService.ListSurveyResponses(1000);
pagedResponsesList = responsesList.ResultData;

これはおそらく簡単なものですが、これを1行にするために必要な構文は私を免れます。

4

4 に答える 4

7

newまず、最初の 2 行の 2 つのステートメントは実際には必要ありません。これらのnewインスタンスは、最後の 2 行で割り当てたものによって上書きされるからです。

C# を初めて使用する場合は、少なくとも2 つの行に固執することをお勧めします。これにより、少なくとも段階的に何が起こっているかを理解できます。特に、最後の 2 行:

ResponseList responsesList = responseService.ListSurveyResponses(1000);
PagedResponseList pagedResponsesList = responsesList.ResultData;

responsesListコードで後で使用しない場合は、これらの 2 行を次の行に要約できます。理解しやすい思います (基本的には、.ResultDataプロパティを の後ろにチェーンするだけですresponseService.ListSurveyResponses())。

PagedResponseList pagedResponsesList = responseService.ListSurveyResponses(1000).ResultData;
于 2012-05-17T17:20:55.277 に答える
5
var pagedResponsesList = responseService.ListSurveyResponses(1000).ResultData;
于 2012-05-17T17:20:28.380 に答える
1

他の人が示唆しているように、次のようにすべてを 1 行にまとめることができます。

var pagedResponsesList = responseService.ListSurveyResponses(1000).ResultData;

しかし、別のサービスを呼び出していることを考えると、例外処理を実行したり、結果を参照する前に null をチェックしたりする必要があると思います.ResultData

于 2012-05-17T17:24:15.070 に答える
0
PagedResponseList pagedResponsesList=responseService.ListSurveyResponses(1000).ResultData
于 2012-05-17T17:21:48.170 に答える