โดยเฉพาะฉันเขียนใน JavaScript
สมมติว่าฟังก์ชั่นหลักของฉันคือฟังก์ชั่น A ถ้าฟังก์ชั่น A ทำการโทรไปยังฟังก์ชั่น B หลายครั้ง แต่ฟังก์ชั่น B นั้นไม่ได้ใช้ที่อื่นฉันควรวางฟังก์ชั่น B ไว้ในฟังก์ชั่น A หรือไม่
เป็นการปฏิบัติที่ดีหรือไม่? หรือฉันควรจะวาง Function B ในขอบเขตเดียวกับ Function A หรือไม่
คำตอบ:
ฉันมักจะชอบฟังก์ชั่นที่ซ้อนกันโดยเฉพาะอย่างยิ่งใน JavaScript
ฉันคิดว่าฝ่ายค้านจำนวนมากมาจากความจริงที่ว่าโปรแกรมเมอร์ส่วนใหญ่ได้รับการเลี้ยงดูในแบบ C / C ++ / Java หรือได้รับการสอนจากคนอื่นที่เป็น ฟังก์ชั่นแบบซ้อนไม่ได้ดูเป็นธรรมชาติเพราะเราไม่ได้สัมผัสกับมันมากนักเมื่อเราเรียนรู้ที่จะเขียนโปรแกรม ไม่ได้หมายความว่ามันไม่มีประโยชน์
คุณควรจะอยู่ในขอบเขตทั่วโลกด้วยเหตุผลหลายประการ
การซ้อนฟังก์ชั่นตัวช่วยเข้ากับผู้โทรจะเพิ่มความยาวของผู้โทร ความยาวฟังก์ชั่นมักจะเป็นตัวบ่งชี้เชิงลบ ฟังก์ชั่นสั้นง่ายต่อการเข้าใจจดจำแก้จุดบกพร่องและบำรุงรักษา
หากฟังก์ชันตัวช่วยนั้นมีชื่อที่เหมาะสมการอ่านชื่อนั้นก็เพียงพอแล้วโดยไม่จำเป็นต้องดูคำจำกัดความใกล้เคียง ถ้าคุณทำจำเป็นที่จะต้องดูความหมายผู้ช่วยเพื่อให้เข้าใจถึงฟังก์ชั่นการโทรแล้วโทรที่ทำมากเกินไปหรือทำงานอยู่ในระดับมากเกินไปของนามธรรมพร้อมกัน
มีผู้ช่วยทั่วโลกที่มีอยู่ช่วยให้ฟังก์ชั่นอื่น ๆ เรียกมันว่ามันจะเป็นประโยชน์โดยทั่วไปหลังจากทั้งหมด หากผู้ช่วยไม่พร้อมใช้งานคุณถูกล่อลวงให้ตัดและวางหรือลืมและนำกลับมาใช้ใหม่ไม่ดีหรือทำให้ฟังก์ชันอื่นยาวเกินกว่าที่จะเป็น
การซ้อนฟังก์ชันผู้ช่วยจะเพิ่มการล่อลวงให้ใช้ตัวแปรจากขอบเขตของผู้โทรโดยไม่มีการประกาศดังนั้นมันจะไม่ชัดเจนว่าอินพุตและเอาต์พุตของผู้ช่วยเหลือคืออะไร หากฟังก์ชั่นไม่ได้ระบุอย่างชัดเจนว่ามีการทำงานของข้อมูลใดและมีผลกระทบอย่างไรมันมักเป็นสัญญาณของความรับผิดชอบที่ไม่ชัดเจน การประกาศตัวช่วยทำงานแบบสแตนด์อโลนบังคับให้คุณรู้ว่ามันทำอะไรจริงๆ
แก้ไข
นั่นกลายเป็นคำถามที่ถกเถียงกันมากกว่าที่ฉันคิด เพื่อชี้แจง:
ใน JavaScript ฟังก์ชั่นการขยายไฟล์ขนาดใหญ่มักจะเติมเต็มบทบาทของคลาสเนื่องจากภาษาไม่ได้มีกลไกการ จำกัด ขอบเขตอื่น ๆ ฟังก์ชั่นผู้ช่วยแน่นอนควรเข้าไปในคลาสเสมือนดังกล่าวไม่ใช่ข้างนอกพวกเขา
และประเด็นเกี่ยวกับเหนำมาใช้ใหม่ได้ง่ายขึ้นว่าถ้าย่อยไม่กลายเป็นที่ใช้กันอย่างแพร่หลายมากขึ้นคุณยินดีที่จะย้ายออกจากสถานที่ทั้งหมดและวางไว้ในสถานที่ที่เหมาะสมเช่นห้องสมุดยูทิลิตี้สตริงหรือการกำหนดค่าในรีจิสทรีทั่วโลกของคุณ หากคุณไม่ต้องการสั่งซื้อรหัสของคุณเช่นนั้นคุณอาจซ้อนรูทีนย่อยเช่นเดียวกับที่คุณทำกับวิธีการวัตถุปกติในภาษา "บล็อก" เพิ่มเติม
ฉันจะเสนอเส้นทางที่สามเพื่อวางทั้งสองฟังก์ชั่นภายในการปิด มันจะมีลักษณะ:
var functionA = (function(){
function functionB() {
// do stuff...
}
function functionA() {
// do stuff...
functionB();
// do stuff...
}
return functionA;
})();เราสร้างปิดโดยการตัดคำประกาศของฟังก์ชั่นทั้งในIIFE ค่าส่งคืนของ IIFE เป็นฟังก์ชั่นสาธารณะที่เก็บไว้ในตัวแปรของชื่อสำหรับฟังก์ชั่น functionA()ฟังก์ชั่นที่สาธารณะสามารถเรียกในตรงทางเดียวกับถ้ามันถูกประกาศเป็นฟังก์ชั่นระดับโลกเช่น โปรดทราบว่าค่าที่ส่งคืนเป็นฟังก์ชันไม่ใช่การเรียกไปยังฟังก์ชันดังนั้นจึงไม่มี parens ในตอนท้าย
โดยการตัดทั้งสองฟังก์ชั่นเหมือนเช่นนั้นfunctionBอยู่ในขณะนี้ภาคเอกชนอย่างสมบูรณ์และไม่สามารถเข้าถึงนอกของการปิด functionAแต่มองเห็นได้เฉพาะ มันไม่ได้เป็น cluttering namespace โลก, และfunctionAจะไม่ให้เกะกะนิยามของ
การกำหนดฟังก์ชัน B ภายในฟังก์ชัน A ให้ฟังก์ชัน B เข้าถึงตัวแปรภายในของ A
ดังนั้นฟังก์ชั่น B ที่กำหนดไว้ในฟังก์ชั่น A จะให้การแสดงผล (หรืออย่างน้อยก็น่าจะเป็นไปได้) ที่ B กำลังใช้งานหรือแก้ไขตัวแปรท้องถิ่นของ A ในขณะที่การกำหนด B ด้านนอกของ A ทำให้มันชัดเจนว่า B ไม่ได้
ดังนั้นเพื่อความชัดเจนของรหัสฉันจะกำหนด B ภายใน A เฉพาะเมื่อ B จำเป็นต้องเข้าถึงตัวแปรโลคอลของ A หาก B มีจุดประสงค์ที่ชัดเจนซึ่งเป็นอิสระจาก A ฉันจะนิยามนอก B แน่นอน
คุณไม่ควรวางซ้อนเพราะมิฉะนั้นฟังก์ชันซ้อนจะถูกสร้างขึ้นใหม่ทุกครั้งที่คุณเรียกใช้ฟังก์ชันภายนอก
เนื่องจากฟังก์ชั่น B ไม่ได้ใช้ที่อื่นคุณอาจทำเพียงฟังก์ชั่นด้านในของฟังก์ชั่น A เพราะนั่นเป็นเหตุผลเดียวที่ทำให้มันมีอยู่