对8年程序员大佬进行代码review，吓一跳~

东哥2024年08月04日约 3 分钟

嗨，我是东哥。

今天刚好看到一个帖子，一位程序员对一位拥有8年资历的程序员大佬进行代码review时表示被吓到了

话不多说，先上代码截图：

这段代码的需求是查询本月的客户返利信息。

简单来说，就是看看哪些客户在本月得到了返利，并且将这些数据提取出来。大佬写的这段代码实现了需求，但写法真是别具一格，独树一帜。

首先，这段代码使用了几个动态SQL片段，主要包括：

乍一看，这代码实现了需求，但问题却藏在细节里。

咱们程序员在编写SQL查询时，往往会考虑查询性能，特别是如何利用索引来提高效率。然而，这位大佬在列上使用了函数（YEAR() 和 MONTH()），导致了无法利用索引的问题。

技术评论：在列上使用函数会阻止优化器利用索引，因为索引是基于列的原始值建立的，而函数改变了这些值的表示方式。这就导致无法直接匹配索引中的条目，需要执行全表扫描来查找匹配的行，从而大大增加了查询所需的时间和资源。

为了解决这个问题，可以将日期过滤条件改为范围查询，这样就能充分利用索引。例如：

AND crr.rebate_confirm_date >= DATE_FORMAT(NOW() ,'%Y-%m-01')AND crr.rebate_confirm_date < DATE_ADD(DATE_FORMAT(NOW() ,'%Y-%m-01'), INTERVAL 1 MONTH)

这种写法可以确保查询条件能用到索引，大大提高查询效率。这样一来，既能满足需求，又能保证代码的执行性能。

很多人看到这里可能会说，这样写代码会阻止优化器利用索引，那是不是就不能这么写了？

其实不然。这位大佬的思路其实是很有意思的。

他采用这种写法，可能是在权衡了实现难度和性能之后的选择。简单直接的写法在开发初期可以快速实现需求，并且在数据量不大时，性能影响并不会很明显。

但是，作为程序员，特别是有多年经验的大佬，在实现功能的同时，也需要考虑到代码的可维护性和性能优化。

这段代码的写法，虽然达到了需求，但在后期的性能优化上会带来不小的挑战。

有人调侃说，这是在“养寇自重”，为后期优化做埋点。

其实，这是一种比较片面的看法。好的代码不仅要实现需求，还要考虑到未来的扩展和优化。

程序员不仅仅是“写代码的人”，更是“设计系统的人”。在代码的每一行中，如何做到既满足当前需求，又为未来的优化留出余地，是我们需要不断思考的问题。

在网上，关于这段代码的讨论也是热火朝天。

有网友说：“你刚毕业的吧？这么较真，都迟早是要失业的人，别把自己太当回事。”

确实，作为新手，可能会对每一行代码都精雕细琢，追求极致的性能优化。但随着经验的积累，我们也需要学会在复杂度和性能之间找到平衡。

东哥觉得，这段代码的写法虽然有些问题，但也反映了程序员在不同阶段的思维差异。

我们需要尊重每一位程序员的选择，同时也要不断学习和改进，毕竟代码是一个不断迭代优化的过程。

希望大家能在评论区分享你们的看法和经验，让我们共同进步。

目前，对编程、职场感兴趣的同学，大家可以联系我微信：golang404，拉你进入“程序员交流群”。

来自同一个声音