Maks atau Default?

Apa cara terbaik untuk mendapatkan nilai Max dari kueri LINQ yang mungkin tidak menghasilkan baris? Jika saya lakukan

Dim x = (From y In context.MyTable _
         Where y.MyField = value _
         Select y.MyCounter).Max

Saya mendapatkan kesalahan saat kueri tidak mengembalikan baris. Saya bisa melakukannya

Dim x = (From y In context.MyTable _
         Where y.MyField = value _
         Select y.MyCounter _
         Order By MyCounter Descending).FirstOrDefault

tapi itu terasa agak tumpul untuk permintaan sesederhana itu. Apakah saya kehilangan cara yang lebih baik untuk melakukannya?

UPDATE: Inilah kisah belakangnya: Saya mencoba mengambil konter kelayakan berikutnya dari tabel anak (sistem warisan, jangan mulai saya ...). Baris kelayakan pertama untuk setiap pasien selalu 1, yang kedua adalah 2, dll. (Jelas ini bukan kunci utama tabel anak). Jadi, saya memilih nilai penghitung maks yang ada untuk seorang pasien, dan kemudian menambahkan 1 padanya untuk membuat baris baru. Ketika tidak ada nilai anak yang ada, saya perlu permintaan untuk mengembalikan 0 (jadi menambahkan 1 akan memberi saya nilai penghitung 1). Perhatikan bahwa saya tidak ingin bergantung pada jumlah mentah baris anak, jika aplikasi warisan memperkenalkan celah dalam nilai penghitung (mungkin). Buruk saya karena mencoba membuat pertanyaan terlalu umum.

Karena DefaultIfEmptytidak diimplementasikan dalam LINQ ke SQL, saya melakukan pencarian pada kesalahan itu kembali dan menemukan artikel menarik yang berhubungan dengan set nol dalam fungsi agregat. Untuk meringkas apa yang saya temukan, Anda dapat mengatasi batasan ini dengan melemparkan ke nullable dalam pilih Anda. VB saya agak berkarat, tapi saya kira akan seperti ini:

Dim x = (From y In context.MyTable _
         Where y.MyField = value _
         Select CType(y.MyCounter, Integer?)).Max

Atau dalam C #:

var x = (from y in context.MyTable
         where y.MyField == value
         select (int?)y.MyCounter).Max();

Saya baru saja mengalami masalah yang sama, tetapi saya menggunakan metode ekstensi LINQ pada daftar daripada sintaks kueri. Casting untuk trik Nullable juga berfungsi di sana:

int max = list.Max(i => (int?)i.MyCounter) ?? 0;

Kedengarannya seperti kasing untuk DefaultIfEmpty(kode yang belum diuji mengikuti):

Dim x = (From y In context.MyTable _
         Where y.MyField = value _
         Select y.MyCounter).DefaultIfEmpty.Max

Pikirkan tentang apa yang Anda tanyakan!

