การสร้างใหม่: มันไม่ใช่แค่คำแฟนซีสำหรับล้างรหัสของคุณใช่ไหม [ปิด]

ก่อนที่หนังสือของมาร์ตินฟาวเลอร์ "Refactoring: การปรับปรุงการออกแบบของรหัสที่มีอยู่" ออกมาเราเคยเรียกการเปลี่ยนแปลงครั้งใหญ่กับรหัส IMO เทคนิคการเปลี่ยนโฉมใหม่เป็นสามัญสำนึก / สิ่งที่เห็นได้ชัดที่เราทำตลอดไป

คุณคิดว่าการปรับโครงสร้างนั้นมีอะไรใหม่หรือไม่? อาจเป็นเพียงวิธีหนึ่งในการหลอกลวงให้ผู้บริหารจัดสรรเวลาสำหรับการล้างโค้ดหรือไม่

Refactoring เก่ากว่าเนินเขาดังนั้นไม่เลยไม่มีอะไรใหม่

และการทำให้ใหม่ไม่ได้ทำความสะอาด เป็นไปได้ แต่ไม่ จำกัด เพียงการทำความสะอาด

มันเป็นการปรับสถาปัตยกรรมของแอปพลิเคชั่นของคุณ (ไม่ว่าจะเล็กหรือใหญ่) ในขณะที่ยังคงพฤติกรรม

ซึ่งหมายความว่าในขณะที่บางส่วนของแอปพลิเคชันของคุณอาจสะอาดและดีอย่างสมบูรณ์เมื่อวานนี้คุณลักษณะใหม่ของวันนี้ต้องมีการปรับแต่งส่วนนั้นเพื่อรองรับคุณสมบัติใหม่

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

สิ่งนี้กล่าวว่าไม่ว่าการเปลี่ยนแปลงใดที่เกิดขึ้นกับโค้ดหนึ่งควรรันการทดสอบของเขาเสมอ ... ในกรณี

มันเป็นเพียงการจัดรหัสให้เรียบร้อย โดยพื้นฐานแล้วโปรแกรมเมอร์ (โดยเฉพาะมาร์ตินฟาวเลอร์) สังเกตว่าพวกเขามีแนวโน้มที่จะทำงานแบบเดียวกันทุกครั้งที่พวกเขาจัดระเบียบโค้ด พวกเขากำหนดและระบุวิธีการจัดระเบียบและปัญหาเกี่ยวกับรหัสและ presto! การสร้างใหม่เกิดขึ้น

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

ไม่มีเวทย์มนตร์ในการปรับโครงสร้าง มันเป็นเพียงศัพท์แสงชุดใหม่เพื่ออธิบายการฝึกฝนแบบเก่า

เราทำสามสิ่งที่แตกต่างกันใน บริษัท ของเราด้วยเวลาที่จัดสรรสำหรับทั้งสาม:

• การปรับโครงสร้างใหม่:ประกอบด้วยการเปลี่ยนแปลงโครงสร้างรหัสดังนั้นการรักษาพฤติกรรม

ตัวอย่าง: แบ่งวิธี 100 บรรทัดที่น่าเกลียดและอ่านไม่ได้ซึ่งทำสี่สิ่งออกเป็นสี่วิธีที่สามารถนำกลับมาใช้ใหม่ได้ 25 บรรทัดในแต่ละวิธี

• การล้างข้อมูล:ประกอบด้วยการทำการแก้ไขเล็กน้อยเพื่อให้โค้ดอ่านได้ง่ายขึ้นโดยไม่แก้ไขพฤติกรรมหรือโครงสร้างของมัน

ตัวอย่าง: การลบรหัสความคิดเห็นหลังจากตรวจสอบให้แน่ใจว่าไม่จำเป็นต้องใช้รหัสนี้อีกต่อไป

• การบังคับใช้กฎ StyleCop / FxCop:ประกอบด้วยการตรวจสอบว่ารหัสตรงกับชุดเริ่มต้นของกฎ StyleCop หรือ FxCop หรือไม่หากไม่ให้แก้ไขเพื่อให้ตรงกับกฎเหล่านั้น

ตัวอย่าง: การเพิ่มCulture.Invariantในstring.Format(หรือวัฒนธรรมอื่นที่มีความเหมาะสมมากขึ้น)

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

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

มันเป็นขั้นตอนมากมาย - เล็กและใหญ่ - หวังว่าจะส่งผลให้โปรแกรมดีขึ้น

