コメント。
とりあえずコメントは随時整理しませんか。
わりとまじで。
他社から回ってきた案件に超高確率であるのが、修正前コードがコメントアウトされてるクソソース。
コメント添えてどういう意図で残してるのかがわかるなら優秀な方という部類ですね。
というかリリースする前に不要コメント全部消しとけよと思わなくもなく。
というかそんなクズ文字列コミットしてんじゃねぇよと言いたいところでもありつつ。
酷いのになると普通にTODOコメント残ってたりする。
TODO残してリリースしたんかーい、って叫びたくなる。
元コードがどんなコードだろうがコメントアウトされてる時点で要件を満たしてないかバグが内在してるゴミコード以外の何物でもないわけで、残す意味が欠片も存在しない。
意図や意味が存在しないものを動作しないとはいえ実装上に残すとかプロにあるまじき怠慢です。
この怒りを他業種にも伝わるように書くとするなら……、全く整理されてないデスクを指差されて、「今日からあそこに席替えだから」って言われるのと同じ感じ。
それも単に散らかってるだけじゃなくて前のプロジェクトで使ってたんだろうなって資料が残ってると思えばその隣には投げ捨てられてぐちゃぐちゃになった新聞だとか雑誌が散乱してて挙句なぜか椅子の上に食いかけで放置された萎びたパンが打ち捨てられてるレベル。
そんな場所に移動させられて「椅子と机あるんだから仕事できるでしょ」って言われるような怒りを感じる。
まあ、かなり大げさに言いましたけど次の作業者の作業の阻害要因になり得るという意味では程度の違いこそあれ同じようなものですねと言い張りたい。
ちなみにそういうソースでコーディング上真っ先に上がる問題点は可読性の低下ですね。
あと、そういうソースを平気であげてくるところほどスタンダードから離れた処理を書きたがる。
無駄に入り組んだ三項演算子とかブロック分けされてないif文とか
if (boolean) return value;
とかコメント無しのビット演算だとか。
do {
if (boolean) break;
}while(0);
この辺りも嬉々として使ってるのってCかC++の一部のナードだけだろと。
一部のスクリプト言語では『スマートに書けるでしょ？』ってドヤ顔してるようなコードが別言語だと非推奨だったりもよくある話。
ちゃんと勉強しよう？
そして、最低限実装は最大限綺麗にしよう？
具体的には、不要コメントの削除と説明コメントのメンテくらいはしよう？
わりとまじで。
