Kapan lebih disukai menggabungkan fungsi Tambah / Edit, dan kapan memisahkannya?

Saya secara teratur menemukan situasi di mana saya perlu Menambah atau Mengedit item, dan kadang-kadang saya menggunakan metode terpisah untuk Tambah dan Edit, dan di lain waktu saya menggabungkannya menjadi satu metode.

Apakah satu metode lebih disukai daripada yang lain? Jika demikian, mengapa?

public void AddItem()
{
    ShowEditingPopup(new Item(), "Add Item");
}

public void EditItem(Item item)
{
    ShowEditingPopup(item, "Edit Item");
}

ATAU

public void EditItem(Item item)
{
    ShowEditingPopup(
        (item ?? new Item()), 
        string.format("{0} Item", (item == null ? "Add " : "Edit "))
    );
}

dimana ShowEditingPopupdidefinisikan sebagai

public void ShowEditingPopup(object popupDataContext, string popupTitle)
{
    PopupDataContext = popupDataContext;
    PopupTitle = popupTitle;
    IsPopupVisible = true;
}

Sunting: Hanya untuk memperjelas, saya tidak Menyimpan item, saya membukanya untuk diedit. Saya hampir selalu menerapkan metode Simpan generik untuk menyimpan ke database

Sunting # 2: Sampel kode yang diedit sehingga lebih akurat mencerminkan jenis situasi yang saya maksud

Pikirkan konteks Anda sejenak ... Tidak ada perbedaan antara Menambahkan elemen baru atau mengedit yang sudah ada. Kamu hanya SaveChanges(). Status elemen individual memberi tahu konteks apakah elemen baru ditambahkan atau elemen yang sudah diedit.

EDIT: Ok ... Nah dalam hal itu, saya punya Aksi Kontrol terpisah (MVC3) untuk setiap tindakan, namun, hanya 1 tampilan ...

Katakanlah saya menambahkan atau mengedit Personentitas. Saya hampir selalu memiliki dua model untuk menangani proses add / edit:

• AddPersonModel
• EditPersonModel

Mereka hampir selalu mewarisi dari kelas dasar abstrak umum:

• AddEditPersonModelBase

Jadi kelas dasar mungkin memiliki Commitmetode abstrak , yang ditimpa oleh kelas turunan untuk memulai Personentitas baru , atau untuk menyimpan perubahan ke Personentitas yang ada .

Logika bisnis yang umum untuk menambahkan dan mengedit masuk dalam kelas dasar (misalnya, untuk mengedit alamat), tetapi selalu ada sedikit logika bisnis yang berbeda jika Anda menambahkan atau mengedit, dan yang masuk dalam subclass. Misalnya, Anda mungkin ingin memberi mereka nomor ID tambahan yang unik, tetapi Anda tidak benar-benar membuatnya sampai Anda menyimpan entitas. Dalam hal ini, hanya Anda yang EditPersonModelmemiliki properti itu.

Demikian pula, Anda akan memiliki Tampilan terpisah (dan ViewModels) untuk menambahkan atau mengedit Person, tetapi masuk akal untuk menambahkan dan mengedit varian yang diwarisi dari kelas dasar yang sama karena banyak yang akan menjadi umum.

Saya pribadi cenderung mengikuti prinsip menjadi semantik . Ini berarti bahwa saya biasanya tidak menyediakan formulir pengeditan all-in-one. Dengan kata lain, pengguna biasanya tidak ingin mengedit seluruh informasi suatu entitas . Sebaliknya proses pengeditan mereka lebih semantik dan dapat diklasifikasikan. Misalnya, untuk entitas pengguna , orang biasanya mengedit kata sandi dan mengedit profil . Jadi, saya membuat dua fungsi untuk diedit seperti:

public bool EditPassword(string userName)
{
    // code here
}

public bool EditProfile(Profile newProfile)
{
    // code here    
}

Atau, pengguna biasanya mengedit artikel dengan cara ini:

public bool PutArticleIntoCategories(int articleId, List<Category> categories)
{

}

public bool EditArticleSummary(int articleId, string newSummary)
{

}

Singkatnya, saya biasanya membuat form pembuatan all-in-one, tapi saya cenderung menggunakan form edit yang kecil untuk setiap operasi edit semantik. Jadi, saya biasanya tidak menggabungkannya. Namun, ini tidak berarti bahwa Anda tidak dapat menggabungkan mereka. Setiap kali Anda ingin memberikan bentuk edit besar dan besar yang menyerupai bentuk kreasi, saya pikir penggabungan dapat bekerja lebih baik.

Ini adalah masalah seberapa baik Anda memahami Prinsip Tanggung Jawab Tunggal . Akan lebih bersih dan lebih mudah untuk membaca, memahami, dan memelihara jika tanggung jawab dipisahkan ke dalam metode / tindakan yang terpisah. Namun, ini dari perspektif pengkodean.

Jika saya harus memilih satu pendekatan dari contoh Anda, saya akan pergi untuk pendekatan pertama .

Perspektif pengguna tergantung pada seberapa mudah menambahkan item baru atau mengedit item yang ada (jumlah klik adalah salah satu cara saya mengukur) dan mungkin untuk mengikuti beberapa aturan emas.

Jika saya harus mengimplementasikan fungsionalitas (UI / Formulir), saya akan memiliki satu formulir ketika menambah dan mengedit keduanya memiliki fetures yang sama, dan dua bentuk terpisah ketika mereka berbeda.