「マスク氏、コードレビュー結果で50人を解雇」記事へのコメント

社畜宣言したのに首かぁ

[Anonymous Coward]

つらいですね・・・。

人を有能・無能の軸と働き者・怠け者の軸で4種類に分けた時、組織にとって最も有害なのは無能な働き者だという言葉があります。
今回首にされた人は無能な働き者扱いされちゃったわけで、せつなすぎる。

ご主人様のために働きますワン！って張り切ってた猟犬が捨てられてクンクン泣いてるイメージが脳裏にちらつく・・・。

Re:

[jizou (5538)]

コードレビューごときで落とされるようなものしか書けない人は、
バグの温床になるコードを生み出し続けるので、
Twitterのような会社にはいらない人でしょうね...

ハードワークで、不毛なコードを生成し続けられると会社としても損失でしょう。
高給をはらう分いい人材を求めるのが、Twitterならできそうです。

Re:社畜宣言したのに首かぁ

[Anonymous Coward]

まあ今回解雇された人が本当にレベル低かったのかは分からんけど、
（低いと言ってもTwitter社員なので普通のプログラマーよりは全然凄そう？）
世の中にはコードレビュー見たらクビにしたくなるような奴はごまんと居るから、
コードレビュー結果でクビにしましたって言われるとそれは仕方ないと思ってしまうね…orz

# コンストラクタに平気でコンストラクタってコメント書くような奴とかだよ！

Re:

[Anonymous Coward]

私のことですね
コンストラクタだと書くことない場合多いので……

Re:

[Anonymous Coward]

書くことないからってコードを見れば自明なことを書くのはNG。
特に「コンストラクタにコンストラクタってコメントする」は、リーダブルコードの悪いコメントの例で最初に出てくるような奴なんで、さすがに直しましょう。。。

＃ リーダブルコード読みました！って人が平気でこれをやる。

Re:

[Anonymous Coward]

昔は，勝手にクラスのひな型に，このコメントつけるIDEがあった気がします．

Re:

[Anonymous Coward]

コンストラクタに平気でコンストラクタ

ほんの一行であり蛇蝎のごとく嫌うほどのものでは無いかと。

それどころか、コメントだけ追っているときは、コメントに「コンストラクタ」と明記されている方が読みやすいかと思いますが。

Re:

[Anonymous Coward]

#Region こんすとらくた
public Hoge(){ ～略～ }
#EndRegion

たぶんオーバーロード増えた時用だろうなと思うことはあるｗ