Apakah tidak masalah bagi kelas untuk menggunakan metode publiknya sendiri?

Latar Belakang

Saat ini saya memiliki situasi di mana saya memiliki objek yang dikirim dan diterima oleh perangkat. Pesan ini memiliki beberapa konstruk, sebagai berikut:

public void ReverseData()
public void ScheduleTransmission()

The ScheduleTransmissionMetode perlu untuk memanggil ReverseDatametode setiap kali disebut. Namun, ada saat-saat di mana saya perlu menelepon secara ReverseDataeksternal (dan saya harus menambahkan di luar namespace sepenuhnya ) dari mana objek tersebut dipakai dalam aplikasi.

Adapun "menerima" Maksud saya ReverseDataakan dipanggil secara eksternal dalam object_receivedevent-handler untuk membatalkan data.

Pertanyaan

Apakah secara umum dapat diterima suatu objek untuk memanggil metode publiknya sendiri?

Saya akan mengatakan itu tidak hanya dapat diterima tetapi dianjurkan terutama jika Anda berencana untuk mengizinkan ekstensi. Untuk mendukung ekstensi ke kelas dalam C #, Anda perlu menandai metode sebagai virtual per komentar di bawah ini. Anda mungkin ingin mendokumentasikan ini, sehingga seseorang tidak terkejut ketika mengganti ReverseData () mengubah cara kerja ScheduleTransmission ().

Itu benar-benar bermuara pada desain kelas. ReverseData () terdengar seperti perilaku mendasar kelas Anda. Jika Anda perlu menggunakan perilaku ini di tempat lain, Anda mungkin tidak ingin memiliki versi lain. Anda hanya perlu berhati-hati agar Anda tidak membiarkan detail khusus untuk ScheduleTransmission () bocor ke dalam ReverseData (). Itu akan menciptakan masalah. Tetapi karena Anda sudah menggunakan ini di luar kelas, Anda mungkin sudah memikirkannya.

Benar.

Visibilitas metode memiliki tujuan tunggal untuk mengizinkan atau menolak akses ke metode di luar kelas atau di dalam kelas anak; metode publik, terlindungi dan pribadi selalu dapat dipanggil di dalam kelas itu sendiri.

Tidak ada yang salah dalam memanggil metode publik. Ilustrasi dalam pertanyaan Anda adalah contoh sempurna dari situasi di mana memiliki dua metode publik persis apa yang harus Anda lakukan.

Namun, Anda dapat dengan cermat memperhatikan pola-pola berikut:

