こんなコードは嫌だ、古い書き方のコード駆逐したい（とりあえず９つ） - Qiita

時代は令和ぞ、何を書いとるんや 転職してきた若いプログラマが変なコード書いている。 どうやら前社の社内研修で教わったとのこと。 さて、何を教わったのだろうか。 ※一応TypeScriptで書きましたが別にC#でも言えることです。 ※CやC++やアセンブラのことは全く知らないので、そのあたり詳しい人は今どんな書き方か記事書いていただけると勉強になります。 １．変数名が雑 クラス、関数、変数、どれも命名は難しいものです。1 大体が英語で大変です。けど頑張ってわかりやすい名前つけるようにしています。 本読んで勉強してください。Google翻訳使ってください。 10行程度の短い関数ならretでもdataとか適当な名前でもいいけど 長くなるようならちゃんと名前つけてるようにしたほうがいいです。 わかりやすい変数名をつけることでひと目で、その変数の役割が理解出来ます。 // Goodってなんやねん！な

[sangping]

コメント欄が本番

[rch850]

yoda 記法禁じたら 10 <= a && a <= 20 書けなくなって可読性落ちたの思い出した

[uskey]

要はバランスおじさん「要はバランス」

[rAdio]

この手の「指摘」を嬉々としてやっちゃうのは、技術者の麻疹みたいなもんよね……。ただ、こういう精神性のまま「提案」「政治」系スキルを伸ばさず歴を重ね、技術も実績も特別なものがないと詰んだりしそう。

[greenbow]

コメント欄で言われてるように代入文には括弧つけてほしい派。何を代入してるのか一瞬迷う。 const isHoge = (hoge == 'Hoge');

[njgj]

叩かれながらも、閉じずにコメント返してるのえらい。（最初の頃はイキった返信しちゃってたけど）

[ghostbass]

大体正しいと思うよ。括弧については副作用防ぐ目的があったりするので微妙。

[Wafer]

これをやりたがるのは駆け出しの初学者

[Amigomr]

言わんとすることはわかるし、同意もするけど、まぁ、言い方の問題だよね。「老害」ねぇ... 自分もいい歳になってきて、年下からこう思われてるんかなとか思うと、なんだか切なくなった。

[htnmiki]

コーディング規約無しで晒し上げる職場って心理的安全性が低そうだよね

[masayoshinym]

“どれだけひどかったかは編集履歴から確認できます。”この追記を読んでこの人は信用できそうって思った。

[wwolf]

「言語として用意されている機能を使わない」は標準ライブラリとかならいいんだけど、マニアックな知識マウント始める人が出てくるから要注意なんですよ

[yuzuk45]

gotoとな・・・VB6ｴ････( ；∀；)

[hdampty7]

若い時はこういうの拘りあったなぁ、自分も。でも細部はどうでもいいのよ、結局は。isGoodが駄目なんじゃなくて、プロジェクト全体で同じ意味でisGoodが使われてるとか、そういうのが重要なんだよ。

[hogeaegxa]

最近仕事でCOBOL書いたけど、あの言語だと上から下に抜ける用途なら、IF文とフラグで頑張るより圧倒的にGOTOのほうが見やすいんだよね。今時の言語で使う場面は思い浮かばないけど

[tattyu]

最適化で触りたくなるけどここで絶対にやってはいけない処理とかあえて残す事ある。ローカル変数名も自分の責任の範囲下でそんな長いコードでも無ければなんでもいいと思ってる。Linqで書いたらGCAlloc起きるじゃん。

[nakag0711]

とりあえずifやforの後にはスペースをいれたまえ、話はそれからだ

[rasterson]

転職を繰り返していると細かいところは気にならなくなってくる。行く先で逆のこと言われることもあるし。同意できるのは、スコープ狭くすること程度かな。あっ、gotoは論外だけどね。

[Katame]

ヨーダ記法は使っていたかも。。 equalsでnullにアクセスするとぬるぽが発生するので定数先にした記憶あります。 今思ったら先にnullチェックしとけば良かったのに、、、

[mas-higa]

for (i = 0; i < n; ++i) じゃなく foreach 使いましょう…みたいな話かと思ったら違った。

[Nekomajin]

変数スコープについては、ケースバイケースじゃないかな。必要最低限のスコープで作ったとき、同じ名前の変数名をうっかり宣言したときに、ややこしい処理だとバグの温床になるような。

[su_zu_ki_1010]

コーディング標準って難しいね～。

[yojik]

老害を老害って理由でディスってると、いつか自分も老害になってしまうぞ （自分に呪いをかけないで！)

[ducktoon]

みんなあたりまえのように言うけど、どんなコード書くのかもわからないような人が知らない間に転職してきてるような職場にガイドラインやlint的な意識あるわけないよ

[kuwalab]

まあこれは荒れるよね

[dekasasaki]

私はこの手のイキりを「IT厨二病」と名付けています。

[misopi]

自分は歳をとらないと思っている人

[zilog80]

分かる!

[akiramaz]

置いた理由を知らないならフェンスをどけてはいけない

[msukasuka]

“記事炎上しててピエン”←自己紹介草

[atsushifx]

リーダブルコード、じゃないな。ピープルウェア、プログラミングの心理学といったリーダーシップのための本を読むべき。こういった炎上記事はマネジメントの問題。基本、G.M.ワインバーグの本はおさえておきたい

[tetsu23]

いろいろ耳が痛いw

[z1h4784]

自分のやり方を他所に持ち込もうとしている老害はあなた本人では？という記事だった。「言語らしい書き方」が素晴らしいならrubyは何でもかんでもワンライナーで書けばいいのかな

[uzusayuu]

こういうのは言語ごとにスタイルガイド決めてそれに従わないと、レビュアーごとのぶれがでそう。

[itotto]

いろんな意味でかわいそう

[toro-chan]

私はどっちでもいいので、こういう人がいたら逆らわずに修正してる。それより「昔のコードをコメントアウトして残す」ってまだあるんだ、と笑った。今はバグ票なんかないのだろうか。3年に1回ぐらい使うんだよな。。

[cuttoff19]

コメント欄

[masa_w]

1、2年目の若手イケイケエンジニアが書きそうな記事。昔のこと思い出してこっちまで恥ずかしくなる

[grover]

割とよくある指摘が多いので、あんまりおじい関係なかった。

[augsUK]

「さて、老害は何を教えたのだろうか」から始まる自称技術文書か