เหตุใดจึงต้องใช้วิธีสาธารณะในชั้นเรียนภายใน

มีรหัสจำนวนมากในหนึ่งในโครงการของเราที่มีลักษณะดังนี้:

internal static class Extensions
{
    public static string AddFoo(this string s)
    {
        if (s == null)
        {
            return "Foo";
        }

        return $({s}Foo);
    }
}

มีเหตุผลที่ชัดเจนในการทำสิ่งนี้นอกเหนือจาก "ง่ายกว่าที่จะทำให้เป็นสาธารณะในภายหลัง"

ฉันสงสัยว่ามันสำคัญเฉพาะในกรณีขอบแปลก ๆ (การสะท้อนใน Silverlight) หรือไม่เลย

UPDATE: คำถามนี้เป็นเรื่องของการบล็อกของฉันในกันยายน 2014 ขอบคุณสำหรับคำถามที่ยอดเยี่ยม!

มีการถกเถียงกันอย่างมากในคำถามนี้แม้จะอยู่ในทีมคอมไพเลอร์เองก็ตาม

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

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

ความคิดเห็นของฉันคือ: ทำเครื่องหมายสมาชิกเช่นสาธารณะ

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

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

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

น่าเสียดายที่มันไม่ได้ผลดีเสมอไป ตัวอย่างเช่นชั้นภายในที่ดำเนินการอินเตอร์เฟซที่ภายในยังคงต้องมีสมาชิกในการดำเนินการทำเครื่องหมายว่าเป็นที่สาธารณะเพราะพวกเขาเป็นส่วนหนึ่งของพื้นผิวสาธารณะของชั้นเรียน

ถ้าชั้นเป็นinternalมันไม่สำคัญว่าจากมุมมองการเข้าถึงไม่ว่าคุณจะทำเครื่องหมายวิธีการหรือinternal แต่ก็ยังดีที่จะใช้ชนิดที่คุณจะใช้ถ้าชั้นเป็นpublicpublic

ในขณะที่บางคนบอกว่าสิ่งนี้ช่วยลดการเปลี่ยนจากinternalเป็นpublicเป็น มันยังทำหน้าที่เป็นส่วนหนึ่งของคำอธิบายของวิธีการ Internalโดยทั่วไปแล้ววิธีการนั้นจะถือว่าไม่ปลอดภัยสำหรับการเข้าถึงที่ไม่มีการแยกส่วนในขณะที่publicวิธีการนั้นถือว่าเป็นเกมฟรี (ส่วนใหญ่)

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

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

ฉันสงสัยว่า "ง่ายกว่าที่จะทำให้เป็นสาธารณะในภายหลัง" ใช่ไหม.

กฎการกำหนดขอบเขตหมายความว่าวิธีการนั้นจะสามารถมองเห็นได้เท่านั้นinternalดังนั้นจึงไม่สำคัญว่าจะมีการทำเครื่องหมายpublicหรือinternalไม่

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

ในบางกรณีอาจเป็นได้ว่าประเภทภายในใช้ส่วนต่อประสานสาธารณะซึ่งหมายความว่าวิธีการใด ๆ ที่กำหนดไว้บนส่วนต่อประสานนั้นจะต้องประกาศเป็นสาธารณะ

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

internalบอกว่าสมาชิกสามารถเข้าถึงได้จากภายในแอสเซมบลีเดียวกันเท่านั้น ชั้นเรียนอื่น ๆ ในการชุมนุมที่สามารถเข้าถึงinternal publicสมาชิก แต่จะไม่สามารถเข้าถึงprivateหรือprotectedสมาชิกinternalหรือไม่

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

internal class SslStreamEx : System.Net.Security.SslStream
{
    public override void Close()
    {
        try
        {
            // Send close_notify manually
        }
        finally
        {
            base.Close();
        }
    }
}

วิธีการจะต้องเป็นpublicและฉันคิดว่ามันไม่มีเหตุผลที่จะกำหนดวิธีการinternalยกเว้นว่าพวกเขาจะต้องเป็นจริงอย่างที่ Eric Lippert กล่าว

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

มีความแตกต่าง ในโครงการของเราเราได้สร้างคลาสภายในจำนวนมาก แต่เราทำการทดสอบหน่วยในชุดประกอบอื่นและในข้อมูลประกอบของเราเราใช้ InternalsVisibleTo เพื่อให้ชุด UnitTest เรียกชุดชั้นในได้ ฉันสังเกตว่าคลาสภายในมีตัวสร้างภายในเราไม่สามารถสร้างอินสแตนซ์โดยใช้ Activator.CreateInstance ในชุดทดสอบหน่วยด้วยเหตุผลบางอย่าง แต่ถ้าเราเปลี่ยนนวกรรมิกให้เป็นสาธารณะ แต่คลาสก็ยังคงอยู่ภายในมันทำงานได้ดี แต่ฉันคิดว่านี่เป็นกรณีที่หายากมาก (เช่น Eric พูดในโพสต์ต้นฉบับ: Reflection)

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

แบบนี้:

public sealed class MyCurrentlySealedClass
{
    protected void MyCurretlyPrivateMethod()
    {
    }
}

ตาม "รูปแบบ" ที่ฉันได้กล่าวมาข้างต้นนี้ควรจะดีอย่างสมบูรณ์ มันเป็นไปตามความคิดเดียวกัน มันทำงานเป็นprivateวิธีการเนื่องจากคุณไม่สามารถรับชั้นเรียน แต่ถ้าคุณลบsealedข้อ จำกัด มันก็ยังใช้ได้: คลาสที่สืบทอดมาสามารถดูวิธีนี้ได้ซึ่งเป็นสิ่งที่ฉันต้องการบรรลุ แต่คุณได้รับการเตือน: หรือCS0628 CA1047ทั้งสองอย่างเกี่ยวกับอย่าประกาศprotectedสมาชิกในsealedชั้นเรียน ยิ่งกว่านั้นฉันได้พบข้อตกลงอย่างสมบูรณ์เกี่ยวกับว่ามันหมดสติ: คำเตือน 'สมาชิกที่ได้รับการป้องกันในชั้นเรียนปิดผนึก' (ชั้นเดียว)

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