リファクタリング一覧

[saruhei]

Class名に動詞

https://github.com/exKAZUu/RepositoryProbe/tree/master/src/main/java/net/exkazuu

やめましょう

getSourceCodeAbsolutePath

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/FileManager.xtend#L20

メソッドが２つあります。

空の中括弧

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/FileManager.xtend#L20

while(hogehoge);

で良いんじゃなかろうか

URI直書きはやめましょう

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/GetNumberOfDelevopersAndTestes.xtend#L10-L12

外出ししたほうがいいんじゃないかな？

if(! hogehoge){} にする

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/GetNumberOfDelevopersAndTestes.xtend#L17

//test//

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/GetNumberOfDelevopersAndTestes.xtend#L17

読み逃しがでる気がします

clone

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/GitManager.xtend#L19

• ２つあります
• runCloneじゃなかろうか

runBlameとかにする

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/GitManager.xtend#L19

blameだけ統一感ががないのが気になる

変数名split?

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/GitManager.xtend#L76

splitResult?

getAuthorName

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/GitManager.xtend#L76

単数のは１個のstring返すので良いんじゃないだろうか＊

mainが長い

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/Main.xtend

やってることは本当に起動処理だけ？

nextPage

https://github.com/exKAZUu/RepositoryProbe/blob/master/src/main/java/net/exkazuu/Main.xtend#L16

clickNextPageに見えます

とりあえずここまで

[gumfum]

clone
runBlameとかにする
変数名split?
getAuthorName

=> New ticket: #88