โปรแกรมเมอร์รุ่นเยาว์คนแรกทำ IDE แบบฝึกหัดและนักพัฒนาอาวุโสคอยสอนพวกเขาในรูปแบบการต่อต้านนี้

ปัญหา

• ความไม่แน่นอน
• การซ่อนข้อมูล
• แบบจำลองภาวะโลหิตจาง
• ล้มเหลวอย่างรวดเร็ว
• ความซื่อสัตย์
• รหัสซ้ำ

1. หลีกเลี่ยง Setters
2. ตั้งค่าแอตทริบิวต์ที่จำเป็นในการเริ่มต้นส่วนตัว

ไม่ถูกต้อง

การกลายพันธุ์ทำให้เกิดปัญหามากมาย

ข้อมูลที่ซ่อนถูกละเมิด

ขวา

การตรวจจับ

ขั้นตอนแรกคือห้ามแอตทริบิวต์สาธารณะ (หากภาษาอนุญาต)

ประการที่สองเราจะค้นหาเมธอดsetXXXX ()วิเคราะห์โครงสร้างวิธีการ (ควรเป็นการกำหนดให้แอตทริบิวต์xxxx )

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

ตัวอย่าง

• DTO

การตั้งค่าคุณลักษณะที่มีความปลอดภัยสำหรับที่ไม่จำเป็นแอตทริบิวต์

พฤติกรรมที่สำคัญคือสิ่งที่ทำให้วัตถุหนึ่งแตกต่างจากอีกวัตถุหนึ่ง

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

รูปแบบบางอย่างเช่นBuilderต้องการการตั้งค่าชิ้นส่วนด้วยวิธีการควบคุมที่เพิ่มขึ้น การตรวจสอบจะเสร็จสิ้นในตอนท้ายและการเปรียบเทียบเอนทิตีที่แท้จริงต้องการ

การตั้งค่าโดยไม่ตั้งใจมีข้อเสียและข้อควรพิจารณามากมายที่กล่าวไปแล้ว

แท็ก

• การกลายพันธุ์
• การซ่อนข้อมูล

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

ความสัมพันธ์

Code Smell 01 - Anemic Models

ข้อมูลเพิ่มเติม

นี่คือการอภิปรายเต็มรูปแบบเกี่ยวกับSetters

นางแบบเปลือย - ส่วนที่ 1: ผู้ตั้งถิ่นฐาน

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

Rebecca Wirfs-Brock

บทความนี้เป็นส่วนหนึ่งของ CodeSmell Series

วิธีค้นหาส่วนที่มีกลิ่นเหม็นของรหัสของคุณ