Apa anti-pola SQL yang paling umum? [Tutup]

Kita semua yang bekerja dengan database relasional telah belajar (atau sedang belajar) bahwa SQL berbeda. Menggali hasil yang diinginkan, dan melakukannya dengan efisien, melibatkan proses yang membosankan yang sebagian ditandai dengan mempelajari paradigma yang tidak dikenal, dan menemukan bahwa beberapa pola pemrograman kita yang paling dikenal tidak bekerja di sini. Apa antipatterns umum yang Anda lihat (atau diri Anda berkomitmen)?

Saya secara konsisten kecewa dengan kecenderungan kebanyakan programmer untuk mencampurkan logika UI mereka di lapisan akses data:

SELECT
    FirstName + ' ' + LastName as "Full Name",
    case UserRole
        when 2 then "Admin"
        when 1 then "Moderator"
        else "User"
    end as "User's Role",
    case SignedIn
        when 0 then "Logged in"
        else "Logged out"
    end as "User signed in?",
    Convert(varchar(100), LastSignOn, 101) as "Last Sign On",
    DateDiff('d', LastSignOn, getDate()) as "Days since last sign on",
    AddrLine1 + ' ' + AddrLine2 + ' ' + AddrLine3 + ' ' +
        City + ', ' + State + ' ' + Zip as "Address",
    'XXX-XX-' + Substring(
        Convert(varchar(9), SSN), 6, 4) as "Social Security #"
FROM Users

Biasanya, programmer melakukan ini karena mereka berniat untuk mengikat dataset mereka langsung ke grid, dan itu hanya nyaman untuk memiliki format SQL Server sisi server daripada format pada klien.

Query seperti yang ditunjukkan di atas sangat rapuh karena mereka memadatkan layer data dengan layer UI. Selain itu, gaya pemrograman ini secara menyeluruh mencegah prosedur tersimpan tidak dapat digunakan kembali.

Inilah 3 teratas saya.

Nomor 1. Gagal menentukan daftar bidang. (Sunting: untuk mencegah kebingungan: ini adalah aturan kode produksi. Ini tidak berlaku untuk skrip analisis satu kali - kecuali saya penulisnya.)

SELECT *
Insert Into blah SELECT *

seharusnya

SELECT fieldlist
Insert Into blah (fieldlist) SELECT fieldlist

Nomor 2. Menggunakan kursor dan loop sementara, ketika loop sementara dengan variabel loop akan dilakukan.

DECLARE @LoopVar int

SET @LoopVar = (SELECT MIN(TheKey) FROM TheTable)
WHILE @LoopVar is not null
BEGIN
  -- Do Stuff with current value of @LoopVar
  ...
  --Ok, done, now get the next value
  SET @LoopVar = (SELECT MIN(TheKey) FROM TheTable
    WHERE @LoopVar < TheKey)
END

Nomor 3. DateLogic melalui tipe string.

--Trim the time
Convert(Convert(theDate, varchar(10), 121), datetime)

Seharusnya

--Trim the time
DateAdd(dd, DateDiff(dd, 0, theDate), 0)

Saya telah melihat lonjakan baru-baru ini "Satu permintaan lebih baik dari dua, oke?"

SELECT *
FROM blah
WHERE (blah.Name = @name OR @name is null)
  AND (blah.Purpose = @Purpose OR @Purpose is null)

Permintaan ini membutuhkan dua atau tiga rencana eksekusi yang berbeda tergantung pada nilai parameter. Hanya satu paket eksekusi yang dihasilkan dan dimasukkan ke cache untuk teks sql ini. Paket itu akan digunakan terlepas dari nilai parameter. Ini berakibat pada kinerja buruk yang terputus-putus. Jauh lebih baik untuk menulis dua kueri (satu permintaan per rencana eksekusi yang dimaksudkan).

• Bidang kata sandi yang dapat dibaca manusia , egad. Cukup jelas.

• Menggunakan LIKE terhadap kolom yang diindeks , dan saya hampir tergoda untuk mengatakan LIKE secara umum.

• Daur ulang nilai-nilai PK yang dihasilkan SQL.

• Tidak ada yang mengejutkan menyebutkan meja dewa . Tidak ada yang mengatakan "organik" seperti 100 kolom bendera bit, string besar dan bilangan bulat.

