函数返回值传参与直接传参处理数据哪种方式更符合最佳实践？

方案2是更合理的实现，这绝非单纯的个人编码偏好选择，行业内有明确的通用编码最佳实践可以参考，两种方案的性能差异在绝大多数业务场景下完全可以忽略。

通用最佳实践判断依据 #

• 接口设计合理性：方案1本质是反模式的“出参通过入参传递”，要求调用方提前感知函数内部需要多少个数组、每个数组对应什么含义，把函数自身的实现成本转嫁给了所有调用方。如果后续需求调整，需要新增周六、周日的统计维度，或是调整工作日排序规则，方案1需要修改所有调用该函数位置的变量声明、传参顺序，漏改任意一处就会出现参数错位的隐蔽bug；方案2把排序逻辑完全封装在FindDayFromDate内部，调用方只需要拿到聚合后的排序结果即可，后续内部逻辑调整不会影响调用方代码，维护成本低得多。
• 误操作风险控制：你的判断是对的，方案1中5个独立声明的数组变量散落在当前作用域，后续任意代码都可以随意修改、重写这些变量，出问题时排查链路很长；同时多同类型参数挨个传递时，非常容易出现传参顺序写反（比如把tuesday和wednesday的位置传错）的低级错误，这类错误在语法检查阶段根本无法识别。方案2用const声明统一的聚合结果sortedDays，从根源上避免了变量被意外重赋值的问题，也不存在多参数传参顺序错位的风险。
• 代码精简度与可读性：方案1需要额外写5个变量声明，两次重复传递5个参数，冗余代码多；方案2的逻辑是完全线性的：输入历史行情数据→拿到按天排序的聚合结果→计算每日均价，读代码的人不需要反复核对参数顺序，逻辑链路非常清晰。

两种方案的示例代码对比如下：

方案1 #

// Arrays to hold what stock price belongs to what day of the week - Note these arrays are used some way down the line
var monday = [], tuesday = [], wednesday = [], thursday = [], friday = []

// Sort each stock date into a day of the week
FindDayFromDate(historyData, monday, tuesday, wednesday, thursday, friday);

// Find the average price for each day of the week
const avgPrices = FindAveragePricePerDay(monday, tuesday, wednesday, thursday, friday);

方案2 #

// Sort each stock date into a day of the week - sortedDays contains arrays of monday...friday
const sortedDays = FindDayFromDate(historyData);

// Find the average price for each day of the week - Expand sortedDays inside function to access monday...friday
const avgPrices = FindAveragePricePerDay(sortedDays);

性能差异说明 #

两种写法不存在有实际业务影响的性能差异。方案1是在外部作用域声明5个数组传入函数修改，方案2是在函数内部声明5个数组、组装成聚合结构返回，仅多了一层外层结构的引用封装开销，这类开销在JS等高级语言中是纳秒级的。哪怕是处理十万、百万级的行情数据，这点开销和遍历数据、计算均价的耗时相比连零头都算不上，完全没必要为了这点感知不到的性能牺牲代码的可维护性。

额外提一个小优化点：用方案2时可以把sortedDays设计为带明确键名的对象结构（比如{ monday: [], tuesday: [], ... }），而不是固定顺序的数组，能彻底避免内部取数时顺序错位的问题。

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