C#可选参数时重载

结论先说

可以构成合法重载，同一命名空间下同时放两份代码，编译完全没问题，不需要改名。

但有一处非常隐蔽、容易踩坑的调用歧义问题，我给你拆开讲明白。

1、C#重载判定规则

两个方法算作重载，判定依据：参数列表（数量、类型、顺序）不一样；返回值、out参数名称不算在内。

对比两个签名：

复制代码

//版本1
TraceAllstream(Point3d, STRtree<...>, STRtree<...>, out HashSet, out HashSet)

//版本2
TraceUpstream(Point3d, STRtree<...>, STRtree<...>, out HashSet, out HashSet, Func<ObjectId,bool> pipeFilter = null)

⚠️等等：方法名都不一样！一个叫 TraceAllstream，一个叫 TraceUpstream。

名字不同 → 根本就不是重载关系！是两个完全独立、互不干扰的函数。

2、如果你本意是：给 TraceUpstream 做重载

也就是两个都叫 TraceUpstream：

复制代码

//A版 无过滤
TraceUpstream(Point3d, STRtree<...>, STRtree<...>, out HashSet, out HashSet)

//B版 带可选委托
TraceUpstream(Point3d, STRtree<...>, STRtree<...>, out HashSet, out HashSet, Func<ObjectId, bool> pipeFilter = null)

✅ 这一对，属于合法重载，编译通过，可以共存。

但是！隐藏大坑 ------ 可选参数带来的歧义陷阱

你调用：

复制代码

TraceUpstream(pt, tree1, tree2, out lines, out points);

编译器可以匹配到A版；

也理论上可以匹配B版（省略掉最后一个可选参数 pipeFilter）。

虽然C#编译器在这种场景，优先选择参数列表精确匹配的版本（A版） ，目前不会报编译报错。

但这种写法属于「重载 + 可选参数」的经典高风险组合，后期维护极易出玄学BUG。

风险场景举例

以后你再新增一个重载：

复制代码

TraceUpstream(Point3d, STRtree<...>, STRtree<...>, out HashSet, out HashSet, int someNum=0);

这时无参调用就会出现二义性编译报错，直接红波浪线。

两条稳妥方案（二选一）

方案1（推荐）：只保留一个 TraceUpstream 方法

就留带可选委托那一个版本，删掉不带过滤的副本。

复制代码

public static void TraceUpstream(
    Point3d startPoint,
    STRtree<LineEndIndexItem> lineEndTree,
    STRtree<PointIndexItem> pointTree,
    out HashSet<ObjectId> upstreamLines,
    out HashSet<ObjectId> upstreamPoints,
    Func<ObjectId, bool> pipeFilter = null)

原来所有老的调用代码一行不用改；不传过滤函数，pipeFilter=null，内部不走过滤逻辑，等价原版功能。
没有重载冲突、没有二义性隐患，维护最简单。

这也是最适合你当前拓扑回溯代码的做法。

方案2：保留重载，但不要用可选参数

如果你硬要两个重载共存，把后面那个的 =null 默认值删掉：

复制代码

//版本1：无过滤
TraceUpstream(Point3d, STRtree<...>, STRtree<...>, out HashSet, out HashSet);

//版本2：强制必须传过滤委托，不允许省略
TraceUpstream(Point3d, STRtree<...>, STRtree<...>, out HashSet, out HashSet, Func<ObjectId, bool> pipeFilter);

这样编译器永远不会产生匹配歧义，安全。调用不需要过滤时就调用第一个。

回到你贴出来那两个函数名字

TraceAllstream 和 TraceUpstream：

二者名称不一样，互不影响，不存在任何重载、冲突问题，放心共存。

如果你是笔误，两个都打算命名为 TraceUpstream，就优先选择方案1：单方法 + 可选委托，抛弃重载副本，最省心避坑。

先说核心结论

C# 编译不会报错，能跑；但是确实有隐性风险，属于不推荐的写法。
C++ 和 C# 的规则不一样，这就是为什么 C++ 这种写法直接二义性报错，而 C# 默默放过了你。