• Lalu ada pola "I miss .ini file" : menyimpan CSV, string yang dibatasi pipa, atau data parse lain yang diperlukan dalam bidang teks besar.

• Dan untuk MS SQL server penggunaan kursor sama sekali . Ada cara yang lebih baik untuk melakukan tugas kursor apa pun.

Diedit karena ada begitu banyak!

Tidak harus menggali lebih dalam untuk itu: Tidak menggunakan pernyataan yang disiapkan.

Menggunakan alias tabel yang tidak berarti:

from employee t1,
department t2,
job t3,
...

Membuat membaca pernyataan SQL besar jauh lebih sulit daripada yang seharusnya

var query = "select COUNT(*) from Users where UserName = '" 
            + tbUser.Text 
            + "' and Password = '" 
            + tbPassword.Text +"'";

1. Masukan pengguna secara buta mempercayai
2. Tidak menggunakan kueri parameterisasi
3. Kata sandi bersih

Bugbears saya adalah tabel Akses kolom 450 yang telah disatukan oleh putra berusia 8 tahun dari sahabat anjing groomer Direktur Pelaksana dan tabel pencarian cerdik yang hanya ada karena seseorang tidak tahu bagaimana menormalkan struktur data dengan benar.

Biasanya, tabel pencarian ini terlihat seperti ini:

ID INT,
Nama NVARCHAR (132),
IntValue1 INT,
IntValue2 INT,
CharValue1 NVARCHAR (255),
CharValue2 NVARCHAR (255),
Date1 DATETIME,
Date2 DATETIME

Saya telah kehilangan hitungan jumlah klien yang pernah saya lihat yang memiliki sistem yang mengandalkan kekejian seperti ini.

Yang paling saya sukai adalah

1. Menggunakan spasi saat membuat tabel, sprocs dll. Saya baik-baik saja dengan CamelCase atau under_scores dan tunggal atau bentuk jamak dan huruf besar atau huruf kecil tetapi harus merujuk ke tabel atau kolom [dengan spasi], terutama jika [spasi aneh] (ya, Saya mengalami ini) benar-benar membuat saya jengkel.

2. Data yang dinormalisasi. Sebuah tabel tidak harus dinormalisasi secara sempurna, tetapi ketika saya bertemu dengan tabel karyawan yang memiliki informasi tentang skor evaluasi mereka saat ini atau apa pun yang utama, itu memberi tahu saya bahwa saya mungkin perlu membuat tabel terpisah di beberapa titik dan kemudian coba sinkronkan. Saya akan menormalkan data terlebih dahulu dan kemudian jika saya melihat tempat di mana denasionalisasi membantu, saya akan mempertimbangkannya.

3. Terlalu sering menggunakan pandangan atau kursor. Tampilan memiliki tujuan, tetapi ketika setiap tabel dibungkus dalam tampilan itu terlalu banyak. Saya harus menggunakan kursor beberapa kali, tetapi secara umum Anda dapat menggunakan mekanisme lain untuk ini.

4. Mengakses. Bisakah suatu program menjadi anti-pola? Kami memiliki SQL Server di tempat kerja saya, tetapi sejumlah orang menggunakan akses karena ketersediaannya, "kemudahan penggunaan" dan "keramahan" untuk pengguna non-teknis. Ada terlalu banyak di sini untuk dituju, tetapi jika Anda berada di lingkungan yang sama, Anda tahu.

gunakan SP sebagai awalan dari nama prosedur toko karena ia akan mencari lebih dulu di lokasi prosedur sistem daripada yang kustom.

Terlalu sering menggunakan meja sementara dan kursor.

Untuk menyimpan nilai waktu, hanya zona waktu UTC yang harus digunakan. Waktu setempat tidak boleh digunakan.

menggunakan @@ IDENTITY alih-alih SCOPE_IDENTITY ()

Dikutip dari jawaban ini :

• @@ IDENTITY mengembalikan nilai identitas terakhir yang dihasilkan untuk tabel apa pun di sesi saat ini, di semua ruang lingkup. Anda harus berhati-hati di sini, karena melintasi batas. Anda bisa mendapatkan nilai dari pemicu, alih-alih pernyataan Anda saat ini.
• SCOPE_IDENTITY mengembalikan nilai identitas terakhir yang dihasilkan untuk tabel apa pun di sesi saat ini dan ruang lingkup saat ini. Umumnya apa yang ingin Anda gunakan.
• IDENT_CURRENT mengembalikan nilai identitas terakhir yang dihasilkan untuk tabel tertentu dalam sesi apa pun dan ruang lingkup apa pun. Ini memungkinkan Anda menentukan tabel yang Anda inginkan nilainya, jika kedua di atas tidak cukup apa yang Anda butuhkan (sangat jarang). Anda bisa menggunakan ini jika Anda ingin mendapatkan nilai IDENTITAS saat ini untuk tabel yang Anda belum memasukkan catatan.

