Was sind die häufigsten SQL-anti-patterns?

Hier sind meine top 3.

Nummer 1. Die Nichtangabe einer Feldliste. (Edit: um Verwechslungen vorzubeugen: dies ist eine Produktion von code-Regel. Es gilt nicht für one-off-analysis-Skripts - es sei denn, ich bin der Autor.)

SELECT *
Insert Into blah SELECT *

sollte

SELECT fieldlist
Insert Into blah (fieldlist) SELECT fieldlist

Nummer 2. Mit einem cursor und die while-Schleife als eine while-Schleife mit einer Schleife Variablen zu tun.

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

Nummer 3. DateLogic durch string-Typen.

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

Sollte

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

Habe ich gesehen, den letzten spike "Eine Abfrage ist besser als zwei, amiright?"

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

Diese Abfrage erfordert zwei oder drei verschiedene Ausführungspläne abhängig von den Werten der Parameter. Nur ein Ausführungsplan generiert wird, und stecken in den cache für diesen sql-text. Dieser plan wird verwendet, unabhängig vom Wert der Parameter. Dies führt zeitweise schlechte Leistung. Es ist viel besser, zu schreiben, zwei Abfragen zu einer Abfrage pro soll Ausführungsplan).

InformationsquelleAutor der Antwort

Nicht tief zu Graben: Nicht mit prepared statements.

InformationsquelleAutor der Antwort

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

1. Blind Vertrauen Benutzereingaben
2. Nicht mit parametrisierte Abfragen
3. Klartext-Passwörter

InformationsquelleAutor der Antwort

Diejenigen, die ich am meisten hasse sind

1. Mithilfe von Leerzeichen beim erstellen von Tabellen, sprocs etc. Ich bin fein mit CamelCase oder under_scores und singular-oder Pluralformen und groß-oder Kleinbuchstaben, aber mit verweisen auf eine Tabelle oder Spalte [mit Leerzeichen], besonders wenn es [ es ist seltsam spaced] (ja, ich habe in diesem) wirklich reizt mich.

2. Denormalisierte Daten. Eine Tabelle muss nicht perfekt normalisiert, aber wenn ich in eine Tabelle der Mitarbeiter, die Informationen über Ihre aktuelle Bewertung-score oder Ihre primären nichts, es sagt mir, dass ich wahrscheinlich benötigen, um eine separate Tabelle zu einem bestimmten Punkt und dann versuchen, Sie zu halten synchronisiert. Ich werde normalisieren Sie die Daten zuerst, und dann, wenn ich einen Platz sehen, wo Denormalisierung hilft, ich werde es prüfen.

3. Überbeanspruchung der Blick auf oder Cursor. Blick auf einen Zweck haben, aber wenn jeder Tisch ist verpackt in einer Sicht es ist zu viel. Ich habe zu verwenden von Cursorn, ein paar mal, aber im Allgemeinen Sie können andere Mechanismen für diese.

4. Zugang. Kann ein Programm sein, ein anti-pattern? Wir haben SQL-Server in meiner Arbeit, aber eine Anzahl von Menschen den Zugang aufgrund seiner Verfügbarkeit, "ease of use" und "Freundlichkeit" für nicht-technische Anwender. Es ist zu viel hier zu gehen, aber wenn Sie schon in einer ähnlichen Umgebung, die Sie kennen.

InformationsquelleAutor der Antwort

Übermäßigen Gebrauch von temporären Tabellen und Cursor.

InformationsquelleAutor der Antwort

mithilfe von @@IDENTITY statt SCOPE_IDENTITY()

Zitiert aus diese Antwort :

• @@IDENTITY liefert den letzten identity-Wert erzeugt für jede Tabelle in der aktuellen Sitzung in allen Gültigkeitsbereichen. Sie müssen hier vorsichtig sein, da es über scopes. Sie könnten einen Wert aus einem trigger, der anstelle von Ihrem aktuellen statement.
• SCOPE_IDENTITY liefert den letzten identity-Wert erzeugt für jede Tabelle in der aktuellen Sitzung und der aktuelle Bereich. In der Regel, was Sie verwenden möchten.
• IDENT_CURRENT gibt den letzten Identitätswert generiert, die für eine bestimmte Tabelle in einer Sitzung oder einem Gültigkeitsbereich. Diese können Sie angeben, welche Tabelle Sie wollen den Wert aus, falls die beiden oben sind nicht ganz das, was man braucht (sehr selten). Könnte man dies verwenden, wenn Sie möchten, Holen Sie sich die aktuellen IDENTITY-Wert für eine Tabelle, die Sie eingesetzt haben, einen Datensatz in.

InformationsquelleAutor der Antwort

select some_column, ...
from some_table
group by some_column

davon aus, dass das Ergebnis sortiert werden, indem some_column. Ich habe gesehen, dass dies etwas mit Sybase, wo die Annahme hält (für jetzt).

InformationsquelleAutor der Antwort

• Den FROM TableA, TableB WHERE syntax für JOINS anstatt FROM TableA INNER JOIN TableB ON

• Annahmen, die von einer Abfrage zurückgegeben werden, sortiert eine bestimmte Art und Weise, ohne dass man eine ORDER BY-Klausel, nur weil das war die Art und Weise es zeigte sich während der Tests in der Abfrage-tool.

InformationsquelleAutor der Antwort

Lernen SQL, die in den ersten sechs Monaten Ihrer Karriere, und nie lernen etwas anderes für die nächsten 10 Jahre. Insbesondere nicht lernen oder effektiv mit windowing/analytische SQL-Funktionen. Insbesondere die Verwendung von over() und die partition durch.

