[B! コードレビュー] 些末なコードレビュー

yuiseki 元記事は単にコメント読んであーこういうのあるあるって話だったのに突然のマジぉこ渋い……寝起きではてブを見るのは良くないんだなと思った

news

2014/03/13 リンク

andalusia 「ここは難しいのでとりあえず放置」「なぜやってるのか不明」なんてコメントが残っているということは、むしろ実質的なレビューが全くされていない証拠だと思うんですよねー。

2014/03/13 リンク

junglejungle 納品物と社内コードレビューがごっちゃになっててなんだかなー。はてなユーザーはコードレビュワーじゃないぞ。

2014/03/13 リンク

tzt まぁ、アカウントやデータを預けてる客の立場からすれば、大丈夫かいな、という気はするわな。こういうところまで気を配るのもサービスのうちなんだが、詰めが甘いところがいかにもはてならしいな、と。

2014/03/13 リンク

kkobayashi 技術者の悪いとこが出てるエントリー。少なくとも客に見せるコメントではないだろって話だと思ってたが。／はてブタイムスタンプ事件の時も似たようなスタンスだったし治らんやろうな

2014/03/13 リンク

giraffe-htn これが『はてなの価値観』であり、それ以上でもそれ以下でもない。誰かの常識は誰かの非常識である、というだけの話。

2014/03/13 リンク

Hash 別に怒るほどのことではない冷やかしにマジレスしてる感じする. 些末な怒り

2014/03/13 リンク

hevohevo id:naoya 追記読みました。ありがとう。私の趣旨は「誤解を招くようなコメントを残しておくのは企業イメージ的に損じゃない？　それも込みでわざと残しているなら問題ないけれどね（個人的には親近感わく）」です。

2014/03/13 リンク

Futaro99 なんか知らん奴に笑われて腹がたったという話でしょ

2014/03/13 リンク

nikkie-ftnext #pycharity massaさんトーク。機械ができること、人がやること切り分ける

あとで読む

2020/07/04 リンク

isdh やらなければいけないレビューは、「その設計は拡張に対して開いていない」「これだとエッジケースが想定外だから不具合につながる」「このテストでは後で要求仕様を解釈しづらい」とか。(改)

2019/10/25 リンク

teckl ちょっと長めのコードレビューしていて、動作面など問題なさすぎたので寂しくて些末なコメントばかりしてしまって反省…

2017/08/30 リンク

massa142 “レビュワーにも知識とスキル、そして真摯な態度が要求される”

2016/12/09 リンク

phakchi0830 “HBFav ”

コードレビュー

2016/09/02 リンク

xxxxxeeeee いいレビューみたいなのってむずいですよね、本気で工数かけて向き合わないとできない

2014/07/16 リンク

nubes “だいたい議論が紛糾しているのは「コードのインデント幅が違う」とか、... とか、そんな些末なことばかりである、ということが多い。”

review

2014/04/05 リンク

nminoru 規模にもよるのだろうけどコードレビューの段階で「その設計は拡張に対して…」とか「これではエッジケースが…」とかの指摘が出てくるということは、設計がないのか、設計のレビューをやらない

2014/03/22 リンク

nobodyplace どんなことでも森を見なくちゃダメ、ということかな / エンジニア的な意味では：合わせて読みたい→http://blog.livedoor.jp/lalha/archives/50495777.html

2014/03/19 リンク

coppieee 問題はユーザに見れる所(JS)にコメントが書かれていたところかな。プログラマーじゃない人がコメントだけ読めばひどいと思われる可能性がある。無駄な炎上を避けるために、削除していたほうが無難？

2014/03/18 リンク

masu_mi ある時間・場所を何に使うか?って意味付け問題は全体の目的で決まる。で、決めた役割に見合わない活動は効率を阻害する。だから、起きない様に共通認識・規律を作って運用しようって話題かなーって(コードレビュー版)

2014/03/18 リンク

hhungry 「ここは難しいのでとりあえず放置」「なぜやってるのか不明」なんてコメントがグーグルのプロダクトに入ってたら幻滅しちゃうよね。ちょっと意識が低い。

2014/03/16 リンク

masa-wo 明日は我が身、かもなあ。

2014/03/15 リンク

