只是因为某些东西有效,这是否意味着你不必重构？

Question

我刚看到WP代码库中的这行代码.

if ( $user_id = wp_validate_auth_cookie( '', apply_filters( 'auth_redirect_scheme', '' ) ) ) {
}

嗯.是啊.if语句中的两个方法调用和赋值语句.

所以,我的猜测是,没有人会重构这个.

有什么理由吗？

Answer 1

要回答标题中的问题:

只是因为某些东西有效,这是否意味着你不必重构？

如果确实如此,我们永远不会重构.因为重构发生在绿色上.也就是说,它发生在所有测试都通过时,按照定义 - 代码有效.如果你用红色"重构",你就不会重构.你正在玩你的代码.你冒着可能不应该冒的风险.重构正在改进现有代码的设计,而不会改变其行为.如果您的代码不起作用,如果您没有测试证明它正在运行,那么您所做的更改可能会改变其行为.