Window-Funktionen, wie Aggregat
Funktionen, führen eine aggregation auf eine
definierte Gruppe (eine Gruppe) von Zeilen, aber
anstelle der Rückgabe ein Wert pro
Gruppe Fenster-Funktionen zurückgeben kann
mehrere Werte für jede Gruppe.

Sehen O ' Reilly SQL Cookbook Anhang A für ein schöner überblick über die windowing-Funktionen.

InformationsquelleAutor der Antwort

Konträre Ansicht: über-obsession mit der Normalisierung.

Meisten SQL/RBDBs Systeme bieten eine Vielzahl von Funktionen (Transaktionen, Replikation) , sind sehr nützlich, auch mit nicht normalisierte Daten. Speicherplatz ist Billig, und manchmal kann es einfacher (einfacher code, schnellere Entwicklungszeit) zu manipulieren /filter /Suche abgerufenen Daten, als es ist, schreiben, bis 1NF-schema, und befassen sich mit all den ärger enthaltenen (komplexen Verknüpfungen, versaute subselects, etc).

Habe ich festgestellt, die über-normierte Systeme sind Häufig vorzeitige Optimierung, insbesondere während der frühen Entwicklungsstadien.

(weitere Gedanken über Sie... http://writeonly.wordpress.com/2008/12/05/simple-object-db-using-json-and-python-sqlite/)

InformationsquelleAutor der Antwort

1) ich weiß nicht, es ist eine "offizielle" anti-pattern, aber ich mag nicht und versuchen zu vermeiden, string-Literale, wie Magie Werte in einer Spalte in der Datenbank.

Ein Beispiel aus dem MediaWiki-Tabelle "Bild":

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",

(Ich habe nur bemerkt, verschiedene Gehäuse, eine andere Sache zu vermeiden)

Entwerfe ich solche Fälle als int-lookups in den Tabellen ImageMediaType und ImageMajorMime mit int primary keys.

2) Tag/string-Umwandlung, stützt sich auf bestimmte NLS-Einstellungen

CONVERT(NVARCHAR, GETDATE())

ohne format identifier

InformationsquelleAutor der Antwort

• Veränderter Blick - Ein Blick verändert sich zu oft und ohne Vorherige Ankündigung oder Grund. Die änderung wird entweder bemerkte am meisten unpassenden Zeit oder noch schlimmer falsch sein und nie bemerkt. Vielleicht Ihrer Anwendung unterbrochen wird, weil jemand dachte, einen besseren Namen für diese Spalte. Als eine Regel sollte Ansichten erweitern die Nützlichkeit des Basis-Tabellen unter Beibehaltung eines Vertrages mit Verbrauchern. Probleme lösen, aber nicht durch hinzufügen von features oder schlimmer Verhalten zu ändern, für das erstellen einer neuen Ansicht. Zu mildern, nicht teilen Sie Ansichten mit anderen Projekten und verwenden Sie CTEs wenn Plattformen ermöglichen. Wenn Ihr Geschäft hat ein DBA kann man wohl nicht ändern Ansichten, aber alle Ihre Ansichten veraltet und oder nutzlos in diesem Fall.

• Den !Paramed - Können in einer Abfrage mehr als einen Zweck? Wohl aber die nächste person, die liest es nicht wissen, bis Sie die Tiefe meditation. Auch wenn Sie diese nicht brauchen gerade jetzt sind die Chancen Sie haben, auch wenn es "nur" zum Debuggen. Hinzufügen von Parametern verringert Wartung Zeit und halten die Dinge TROCKEN. Wenn Sie eine where-Klausel verfügen, sollten Sie die Parameter haben.

• Fall für keinen FALL -

  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  

InformationsquelleAutor der Antwort

Setzen Sachen in temporäre Tabellen, vor allem Menschen, die Schalter von SQL Server zu Oracle haben die Angewohnheit, übernutzung temporäre Tabellen. Nur mit geschachtelten select-Anweisungen.

InformationsquelleAutor der Antwort

Entwickler schreiben von Abfragen, die zimmerreserviereung, ohne das eine gute Idee, was macht SQL-Anwendungen (individuelle Abfragen und multi-user-Systeme) schnell oder langsam. Dies beinhaltet die Unwissenheit über:

• physischen I/O-Minimierung Strategien, gegeben, dass die meisten Abfragen' Flaschenhals ist I/O-CPU -
• perf Auswirkungen der verschiedenen Arten von physischen Speicher zugreifen (z.B. viele sequentielle I/O wird schneller sein als viele kleine random-I/O, obwohl weniger also, wenn Sie Ihre physischen Speicher ist eine SSD!)
• wie hand-optimieren einer Abfrage, wenn das DBMS erzeugt eine schlechte Abfrage-plan
• wie zu diagnostizieren schlechte Datenbank-performance, wie man "debug" eine langsame Abfrage, und, wie zu Lesen, die einen Abfrage-plan (oder zu ERKLÄREN, je nach Ihrer DBMS-Wahl)
• locking-Strategien zur Optimierung von Durchsatz und vermeiden von deadlocks in multi-user-Anwendungen
• Bedeutung, die Dosierungs-und andere tricks zu handhaben, die Verarbeitung der Datenbestände, die
• Tabelle und index-design, beste balance, Raum und Leistung (z.B. Abdeckung von Indizes, halten Indizes, kleine, wo möglich, Verringerung der Datentypen auf die minimale benötigte Größe, etc.)

InformationsquelleAutor der Antwort

Mit Primärschlüssel als ein Surrogat für die Aufnahme von Adressen und die Verwendung von Fremdschlüsseln als Surrogat für Zeiger eingebettet in den Aufzeichnungen.

InformationsquelleAutor der Antwort