การเขียนโปรแกรมเชิงวัตถุ: getters / setters หรือชื่อตรรกะ

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

ฉันมีชั้นเรียนCharacterStylesที่มีตัวแปรสมาชิกbold, italic, underline(และอื่น ๆ บางอย่าง แต่ฉันจะปล่อยให้พวกเขาออกไปให้มันง่าย) วิธีที่ง่ายที่สุดที่จะช่วยให้ส่วนอื่น ๆ ของโปรแกรมที่จะเข้าถึงตัวแปรเหล่านี้จะเขียนวิธีการ getter / หมาเพื่อให้คุณสามารถทำและstyles.setBold(true)styles.setItalic(false)

แต่ฉันไม่ชอบสิ่งนี้ ไม่เพียงเพราะผู้คนจำนวนมากพูดว่าผู้ได้รับ / setters ทำลายการห่อหุ้ม (มันเลวร้ายจริงๆเหรอ?) แต่ส่วนใหญ่เป็นเพราะมันไม่สมเหตุสมผล ฉันคาดหวังที่จะจัดสไตล์ตัวละครผ่านวิธีการหนึ่งstyles.format("bold", true)หรือบางอย่างเช่นนั้น แต่ไม่ผ่านวิธีเหล่านี้ทั้งหมด

แม้ว่าจะมีปัญหาหนึ่งข้อ เนื่องจากคุณไม่สามารถเข้าถึงตัวแปรสมาชิกวัตถุโดยเนื้อหาของสตริงใน C ++ ฉันจะต้องเขียนคอนเทนเนอร์ if-statement / switch ขนาดใหญ่สำหรับสไตล์ทั้งหมดหรือฉันจะต้องเก็บสไตล์ไว้ในอาเรย์แบบเชื่อมโยง ( แผนที่).

ฉันไม่สามารถเข้าใจได้ว่าวิธีที่ดีที่สุดคืออะไร ช่วงเวลาหนึ่งฉันคิดว่าฉันควรเขียนตัวรับสัญญาณ / ผู้ตั้งค่าและช่วงเวลาถัดไปที่ฉันเอนตัวไปทางอื่น คำถามของฉันคือคุณจะทำอย่างไร แล้วทำไมคุณถึงทำอย่างนั้น?

ใช่ getters / setters แบ่งการห่อหุ้ม - โดยทั่วไปแล้วมันเป็นเพียงเลเยอร์พิเศษระหว่างการเข้าถึงฟิลด์ต้นแบบโดยตรง คุณอาจเข้าถึงได้โดยตรง

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

DrDobbs มีบทความที่กล่าวเพิ่มเติม

สำหรับปัญหาของคุณฉันมี 2 วิธีคือ setStyle และ clearStyle (หรืออะไรก็ตาม) ที่ใช้ enum ของสไตล์ที่เป็นไปได้ คำสั่ง switch ภายในเมธอดเหล่านี้จะใช้ค่าที่เกี่ยวข้องกับตัวแปรคลาสที่เหมาะสม ด้วยวิธีนี้คุณสามารถเปลี่ยนการเป็นตัวแทนภายในของสไตล์เป็นอย่างอื่นถ้าคุณตัดสินใจที่จะเก็บไว้เป็นสตริง (สำหรับใช้ใน HTML เป็นต้น) - สิ่งที่ต้องการให้ผู้ใช้ทุกคนในชั้นเรียนของคุณเปลี่ยนด้วยหากคุณใช้ รับ / ตั้งค่าคุณสมบัติ

คุณยังคงสามารถเปิดใช้งานสตริงได้หากคุณต้องการรับค่าตามอำเภอใจมีทั้งคำสั่ง if-then ขนาดใหญ่ (ถ้ามีจำนวนน้อย) หรือแผนที่ของค่าสตริงเป็นตัวชี้วิธีโดยใช้std :: mem_fun (หรือstd :: ฟังก์ชั่น ) ดังนั้น "ตัวหนา" จะถูกเก็บไว้ในแผนที่ที่สำคัญที่มีค่าเป็น sts :: mem_fun เป็นวิธีการที่ตั้งค่าตัวแปรตัวหนาเป็นจริง (ถ้าสตริงเหมือนกันกับชื่อตัวแปรสมาชิกคุณสามารถใช้แมโคร stringifyingเพื่อลดปริมาณของรหัสที่คุณจำเป็นต้องเขียน)

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

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

สำหรับตัวอย่าง pseudocode:

class TextWriter : TextDrawingInterface {
    public:
        void WriteString(string x) {
            // write some text somewhere somehow
        }
}

class BoldDecorator : TextDrawingInterface {
    public:
        void WriteString(string x) {
            // bold application on
            m_textWriter.WriteString(x);
            // bold application off
        }

        ctor (TextDrawingInterface textWriter) {
            m_textWriter = textWriter;
        }

    private:
        TextWriter m_TextWriter;
}

และอื่น ๆ สำหรับสไตล์การตกแต่งแต่ละแบบ ในการใช้งานที่ง่ายที่สุดคุณสามารถพูดได้

TextDrawingInterface GetDecoratedTextWriter() {
    return new BoldDecorator(new ItalicDecorator(new TextWriter()));
}

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

ฉันเรียนวิธีแก้ปัญหาที่สอง มันดูหรูหราและยืดหยุ่นมากขึ้น แม้ว่าจะเป็นการยากที่จะจินตนาการถึงประเภทอื่น ๆ ที่ไม่ใช่ตัวหนาตัวเอียงและขีดเส้นใต้ (overline?) แต่ก็เป็นการยากที่จะเพิ่มประเภทใหม่โดยใช้ตัวแปรสมาชิก

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

ฉันคิดว่าคุณคิดมากเกินไป

styles.setBold(true) และstyles.setItalic(false)ก็โอเค