รหัสการจัดรูปแบบแบบสอบถาม SQL

ฉันควรทำลายแบบสอบถาม SQL ในบรรทัดที่แตกต่างกันหรือไม่ ตัวอย่างเช่นในโครงการที่ฉันกำลังดำเนินการเรามีแบบสอบถามที่ใช้คอลัมน์ 1600! 1600 + ตัวอักษรแท็บ ฉันเขียนข้อความค้นหาเช่นนี้:

   "SELECT bla , bla2 , bla FROM bla " . 
     "WHERE bla=333 AND bla=2" . 
      "ORDER BY nfdfsd ...";

แต่พวกเขาเรียกร้องให้ฉันใส่ไว้ในบรรทัดเดียวและบอกว่าสไตล์ของฉันจัดรูปแบบไม่ถูกต้อง ทำไมมันเป็นการปฏิบัติที่ไม่ดี?

สำหรับเหตุผลในการควบคุมแหล่งที่มาเรามีการแพร่กระจายหลังจากทุกส่วนคำสั่งหรือเครื่องหมายจุลภาค ดังนั้นข้างบนของคุณกลายเป็น

SELECT bla 
     , bla2 
     , bla 
FROM   bla 
WHERE  bla=333 
  AND  bla=2
ORDER  BY nfdfsd
        , asdlfk;

(การแท็บและการจัดแนวไม่มีมาตรฐานที่นี่ แต่โดยปกติแล้วจุลภาคมักเป็นการนำหน้า)

แต่ถึงกระนั้นก็ไม่ได้สร้างความแตกต่างประสิทธิภาพ

แบบสอบถามที่มี 1,600 คอลัมน์ดูเหมือนว่าต้องการการตรวจสอบที่จริงจังโดย DBA ที่ดี

หากแบบสอบถามมีความซับซ้อนฉันจะสรุป หากตรงไปตรงมาฉันจะปล่อยให้มันเป็นบรรทัดเดียวเว้นเสียแต่ว่ามันจะยาวเกินไปฉันจะเริ่มห่ออีกครั้ง

มันคือทั้งหมดที่เกี่ยวกับความสามารถในการจัดการและการทำความเข้าใจในสิ่งที่ควรทำดังนั้นการห่อหรือไม่สามารถตัดสินใจได้อย่างง่ายดายเว้นแต่องค์กรของคุณมีกฎการจัดรูปแบบโค้ดบางอย่างเกี่ยวกับเรื่องนี้

Re: มันเป็นการฝึกเขียนโค้ดที่ไม่ดี แทบจะไม่ได้! เป็นการปฏิบัติที่ดีมาก ไม่มีเหตุผลที่ดีที่ฉันรู้ที่จะใช้แบบสอบถามที่ยาวและเหตุผลที่ดีมากมายที่จะจัดรูปแบบใหม่ อย่างที่ฉันได้พูดไปแล้ว DBA ที่มีทักษะอาจต้องทำงานกับมัน

ข้อได้เปรียบเพียงข้อเดียวของการสืบค้นแบบบรรทัดเดียวที่อยู่ในใจคือการสอบถามเหล่านั้นอาจง่ายต่อการ grep นอกเหนือจากนั้นแม้ว่าฉันจะนิ่งงัน โดยส่วนตัวแล้วฉันชอบข้อความค้นหาแบบแยกย่อยที่อ่านง่ายขึ้น

ความคิดเห็นหลายบรรทัดเป็นสิ่งที่ดีเกือบจะสำคัญเมื่อต้องรับมือกับ SQL จำนวนมาก และถ้าภาษาการเขียนโปรแกรมของคุณมีเครื่องหมายคำพูด heredoc มันจะดีกว่า (ผู้แก้ไขหลายคนสามารถเน้นไวยากรณ์ SQL ในพวกเขาได้)

ตัวอย่าง:

$a = SQL<<<
    SELECT a, b, c, d
    FROM Foo f
    WHERE f.a = ?
SQL;

เมื่อทำงานกับข้อความค้นหาหลายสิบบรรทัด (หรือหลายร้อย) ทั้งการเยื้องและช่องว่างทำให้ข้อความทำงานได้

