如何解决C#中的System.StackOverflowException？求优化方案

嘿，这问题太典型了——你这是无限递归搞出来的栈溢出！咱们来拆解一下：

你看你的代码里，GetData方法会调用GetItemDetails，而GetItemDetails又反过来调用GetData，这就形成了一个无限循环的调用链。每一次方法调用都会在栈上创建一个调用帧，随着调用次数越来越多，栈的内存被耗尽，直接触发System.StackOverflowException。而且要提醒你：这个异常是无法被catch块捕获的，CLR在检测到栈溢出时会直接终止进程，所以你的try-catch在这里根本起不到作用。

怎么解决？核心是打破循环 #

首先得明确两个方法的职责：

• GetData的目标是返回MetricItemDetail对象
• GetItemDetails的目标是返回具体的double数值

你需要让GetItemDetails直接去获取那个数值，而不是回头调用GetData。比如把GetItemDetails改成直接查询数据源或者计算数值：

// 修正后的GetItemDetails：直接获取数值，不再调用GetData
public double GetItemDetails(int itemId, int itemTwoId) {
    // 这里写实际获取double值的逻辑，比如从数据库、缓存或者计算得到
    // 举个例子：
    return Database.QueryValue("SELECT value FROM metrics WHERE id = @p1 AND twoId = @p2", itemId, itemTwoId);
}

// GetData保持调用GetItemDetails的逻辑，现在就不会循环了
public MetricItemDetail GetData(int itemId, int itemTwoId) {
    // 你的其他前置代码...
    try {
        var value = GetItemDetails(itemId, itemTwoId);
        return new MetricItemDetail { Value = value }; // 假设你的Class实际是MetricItemDetail，这里调整为对应类名
    } catch (Exception e) {
        Console.WriteLine("{0} Exception caught.", e);
        return new MetricItemDetail();
    }
}

更优的实现建议 #

• 单一职责原则：每个方法只做一件事，GetData专注于组装业务对象，GetItemDetails专注于获取数值，避免职责交叉导致的循环依赖。
• 抽取共享逻辑：如果两个方法有重复的逻辑，比如参数校验、数据源连接，可以把这些逻辑抽成一个私有辅助方法，比如private double FetchRawValue(int itemId, int itemTwoId)，然后两个方法都调用这个辅助方法，而不是互相调用。
• 避免无意义递归：递归适合处理树遍历、分治算法等场景，这种业务逻辑的循环调用完全是逻辑错误，一定要避免。

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