ฉันเห็นด้วยกับ "refactoring เป็นคำแฟนซีสำหรับล้างรหัสของคุณ" แต่ไม่ใช่ด้วย "เพียงแค่" ผู้คนใช้คำแปลก ๆ ด้วยเหตุผล: บางครั้งเพราะพวกเขาต้องการดูฉลาดและบางครั้งก็เพราะพวกเขาถ่ายทอดความหมายที่มากขึ้นหรือแม่นยำมากขึ้นและการปรับโครงสร้าง IMHO (แม้ว่าบางครั้งจะใช้ผิดวัตถุประสงค์) ก็อ้างถึงหลัง

"ล้างข้อมูล" อาจหมายถึงอะไรก็ได้ตั้งแต่ "ฟอร์แมตใหม่" ไปจนถึง "เขียนใหม่ชิ้นใหญ่ ๆ "

"การปรับโครงสร้างใหม่" หมายถึงบางสิ่งบางอย่างโดยเฉพาะเช่น "การเปลี่ยนแปลงเล็ก ๆ น้อย ๆ ของรหัสที่ออกแบบมาเพื่อรักษาฟังก์ชันการทำงานเดียวกันในขณะที่เปลี่ยนเป็นการออกแบบที่ดีกว่า" และมีเนื้อหาของแนวปฏิบัติที่ดีที่สุดในการเรียงลำดับของสิ่งที่คุณทำ: บางอย่างเป็นการเฉพาะกิจ แต่มีหลักการทั่วไปเช่นการใช้การทดสอบหน่วยการแยกส่วนของฟังก์ชั่นลงในฟังก์ชั่นใหม่หรือชั้นเรียน ฯลฯ ซึ่งผู้คนสามารถและควรเรียนรู้ .

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

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

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

สำหรับฉันการปรับโครงสร้างส่วนบุคคลคือสิ่งที่ผู้คนเรียกตอนนี้ว่า "การสร้างรหัสซ้ำอัตโนมัติ" เช่น: การสนับสนุนเทคนิคการปรับโครงสร้างใหม่ภายใน IDE นี่เป็นการปรับปรุงที่แท้จริงของสิ่งที่ฉันทำก่อนหน้านี้ (ซึ่งแน่นอนว่าเจ็บปวดมาก) ฉันสามารถทำการเปลี่ยนแปลงในชั้นเรียนหนึ่งและไม่ต้องกังวลว่าสิ่งนี้จะส่งผลกระทบต่อส่วนที่เหลือของซอฟต์แวร์อย่างไร ฉันคิดว่ามาร์ตินทำการปรับเปลี่ยนเทคนิคอย่างเป็นทางการจนถึงจุดที่สามารถแสดงเป็นอัลกอริธึมและนำไปใช้ใน IDEs ต่างๆ

ดังนั้นหากคุณเข้าใจการปรับโครงสร้างใหม่เป็นกระบวนการแสดงว่าไม่มีอะไรใหม่ หากคุณเห็นว่ามันเป็นระบบอัตโนมัติก็ใช่ว่าจะเป็นการปรับปรุงครั้งใหญ่ ลองเปลี่ยนชื่อคลาสหลักสองสามตัว (จริงๆไม่ใช่ผ่านตัวเลือก refactoring ของ IDE) ในโครงการขนาดใหญ่พอสมควรเพื่อดูว่าทำไม :)

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

รหัส "การล้างข้อมูล" เป็นสิ่งที่เราทำอย่างต่อเนื่องโดยไม่มีการจัดสรรเวลาเป็นพิเศษ สำหรับฉัน "cleanup" มักจะเปลี่ยนชื่อทำความสะอาดความคิดเห็น ฯลฯ

ฉันว่าไม่

อาจมีการล้างข้อมูลในกระบวนการ refactoring แต่ไม่ใช่สาระสำคัญ

การล้างข้อมูลมาพร้อมกับสมมติฐานว่ารหัสก่อนหน้าไม่สะอาด ในความเป็นจริงนักพัฒนา refactor รหัสของพวกเขาแม้รหัสเดิมสะอาดอยู่แล้ว

DRYเป็นไดรฟ์หลักที่อยู่เบื้องหลังการปรับโครงสร้างใหม่

เมื่อเพิ่มรหัสใหม่ลงในฐานรหัสที่มีอยู่การปรับโครงสร้างจะทำตามธรรมชาติเนื่องจากหลักการของ DRY

แค่ 0.02 ของฉัน

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

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

คำว่า 'refactoring' ยืมมาจากพีชคณิตอย่างหรูหรา มันหมายถึงการทำให้คำศัพท์ง่ายขึ้นเพื่อให้ได้ผลลัพธ์เดียวกัน ไม่เพียง แต่สง่างามมันเป็นการปฏิวัติ - มันต้องใช้วิธีการที่ จำกัด อย่างหนักในโค้ดของคุณในระดับที่ทำให้หลายคนประหลาดใจ ดังนั้นคำนี้จึงสำคัญและเป็นประโยชน์

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

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

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