[問題] class 行數過長問題

作者: shadow3x3x3 (UCCU)   2016-06-16 08:56:16
第一次在本版提問問題,還請多多指教 !
最近寫Ruby的時候其實都使用了Rubocop https://goo.gl/py0lAk
來檢測自己程式的壞味道
大部分檢測出的問題都可以想辦法修改
可是目前最讓我棘手的是Ruby的class不能寫超過100行這項原則
雖然我還挺認同這項原則的,但是實際要遵從時還是相當困難
例如我已經想破頭都想不出到底如何將我超過百行的class在精簡(已經繼承一父類別)
於是為了解決百行內問題,我就將一些方法移到module裡,再從class中include進來
可是拆出的module我也只命名成xxx_util,甚至module方法裡還用到class變數(@foo...e
tc
目前的做法感覺相當不健康,但另外我還能想到的做法就是將類別關係更加細分
拆成更多父類別一層層繼承下來...
又或者是我Ruby Code寫得太冗長應該先從改善Ruby Code下手嗎?
想問問各位是如何遵從單class不超過百行的原則,或是有沒有正確的做法是我應該去實
現?
謝謝各位!
作者: Neisseria (Neisseria)   2016-06-16 10:26:00
比起行數,class 的 SOLID 原則更應該要注意有時候一個 method 可能就 30-40 行了一個 class 只能 100 行實在有點少 =.=一個 method 30 行可能也是有點多,可能要 refactor有時 refactor 成 private method,行數不一定會減少
作者: mars90226 (火星人)   2016-06-16 15:21:00
100行不是硬性規則,只是要提醒你行數少表示程式清晰如果真的無法低於100行,就放著吧,繼續改可能反而減少程式可讀性
作者: shadow3x3x3 (UCCU)   2016-06-16 22:02:00
其實Rubocop也有規範一個method不超過10行,這方面還算是好控制另外確實拆開來可讀性會變差一些,或許還是case by case吧
作者: rayway30419 (RayWay)   2016-06-17 01:02:00
你可以自己定義rubocop我用default的lint兩天就反安裝了

Links booklink

Contact Us: admin [ a t ] ucptt.com