コードレビューにおけるレビュアー側のアンチパターン
tl;drコードレビューが上手く回って無くてチームが疲弊して辛かったよレビュアーの言い方を変えるだけで大体解決するよ立場とかで例外を許さず、みんながレビューしてレビューされると良いよはじめにあるプロジェクトでGitHubのPRベースでのコードレビューを導入をしました。いかんせんチーム開発が初めてレベルの新人さんが多く、何かと苦労しました。特にレビュイーに対して不効率な指摘はそのまま指示の不明確さに繋がり、チーム全体の開発生産性を下げるので、レビュアーはレビュイー以上に気を使う必要があると感じました。下手をすると、レビュイーのメンタルが弱って闇堕ちするので、チームメンバーの最も大人な人がメンタルケアしたりします。大人な人は大体がリーダー格なので、その人の時間が奪われると何かと開発現場が疲弊しちゃいますね。コードレビューってそんなに難しいものだっけと思ったりもしますが、反省の意味も込めて実際に
コードレビューの高まった言葉 - 職質アンチパターン
ブログ間違った，普段こういう事はこっちに書いてます． http://moznion.hatenadiary.com 最近自分がコードレビューで使いがち，あるいは表立って使ってないんだけど内心評す時に使う言葉が色々とあり，まとめてみることとした．参考にしない方が良いと思う． 左は言葉，右は説明． 屈強 - コードが力強い時に使う．例えば長い一枚スクリプトとか，コメントが一切ないバッチ処理とか．やや批判的な意味合いで使うことが多い． マッチョ - 屈強と同じ文脈で使いがち 屈強だけどしなやか - 屈強だけどしなやかな時に使う．好意的な屈強さと言える． モノリス - 長大なトランザクションスクリプト見た時とかに使う．やや批判的． 言い訳ないですか - 後で直していくぞ！ というメンタルの時に書かれたコードのコメントが案外少ない時に使う言葉．言い訳は無いよりあった方が良い．実際には「もうちょっと言
[翻訳] コードレビューについて
この記事は::..: glen.nu :.: ramblings :.: on code review :.::の意訳記事です。@9len氏の許可を受けて投稿しています。誤り・修正などがありましたら、@iwashi86までご連絡いただけますと幸いです。 This article originally appeared in English at :..:: GLEN D SANFORD :.: RAMBLINGS :.: ON CODE REVIEW ::..: and has been translated with @9len’s permission for posting to this blog in Japanese. この記事は2014年3月に書いている。Twitterでユーザ検索チームを私が率いていたころの話だ。この記事は、コードレビューに関するセオリー・アプローチを体系化
![[翻訳] コードレビューについて](https://cdn-ak-scissors.b.st-hatena.com/image/square/4b2b513cd4b9941cd623836a282bad7afa76c330/height=288;version=1;width=512/https%3A%2F%2Fiwashi.co%2Fassets%2Fimages%2Fogp-min.png)
つらくないコードレビューの運用 - Speaker Deck
All slide content and descriptions are owned by their creators.
次のAmazonレビューを確認してください: SANYO ICレコーダー ポータブルラジオレコーダー ICR-RS110MF(S)
SANYO の IC レコーダーは優れものである。先ず、音が良い。30年数前に数十万円を投じてサロンの一角を占領しているハイファイ・セットの機能のすべてが小さなボディに凝縮されている。電波時計が内蔵されているので秒単位の正確さでAM、FM番組のタイマー録音ができる。内蔵マイクで野外の小鳥の鳴き声のステレオ録音ができる。２GBのマイクロSDを使うと35時間の音楽のステレオ録音ができる。70分のCDならば30枚である。 私は、家内が脳内出血の後遺症で半身不随となり、病院でベッド生活をしている。初夏には近くの山でウグイスの鳴き声を録音してイヤホンで聞かせた。最近は病院の許可を得てスピーカー（クレードル）を枕元に置けるようになったので、専ら音楽を聞かせている。 IC レコーダを2台購入し、1台は自宅においてFMのタイマー録音、CDのコピイ、編集したマイクロSDの作成に使い、もう1台を家内の枕元に置
10年間で最も人気を集めた“至極の感想”は はてなブックマーク「Amazonレビュー」ランキング - はてなニュース
読んだ本に衝撃を受けたから、その感想にも熱がこもってしまう。遊んだゲームがあまりにも素晴らしくて、つい遊び心の詰まった文章をつづってしまう。Amazon.co.jpのカスタマーレビューには、時に読む者の心をぐっと引き寄せる“至極”の紹介文が投稿されます。その熱意に押されるように、気が付けば購入ボタンをクリックしていることもしばしば。はてなブックマーク10周年の記念企画として、今回は過去10年間でブックマーク数が多かった「Amazonレビュー」の上位50エントリーを発表します。（集計期間：2005年1月1日～2015年6月30日） ▽ Amazon.co.jp： 色彩を持たない多崎つくると、彼の巡礼の年のドリーさんのレビュー 1位に輝いたのは、2013年に発売された村上春樹さんの小説『色彩を持たない多崎つくると、彼の巡礼の年』に寄せられたレビュー。投稿者のドリーさんは星1つの評価と辛口ですが
コードレビューのベストプラクティス | POSTD
Wiredrive では、私たちはかなりの数のコードレビューを行います。しかし、ここで働き始める前には私はコードレビューなどしたことがありませんでした。今回は、私がコードレビューをする時に何に注目するようにしているかや、私の考え出したベストなコードレビューのやり方をお話したいと思います。 コードレビューとは、簡単に言うと2人以上の開発者で問題を引き起こしそうなコードの修正について話し合うことです。コードレビューをすることのメリットについては多くの記事で語られており、知識を共有できること、コードのクオリティが上がること、開発者が成長できることなどが挙げられています。しかし、レビューを行う上で、どのように進めていくかという具体的なことについてはあまり多く語られてないように私は思いました。 レビューで何に注目するか アーキテクチャ/デザイン 単一責任原則 : 1つのクラスは変更する理由が2つ以上
人生はすべて「プレゼン」と「レビュー」である - リクナビNEXTジャーナル
Photo by University of the Fraser Valley こんにちは、はせ おやさいです。社会に出て早◯年、いろんな職場を経験しましたし、いろんな人と一緒に仕事をしてきました。 今日は「ある強烈な上司」のもとで働いていたとき教えてもらった、一生ものの考え方について書いてみようと思います。 ■ 一緒に働いていた「ある強烈な上司」の話 その「ある強烈な上司」は徹頭徹尾ロジカルな人。100％の再現性がないものを嫌い、感覚だけで何かを判断することを嫌う、とてもアクの強いタイプでした。 当時手掛けていたコンテンツのデザイン案を複数種類持っていき、「お前はどっちがいいと思う？」と聞かれたとき、「うーん、なんとなくこっちですかね、かわいいから」とウッカリ答えてしまったことがありました。 わたしの返答を聞いた彼の表情はみるみる変わり、「なんだ、その説明は！ その『かわいい』というも
コードレビューについて : 小野和俊のブログ
伊藤直也さんが「些末なコードレビュー」というエントリを書いて話題になっている。このエントリで伊藤さんはコードレビューの話と、はてなのJavaScriptの話と2つの話題に触れている。前者のコードレビューについてはアプレッソでは8年ほど前から「コードレビューを通っていないコードはコミット不可」というルールですべてのソースコードに対してコードレビューを必須にしてきた関係で私も思うところがあるので、エントリを書いてみようと思う。 伊藤さんが例示しているように、インデントやreturnの省略などの話は好みの問題であり、議論してもソフトウェアの改善につながらない。なのでコードレビューでこうした宗教論争が起こるようなら、コーディング規約を見直すべきだ。「無駄に悩んだり議論したりすることを減らす」ことはコーディング規約の主たる効果のひとつだと言える。 コードレビューに慣れないチームが、何の考えもナシにコ
レビュータイムの導入・消滅・再導入 - $shibayu36->blog;
今日こんなかんじの会話があって、レビュータイム導入した時のことを思い出したので、適当に書こうと思う。 ひさいちレビュー、必ず通すみたいなの良いのか悪いのか— ひさいち (@hisaichi5518) 2014年3月13日 @hisaichi5518 マジレスすると、そのような体制にしておくとスケールしないので、最初の段階では必ず通すというルールにしつつ、他の人がレビューしても大丈夫に出来るように、レビューの練習を同時にしていってもらうとしないといけなさそう— 柴崎優季 (@shiba_yu36) 2014年3月13日 @hisaichi5518 今のチームで新人が入った時は、レビュータイムというのを必ず設けてその時間には最低限どれか一つレビューするというのをやってもらってる。でも慣れるまではこれまでチームにいる人がレビューしないとmergeしないということにしてる。— 柴崎優季 (@shi
些末なコードレビュー - naoyaのはてなダイアリー
朝起きて布団から出るのがつらいので、HBFav をつらつらと眺めていた。 あるサービスの JavaScript が重いとか、そのコードが難読化されてないとか、担当者とおぼしき人間が書いたコメントがそのまま残ってるから消しましょうよとか、そんなことが書かれていた。JavaScript が重い、という話は結局そのサービスの JavaScript が重かったのではなく、ユーザーが自分で導入した広告が重いというだけの話だった。 コードが難読化されていない、趣味の製品ではなく会社の製品なのでコメントそのまま残ってるから消しましょう･･･実にくだらない。 ところで話は変わってコードレビューについて。 コードレビューに慣れないチームが、何の考えもナシにコードレビューを始めるととにかく気になったこと大小様々な指摘が行われることになる。一見、いろいろな指摘が出て議論が活発になっているように見えるが、だいたい
コードレビューを円滑に行いたい (#cross2014 のお話) - $shibayu36->blog;
id:antipopさんやid:studio3104さんに機会をもらえて、CROSS 2021に参加させてもらい、はてなでのレビューの話を軽くさせてもらった。はてなからは僕とid:hakobe932さんとで参加した。 http://blog.kentarok.org/entry/2014/01/18/204552 2014/1/17 #cross2014 コードレビューCROSS 〜ぶつかり稽古 2014初場所〜 - Togetter それで、今回参加して他の会社の人のレビューの話も聞いて、あーそれはあるあるとか、そういう問題解決するためにこういうことしてますとか、他の会社ではこういう時どうしているんだろとか、幾つかおもうところがあったので、もう少しレビューのことについて書いてみる。 レビューと関係性問題 レビュアーとレビュイーの関係に関して - 職質アンチパターン コードレビューと関係性
Amazonレビューをお金を払って書いてもらったらかなり問題になっていた件について - きんどう
この記事はKindle作家の”七夕賢”さんからゲストポストいただきました kindleストアで自著「きっと世界でいちばんやさしい組織論」と、その英訳版を出版しています七夕賢と申します。現在は英訳版を世界に向けてプロモーション活動中です。 アメリカではKindle本の「レビュー」をお金で買うことができます。しかし、この行為はガイドライン違反であり、そのことで、いま向こうで議論が巻き起こっています。 そこで今回は、自著のプロモーション活動の中で実際に私が経験したことについて書きます。 海外ではAmazonレビューをお金で買える 今月の十三日から三日間、私がkindleストアで出版した英語の本の無料キャンペーンを行います。そのための準備として、kindle本の情報を扱う海外のサイトに対しての告知を行っているところです。無料で載せてくれる所も多いですが、評価の高いレビューがたくさんついていなければ
コードレビューツール 6選 どれが最適？ | Act as Professional - hiroki.jp by HIROCASTER
Pythonで書かれたレビューツールです。VMware社内で利用されていることで有名なツールです。 プレコミットレビューという概念のレビューツールです。つまり、コミット前にレビューをするという事が前提になっているツールです。よって、結果的に差分を重点的に確認していくツールのつくりになっています。 rietveld rietveld – Code Review, hosted on Google App Engine – Google Project Hosting Google社内で使われているコードレビューツールである「Mondrian」のオープンソース版です。基本的にGoogle App Engineで動くことが前提になっています。 GAEの上のコードのデータを置くということがオトナの事情的に難しいかもしれませんが、検討してみてください。 Phabricator Phabricator
コードレビューについて - camlspotter’s blog
このところ立て続けにコードレビューについて話をする機会があったので 私が経験した最高のレビュー体制を簡単にまとめておこうと思います。 利点 何故必要か 何が嬉しいのか コスト うまく回すためには何が必要か 細かい運営方法 はっきり言って当たり前の事しか書きません。 私も当時は当たり前のことだと思っていましたから、特に気にもしていなかったのです。 ただ見聞するところによると、これをちゃんとやっているところはとても少ないようです。 ウォールストリート系のファンドでもろくにレビューしてないとかどういうことなんでしょう。 だから時々会社が吹っ飛ぶんですね… 結局は、ああだ、こうだ各論を言っても、ちゃんとやれるのか、それ一点に尽きてしまう話なのですが… 利点 レビューを何のためにするか、それはまず第一に自分達の書いているコードに潜在するバグによる損失をできるだけ少なくすることでしょう。 型システムや
シンプルで格好いい。親切なコードレビューシステム·Barkeep MOONGIFT
BarkeepはGitリポジトリに対応したユーザビリティ高いコードレビューシステムです。 会社でプログラミングを行っているとそのコードの品質はばらつきが出てきます。そうするとバグが多くなったり、予期しない問題に直面したりします。それを防ぐのに有効なのがコードレビューです。Barkeepはユーザフレンドリーなコードレビューシステムになっています。 メイン画面です。コミットログが並んでいます。 詳細です。差分が表示されています。 サイドバイサイド。アニメーションしながら表示されて格好いいです。 コードをダブルクリックするとコメントできます。 コメントしました。 一つにまとまっている場合もコメントできます。 レビュー依頼もできます。 ステータスです。レビューされている、されていないといった情報が一目で分かります。 検索結果です。 こちらはプロフィール。 Barkeepは検索における入力補完やフィ
Tiempo es dineroさんのアマゾンレビューが大作過ぎる: administratorの日記
素晴らしいレビュアー発見。 ここまで来ると単なるレビューじゃなくて完全に著作物性が肯定される気がします。 ということで転載はちょっとアレなんで、特に気に入ったものをいくつか外部リンクでご紹介します。 シュアファイア LX2 ルママックス（LEDライト？） タイトルからして素晴らしい。 appleは是非この人を広報に採用するべきでしょう。 ロックスター エナジードリンク 250ml×30本 558 人中、490人の方が、｢このレビューが参考になった｣と投票。 そりゃそうでしょう。 レッドブルのレビューなのかロックスターのレビューなのか読んでるうちに混乱してくる程グングン引き込む筆力。 素晴らしい。 僕らの漫画 (ビッグ コミックス〔スペシャル〕) 何でしょう。 漫画の内容には一切触れていないのに「買いたい」と思わせてしまうこの文章。 作品への計り知れぬ愛すら感じます。 日本人が忘れてしまった
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
コードレビューガイドライン #loupestudy