エンジニア立ち居振舞い：重箱の隅をつつかない - 面白コンテンツ探求日記

お題「エンジニア立ち居振舞い」 自分は重箱の隅をつつかないというのを意識してる。 重箱の隅をつつく問題はコードレビューの現場でよく聞く。レビューの場で所詮「書き方レベル」の指摘が横行してしまうというやつ。 誰しも綺麗なコードを追求したい気持ちはあると思うし、自分もそうなんだけど、あまり良くないなと思ってやらない事にしてる。理由は3つ。 時間の無駄 指摘される側の精神衛生上よくない そもそも意味ない 細かい指摘でも修正してマージするまでには結構時間がかかる。コードを直し、手元でビルド・動作確認し、pushしてCIを回し、「修正しました」と報告し、LGTMが付いてやっとマージできる。このプロセスが日に何度も発生すると確実に時間を食ってしまうし、Nitsな内容を何度も何度も受けると精神的にも疲弊してしまう。それで生産性が下がってしまえばもっと大きな問題になる。 こうした指摘をしたくなる時、「コー

[Sixeight]

逆に勝手に直しまくってたらめちゃくちゃ険悪な感じになったことがあるので、「どうでもいいことはどうでもいい」という諦めが必要。

[hachibeechan]

プロジェクトの一番最初にlintの設定をみんなで決めて、lintでカバーしきれない部分はほっとくってのが一番費用対効果が高いよねというのは最近よく感じる

[te2u]

重箱の隅をつつけるコードは本筋は問題ない、いいコード。自分の問題は、指摘事項があるときに、その数が増えると重箱の隅をつつくような指摘が含まれてしまうこと。

[lenore]

統一感のある名前付けが必要ならばあらかじめ規約を公開するべきだし、統一感のある記法にしたければツールを導入すべき。そうすることでレビューのひっかかりも短縮するし、規約の間違いも改善される

[morototo]

妥協は大事

[pixmap]

重箱の隅で悩まないよう、共有すべきルールやドキュメントを整えておくのがいいと思う。その上で直せというのではなく、ドキュメントを見てねと伝える。

[beerbeerkun]

こういう雰囲気で仕事したいね。

[aike]

プログラミング言語やフレームワークが進化して昔よりも圧倒的に理解しやすいコードになったので、相対的にコーディング規約レベルの可読性担保の必要性が下がったのもあると思う。

[itochan315]

どうでもよくなりたい

[hi-hats]

“レビューの場で所詮「書き方レベル」の指摘が横行してしまうというやつ” まぁ自分でも分かってて言われるときのモニョっとしたアレはありますね

[suthio]

“もっと根本的な設計だったり、技術選定の失敗といったより大きな問題だった。些細なコードの書き方にこだわって時間を使うより、こうした大きな問題の改善に力を注ぐのがよっぽど良い。”

[peperon_brain]

ここでいう「エンジニア」は「ソフトウェアエンジニア」のことですよね？生産管理とか土木のエンジニアは対象になってないですよね？(重箱の隅をつつく)

[mizukmb]

こういうのはbotにやらせるのが良いって最近知った(重箱の隅をつつかせるという意味)

[haya14busa]

エンジニア立ち振舞い：機械的に判別できる重箱の隅は機械につついてもらう (reviewdog どうぞ)(宣伝) エンジニア立ち振舞い：機械的に直せる重箱の隅は機械に直してもらう(gofmt, fixer, ...)

[yoiIT]

コードならともかく、要件定義書のてにをはを指摘し始める奴とかどうにかならんか

[n_231]

コーディング規約とかガイドラインとか、たとえオレオレでも有ればこういうときに便利。

[koogawa]

ちょっとの修正でも直すのに時間かかるのすごくわかる。。レビューされる側が違うタスクに着手してる場合は特に / なので迅速なレビューを心掛けたい

[knjname]

もうスケジュール的に直せないところでレビュー依頼きて重箱の隅つつくしかないケースもあるぞ / エンジニア立ち居振舞い：重箱の隅をつつかない - 面白コンテンツ探求日記

[kuborn]

まあ大抵の場合は動いてればいいですしね。細かなレビューをしてほしいときと、プルリクのためのレビューを分ければ済む話かな。

[nntsugu]

lintやformatterでフォローできない分はスルーするというルールでいいんじゃない？本質から外れるところは自動化して人間じゃないものに指摘させる＆ヘイトを集めるのが良さげ。　パフォーマンスや読みやすさの観点であ

[Lat]

実際に発言者が手を動かすのなら大歓迎だが、それをコミットするかと言われるとそれはまた別の問題だ。

[ekshinyah]

“指摘される側の精神衛生上よくない”

[AC-10]

ソフトウェアエンジニアって書きなさい(重箱の隅) ／ しれっと直すのは良くないと思う

[kksg]

レビューされる側が気付いてない問題があるかもなので、一応指摘しつつどっちでもいいという立場でやってる

[psfactory]

エンジニア立ち居振舞い：重箱の隅をつつかない - 面白コンテンツ探求日記

[taketyan]

瑣末な自分の好みだという前提で書いておく派。全体的には同意するけどこれ長期的には問題あると思う... > “後日自分がそのコードを触る時にしれっと直すようにしてる”

[voidy21]

妥協のレベル感と好みのレベル感は揃えるの難しい

[mr_yamada]

すごく優秀なんだけど、細かい仕事がびっくりするほど雑な部下がいて、注意の仕方はいつもいろいろと考えさせられる。

[miyadai454]

よくわかる

[yoshikidz]

言い方が大半を占めるよね、、、ほんと面白がって指摘する人いるけれど、ほんと自分のことだけで、人のことわかってないなーって思う

[basyura]

度合いによるけど、可読性が低いものをスルーしてると悪くなることは当たり前に有るけど、そこに限らず良くなることは皆無なんだよなぁ派（少ない経験談より）