c# - 怎么写才好看？

Question

嗨，我是 c# 的新手，我想问一下如何更漂亮地编写这段代码

public void Update(Product pr)
    {
        Product prod = GeProductById(pr.ProductID);
        prod.Name       = pr.Name;
        prod.Count      = pr.Count;
        prod.InputPrice = pr.InputPrice;
        prod.InputDate  = pr.InputDate;
        prod.OutPrice   = pr.OutPrice;
        prod.InputPriceByCurrency   = pr.InputPriceByCurrency;
        prod.InputPriceCurrency     = pr.InputPriceCurrency;
        prod.ComeOwner  = pr.ComeOwner;
        prod.GroupID    = pr.GroupID;
        prod.Discount   = pr.Discount;

        _context.SubmitChanges();
    }

它所做的只是复制除 id 之外的所有属性。可以短时间写吗？谢谢，对不起我的英语不好

Answer 1

这似乎还有更多。为什么不能将 pr 按原样保存到数据存储中？看起来您使用 ORM 的效率可能很低，尽管可能有充分的理由我们只是无法从这段代码的范围中看到这一点。

更笼统地说，你可能会幸运地得到类似 automapper 的东西在这种情况下。不过，老实说，我认为只要对您使用 ORM 的方式进行一些重新设计，这里就没有必要了。

根据目前为止的评论，我还想指出，像这样冗长乏味的映射函数本质上并没有错。即使它是 20 个属性或 200 个属性。如果它只写了一次，很容易阅读和理解它在做什么，做它正在做的事情等等，那么它就不是一段糟糕的代码。代码并不总是很漂亮，也不需要如此。很容易想做一些更聪明的事情来让代码更酷或更有趣，但在以后支持该代码时，聪明并不总是更好。