在我的rails应用程序中,我有一个这样的方法：
def cart
if user_signed_in?
@user = current_user
if @user.cart.present?
@cart = @user.cart
else
@cart = false
end
else
cart_id = session[:cart_id]
if cart_id.present?
@cart = Cart.find(cart_id.to_i)
else
@cart = false
end
end
end
Rubocop标记了这个方法,因为Method有太多行.为什么写一个行太多的方法不好？如果我们必须做很多工作怎么办？我怎样才能重新考虑这一点并编写好的代码？
解决方法
一种方法是你可以使用
ternary operator重构它,但代价是可读性.
def cart
if user_signed_in?
@user = current_user
@cart = @user.cart.present? ? @user.cart : false
else
cart_id = session[:cart_id]
@cart = cart_id.present? ? Cart.find(cart_id.to_i) : false
end
end
其次,如果你被迫编写一个很长的方法,这意味着你的面向对象设计有问题.也许,您需要为额外功能设计一个新类,或者您需要通过编写多个方法将功能拆分到同一个类中,这些方法在组合时可以完成单个long方法的工作.
Why is it bad to write a method with too many lines?
就像一篇带有大段落的文章难以阅读一样,同样使用较长方法的程序难以阅读,并且不太可能重复使用.您划分代码的块越多,代码的模块化,可重用性和可理解性就越高.
What if we have to do a lot of work in it?
如果你必须做很多工作;这肯定意味着你以一种不好的方式设计你的课程.也许,您需要设计一个新类来处理此功能,或者您需要将此方法拆分为较小的块.
How can I re-factor this and write good code?
为此,我强烈推荐给你一本名为：Refactoring by Martin Fowler的好书,他在解释如何,何时以及为何重构你的代码时非常惊人.