Apakah cara memanggil fungsi ini merupakan praktik yang buruk?

Saya memiliki kode berikut:

public void moveCameraTo(Location location){
    moveCameraTo(location.getLatitude(), location.getLongitude());
}

public void moveCameraTo(double latitude, double longitude){
    LatLng latLng = new LatLng(latitude, longitude);
    moveCameraTo(latLng);
}

public void moveCameraTo(LatLng latLng){
    GoogleMap googleMap =  getGoogleMap();
    cameraUpdate = CameraUpdateFactory.newLatLngZoom(latLng, INITIAL_MAP_ZOOM_LEVEL);
    googleMap.moveCamera(cameraUpdate);
}

Saya berpikir bahwa dengan cara ini saya menghilangkan tanggung jawab untuk mengetahui apa yang ada LatLngdi kelas lain, misalnya.

Dan Anda tidak perlu menyiapkan data sebelum memanggil fungsi.

Bagaimana menurut anda?

Apakah pendekatan ini memiliki nama? Apakah ini benar-benar praktik yang buruk?

Anda menggunakan fitur kelebihan metode bahasa untuk menawarkan cara alternatif penelepon untuk menyelesaikan ketergantungan metode pada informasi posisi. Anda kemudian mendelegasikan ke metode lain untuk menyelesaikan pekerjaan memperbarui kamera yang tersisa.

Aroma kode di sini adalah jika Anda terus memperluas rangkaian metode memanggil metode. Metode pengambilan lokasi memanggil metode pengambilan ganda yang memanggil metode pengambilan latLng yang akhirnya memanggil sesuatu yang tahu cara memperbarui kamera.

Rantai panjang hanya sekuat rantai terlemah mereka. Setiap ekstensi rantai meningkatkan jejak kode yang harus bekerja atau hal ini rusak.

Ini adalah desain yang jauh lebih baik jika setiap metode mengambil jalan sesingkat mungkin untuk menyelesaikan masalah yang telah disajikan kepadanya. Itu tidak berarti masing-masing harus tahu cara memperbarui kamera. Masing-masing harus menerjemahkan jenis parameter posisi mereka ke satu jenis seragam yang dapat diteruskan ke sesuatu yang tidak tahu cara memperbarui kamera ketika disajikan jenis yang satu ini.

Lakukan seperti itu dan Anda dapat menghapus satu tanpa merusak setengah dari yang lainnya.

Mempertimbangkan:

public void moveCameraTo(Location location){
    moveCameraTo( new LatLng(location) );
}

Ini membuat berurusan dengan masalah garis lintang dan bujur LatLng. Biaya penyebarannya adalah pengetahuan LatLngsekitar. Itu mungkin tampak mahal tetapi dalam pengalaman saya itu adalah alternatif yang lebih disukai daripada obsesi primitif yang adalah apa yang menghindari membangun objek parameter membuat Anda terjebak.

Jika Locationdapat dilakukan kembali tetapi LatLngtidak, pertimbangkan untuk menyelesaikan ini dengan menambahkan pabrik ke Location:

moveCameraTo( location.ToLatLng() );

Ini juga menghindari obsesi primitif.

Tidak ada yang salah dengan solusi Anda.

Tetapi preferensi pribadi saya adalah bahwa metode-metode itu tidak berguna. Dan hanya menyulitkan antarmuka objek apa pun yang menjadi bagiannya.

Tidak void moveCameraTo(double latitude, double longitude)benar-benar menyederhanakan kode, karena saya tidak melihat masalah hanya memanggil moveCameraTo(new LatLng(latitude, longitude));tempat itu. Metode ini juga berbau obsesi primitif.

Itu void moveCameraTo(Location location)bisa lebih baik diselesaikan dengan membuktikan Location.ToLatLng()metode dan panggilan moveCameraTo(location.ToLatLng()).

jika ini adalah C # dan jika metode seperti itu benar-benar diperlukan, saya lebih suka mereka sebagai metode ekstensi daripada metode contoh. Penggunaan metode ekstensi akan menjadi sangat jelas jika Anda mencoba abstrak dan uji unit contoh ini. Karena akan jauh lebih mudah untuk hanya memalsukan metode tunggal daripada banyak kelebihan dengan konversi sederhana.

