Sonar警告：将unique_ptr引用替换为原始指针或引用的疑问

首先明确：Sonar的这个警告点得很准，传递const std::unique_ptr<T>&作为函数参数，绝大多数时候都是没必要的，属于接口设计上的冗余。

为什么不推荐传const unique_ptr<T>&？ #

unique_ptr的核心职责是管理对象所有权，它的存在是为了明确“谁负责释放对象”。如果你的runQuery函数只是要调用N1qlClientProxy的方法，完全不涉及所有权转移、指针重置这类操作，那传递unique_ptr的引用就属于“暴露不必要的实现细节”——别人看到这个参数，会下意识疑惑：这个函数会不会修改指针？会不会要接管所有权？反而模糊了函数的真实意图。

应该怎么改？优先传对象引用 #

如果runQuery不需要处理空指针（也就是调用方保证unique_ptr一定非空），直接把函数参数改成const hri::search::n1ql::N1qlClientProxy&，调用时传*your_unique_ptr即可。这种方式语义最清晰：函数只需要对这个对象做只读访问，和所有权半毛钱关系没有。

如果函数需要处理空指针的情况（调用方可能传空的unique_ptr），那可以把参数改成const hri::search::n1ql::N1qlClientProxy*，调用时传your_unique_ptr.get()，函数内部先检查指针是否为空再操作。

什么时候传const unique_ptr<T>&是合理的？ #

只有当函数确实需要操作unique_ptr本身的时候才用，比如：

• 函数需要重置这个指针（比如把它换成另一个对象）
• 函数需要把指针的所有权转移到另一个unique_ptr（不过这种情况通常用非const引用或者直接移动unique_ptr）
• 函数需要检查指针是否为空，同时后续可能要修改指针（但这种场景也可以用非const指针代替）

但你的runQuery是接收const引用，说明连修改指针的权限都没有，那传unique_ptr引用就更没意义了——既不能改指针，又暴露了所有权细节，纯粹多此一举。

总结 #

直接改runQuery的参数类型，换成对象的const引用（或原始指针，如果需要处理空值），调用时解引用unique_ptr传递。Sonar的警告不是吹毛求疵，是帮你优化接口的语义清晰度，降低代码耦合。

内容的提问来源于stack exchange，提问作者Hani Gotc Reinstate Monica