ดูเหมือนว่านี่เป็นเรื่องเกี่ยวกับการกำหนดเคียวรีขนาดใหญ่โดยเฉพาะในภาษาการเขียนโปรแกรมโดยเฉพาะเมื่อคุณใส่เคียวรีไว้ในสตริงตามตัวอักษรและต่อกัน

หากเป็นภาษาที่รวบรวมก็ไม่ควรสร้างความแตกต่างเลย - หนึ่งในการปรับแต่งแรกที่คอมไพเลอร์จะทำคือการเชื่อมต่อสตริงตัวอักษรเข้าด้วยกันโดยอัตโนมัติ

สำหรับไวยากรณ์คุณควรพิจารณาย้ายแบบสอบถามนอกรหัสของคุณ - เก็บไว้ในไฟล์ทรัพยากร. sql แยกต่างหากและให้ซอฟต์แวร์ของคุณอ่านไฟล์นั้น ใช้คำสั่งที่เตรียมไว้สำหรับตัวแปรหากไม่ใช่เคียวรีที่สร้างขึ้นแบบไดนามิก (เช่นคำสั่งที่เพิ่ม ฯลฯ ขึ้นอยู่กับพารามิเตอร์บางอย่าง) หากมันถูกสร้างขึ้นแบบไดนามิกคุณสามารถเพิ่มตัวแปรทดแทนของคุณเองโดยใส่พารามิเตอร์เพิ่มเติมได้ทุกที่และทุกเวลาที่ต้องการ

สำหรับคอลัมน์ 1600 ฉันขอแนะนำให้สร้างมุมมองอย่างจริงจังแทนที่จะเป็น

SELECT column1, column2, .... column1600 from X where Y

คุณจะได้รับ

SELECT * จาก viewX WHERE y

กระชับมากขึ้นในรหัสของคุณเอง

ฉันมักจะใช้รูปแบบที่วางโดย @glasnt เพื่อแก้ไขปัญหาการสืบค้นที่ซับซ้อน แต่มักจะมีการสืบค้นในบรรทัดเดียว

สิ่งนี้อาจไม่ตอบคำถามของคุณ แต่ฉันขอแนะนำอย่างยิ่งให้แบ่งคำถามของคุณออกเป็นข้อสงสัยเล็ก ๆ เห็นได้ชัดว่าสิ่งนี้ขึ้นอยู่กับแบบสอบถาม แต่ยิ่งคุณเพิ่มคำสั่งและเข้าร่วมในแบบสอบถามมากเท่าไรเครื่องยนต์ SQL ก็ยิ่งน้อยลงเท่านั้นที่สามารถเพิ่มประสิทธิภาพการสืบค้นของคุณ

ผู้จำหน่ายฐานข้อมูลของคุณควรมีเครื่องมือเช่นการอธิบายของ MySQL (หรือการตั้งค่า SHOWPLAN_ALL ของ MSSQL) ซึ่งจะแสดงให้คุณเห็นว่าฐานข้อมูลกำลังทำอะไรอยู่เบื้องหลังเพื่อเพิ่มประสิทธิภาพการสืบค้นของคุณทุกครั้งที่ฐานข้อมูลมีการสร้างตารางชั่วคราว ความล่าช้าอย่างมากเมื่อคุณพูดถึงผู้ใช้หลายคนพร้อมกัน

ด้วยการย้ายสิ่งที่อาจดูเหมือนลอจิกเล็กน้อยออกจาก SQL และในโค้ดของคุณคุณสามารถเพิ่มประสิทธิภาพได้อย่างน่าทึ่ง - SQL นั้นยอดเยี่ยมในการใช้งานที่ง่าย

ประโยชน์ที่ชัดเจนสำหรับสิ่งนี้เนื่องจากอาจเกี่ยวข้องกับคุณคือข้อความค้นหาของคุณซับซ้อนน้อยกว่าและอ่านง่าย - ง่ายต่อการจัดการ (ไม่ใช่> 1600 คอลัมน์) และเร็วกว่า ชนะแน่นอนทุกรอบ

หวังว่าจะช่วย :)