Menggunakan kembali bidang 'mati' untuk sesuatu yang tidak dimaksudkan (misalnya menyimpan data pengguna dalam bidang 'Faks') - sangat menggoda sebagai perbaikan cepat!

select some_column, ...
from some_table
group by some_column

dan dengan asumsi bahwa hasilnya akan diurutkan berdasarkan some_column. Saya telah melihat ini sedikit dengan Sybase di mana asumsi itu berlaku (untuk saat ini).

SELECT FirstName + ' ' + LastName as "Full Name", case UserRole when 2 then "Admin" when 1 then "Moderator" else "User" end as "User's Role", case SignedIn when 0 then "Logged in" else "Logged out" end as "User signed in?", Convert(varchar(100), LastSignOn, 101) as "Last Sign On", DateDiff('d', LastSignOn, getDate()) as "Days since last sign on", AddrLine1 + ' ' + AddrLine2 + ' ' + AddrLine3 + ' ' + City + ', ' + State + ' ' + Zip as "Address", 'XXX-XX-' + Substring(Convert(varchar(9), SSN), 6, 4) as "Social Security #" FROM Users

Atau, menjejalkan semuanya menjadi satu baris.

• The FROM TableA, TableB WHEREsintaks untuk BERGABUNG daripadaFROM TableA INNER JOIN TableB ON

• Membuat asumsi bahwa permintaan akan dikembalikan disortir dengan cara tertentu tanpa memasukkan klausa ORDER BY, hanya karena itulah yang muncul selama pengujian di alat kueri.

Belajar SQL dalam enam bulan pertama karir mereka dan tidak pernah belajar hal lain selama 10 tahun ke depan. Khususnya tidak belajar atau secara efektif menggunakan fitur windowing / analitik SQL. Khususnya penggunaan over () dan partisi oleh.

Fungsi jendela, seperti fungsi agregat, melakukan agregasi pada kumpulan baris yang ditentukan (grup), tetapi alih-alih mengembalikan satu nilai per grup, fungsi jendela dapat mengembalikan beberapa nilai untuk setiap grup.

Lihat O'Reilly SQL Cookbook Lampiran A untuk tinjauan umum yang bagus dari fungsi windowing.

Saya perlu menempatkan favorit saya saat ini di sini, hanya untuk membuat daftar lengkap. Antipattern favorit saya tidak menguji kueri Anda .

Ini berlaku ketika:

1. Kueri Anda melibatkan lebih dari satu tabel.
2. Anda pikir Anda memiliki desain yang optimal untuk kueri, tetapi jangan repot-repot menguji asumsi Anda.
3. Anda menerima kueri pertama yang berfungsi, tanpa petunjuk apakah itu hampir dioptimalkan.

Dan setiap tes dijalankan terhadap data atipikal atau tidak mencukupi tidak masuk hitungan. Jika ini adalah prosedur yang tersimpan, masukkan pernyataan tes ke dalam komentar dan simpan, dengan hasilnya. Kalau tidak, masukkan ke dalam komentar dengan kode hasilnya.

Penyalahgunaan Tabel Sementara.

Khususnya hal semacam ini:

SELECT personid, firstname, lastname, age
INTO #tmpPeople
FROM People
WHERE lastname like 's%'

DELETE FROM #tmpPeople
WHERE firstname = 'John'

DELETE FROM #tmpPeople
WHERE firstname = 'Jon'

DELETE FROM #tmpPeople
WHERE age > 35

