エントリーの編集
![loading...](https://b.st-hatena.com/bdefb8944296a0957e54cebcfefc25c4dcff9f5f/images/v4/public/common/loading@2x.gif)
エントリーの編集は全ユーザーに共通の機能です。
必ずガイドラインを一読の上ご利用ください。
記事へのコメント98件
- 注目コメント
- 新着コメント
![sugawara1991 sugawara1991](https://cdn.profile-image.st-hatena.com/users/sugawara1991/profile.png)
sugawara1991
「やめた方が良い」ではなくて筆者的に「大罪」「やるな」のオンパレードで肩透かし感。嘘とは言わんがタイトルで「やりがちだし理由も分かるけど避けかたあるからやめよう?」という建設的な内容を勝手に期待してた
![k-motoyan888 k-motoyan888](https://cdn.profile-image.st-hatena.com/users/k-motoyan888/profile.png)
k-motoyan888
酷いコードを書く人は教育すればいいし、酷いコードを書く人が権力を握っている・意固地でいられるのならそれはテクの問題ではなく組織の問題なんだろうなぁと、こういった記事が出てくる度に不毛な気持ちになる。
![MzdA0w73tg MzdA0w73tg](https://cdn.profile-image.st-hatena.com/users/MzdA0w73tg/profile.png)
MzdA0w73tg
こういう恨み節をグチグチと書き連ねるサービスになってしまったのねQiita。「こうならないように自動的にこうしてくれるライブラリ書きました!使い方は……」とかっていう方が建設的だと思うけど……。
![otchy210 otchy210](https://cdn.profile-image.st-hatena.com/users/otchy210/profile.png)
otchy210
概ね同意できるがいくつか疑問符が付く感じだった。まあ過去の経験や環境によって感覚が違うことはあり得る。Web で再帰が必要ないと言い切っちゃうのは、DOM 辿って何かするとか書いた事無いのかな、とか。
![quick_past quick_past](https://cdn.profile-image.st-hatena.com/users/quick_past/profile.png)
quick_past
一度に表示される文章量的にきついかもしれないけど、具体的な例があるともっとよかったかも。メソッドの切り分けに関しては、細分化させる上でメソッド名に悩み始めたらそれは適切な分け方じゃないと思ってる。
![oakbow oakbow](https://cdn.profile-image.st-hatena.com/users/oakbow/profile.png)
oakbow
再帰、ごくたまーに使うと思うけどな…。書かれてる内容はだいたい合ってそうだけど、ブコメ見る限りこのくらいでも軋轢があるものなので、rubocopみたいに機械的に指摘できるものを使うといいなあ。
![shun_libra shun_libra](https://cdn.profile-image.st-hatena.com/users/shun_libra/profile.png)
shun_libra
お説ご尤もだけど「撃っていいのは撃たれる覚悟のある奴だけだ」というのはお忘れなきよう。コードレビューに限らず、将来ベテランになってこんな事も…と現場で殴られて善処できる自信があるなら、まあどうぞ。
![kathew kathew](https://cdn.profile-image.st-hatena.com/users/kathew/profile.png)
kathew
"外部キーを使わない"自称ベテランに多い。そして、コードで担保すれば良いという。使わないのはデータの取り扱いが不便(親子を意識する必要があるから)だからというが‥‥。不整合のメンテコストが馬鹿にならない
![Iridium Iridium](https://cdn.profile-image.st-hatena.com/users/Iridium/profile.png)
Iridium
「別にどうとも思わないがお前に指摘されたからやめてやるよ」と思われるのはマズい。効果と弊害を評価しないと、本質的な理解にならない。評価によってはそのときどう書くべきかは違った結論になる。
注目コメント算出アルゴリズムの一部にLINEヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています
![アプリのスクリーンショット](https://b.st-hatena.com/bdefb8944296a0957e54cebcfefc25c4dcff9f5f/images/v4/public/entry/app-screenshot.png)
いまの話題をアプリでチェック!
- バナー広告なし
- ミュート機能あり
- ダークモード搭載
2019/01/20 リンク