1. Metode Hello()memanggil World(string, int, int)seperti ini:

   Hello()
   {
       this.World("Some magic value here", 0, 100);
   }

   Hindari pola ini. Sebagai gantinya, gunakan argumen opsional . Argumen opsional membuat mudah ditemukan: pemanggil yang mengetik Hello(tidak perlu tahu bahwa ada metode yang memungkinkan untuk memanggil metode dengan nilai default. Argumen opsional juga mendokumentasikan diri. World()tidak menunjukkan kepada pemanggil apa nilai default aktual.

2. Metode Hello(ComplexEntity)memanggilWorld(string, int, int)seperti ini:

   Hello(ComplexEntity entity)
   {
       this.World(entity.Name, entity.Start, entity.Finish);
   }

   Sebaliknya, gunakan kelebihan beban . Alasan yang sama: kemampuan menemukan yang lebih baik. Penelepon dapat melihat sekaligus semua kelebihan beban melalui IntelliSense dan memilih yang benar.

3. Metode hanya memanggil metode publik lainnya tanpa menambahkan nilai substansial.

   Berpikir dua kali. Apakah Anda benar-benar membutuhkan metode ini? Atau haruskah Anda menghapusnya dan membiarkan penelepon memanggil metode yang berbeda? Petunjuk: jika nama metode ini kelihatannya tidak benar atau sulit ditemukan, Anda mungkin harus menghapusnya.

4. Metode memvalidasi input dan kemudian memanggil metode lain:

   Hello(string name, int start, int end)
   {
       if (name == null) throw new ArgumentNullException(...);
       if (start < 0) throw new OutOfRangeException(...);
       ...
       if (end < start) throw new ArgumentException(...);
   
       this.World(name, start, end);
   }

   Sebaliknya, Worldharus memvalidasi parameternya sendiri, atau bersifat pribadi.

Jika sesuatu bersifat publik, itu dapat dipanggil kapan saja oleh sistem apa pun. Tidak ada alasan Anda tidak bisa menjadi salah satu dari sistem itu juga!

Di perpustakaan yang sangat optimal, Anda ingin menyalin idiom yang diajukan java.util.ArrayList#ensureCapacity(int).

PastikanKapasitas

• ensureCapacity bersifat publik
• ensureCapacity memiliki semua batas yang diperlukan memeriksa dan nilai default, dll.
• ensureCapacity panggilan ensureExplicitCapacity

memastikanKapasitasInternal

• ensureCapacityInternal bersifat pribadi
• ensureCapacityInternal memiliki pengecekan kesalahan minimal karena semua input berasal dari dalam kelas
• ensureCapacityInternal Panggilan JUGA ensureExplicitCapacity

memastikanKapasitas Eksplisit

• ensureExplicitCapacity adalah JUGA pribadi
• ensureExplicitCapacitytidak memiliki pengecekan kesalahan
• ensureExplicitCapacity melakukan pekerjaan yang sebenarnya
• ensureExplicitCapacitytidak dipanggil dari mana pun kecuali oleh ensureCapacitydanensureCapacityInternal

Dengan cara ini, kode yang Anda percayai mendapat akses istimewa (dan lebih cepat!) Karena Anda tahu inputnya baik. Kode yang tidak Anda percayai melewati kekakuan tertentu untuk memverifikasi integritasnya dan membom atau memberikan default atau menangani input yang buruk. Keduanya menyalurkan ke tempat yang benar-benar berfungsi.

Namun, ini digunakan di ArrayList, salah satu kelas yang paling banyak digunakan di JDK. Sangat mungkin kasing Anda tidak memerlukan tingkat kerumitan dan kekakuan itu. Singkat cerita, jika semua ensureCapacityInternalpanggilan diganti dengan ensureCapacitypanggilan, kinerjanya masih akan sangat, sangat bagus. Ini adalah optimasi mikro mungkin hanya dilakukan setelah pertimbangan luas.

Beberapa jawaban baik telah diberikan, dan saya juga akan setuju dengan mereka bahwa ya, suatu objek dapat memanggil metode publiknya dari metode lainnya. Namun, ada sedikit peringatan desain yang harus Anda perhatikan.

Metode publik biasanya memiliki kontrak "mengambil objek dalam keadaan konsisten, melakukan sesuatu yang masuk akal, meninggalkan objek dalam keadaan konsisten (mungkin berbeda)". Di sini, "konsisten" dapat berarti, misalnya, bahwa Lengtha List<T>tidak lebih besar dari itu Capacitydan yang mereferensikan unsur-unsur pada indeks dari 0keLength-1 tidak akan melempar.

Tetapi di dalam metode objek, objek mungkin dalam keadaan tidak konsisten, jadi ketika Anda memanggil salah satu metode publik Anda, itu mungkin melakukan hal yang sangat salah, karena itu tidak ditulis dengan kemungkinan seperti itu dalam pikiran. Jadi, jika Anda berencana untuk memanggil metode publik Anda dari metode lain, pastikan bahwa kontrak mereka adalah "mengambil objek di beberapa negara, melakukan sesuatu yang masuk akal, meninggalkan objek dalam (mungkin berbeda) (mungkin tidak konsisten - tetapi hanya jika awal negara tidak konsisten) ".

Contoh lain ketika memanggil metode publik di dalam metode publik lain sama sekali baik-baik saja adalah pendekatan CanExecute / Execute. Saya menggunakannya ketika saya membutuhkan validasi dan pelestarian yang tidak berubah-ubah .

Tetapi secara umum saya selalu berhati-hati tentang itu. Jika suatu metode a()disebut metode dalam b(), itu berarti metode tersebut a()adalah detail implementasi dari b(). Sangat sering itu menunjukkan bahwa mereka termasuk level abstraksi yang berbeda. Dan fakta bahwa mereka berdua bersifat publik membuat saya bertanya-tanya apakah itu merupakan pelanggaran prinsip tanggung jawab tunggal atau tidak.

Maaf, tetapi saya harus tidak setuju dengan sebagian besar jawaban 'ya Anda bisa' dan mengatakan bahwa:

Saya akan mencegah kelas memanggil satu metode publik dari yang lain

Ada beberapa masalah potensial dengan praktik ini.

1: Infinite loop di kelas yang diwarisi

Jadi kelas dasar Anda memanggil method1 dari method2 tetapi kemudian Anda, atau orang lain, mewarisi method1 dan menyembunyikan method1 dengan metode baru yang memanggil method2.

2: Acara, Pencatatan dll.

misal saya punya metode Add1 yang memancarkan suatu peristiwa '1 ditambahkan!' Saya mungkin tidak ingin metode Add10 meningkatkan kejadian itu, menulis ke log atau apa pun, sepuluh kali.

3: threading dan deadlock lainnya

Misalnya, InsertComplexData membuka koneksi db, memulai transaksi, mengunci tabel, Kemudian memanggil InsertSimpleData, dengan membuka koneksi, memulai transaksi, menunggu tabel dibuka ...

Saya yakin ada lebih banyak alasan, salah satu jawaban lain menyentuh 'Anda mengedit metode1 dan terkejut metode2 mulai berperilaku berbeda'

Secara umum, jika Anda memiliki dua metode publik yang berbagi kode, lebih baik membuat keduanya memanggil metode pribadi daripada satu panggilan yang lain.

Edit ----

Mari kita memperluas kasus khusus di OP.

kami tidak memiliki banyak detail tetapi kami tahu bahwa ReverseData dipanggil oleh event handler dari beberapa jenis serta metode ScheduleTransmission.

Saya berasumsi bahwa membalikkan data juga mengubah keadaan internal objek

Mengingat kasus ini, saya akan berpikir bahwa keselamatan utas akan menjadi penting dan karenanya keberatan ketiga saya terhadap praktik ini berlaku.

Untuk membuat thread ReverseData aman, Anda dapat menambahkan kunci. Tetapi jika ScheduleTransmission juga perlu diamankan, Anda ingin berbagi kunci yang sama.

Cara termudah untuk melakukan ini adalah dengan memindahkan kode ReverseData ke metode pribadi dan kedua metode publik menyebutnya. Anda kemudian dapat meletakkan pernyataan kunci di Metode Publik dan berbagi objek kunci.

Jelas Anda dapat berdebat "itu tidak akan pernah terjadi!" atau "Saya bisa memprogram kunci dengan cara lain" tetapi poin tentang praktik pengkodean yang baik adalah untuk menyusun kode Anda dengan baik sejak awal.

Dalam istilah akademis saya akan mengatakan ini melanggar huruf L dalam huruf padat. Metode publik lebih dari sekadar dapat diakses secara publik. Mereka juga dapat dimodifikasi oleh pewarisnya. Kode Anda harus ditutup untuk modifikasi yang berarti Anda harus memikirkan pekerjaan apa yang Anda lakukan baik dalam metode publik maupun yang dilindungi.

Inilah satu lagi: Anda juga berpotensi melanggar DDD. Jika objek Anda adalah objek domain, metode publiknya haruslah istilah Domain yang berarti sesuatu bagi bisnis. Dalam hal ini sangat tidak mungkin bahwa 'beli selusin telur' sama dengan 'beli 1 telur 12 kali' bahkan jika itu dimulai dengan cara itu.