UPDATE People
SET firstname = 'Fred'
WHERE personid IN (SELECT personid from #tmpPeople)

Jangan membangun tabel sementara dari kueri, hanya untuk menghapus baris yang tidak Anda butuhkan.

Dan ya, saya telah melihat halaman kode dalam bentuk ini di DB produksi.

Pandangan pelawan: obsesi berlebihan dengan normalisasi.

Sebagian besar sistem SQL / RBDBs memberikan satu banyak fitur (transaksi, replikasi) yang cukup berguna, bahkan dengan data yang tidak diformalkan. Ruang disk murah, dan kadang-kadang bisa lebih sederhana (kode lebih mudah, waktu pengembangan lebih cepat) untuk memanipulasi / memfilter / mencari data yang diambil, daripada menulis skema 1NF, dan menangani semua kerepotan di dalamnya (gabungan yang kompleks, sub-pilihan yang tidak menyenangkan) , dll).

Saya telah menemukan bahwa sistem yang terlalu dinormalisasi seringkali merupakan optimasi prematur, terutama pada tahap pengembangan awal.

(lebih banyak pemikiran tentang itu ... http://writeonly.wordpress.com/2008/12/05/simple-object-db-using-json-and-python-sqlite/ )

Saya hanya menyatukan ini, berdasarkan pada beberapa respons SQL di sini pada SO.

Ini adalah antipattern yang serius untuk berpikir bahwa pemicu adalah untuk database seperti event handler ke OOP. Ada persepsi bahwa sembarang logika lama dapat dimasukkan ke dalam pemicu, untuk dipecat ketika transaksi (peristiwa) terjadi di atas meja.

Tidak benar. Salah satu perbedaan besar adalah bahwa pemicu sinkron - dengan balas dendam, karena mereka sinkron pada operasi yang ditetapkan, bukan pada operasi baris. Di sisi OOP, justru sebaliknya - peristiwa adalah cara yang efisien untuk menerapkan transaksi asinkron.

Prosedur atau Fungsi Tersimpan tanpa komentar ...

1) Saya tidak tahu ini anti-pola "resmi", tapi saya tidak suka dan mencoba untuk menghindari string literal sebagai nilai ajaib dalam kolom database.

Contoh dari tabel 'gambar' MediaWiki:

img_media_type ENUM("UNKNOWN", "BITMAP", "DRAWING", "AUDIO", "VIDEO", 
    "MULTIMEDIA", "OFFICE", "TEXT", "EXECUTABLE", "ARCHIVE") default NULL,
img_major_mime ENUM("unknown", "application", "audio", "image", "text", 
    "video", "message", "model", "multipart") NOT NULL default "unknown",

(Saya hanya melihat casing yang berbeda, hal lain yang harus dihindari)

Saya merancang kasus-kasus seperti pencarian int ke dalam tabel ImageMediaType dan ImageMajorMime dengan kunci primer int.

2) konversi tanggal / string yang bergantung pada pengaturan NLS tertentu

CONVERT(NVARCHAR, GETDATE())

tanpa pengidentifikasi format

Subquery identik dalam kueri.

• Tampilan yang Diubah - Tampilan yang terlalu sering diubah dan tanpa pemberitahuan atau alasan. Perubahan akan diketahui pada waktu yang paling tidak tepat atau lebih buruk salah dan tidak pernah diperhatikan. Mungkin aplikasi Anda akan rusak karena seseorang memikirkan nama yang lebih baik untuk kolom itu. Sebagai aturan, pandangan harus memperluas kegunaan tabel dasar sambil mempertahankan kontrak dengan konsumen. Perbaiki masalah tetapi jangan menambahkan fitur atau mengubah perilaku yang lebih buruk, untuk itu buat tampilan baru. Untuk mengurangi jangan berbagi pandangan dengan proyek lain dan, gunakan CTE ketika platform memungkinkan. Jika toko Anda memiliki DBA, Anda mungkin tidak dapat mengubah tampilan tetapi semua tampilan Anda akan usang dan atau tidak berguna dalam hal ini.

• The! Paramed - Dapatkan kueri memiliki lebih dari satu tujuan? Mungkin tetapi orang berikutnya yang membacanya tidak akan tahu sampai meditasi mendalam. Bahkan jika Anda tidak membutuhkannya sekarang kemungkinan Anda akan, bahkan jika itu "hanya" untuk debug. Menambahkan parameter menurunkan waktu perawatan dan menjaga keadaan tetap KERING. Jika Anda memiliki klausa di mana Anda harus memiliki parameter.

