検索でtext_field_tagの属性名がnicknameになっていた。将来性を考えるなら、
最初からkeywordにしておくべきだろう。
メソッドを組むにしても、常にこれを意識せよ。メソッドに分けていいのか、
分けない方がいいか。このメソッド内にかくべきかそうでないか。
例えば、検索でpaginate(:user, ***)のuserの部分は同じだから、
def seach pagenate(:user, search_by_type(search_type)) ・・・・ end
としていいか・・・・・。いやいや、将来user以外の検索ができるかもだから
def search search_by_type(search_type) end
としようとなる。凝縮度、結合度からみても後者が綺麗