Pertanyaan yang diberi tag «code-smell»

Saya seorang insinyur listrik dan saya tidak tahu apa yang saya lakukan. Harap simpan pengelola kode saya di masa mendatang. Baru-baru ini saya telah mengerjakan sejumlah program yang lebih kecil (dalam C #) yang fungsinya secara logis "prosedural". Sebagai contoh, salah satunya adalah program yang mengumpulkan informasi dari database yang …

12 c#  object-oriented  code-quality  code-smell  procedural 

Kami memiliki sistem mengerikan yang ditulis dalam ASP.NET 2.0 sehingga kami perlu menambahkan beberapa fungsionalitas. Masalahnya adalah bahwa suatu produk tertentu memiliki fitur UI yang harus dinyalakan untuk bisnis yang dimulai setelah tanggal tertentu (dan yang lainnya dimatikan), sementara halaman tersebut harus tampak sama untuk bisnis yang sudah ada. Saya …

11 anti-patterns  code-smell  ui 

Saya membaca beberapa posting blog pagi ini, dan menemukan ini : Jika satu-satunya kelas yang pernah mengimplementasikan antarmuka Pelanggan adalah CustomerImpl, Anda tidak benar-benar memiliki polimorfisme dan substitusi karena pada praktiknya tidak ada yang menggantikan pada saat runtime. Itu adalah generalitas palsu. Itu masuk akal bagi saya, karena mengimplementasikan antarmuka …

11 design  unit-testing  code-smell  polymorphism 

Saya sudah melakukan TDD selama setahun sekarang, saya merasa cukup baik tentang itu, saya suka suite tes saya dan semuanya. Namun saya perhatikan bahwa belakangan ini saya telah melakukan banyak verifikasi panggilan palsu. Sebagai contoh, saya akan memiliki Layanan yang akan memiliki Repositori disuntikkan - dalam pengujian unit saya, saya …

11 unit-testing  tdd  code-smell  anti-patterns 

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 …

10 design  design-patterns  clean-code  code-smell  bad-code 

Saya telah mewarisi beberapa kode buruk yang telah saya sertakan contoh singkat di bawah ini. Apakah ada nama untuk pola khusus ini? Apa sajakah rekomendasi untuk refactoring ini? // 0=Need to log in / present username and password // 2=Already logged in // 3=Inactive User found // 4=Valid User found-establish …

10 coding-style  code-quality  coding-standards  code-reviews  code-smell 

Jadi saya baru-baru ini membuat beberapa refactoring utama pada kode saya. Salah satu hal utama yang saya coba lakukan adalah membagi kelas saya menjadi objek data dan objek pekerja. Ini terinspirasi, antara lain, oleh bagian dari Kode Bersih ini : Hibrida Kebingungan ini kadang-kadang menyebabkan struktur data hibrida yang tidak …

10 java  object-oriented-design  data-structures  clean-code  code-smell 

Apa saja gejala dalam basis kode yang menunjukkan bahwa pendekatan event-listener diperlukan? Tampak bagi saya bahwa ketika ada kelas yang perlu dipanggil dengan banyak, tidak didefinisikan pada set waktu desain kelas lain, Anda memerlukan semacam kerangka kerja pensinyalan, tetapi saya ingin mendengar situasi apa yang ada yang akan ditingkatkan dengan …

10 architecture  design-patterns  code-smell 

Saya memiliki satu set fungsi tujuan tunggal yang saya butuhkan dalam dua pengontrol terpisah. Saat ini saya hanya memiliki kode duplikat dan saya ingin menyingkirkannya. Kode ini adalah bagian dari pengontrol dan tidak termasuk dalam lapisan layanan saya. Di mana Anda akan meletakkannya?

10 mvc  code-smell 

Saya membaca The Art of Unit Testing oleh Roy Osherove. Saya di bagian 7.2. Menulis tes yang dapat dipertahankan di mana penulis memiliki catatan tentang bau kode ini: CATATAN: Saat Anda menyatakan status internal agar terlihat oleh tes luar, apakah itu dapat dianggap sebagai bau kode (tanda bahwa ada sesuatu …

9 unit-testing  code-quality  code-smell 

Dalam aplikasi Delphi 2007 kami, kami menggunakan banyak konstruksi berikut FdmBasic:=TdmBasicData(FindOwnerClass(AOwner,TdmBasicData)); FindOwnerClass melakukan perjalanan hierarki pemilik komponen saat ini ke atas untuk menemukan kelas tertentu (dalam contoh TdmBasicData). Objek yang dihasilkan disimpan dalam variabel Field FdmBasic. Kami menggunakan ini terutama untuk melewatkan datamodules. Contoh: Saat membuat laporan, data yang dihasilkan …

9 code-smell 

Hari ini di tempat kerja, salah seorang kolega saya meninjau kode saya, dan menyarankan agar saya menghapus properti hanya set dan menggunakan metode sebagai gantinya. Ketika kami berdua sibuk dengan hal-hal lain, dia menyuruh saya untuk melihat Property Designbagian dari buku "Kerangka Desain Pedoman". Dalam buku itu penulis hanya berkata …

9 design  code-quality  code-reviews  code-smell 

Operator null-penggabungan dalam c # memungkinkan Anda untuk mempersingkat kode if (_mywidget == null) return new Widget(); else return _mywidget; Ke: return _mywidget ?? new Widget(); Saya terus menemukan bahwa operator yang berguna yang ingin saya miliki di C # akan menjadi salah satu yang memungkinkan Anda untuk mengembalikan properti …

9 c#  code-smell  null 

Tutup. Pertanyaan ini di luar topik . Saat ini tidak menerima jawaban. Ingin meningkatkan pertanyaan ini? Perbarui pertanyaan sehingga sesuai topik untuk Rekayasa Perangkat Lunak Stack Exchange. Ditutup 4 tahun yang lalu . Proyek perangkat lunak yang sedang saya kerjakan melibatkan saya dan programmer lain. Proyek ini melibatkan backend mesin …

8 code-smell 

Saya memiliki tiga sensor dalam satu paket yang semuanya harus dikalibrasi yang akan saya sebut sens1, sens2, dan sens3. Kalibrasi untuk sens1 dan sens2 identik, tetapi kalibrasi untuk sens3 membutuhkan parameter tambahan. Pertanyaan saya adalah, "Apa cara terbaik untuk menangani tiga objek yang hampir identik dengan tetap mempertahankan keterbacaan?" Pikiran …

8 code-smell  polymorphism