同事的代码看得我脑壳疼，举三个代码例子，看看问题出在哪里

一个同事写得代码我看了以后脑瓜子疼，然后我找了几个我认为的槽点把他给叫了过来，问他这么写有什么意义，然后他竟然反问我：“这么写有什么问题吗？”。结果旁边另外一个同事看了一眼他写的代码，然后笑呵呵得问我：“对啊，这么写有什么问题吗？”。

我一抓脑瓜子，想了想，然后无奈道：“哇，你们现在都这么写代码吗？”。

我找了同事代码中三个比较经典的代码例子来说这个问题。

第一段代码，其实就是给一个Boolean变量A赋值的过程，这个变量A是根据另外一个Boolean变量B的变化而变化的，那么按照他代码里写的逻辑，是不是完全只需要直接使用另外一个B的值就可以了？

当然，有可能B的访问范围没有A的访问范围大，那的确需要将B的值赋值给A，那其实直接赋值就可以了，无需使用If语句来判断B的值。

第二段代码，他判断了一个Boolean变量是否等于true，在语法层面，这种写法没有问题，但是能不能直接写成if(A)?

第三段代码，因为场景不同，需要调用的函数（方法）也不同，于是他写了一个if语句用来分别调用不同的函数（方法），这种逻辑是对的，但是因为不同函数（方法）的参数都是一样的，调用的函数（方法）前需要对变量赋值，于是在if语句的成立条件代码块里和if语句的不成立代码块里均对几个变量赋了相同的值。

这么写是没有问题，我只是觉得，相同逻辑部分能不能写在if语句外面呢？

结语

其实旁边那位同事就是看热闹不嫌事大的，我给同事指出的其实是代码风格的问题，有些人的代码风格一旦养成了，未来是解不开的。

所谓“代码能跑，你管我怎么写！”这样的话，一般都是看热闹的人说的，真正关心你成长的同事，不会说这样的话！