Maks {{1, 2, 3, -1, -2, -3} jelas 3. Maks {{}} jelas 2. Tapi apa maks dari himpunan kosong {}? Jelas itu adalah pertanyaan yang tidak berarti. Maks dari set kosong sama sekali tidak didefinisikan. Mencoba mendapatkan jawaban adalah kesalahan matematis. Maks dari setiap set itu sendiri harus menjadi elemen dalam set itu. Set kosong tidak memiliki elemen, sehingga mengklaim bahwa beberapa angka tertentu adalah maks set itu tanpa berada di set itu adalah kontradiksi matematika.

Sama seperti perilaku yang benar bagi komputer untuk melempar pengecualian ketika programmer memintanya untuk membaginya dengan nol, demikian juga perilaku yang benar bagi komputer untuk melempar pengecualian ketika programmer meminta untuk mengambil maks set kosong. Pembagian dengan nol, mengambil maks set yang kosong, menggoyangkan spacklerorke, dan mengendarai unicorn terbang ke Neverland semuanya tidak ada artinya, tidak mungkin, tidak terdefinisi.

Sekarang, apa yang sebenarnya ingin Anda lakukan?

Anda selalu bisa menambahkan Double.MinValueke urutan. Ini akan memastikan bahwa setidaknya ada satu elemen dan Maxakan mengembalikannya hanya jika itu adalah minimum. Untuk menentukan opsi mana yang lebih efisien ( Concat, FirstOrDefaultatau Take(1)), Anda harus melakukan pembandingan yang memadai.

double x = context.MyTable
    .Where(y => y.MyField == value)
    .Select(y => y.MyCounter)
    .Concat(new double[]{Double.MinValue})
    .Max();

int max = list.Any() ? list.Max(i => i.MyCounter) : 0;

Jika daftar memiliki elemen apa pun (mis. Tidak kosong), itu akan mengambil maks bidang MyCounter, yang lain akan mengembalikan 0.

Karena .Net 3.5 Anda dapat menggunakan DefaultIfEmpty () yang meneruskan nilai default sebagai argumen. Sesuatu seperti salah satu cara berikut:

int max = (from e in context.Table where e.Year == year select e.RecordNumber).DefaultIfEmpty(0).Max();
DateTime maxDate = (from e in context.Table where e.Year == year select e.StartDate ?? DateTime.MinValue).DefaultIfEmpty(DateTime.MinValue).Max();

Yang pertama diizinkan ketika Anda query kolom NOT NULL dan yang kedua adalah cara yang digunakan untuk query kolom NULLABLE. Jika Anda menggunakan DefaultIfEmpty () tanpa argumen, nilai default akan ditentukan untuk tipe output Anda, seperti yang Anda lihat di Tabel Nilai Default .

SELECT yang dihasilkan tidak akan begitu elegan tetapi dapat diterima.

Semoga ini bisa membantu.

Saya pikir masalahnya adalah apa yang Anda inginkan terjadi ketika kueri tidak memiliki hasil. Jika ini merupakan kasus luar biasa maka saya akan membungkus kueri dalam blok coba / tangkap dan menangani pengecualian yang dihasilkan oleh kueri standar. Jika tidak ada permintaan untuk mengembalikan hasil, maka Anda harus mencari tahu apa yang Anda inginkan hasilnya dalam kasus itu. Mungkin jawaban @ David (atau yang serupa akan bekerja). Artinya, jika MAX akan selalu positif, maka mungkin cukup untuk memasukkan nilai "buruk" yang dikenal ke dalam daftar yang hanya akan dipilih jika tidak ada hasil. Secara umum, saya akan mengharapkan permintaan yang mengambil maksimum untuk memiliki beberapa data untuk bekerja dan saya akan pergi rute mencoba / menangkap karena jika tidak Anda selalu dipaksa untuk memeriksa apakah nilai yang Anda peroleh sudah benar atau tidak. SAYA'

Try
   Dim x = (From y In context.MyTable _
            Where y.MyField = value _
            Select y.MyCounter).Max
   ... continue working with x ...
Catch ex As SqlException
       ... do error processing ...
End Try

Kemungkinan lain adalah pengelompokan, mirip dengan cara Anda mendekatinya dalam SQL mentah:

from y in context.MyTable
group y.MyCounter by y.MyField into GrpByMyField
where GrpByMyField.Key == value
select GrpByMyField.Max()

Satu-satunya hal (pengujian lagi di LINQPad) beralih ke rasa VB LINQ memberikan kesalahan sintaks pada klausa pengelompokan. Saya yakin padanan konseptualnya cukup mudah ditemukan, saya hanya tidak tahu bagaimana cara mencerminkannya dalam VB.

SQL yang dihasilkan akan menjadi sesuatu seperti:

SELECT [t1].[MaxValue]
FROM (
    SELECT MAX([t0].[MyCounter) AS [MaxValue], [t0].[MyField]
    FROM [MyTable] AS [t0]
    GROUP BY [t0].[MyField]
    ) AS [t1]
WHERE [t1].[MyField] = @p0

SELECT bersarang tampak menjengkelkan, seperti eksekusi kueri akan mengambil semua baris kemudian pilih yang cocok dari set yang diambil ... pertanyaannya adalah apakah SQL Server mengoptimalkan kueri menjadi sesuatu yang sebanding dengan menerapkan klausa mana ke SELECT bagian dalam. Saya sedang melihat itu sekarang ...

Saya tidak berpengalaman dalam menafsirkan rencana eksekusi di SQL Server, tetapi sepertinya ketika klausa WHERE ada di luar SELECT, jumlah baris aktual yang menghasilkan langkah itu adalah semua baris dalam tabel, dibandingkan hanya baris yang cocok. ketika klausa WHERE ada di SELECT bagian dalam. Yang mengatakan, sepertinya hanya 1% biaya dialihkan ke langkah berikut ketika semua baris dipertimbangkan, dan bagaimanapun hanya satu baris yang kembali dari SQL Server jadi mungkin itu bukan perbedaan besar dalam skema besar hal .

terlambat, tapi saya memiliki masalah yang sama ...

Mengulang kode Anda dari posting asli, Anda ingin maks set S didefinisikan oleh

(From y In context.MyTable _
 Where y.MyField = value _
 Select y.MyCounter)

Mempertimbangkan komentar terakhir Anda

Cukuplah untuk mengatakan bahwa saya tahu saya ingin 0 ketika tidak ada catatan untuk dipilih, yang pasti berdampak pada solusi akhirnya

Saya dapat menguraikan kembali masalah Anda sebagai: Anda ingin maks {0 + S}. Dan sepertinya solusi yang diusulkan dengan concat secara semantik adalah yang benar :-)

var max = new[]{0}
          .Concat((From y In context.MyTable _
                   Where y.MyField = value _
                   Select y.MyCounter))
          .Max();

Kenapa Bukan sesuatu yang lebih langsung seperti:

Dim x = context.MyTable.Max(Function(DataItem) DataItem.MyField = Value)

Satu perbedaan menarik yang tampaknya perlu diperhatikan adalah bahwa sementara FirstOrDefault dan Take (1) menghasilkan SQL yang sama (menurut LINQPad,), FirstOrDefault mengembalikan nilai - default - saat tidak ada baris yang cocok dan Take (1) kembali tidak ada hasil ... setidaknya di LINQPad.

Hanya untuk membuat semua orang di luar sana tahu bahwa menggunakan Linq untuk Entitas metode di atas tidak akan berfungsi ...

Jika Anda mencoba melakukan sesuatu seperti

var max = new[]{0}
      .Concat((From y In context.MyTable _
               Where y.MyField = value _
               Select y.MyCounter))
      .Max();

Ini akan menghasilkan pengecualian:

System.NotSupportedException: Node ekspresi LINQ ketik 'NewArrayInit' tidak didukung di LINQ ke Entitas ..

Saya sarankan hanya melakukan

(From y In context.MyTable _
                   Where y.MyField = value _
                   Select y.MyCounter))
          .OrderByDescending(x=>x).FirstOrDefault());