tototti あまりコードレビュー受けた経験ないのだけど、たいていはコメントとかインデントの指摘くらいで終わってるなあと...。

2014/03/15 リンク

atsushieno 元記事は明らかにはてなをけなす意図を含めて書かれているし怒るのは当然だろう。という程度のことがわからないコメントが多すぎる。

2014/03/15 リンク

nilab 些末なコードレビュー - naoyaのはてなダイアリー

2014/03/14 リンク

akachochin ソースコードレビューの心構え。でも、誰もが一度はそういう通過儀礼を経て大人になる。そう、大人になればよいが、世の中にはピーターパンがときおり存在するのだ。

2014/03/14 リンク

rti7743 同意できる

2014/03/14 リンク

efcl ESLintでJavaScriptのコメントをLintする例を書いておきました。 https://github.com/azu/hatena-blog-js-lint

JavaScript

2014/03/14 リンク

mag4n レビューに関しては本質的な部分を見た方がいいのは確か。コメントアウト記述云々はGrunt等使って納品段階で圧縮かけて出力しろやでOK。てか開発用と納品用でちゃんと分けようよ。

2014/03/14 リンク

haishi JavaScript にコメントが残っているからダメだ、なんてのはインデント幅が2じゃない、4にしろ、と言っているようなものである。自転車置き場の議論を読むべき。難読化は、しなければいけないというものではない。仮に難

2014/03/14 リンク

taguch1 元ネタもこのポストも反論ポストもコメントも含めてやっぱりエンジニアは誠実だと思う。考えることをやめない

2014/03/14 リンク

kasumani ■ 些末なコードレビュー朝起きて布団から出るのがつらいので、HBFav をつらつらと眺めていた。 Tags: from Pocket March 14, 2014 at 07:14AM via IFTTT

2014/03/14 リンク

pmint JavaScriptコードは読めないとね。ブラウザー側で他の拡張やブックマークレットと競合するかも知れないし、そういうのを作る人にとっては動いて読める状態じゃないといけない。難読化してはいけないということね。

2014/03/14 リンク

itochan ブコメの「OSS脳」これが全てだと思う。私はこれは良い意味だと思うけど、人により解釈は違うだろう　／　dirty hackに「エラー無視」コメントは問題ないと思う

2014/03/14 リンク

tyoro1210 ダサさ

はてな

2014/03/14 リンク

kfujii これはこれでもっともだけど怒りどころが違うんではという気もして微妙

2014/03/14 リンク

mellow-mikan 「ナンセンスクレーム」の典型例。対応してもユーザーの利便性や品質は何も向上しないけど、クレームを減らすためには必要な作業。ナンセンスクレームを減らすために、あえてユーザーの利便性を落とすこともよくある

2014/03/14 リンク

knjname 非技術系のレッテルをはてなが受けると確実な損害になるのでこれらの類の指摘を否定するしかないのだと思う

2014/03/13 リンク

FTTH 過剰にアットホームな印象を与える、とは思うが「問題」をどこに設定するかだろね。プログラマじゃなくて文学部視点のツッコミ（レビュー指摘）はあってもいいかも。

# |ω・)……

2014/03/13 リンク

規約違反を報告

些末なコードレビュー - naoyaのはてなダイアリー

朝起きて布団から出るのがつらいので、HBFav をつらつらと眺めていた。あるサービスの JavaScript が重... 朝起きて布団から出るのがつらいので、HBFav をつらつらと眺めていた。あるサービスの JavaScript が重いとか、そのコードが難読化されてないとか、担当者とおぼしき人間が書いたコメントがそのまま残ってるから消しましょうよとか、そんなことが書かれていた。JavaScript が重い、という話は結局そのサービスの JavaScript が重かったのではなく、ユーザーが自分で導入した広告が重いというだけの話だった。コードが難読化されていない、趣味の製品ではなく会社の製品なのでコメントそのまま残ってるから消しましょう･･･実にくだらない。ところで話は変わってコードレビューについて。コードレビューに慣れないチームが、何の考えもナシにコードレビューを始めるととにかく気になったこと大小様々な指摘が行われることになる。一見、いろいろな指摘が出て議論が活発になっているように見えるが、だいたい