Saya pikir dengan cara ini saya menghilangkan tanggung jawab untuk mengetahui apa itu LatLng di kelas lain, misalnya.

Saya tidak melihat alasan mengapa ini akan menjadi masalah. Selama kode Anda merujuk kelas yang berisi void moveCameraTo(LatLng latLng), itu masih secara tidak langsung bergantung pada LatLng. Bahkan jika kelas itu tidak pernah langsung dipakai.

Dan Anda tidak perlu menyiapkan data sebelum memanggil fungsi.

Saya tidak mengerti maksud Anda. Jika itu berarti membuat instance baru atau mengubah kelas dari satu ke yang lain, saya tidak melihat masalah dengan itu.

Memikirkan hal itu, saya merasa bahwa apa yang saya katakan juga didukung oleh desain API. NET itu sendiri. Secara historis, banyak kelas .NET mengikuti pendekatan Anda karena memiliki banyak kelebihan dengan parameter berbeda, dan konversi sederhana di dalamnya. Tapi itu sebelum metode penyuluhan ada. Kelas .NET yang lebih modern lebih ringan di API mereka sendiri dan jika ada metode dengan kelebihan parameter, mereka disediakan sebagai metode ekstensi. Contoh yang lebih lama adalah NLog ILogger yang memiliki lusinan beban berlebih untuk menulis log. Bandingkan dengan Microsoft.Extensions.Logging.ILogger yang lebih baru yang memiliki total 3 metode (dan hanya 1 jika Anda menghitung pencatatan sendiri). Tetapi ada banyak pembantu dan berbagai parameter sebagai metode ekstensi .

Saya pikir jawaban ini menunjukkan bahwa beberapa bahasa akan memiliki alat untuk membuat desain seperti ini lebih bagus. Saya tidak tahu banyak tentang Java, jadi saya tidak yakin apakah akan ada yang setara. Tetapi bahkan menggunakan metode statis biasa mungkin bisa menjadi pilihan.

Saya tidak yakin apa nama yang tepat untuk ini, jika ada, tetapi itu adalah praktik yang baik. Anda mengekspos beberapa kelebihan, memungkinkan kelas panggilan untuk menentukan parameter yang mengaturnya ingin digunakan. Seperti yang Anda katakan, kelas lain mungkin tidak tahu apa LatLngobjek itu, tetapi mungkin tahu itu Location.

Memiliki satu metode panggilan yang lain juga penting, karena Anda tidak ingin kode duplikat di seluruh metode tersebut. Seperti yang telah Anda lakukan, pilih satu metode untuk menjadi yang melakukan pekerjaan, dan mintalah metode lain menyebutnya (langsung atau tidak langsung)

Jika Anda keberatan menggunakan jenis metode, itu hanya fitur metode overload yang bagus.

Tetapi tidak ada menghilangkan tanggung jawab untuk mengetahui apa itu LatLng . Karena Anda menginisialisasi LatLng latLng = new LatLng(latitude, longitude). Ini benar-benar ketergantungan kepada LatLng. (Untuk memahami mengapa Inisialisasi adalah masalah ketergantungan, Anda dapat memeriksa Dependency Injection ) Membuat metode kelebihan beban hanya membantu klien yang tidak peduli LatLng. Jika Anda maksudkan ini, itu juga bagus tapi saya rasa itu bukan pendekatan. Ini hanya banyak metode layanan untuk klien.

Jadi, ada dua opsi untuk mendesain arsitektur Anda:

1. Buat banyak metode kelebihan beban dan berikan kepada klien.
2. Buat beberapa metode kelebihan beban yang membutuhkan parameter sebagai antarmuka atau kelas beton.

Saya lari sejauh mungkin dari menciptakan metode yang membutuhkan tipe primitif sebagai parameter (Opsi 1). Karena jika bisnis Anda berubah banyak waktu dan Anda perlu memainkan parameter metode, sangat sulit untuk mengubah dan mengimplementasikan semua fungsi pemanggil.

Alih-alih ini, gunakan antarmuka (Injeksi Ketergantungan). Jika Anda pikir itu biaya dan membutuhkan waktu lebih lama, maka gunakan kelas dan berikan metode ekstensi mapper mereka (Opsi 2).