Dan FirstOrDefaultakan mengembalikan 0 jika daftar Anda kosong.

decimal Max = (decimal?)(context.MyTable.Select(e => e.MyCounter).Max()) ?? 0;

Saya telah mengetuk MaxOrDefaultmetode ekstensi. Tidak banyak, tetapi kehadirannya di Intellisense adalah pengingat yang berguna bahwa Maxpada urutan kosong akan menyebabkan pengecualian. Selain itu, metode ini memungkinkan standar untuk ditentukan jika diperlukan.

    public static TResult MaxOrDefault<TSource, TResult>(this 
    IQueryable<TSource> source, Expression<Func<TSource, TResult?>> selector,
    TResult defaultValue = default (TResult)) where TResult : struct
    {
        return source.Max(selector) ?? defaultValue;
    }

Untuk Entity Framework dan Linq to SQL kita dapat mencapai ini dengan mendefinisikan metode ekstensi yang memodifikasi metode yang Expressionditeruskan ke IQueryable<T>.Max(...):

static class Extensions
{
    public static TResult MaxOrDefault<T, TResult>(this IQueryable<T> source, 
                                                   Expression<Func<T, TResult>> selector)
        where TResult : struct
    {
        UnaryExpression castedBody = Expression.Convert(selector.Body, typeof(TResult?));
        Expression<Func<T, TResult?>> lambda = Expression.Lambda<Func<T,TResult?>>(castedBody, selector.Parameters);
        return source.Max(lambda) ?? default(TResult);
    }
}

Pemakaian:

int maxId = dbContextInstance.Employees.MaxOrDefault(employee => employee.Id);
// maxId is equal to 0 if there is no records in Employees table

Query yang dihasilkan identik, ini berfungsi seperti panggilan normal ke IQueryable<T>.Max(...)metode, tetapi jika tidak ada catatan, ia mengembalikan nilai standar tipe T alih-alih melempar pengecualian.

Saya baru saja mengalami masalah yang sama, tes unit saya lulus menggunakan Max () tetapi gagal ketika dijalankan terhadap database hidup.

Solusi saya adalah memisahkan kueri dari logika yang dilakukan, bukan bergabung dalam satu kueri.
Saya membutuhkan solusi untuk bekerja dalam unit test menggunakan Linq-objects (di Linq-objects Max () bekerja dengan nulls) dan Linq-sql ketika mengeksekusi di lingkungan langsung.

(Saya mengejek Select () di tes saya)

var requiredDataQuery = _dataRepo.Select(x => new { x.NullableDate1, .NullableDate2 }); 
var requiredData.ToList();
var maxDate1 = dates.Max(x => x.NullableDate1);
var maxDate2 = dates.Max(x => x.NullableDate2);

Kurang efisien? Mungkin.

Apakah saya peduli, selama aplikasi saya tidak jatuh di waktu berikutnya? Nggak.