Edge-home-orchestration-go: [MNEDC] Refactoring dalam hal Arsitektur

Dibuat pada 16 Nov 2020  ·  5Komentar  ·  Sumber: lf-edge/edge-home-orchestration-go

Saya pikir fungsi mnedcmgr harus berada di bawah discoverymgr karena mereka mirip dalam hal fungsionalitas.
Bagaimana dengan memindahkan kode mnedcmgr di bawah discoverymgr?
Di bawah ini harus diubah.

  • [x] Ubah nama paket dari mnedcmgr menjadi mnedc atau semacamnya.
  • [ ] Discovermgr harus memanggil StartMNEDCServer() dan StartMNEDCClient().

Setiap komentar dipersilakan!

question refactoring

Komentar yang paling membantu

@t25kim @MoonkiHong : Kami memeriksa modifikasi ini. Tampaknya melibatkan pengimporan siklis yang tidak didukung oleh Go. Jadi kami memeriksa perubahan yang sama dan pengujian.

Semua 5 komentar

@ayush-kr @suresh-lc Bagaimana dengan pendapat Anda (dari sudut pandang kontributor)?

Saya pikir fungsi mnedcmgr harus berada di bawah discoverymgr karena mereka mirip dalam hal fungsionalitas.
Bagaimana dengan memindahkan kode mnedcmgr di bawah discoverymgr?
Di bawah ini harus diubah.

  • [ ] Ubah nama paket dari mnedcmgr menjadi mnedc atau semacamnya.
  • [ ] Discovermgr harus memanggil StartMNEDCServer() dan StartMNEDCClient().

Setiap komentar dipersilakan!

@t25kim Terima kasih atas saran yang bagus. Kami akan memperkirakan kelayakannya untuk memigrasikan fungsi mnedcmgr menjadi discoverymgr dengan memperhatikan kontributor mnedc .

@t25kim @MoonkiHong : Kami memeriksa modifikasi ini. Tampaknya melibatkan pengimporan siklis yang tidak didukung oleh Go. Jadi kami memeriksa perubahan yang sama dan pengujian.

@t25kim @MoonkiHong : Kami memeriksa modifikasi ini. Tampaknya melibatkan pengimporan siklis yang tidak didukung oleh Go. Jadi kami memeriksa perubahan yang sama dan pengujian.

@suresh-lc Terima kasih atas analisis cepat Anda. Seperti yang Anda sarankan, mari kita bahas lebih lanjut tentang pendekatan terbaik setelah Anda menyelesaikan pemeriksaan untuk mengubah hal yang sama dan menguji, terutama tentang _pengimporan siklus yang tidak didukung oleh GoLang_. Kami akan menunggu pembaruan Anda. ^^

Masalah ini masih berlaku sejak PR #178 gabungan terbaru diselesaikan Change package name from mnedcmgr to mnedc or something. .

Apakah halaman ini membantu?
0 / 5 - 0 peringkat