クソコードにならない為に、これだけは守って欲しい７つのこと - Qiita

まえがき 今回書く内容は、ある程度経験あるエンジニアでも、陥りがちなものに絞って書いてみたつもりですので、[重複コードは書かない]などの超あたりまえの事は書いていません。 2017/03/16 最近よく見られてそうなので１つ追記[そもそも継承するな!!!] そもそも継承するな!!! 継承するのは、どうしようもない場合のみにしてください。 その前に、strategyパターンや、compositeパターンなどの他のやり方を考慮してもなお、継承するのが妥当である場合のみにしてください。 基本的に継承しないほうが、スケーラブルだし、テストコードも容易にかけます。 継承はis-a関係 「あー、継承ね。はいはい」で飛ばしてんじゃねーよ。 いやマジで!!! ほぼ全てのエンジニアは[is-a]が何か知っています。 というのも全てのオブジェクト思考の書籍には出てくる概念だからです。 しかし、私の経験上この概

[ebo-c]

「何故かこの行を消すと動かなくなる」はコメントとしては有用か

[koyancya]

良い -> "思いを込めたコメント"

[kastro-iyan]

//おまじない

[FunnyBunnyDizzy]

結論はわかるのだけど、どれもこれも「なぜ」が書かれていないため腹落ち感が少ない。

[kijtra]

// 探さないでください

[pero_0104]

飛ばしてんじゃねーよwwwありがとう飛ばすところだった！！

[edo_m18]

日本語で分かりやすく簡潔に書くことと、プログラムを分かりやすく簡潔に書くことは結構似てると思ってる。

[blueroom555]

DRY原則しか意識しない人多いから。直交性意識しよう。同じ（ような）コードは2回書いていいこともある。わりと頻繁に。

[take-it]

なんかごめんなさい。“このコメントを見ると、エンジニアのレベル感も伝わるし、修正しても良いのだなとわかる”

[torimal3104]

具体例欲しい

[teppodone]

#てぽめも ウンコード

[luege_traum]

コールバックのところ、追記を10回くらい読んでようやく理解した。ここの例として書かれてるコードっぽいやつ、他の項目で書かれてるクソコードの要素が詰め込まれてない？

[yorisilo]

ドメインモデル

[Tomosugi]

次はクソQiitaエントリーにならない為に〜を書いてください。

[esper]

「コールバックは、ロジックと絡めない。」だけ意味が分からない(文意を取れない)。値の受け渡しだけのコードってアクセサのこと？ｗ

[pribetch]

“グルーバル変数”

[at_yasu]

似たようなコードが二つ以上あるのは、まぁしかたないよね、と思うけど、全く同じ処理なら trait 使うとかでいい気もする。

[kuy]

コールバックのところが具体的にどういうことなのかいまいちわからん。

[iguchi1124]

あとでもう一度読みたい

[slumbers99]

共通化とかいって継承しちゃうやつ。

[elzup]

いくつかハッとさせられた

[wkbyshnbtk]

“コメントは綺麗に書こうとするな。思いを込めろ！！！ ”

[vanbraam]

設計とは何か;この中の幾つかは設計を(レビューも含め)ちゃんとやってれば避けられる問題だと思うし,また幾つかはコーディング規約で避けられる問題の様に思う;コメントに対する意見は曖昧すぎ

[indication]

思想はたいせつに

[sona-zip]

is aとhas aの使い分け大事 人クラスを継承した戦士クラスと魔法使いクラス作ってはダメ 転職することがあるのだから人クラスの属性として職業を持つべき そうしないとインスタンスの破棄と生成が必要になる

[psfactory]

クソコードにならない為に、これだけは守って欲しい７つのこと - Qiita

[nilab]

クソコードにならない為に、これだけは守って欲しい７つのこと - Qiita

[kkeisuke]

“コメントは綺麗に書こうとするな。思いを込めろ！！！ ”

[W53SA]

「次は思いを込めたコメント例」

[satohu20xx]

コードで意図が伝わるのがいいと思う。finalは明示的に変更しちゃいけないっていうことを意味するのでこれは変更しちゃだめだよっていう時につけるとか意味を持たせたいと思う方です。

[kaionji]

笑ったw "よくわかんないけど、メインスレッドで呼び出したら、クラッシュしなくなった。"

[snowcrush]

言いたいことはなんとなく分かるし、同意できそうなんだけど、例がちょっとわかりづらい。

[su_zu_ki_1010]

今の部署、これ意識してるかなぁ…（遠い目）（自分のコードを振り返りつつ自省）

[John_Kawanishi]

オブジェ言語かぁ