1

誰かが私のリポジトリコードに関する入力をしてくれて、コードを改善する方法について何か提案をしてくれるかどうか疑問に思っていました。以下に、リポジトリクラスとコントローラを含めます。また、リストの一番上に値を挿入したい場合、それを行うための良い方法は何ですか？

    public static List<ChartApp> ListApplications()
    {
        using (var db = new LatencyDBContext())
        {
            var appNames = db.LoginApplications.Select(item => new ChartApp()
              {
                  LoginApplicationID = item.LoginApplicationID,
                  LoginAppName = item.LoginAppName,
              });
            return appNames.ToList();
        }
    }

そして、コントローラー：

    var listApps = LoginApplicationRepository.ListApplications().OrderBy(item => item.LoginAppName);

        var myCharts = new ChartsViewModel();
        myCharts.AppsForChart = listApps.ToList();
4

1 に答える 1

1

まず、コードの構造は私には問題ないように見えます。特に、リストが通常LoginAppNameで順序付けられる必要がある場合は、リポジトリでOrderByを実行できます。コントローラのlistAppsでToList（）を呼び出す必要はありません。これはリポジトリですでに行われているため、listAppsはすでにリストになっています。

リストの先頭にアイテムを挿入するには、Insertメソッドを使用します。例：

listApps.Insert(0, newAppItem);
于 2013-02-15T00:12:21.127 に答える