如何复用返回定价信息的代码块？解决迭代器返回错误

问题概述 #

需要复用定价计算逻辑以避免重复代码，当前PriceCalculator方法在return u;处报错：Cannot return a value from an iterator，同时希望明确代码复用的方向。

当前代码 #

控制器代码 #

public async Task<IActionResult> RetailTimePriceDisplay(string zero)
{
    var j = await PriceCalculator(zero); //zero是作业文件ID
    ViewBag.priceTrk = j.priceTrk;
    ViewBag.priceVan = j.priceVan; //将这些价格通过ViewBag传递到视图
    ViewBag.rateTravelDrvr = j.rateTravelDrvr;

    return View();
}

待复用的PriceCalculator方法 #

public async IAsyncEnumerable<PricingIndex> PriceCalculator(string zero)
{
    //调用路线信息
        var i = await BingMapRouteCalculate(zero);
        //实例化定价索引对象
        PricingIndex u = new PricingIndex();
        var j = new PricingIndex();

    //获取作业文件[zero]相关定价
    u.rateMile = _context.ProductList.First(n => n.AccountNumber == 4202).ListPrice;
    decimal rateLabor = _context.ProductList.First(n => n.AccountNumber == 4201).ListPrice;
    //使用BingMaps计算'TRK'价格
    u.priceTrk = (i.TravelDistance * u.rateMile) + 100; 
    //使用BingMaps计算'VAN'价格
    u.priceVan = i.TravelDistance * u.rateMile; 

    //使用BingMaps计算'rateTravelDrvr'
    decimal rateTravelDrvr = Math.Ceiling(i.TravelDuration * rateLabor);

    return u;
}

PricingIndex类 #

public class PricingIndex
{
    public decimal rateMile { get; set; }
    public decimal priceVan { get; set; }
    public decimal priceTrk { get; set; }
    public decimal rateTravelDrvr { get; set; }
}

解决步骤 #

1. 修正返回类型错误：IAsyncEnumerable<T>用于异步返回多个元素的序列，但此处只需要返回单个PricingIndex对象，所以将方法返回类型改为Task<PricingIndex>。
2. 清理冗余代码：删除未使用的PricingIndex j实例化语句。
3. 补全属性赋值：将计算得到的rateTravelDrvr赋值给u.rateTravelDrvr，否则视图无法获取该值。
4. 实现代码复用：将修正后的PriceCalculator封装为独立方法（可放在控制器私有方法中，或提取到专门的定价服务类），在所有需要计算定价的地方直接调用即可。

修正后的PriceCalculator方法 #

public async Task<PricingIndex> PriceCalculator(string zero)
{
    //调用路线信息
    var routeInfo = await BingMapRouteCalculate(zero);
    //实例化定价索引对象
    var pricingResult = new PricingIndex();

    //获取作业文件[zero]相关定价
    pricingResult.rateMile = _context.ProductList.First(n => n.AccountNumber == 4202).ListPrice;
    decimal rateLabor = _context.ProductList.First(n => n.AccountNumber == 4201).ListPrice;
    
    //使用BingMaps计算'TRK'价格
    pricingResult.priceTrk = (routeInfo.TravelDistance * pricingResult.rateMile) + 100; 
    //使用BingMaps计算'VAN'价格
    pricingResult.priceVan = routeInfo.TravelDistance * pricingResult.rateMile; 

    //使用BingMaps计算'rateTravelDrvr'并赋值
    pricingResult.rateTravelDrvr = Math.Ceiling(routeInfo.TravelDuration * rateLabor);

    return pricingResult;
}

搜索关键词建议 #

• C# 异步方法返回单个对象 vs 枚举
• C# 业务逻辑代码复用
• ASP.NET Core 服务层封装
• Cannot return a value from an iterator 错误排查

内容的提问来源于stack exchange，提问作者Nick Fleetwood