Not Found

[techboon]

無知は死ねというレビューはよくないです。大抵は悪意からやってる訳ではないので、より最適化され最低限であるべき、そしてこういうやり方があるよ方式の方がお互い気持ち良く仕事できます。

[sugawara1991]

「やめた方が良い」ではなくて筆者的に「大罪」「やるな」のオンパレードで肩透かし感。嘘とは言わんがタイトルで「やりがちだし理由も分かるけど避けかたあるからやめよう？」という建設的な内容を勝手に期待してた

[canadie]

コードじゃなくて人を憎んでるよね？

[taruhachi]

アンチパターンの方が推奨パターンより刺さりやすいのはわかるが、やめた方がいいアンチパターンの羅列は二重否定で分かりにくくなるというアンチパターン。

[ryunosinfx]

新人のソースコードを読むとよく出会えるし、イキったやつのソースコードを読んでも出会える。なぜ、再帰を覚えると人は使ってしまうのか・・・必要もないのに・・・

[stp7]

初心者・イキってる奴でも指摘されたら大抵修正するが、ベテランという名の老害は説明しても絶対修正しようとしない

[tmatsuu]

わかる。インデントやファイルの最後で改行の件はcommit前にdiff見れば気づくはずで、キレイなdiffになっているか確認するクセをつけると良い。

[kencharos]

いい内容だと思うけど、goのinterface{} とScala java のジェネリクスが同じ扱いなのは、ん？という感じ。

[trashtoy]

可読性向上のために再帰を使うことはよくある (変数の再代入を回避するなど)

[taka_pop]

この人自体もあまりプログラミングをやっていないな？感が、ある

[naohero22]

こわいよーw

[dala]

人や状況によって色々意見はあるだろうけど、こういうのは時々見とかないと気が緩むよね…。

[k-motoyan888]

酷いコードを書く人は教育すればいいし、酷いコードを書く人が権力を握っている・意固地でいられるのならそれはテクの問題ではなく組織の問題なんだろうなぁと、こういった記事が出てくる度に不毛な気持ちになる。

[kazuneko]

いろいろ思うところはあるけどクソコード耐性は高いほうがいいんじゃないかな？

[MzdA0w73tg]

こういう恨み節をグチグチと書き連ねるサービスになってしまったのねQiita。「こうならないように自動的にこうしてくれるライブラリ書きました！使い方は……」とかっていう方が建設的だと思うけど……。

[golbaser-neko]

少し燃えてるからなんだろなと思ったけど、言葉が強いだけで大体同意できる内容だったかな。自分も経験があるけど、酷いコードを書く人が多い環境だと強く言いたくなりがちよね。

[n_231]

こんな駄文でクソコードが減るなら今頃世界は理想郷だな

[hdampty7]

一番大事なのはルールの運用方法でレビュー時にＮＧにするためにルールが明文化されるチームにいたときは精神が狂いそうになった。ルールは前もって一か所に分かるように書いておく。これ大事。

[awekuit]

“https://qiita.com/pakkun/items/9bef9132f168ba0befd7”

[houyhnhm]

パターンの概説に価値判断が入っていると訳が分からなくなる。

[mas-higa]

単なる好みや場合によるとしか言えないものも少なくない

[dagama]

かーっ！時間と金とハイパー技術力があればなーっ！かーっ！

[manFromTomorrow]

すごいデカい風呂敷だなーと思って読み始めたんでびっくりした（悪い意味で）

[NOV1975]

アンチパターンというほどシチュエーションが絞られてないのでこのエントリはそれなりに分かっている人があるあるネタとして消費する以外の用途が思いつかない。

[akanehara]

節子、それレビューやない。マウンティングや。

[ghostbass]

まあ大体理解/納得できる。保守・改修の際にそのコードを読むのは君じゃない、っていうのは意識すべき。言い方の問題はあるだろうけど直さない奴はとことん直さないからなあ。

[yantzn]

内容は同意だが。レビュアーとしてよくない

[aroechan]

再帰をやめろは単にお前の読解力が低いだけだろ？

[otchy210]

概ね同意できるがいくつか疑問符が付く感じだった。まあ過去の経験や環境によって感覚が違うことはあり得る。Web で再帰が必要ないと言い切っちゃうのは、DOM 辿って何かするとか書いた事無いのかな、とか。

[quick_past]

一度に表示される文章量的にきついかもしれないけど、具体的な例があるともっとよかったかも。メソッドの切り分けに関しては、細分化させる上でメソッド名に悩み始めたらそれは適切な分け方じゃないと思ってる。

[usaribon]

レビューで自分のポリシーに従いボコボコにするタイプや。

[oakbow]

再帰、ごくたまーに使うと思うけどな…。書かれてる内容はだいたい合ってそうだけど、ブコメ見る限りこのくらいでも軋轢があるものなので、rubocopみたいに機械的に指摘できるものを使うといいなあ。

[shun_libra]

お説ご尤もだけど「撃っていいのは撃たれる覚悟のある奴だけだ」というのはお忘れなきよう。コードレビューに限らず、将来ベテランになってこんな事も…と現場で殴られて善処できる自信があるなら、まあどうぞ。

[kathew]

"外部キーを使わない"自称ベテランに多い。そして、コードで担保すれば良いという。使わないのはデータの取り扱いが不便（親子を意識する必要があるから）だからというが‥‥。不整合のメンテコストが馬鹿にならない

[richard_raw]

クソコード耐性の高い人を辞めさせればいいのでは（適当）

[astromi]

愚痴ってるだけでおもんない

[versatile]

よくないコード＝老害のイキリとシャシャった新人

[Iridium]

「別にどうとも思わないがお前に指摘されたからやめてやるよ」と思われるのはマズい。効果と弊害を評価しないと、本質的な理解にならない。評価によってはそのときどう書くべきかは違った結論になる。

[arearekorekore]

わかるーって思いながら過去のクソコードに前ならえしてる

[sds-page]

違う型をいくらでも再代入できるPythonヤバい。量産型パイソニスタの襲来に備えろ