• Kasing tanpa KASUS -

  SELECT  
  CASE @problem  
    WHEN 'Need to replace column A with this medium to large collection of strings hanging out in my code.'  
      THEN 'Create a table for lookup and add to your from clause.'  
    WHEN 'Scrubbing values in the result set based on some business rules.'  
      THEN 'Fix the data in the database'  
    WHEN 'Formating dates or numbers.'   
      THEN 'Apply formating in the presentation layer.'  
    WHEN 'Createing a cross tab'  
      THEN 'Good, but in reporting you should probably be using cross tab, matrix or pivot templates'   
  ELSE 'You probably found another case for no CASE but now I have to edit my code instead of enriching the data...' END  

Dua yang paling saya temukan, dan dapat memiliki biaya yang signifikan dalam hal kinerja adalah:

• Menggunakan kursor alih-alih ekspresi berbasis set. Saya kira ini sering terjadi ketika programmer berpikir secara prosedural.

• Menggunakan sub-kueri terkait, saat bergabung ke tabel turunan dapat melakukan pekerjaan.

Menempatkan barang di tabel sementara, terutama orang yang beralih dari SQL Server ke Oracle memiliki kebiasaan menggunakan tabel sementara secara berlebihan. Cukup gunakan pernyataan pilih bersarang.

Pengembang yang menulis kueri tanpa memiliki ide bagus tentang apa yang membuat aplikasi SQL (baik permintaan individu dan sistem multi-pengguna) cepat atau lambat. Ini termasuk ketidaktahuan tentang:

• strategi minimisasi fisik I / O, mengingat bahwa hambatan yang paling banyak adalah I / O bukan CPU
• Dampak perf dari berbagai jenis akses penyimpanan fisik (mis. banyak I / O berurutan akan lebih cepat daripada banyak I / O acak kecil, meskipun lebih sedikit jika penyimpanan fisik Anda adalah SSD!)
• cara menyetel kueri jika DBMS menghasilkan rencana kueri yang buruk
• cara mendiagnosis kinerja basis data yang buruk, cara "men-debug" kueri lambat, dan cara membaca rencana kueri (atau MENJELASKAN, tergantung pada DBMS pilihan Anda)
• mengunci strategi untuk mengoptimalkan throughput dan menghindari kebuntuan dalam aplikasi multi-pengguna
• pentingnya batching dan trik lain untuk menangani pemrosesan set data
• desain tabel dan indeks untuk menyeimbangkan ruang dan kinerja terbaik (mis. mencakup indeks, menjaga indeks sedapat mungkin, mengurangi tipe data hingga ukuran minimum yang diperlukan, dll.)

Menggunakan SQL sebagai paket ISAM (Indexed Sequential Access Method) yang dimuliakan. Secara khusus, kursor bersarang alih-alih menggabungkan pernyataan SQL menjadi pernyataan tunggal, meskipun lebih besar. Ini juga dianggap sebagai 'penyalahgunaan pengoptimal' karena sebenarnya tidak banyak yang dapat dilakukan pengoptimal. Ini dapat dikombinasikan dengan pernyataan yang tidak disiapkan untuk inefisiensi maksimum:

DECLARE c1 CURSOR FOR SELECT Col1, Col2, Col3 FROM Table1

FOREACH c1 INTO a.col1, a.col2, a.col3
    DECLARE c2 CURSOR FOR
        SELECT Item1, Item2, Item3
            FROM Table2
            WHERE Table2.Item1 = a.col2
    FOREACH c2 INTO b.item1, b.item2, b.item3
        ...process data from records a and b...
    END FOREACH
END FOREACH

Solusi yang benar (hampir selalu) adalah menggabungkan dua pernyataan SELECT menjadi satu:

DECLARE c1 CURSOR FOR
    SELECT Col1, Col2, Col3, Item1, Item2, Item3
        FROM Table1, Table2
        WHERE Table2.Item1 = Table1.Col2
        -- ORDER BY Table1.Col1, Table2.Item1

FOREACH c1 INTO a.col1, a.col2, a.col3, b.item1, b.item2, b.item3
    ...process data from records a and b...
END FOREACH

Satu-satunya keuntungan untuk versi loop ganda adalah bahwa Anda dapat dengan mudah melihat jeda antara nilai-nilai dalam Table1 karena loop dalam berakhir. Ini bisa menjadi faktor dalam laporan break-control.

Selain itu, pengurutan dalam aplikasi biasanya tidak boleh.

Menggunakan kunci utama sebagai pengganti untuk alamat rekaman dan menggunakan kunci asing sebagai pengganti untuk pointer yang tertanam dalam catatan.