一、先对比 C++ vs C# 的区别

C++

复制代码

void Foo(int a);
void Foo(int a, std::function<bool(int)> filter = nullptr);
Foo(1); // 直接编译报错，二义性

C++ 重载决议：带默认参数的函数，在决议阶段就视作可行候选 ，两个函数都能匹配调用 Foo(1)，直接报歧义。

C#

复制代码

void Foo(int a);
void Foo(int a, Func<bool> filter = null);
Foo(1);

C#重载匹配优先级规则：

优先选择「精确匹配参数数量」的重载。

调用时不给第3个参数，A版（5个参数）参数数量完全对上 → 直接选中A；B版虽然可以省略可选参数，但是优先级更低，不会被选。

👉 所以现在你的代码当下不会出错。

二、但是风险真实存在（3个坑）

坑1：未来再加一个重载，立刻引爆二义性编译错误

假设以后新增：

复制代码

//C版
TraceUpstream(Point3d, STRtree<...>, STRtree<...>, out HashSet, out HashSet, int xxx=0);

现在调用不带第6参数：

复制代码

TraceUpstream(pt, tree1,tree2, out lines,out pts);

候选列表：B版(带Func可选)、C版(带int可选)；两个都可以省略最后一个参数，没有精确匹配版本可以选。

→ 编译器分不清选谁，直接编译报错：调用具有二义性 。

一旦出现，你所有地方都得改。

坑2：重构、反射、动态调用很容易踩坑

如果你后面用反射去调用这个方法，反射并不遵守「精确参数优先」这套重载决议；

你获取方法的时候极有可能绑定到带可选参数的版本，引发意料之外的行为。

坑3：可读性隐患，团队协作容易被误导

别人维护代码，看到两个同名方法，一个5参、一个6参带默认值，第一反应就是重载；

很容易误以为：调用5参版本会自动走6参版本并传入null（事实并不是）。

三、三种整改方案，按推荐顺序

✅方案1（最优，根治所有风险）：只保留唯一一个带可选参数版本，删掉A版无过滤重载

复制代码

public static void TraceUpstream(
    Point3d startPoint,
    STRtree<LineEndIndexItem> lineEndTree,
    STRtree<PointIndexItem> pointTree,
    out HashSet<ObjectId> upstreamLines,
    out HashSet<ObjectId> upstreamPoints,
    Func<ObjectId, bool> pipeFilter = null)

老调用代码完全不用改，不传过滤函数，pipeFilter就是null，内部不执行过滤，等价原版A功能。

没有重载，就永远不存在二义性隐患，也是拓扑回溯这类函数最标准的扩展方式。

✅方案2：保留两个重载，去掉B版的默认值 =null（禁止省略）

复制代码

//A版：无过滤
public static void TraceUpstream(
    Point3d startPoint,
    STRtree<LineEndIndexItem> lineEndTree,
    STRtree<PointIndexItem> pointTree,
    out HashSet<ObjectId> upstreamLines,
    out HashSet<ObjectId> upstreamPoints);

//B版：带过滤，必须手动传委托，不能省略
public static void TraceUpstream(
    Point3d startPoint,
    STRtree<LineEndIndexItem> lineEndTree,
    STRtree<PointIndexItem> pointTree,
    out HashSet<ObjectId> upstreamLines,
    out HashSet<ObjectId> upstreamPoints,
    Func<ObjectId, bool> pipeFilter);

调用过滤版本必须显式传参：

复制代码

TraceUpstream(pt, t1, t2, out lines, out pts, myFilter);

这样两个重载参数数量不一样，调用的时候意图清晰，永远不会出现歧义。

❌方案3：维持现状不动（不推荐）

短期能用，但埋下未来版本炸弹；后续新增重载时，二义性问题就会爆发。

简短总结

当前：A(5参) + B(6参+可选参数)，运行没问题，C#不会报错；
本质：属于「带可选参数的重载」，是一个定时炸弹，不是正确稳健的设计；
拓扑回溯工具类，首选：单方法 + 可选委托参数，删掉无过滤重载版本。