• 人気のコメント(8)
  • 全てのコメント
d4-1977 d4-1977 リファクタリングから学ぶ事って多いって実感しています

2020/03/25 リスト

p1ass p1ass 名前の付け方は大事だよね。でも自分では良し悪しが分からないことが多いのでなるべくレビューで拾ってほしいと思ってる

2020/03/13 リスト

fa11enprince fa11enprince 主観であるけど、名前がJava以上に冗長、calculate_とかノイズ。areaなのはわかっているのでget_でも良い。== falseとかも気になるし、?なしだしRubyぽくない。notificationが抽象的で購入処理をしたら何が起こるのかを書くべき。

2020/03/12 リスト

maecchi maecchi コメントが簡潔=内容がわかりやすいを目指したい

2020/03/12 リスト

GEROMAX GEROMAX ええやないですか、これぐらいのことできない人もいっぱい居ますよ。。

2020/03/12 リスト

eru01 eru01 ループ変数i,j,kの使いこなしはまかせろー

2020/03/12 リスト

regularexception regularexception なんで、Rubyなのにwhenの中で逐一代入してんだろ

2020/03/12 リスト

kagerou_ts kagerou_ts リファクタリング専門チームがあるってことがもう良い。存分にリファクタリングしたい…

2020/03/12 リスト

threetea0407 threetea0407 関数をたくさん作ったら読みやすくなるっての、端的には嘘だと思う。関数は複雑な条件分岐を隠蔽する。

2020/03/12 リスト

yarumato yarumato “crowdworks.jpはサービスインから約8年が経過し、30万行を超えるモノリシックなRailsアプリケーション”

2020/03/12 リスト

saiid saiid 焼畑農業の後始末を延々としていくチームってことか、とんでもなく辛そう。

2020/03/12 リスト

haruten haruten database_open(); //データベースオープン みたいなコメント見たことあるけど、これはもはやフリガナ

2020/03/12 リスト

fukken fukken サンプルだからでもあるだろうが purchase_alcohol が責務過多だな。高レベルのドメイン知識を反映したメソッドはフローチャートのように読めるべき。実装詳細とドメインロジックが混在してる

2020/03/12 リスト

otihateten3510 otihateten3510 うーん。どうにも皆がこの段階で止まってるのが良くない気がする。色んな仕事してると確かにその次の改善に行ってるコード見かけない。/今度書く

2020/03/12 リスト

aroechan aroechan リーダブルコードとかでも書かれている内容。基本的でみんなできていると思っているがすごい大事な話

2020/03/12 リスト

ene0kcal ene0kcal 購入可能を判定するのにIsPurchasableを検討してみたい。

2020/03/12 リスト

nakag0711 nakag0711 昔から関数の長さは一画面までと言われている

2020/03/12 リスト

annoy annoy 自分なら提供可能か判定する関数を切り出す。 あとインスタンスを選ぶロジックは連想配列(map?)にするかな。

2020/03/12 リスト

kemuken kemuken "customer_age <= 19" が気になる。条件の反転なら素直に "< 20" でいいと思う。(元のコードにない数値の記述は「既存のコードを安全に改善する」から外れる気がする)

2020/03/12 リスト

vndn vndn purchase_impossible?()みたいなメソッドは肯定的な名前で切り出して利用する側で否定するほうが好み。個人の趣味かもしれない。

2020/03/12 リスト

kamiya-s-max kamiya-s-max とてもわかりやすい。rubyが例なので個人的に理解やすく、手順ごとに解説があって、考え方をすぐに自分に落としこめられそう

2020/03/12 リスト

関連記事

リファクタリングチームに入ってから学んだ理解しやすいコードを書くための基本的なこと - クラウドワークス エンジニアブログ

こんにちは! 去年の4月に新卒入社してからお酒ばかり飲んでいるエンジニアのd4teです。 4月から11月ま...

ブックマークしたユーザー

  • torimetal2020/03/30 torimetal
  • d4-19772020/03/25 d4-1977
  • somathor2020/03/16 somathor
  • tsumuchan2020/03/14 tsumuchan
  • p1ass2020/03/13 p1ass
  • alcus2020/03/13 alcus
  • dellab722020/03/13 dellab72
  • eri123go2020/03/13 eri123go
  • mayoisen2020/03/13 mayoisen
  • grand_big2020/03/13 grand_big
  • sheepsace2020/03/13 sheepsace
  • khiramatsu2020/03/13 khiramatsu
  • rikkomo82020/03/13 rikkomo8
  • kazkun2020/03/13 kazkun
  • dafujii2020/03/13 dafujii
  • bluescreen2020/03/13 bluescreen
  • mcddx302020/03/13 mcddx30
  • chicken_kozo2020/03/13 chicken_kozo
すべてのユーザーの
詳細を表示します

いま人気の記事

いま人気の記事 - テクノロジー

新着記事 - テクノロジー

同じサイトの新着

クロワッサンが焼けるなんて! - お菓子とネコ、たまに着物

3 users https://rikimarutti.hatenablog.com/

クリスマスっぽいカップケーキ - お菓子とネコ、たまに着物

2 users https://rikimarutti.hatenablog.com/