一个函数有多少行代码比较合适？

要看功能，如果一个方法可以分成好多个子功能的话
尽量将其抽取出来作为单独的方法，
曾经重构过好多这样的代码，
不是写代码的人水平低，
主要还是不负责，懒得多想。

个人觉得函数的写法应该遵循以下两点基本的原则：
1.当发现函数中有重复代码的时候，说明你可以将它封装成一个新的函数了
2.每个函数按照功能命名，即每个函数的实现都应该仅包含函数名所传达的内容。

根据单一职责设计原则 分得越精细越好，而且要看功能，如果一个方法可以分成好多个子功能的话尽量将其抽取出来作为单独的方法，曾经重构过好多这样的代码，不是写代码的人水平低，主要还是不负责，懒得多想。

尽量不超过一屏应该是公认的，因为当你要上下滚动屏幕时会大幅增加你出错的概率。
不过我觉得更多的还是要关注这个函数是否有再被分解的意义，如果代码不需要重用，在功能上也没有显得很独立，那就不必要强行要求分解成更小的函数吧。