写代码时，修改传入参数值的这种行为可取吗？

Question

修改前代码

public IActionResult GetOrders(OrderSearch model)
{
    if (ModelState.IsValid)
    {               
        model.UserId = HttpContextRepository.GetUserId(HttpContext);
        return Json(ServiceIns.GetOrders(model));
    }
    return Json(new Response<string> { ErrMsg = "查询参数不规范" });
}

修改后代码(通过viewmodel方式解决)

class OrderViewModel
{
    public string UserId {get;set;}
}

public IActionResult GetOrders(OrderSearch model)
{
    if (ModelState.IsValid)
    {
        var orderViewModel= new OrderViewModel{ UserId = HttpContextRepository.GetUserId(HttpContext) };
        return Json(ServiceIns.GetOrders(model,orderViewModel));
    }
    return Json(new Response<string> { ErrMsg = "查询参数不规范" });
}

Answer 1

单看这么几行代码无法回答这个问题。
有几个原则楼主可以参考下：
1、增加代码重用性
2、降低代码耦合度，将易变化的部分跟不易变化的部分分离
3、增加可测试性（容易单元测试）
4、增加易读性