タグ

reviewに関するrydotのブックマーク (15)

  • GitHub - joho/awesome-code-review: An "Awesome" list of code review resources - articles, papers, tools, etc

    An experiment to assess the cost-benefits of code inspections in large scale software development (Porter, Siy, Toman & Votta, 1997) Early paper that tested a range of then-current review techniques including multi-stage review and code-review-via-meeting that found you can get most of the benefit in an offline, single pass, with two reviewers. Anywhere, anytime code inspections: using the Web to

    GitHub - joho/awesome-code-review: An "Awesome" list of code review resources - articles, papers, tools, etc
    rydot
    rydot 2019/05/12
  • 役立つコードレビュー 8つのヒント | POSTD

    役立つコードレビュー(CR)のコツは、学校では習いません。アルゴリズム、データ構造、プログラム言語の基礎は習っても、確実に役に立つフィードバックを返す方法をじっくりと教えてくれる人はいないでしょう。 コードレビューは優れたソフトウェアを作り出すには欠かせないプロセスです。レビューを通したコードは、そうでないコードよりも 質が高く、バグが少ない 傾向があります。健全なコードレビュー文化には、副次的な利点もあります。たとえば、 バス因子 を押しとどめる、新メンバーのトレーニングに最適なツールになる、など。また、コードレビューは優れた知識共有の手段でもあります。 前提 まずは、この記事のポイントの前提を提示する必要があるでしょう。それは以下のとおりです。 信頼のおける環境で作業をしている。あるいは、あなたとチームは、あなたの信頼性を高めることを目指して作業している。 コードではないシナリオでフィ

    役立つコードレビュー 8つのヒント | POSTD
  • なぜあなたのPull Requestは読まれないのか - Qiita

    Pull Requestを出してレビューしてもらってから反映。 どこにでもあるありふれた開発フローに付きまとう、どこにでもあるありふれた問題。 「Pull Requestがレビューされない」 もちろん開発フローにレビューが含まれている以上、レビューをしないメンバーにも非がないとは言えませんが、多くの場合はレビューされないPRには問題があるものです。 デカい 兎にも角にもデカいPRは読むのがつらいです。 もちろん要件が明記されていないなど、他にもPRが読みにくくなる原因はたくさんありますが、一番はこれです。 極端な話、1行変更のPRは他に何も書かれていなくても実装内容を察することができますが、10ファイル100行の差分と箇条書き20点の要件が書かれたPRは内容を把握するだけで一苦労です。 しかし、このこと自体は数カ月でもコードを書いていれば自然と勘づくもの。 問題はなぜPRが大きくなってしま

    なぜあなたのPull Requestは読まれないのか - Qiita
  • コードレビューのベストプラクティス | POSTD

    Wiredrive では、私たちはかなりの数のコードレビューを行います。しかし、ここで働き始める前には私はコードレビューなどしたことがありませんでした。今回は、私がコードレビューをする時に何に注目するようにしているかや、私の考え出したベストなコードレビューのやり方をお話したいと思います。 コードレビューとは、簡単に言うと2人以上の開発者で問題を引き起こしそうなコードの修正について話し合うことです。コードレビューをすることのメリットについては多くの記事で語られており、知識を共有できること、コードのクオリティが上がること、開発者が成長できることなどが挙げられています。しかし、レビューを行う上で、どのように進めていくかという具体的なことについてはあまり多く語られてないように私は思いました。 レビューで何に注目するか アーキテクチャ/デザイン 単一責任原則 : 1つのクラスは変更する理由が2つ以上

    コードレビューのベストプラクティス | POSTD
  • 眼鏡なしのコードレビュー | POSTD

    例えば、あなたが驚くほど聡明な開発チームのメンバーで、コードレビューのみに一日の時間を確保しているとします。しかし作業を開始して2時間後、眼鏡を忘れてきてしまい、午前中はぼんやりとしたカラフルな表示を見つめていただけだったということに気づいたとします。さて、あなたはどうしますか? 家まで歩いて10分もかからないし、天気も良ければ、眼鏡を取りに帰るのが一番です。でも朝家を出るとき、攻撃的なスズメバチの群れが眼鏡の置いてある部屋に巣を作って、邪魔されたくない様子だったらどうしますか? そういう時はもちろん、コンタクトレンズを付けてきたふりをして、恥ずかしい思いをしないようにするのがよいでしょう。実際に読むことなく膨大な量のファイルを見分けることができるということを覚えておいて下さい。 参考コード 1 不安の種は隔離するべきだということに誰も異論はないでしょう。そしてもちろん、あらゆるクラスは一

    眼鏡なしのコードレビュー | POSTD
  • CRPGでの最適化と確認 - 指輪世界の第五日記

    プレイヤーがコンピューターRPGに放り込まれたとき、まず、いくらかの時間をかけての学習過程があります。そこでプレイヤーは「よしこんなもんだろう。」というところまで最適化をします。つまり、どんな手順で呪文をかけアイテムを使うのが強いのか、どんな装備でどこそこの敵を何匹狩り、どれだけMPを消費したら町に戻り、どのアイテムを補給して再出撃するか。そのような最適化されたメソッドを編み出すまでが、学習過程です。これは言い換えると、出題されたリソースマネージメントのパズルを解くことです。この問題を解き/メソッドを得、採用してからが反復作業になります。 たいていのゲームは、最初の反復作業のままでは終わりません。この反復作業中になんらかのポイントが貯まっていって、一定値になったら状況を変えるように仕込んであります。CRPGでいえば経験値/貨幣蓄積によるレベルアップその他の成長がそれです。これが次のエリアへ

    CRPGでの最適化と確認 - 指輪世界の第五日記
  • Process Impact -- Page Not Found

  • 岡部健(kenokabe)氏のIQ145本の哲学部分に関しての註釈メモ|esehara shigeo

    つい最近になって、「関数型言語を教える」という名目のようなが販売されて、プログラマ界隈で話題となりました。このは、一見プログラム入門書のように見えますが、著者の哲学的な思惟が含まれており、それらを知らない人間にとっては、判断が付きようがない品物であることは確かで、その中身の是非を知りたいという人々がそれなりにいることに気がつきました。 自分の場合、元々バックグラウンドが哲学や現代思想をかじって遊んでいた人間というのもあり、それらの議論について、ある程度理解しているつもりですので、その辺で気になった部分に関して、メモ書きをしておこう、と思ったのがこの記事の背景です。 ちなみに、如何なる仮説であれ、それらは平等に扱われるべきです。この著書が問題となるのは、それが如何なる文献を参照し、その判断が正しいのかどうなのか、ということが余りにも不明瞭であり、また議論としても、言い切りが多く、果たして

    岡部健(kenokabe)氏のIQ145本の哲学部分に関しての註釈メモ|esehara shigeo
  • コードレビューの話

    新卒エンジニア向けにコードレビューを「する」話をしました。 http://hisaichi5518.hatenablog.jp/entry/2014/10/29/165721

    コードレビューの話
  • ずるいとunfairの違い - 書評 - ずるい!? なぜ欧米人は平気でルールを変えるのか : 404 Blog Not Found

    2009年12月25日21:00 カテゴリ書評/画評/品評Value 2.0 ずるいとunfairの違い - 書評 - ずるい!? なぜ欧米人は平気でルールを変えるのか ディスカヴァーより献御礼。 ずるい!? なぜ欧米人は平気でルールを変えるのか 青木高夫 今年もあと余すところ一週間。仕事納めは28日のところが多いようだが、一日だけ出社するならいっそと事実上今日が今年の最終勤務日という人も少なくないのではないだろうか。ちなみに我が娘たちは日が終業式。 そして来年始業式を迎える前に、必ず読んでおいて欲しいのが書だ。来年、いや次の10年こそは「新たなルール」から逃れようのない(1|10)年になるのだから。 書「ずるい!?なぜ欧米人は平気でルールを変えるのか」は、ルールとは何のためにあるのかを説いた一冊。 目次 - Discover: ショッピングカートより 目次 第1章 なぜ私たちはル

    ずるいとunfairの違い - 書評 - ずるい!? なぜ欧米人は平気でルールを変えるのか : 404 Blog Not Found
  • 些末なコードレビュー - naoyaのはてなダイアリー

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

    些末なコードレビュー - naoyaのはてなダイアリー
  • Practical OCaml : 絶対買うな - camlspotter’s blog

    Practical OCaml を手に入れた。ただで。買おうとしたらやるから買うなと言われて一年。持ってこいといっても誰も持ってこなかったので、取りに行った。こりゃ内容のわりに重いから誰も持ってこないワケだ。 このについて書くこと自体全く持って時間の無駄なのだが、どうやら Practical OCaml を買って OCaml を勉強しようとしている人達がいるらしい。正直それはあまりに不幸なので、注意を喚起するべくここに書評のようなものを書いておく。 Practical OCaml 作者: Joshua B. Smith出版社/メーカー: Apress発売日: 2006/10/20メディア: ハードカバー クリック: 41回この商品を含むブログ (9件) を見るPractical Common Lisp が名著(だそうだ)なだけに大変残念な結果です。 まず結論から 簡単だ: このを買う位

    Practical OCaml : 絶対買うな - camlspotter’s blog
  • Big Sky :: 「実践Vim 〜思考のスピードで編集しよう〜」書評

    アスキー・メディアワークス様より、献して頂きました。 まず先に総評を言ってしまいますが、Vimmer ならば買うべきだと思います。 Big Sky :: Vimテクニックバイブル ~作業効率をカイゼンする150の技 もちろんこれらのも僕を喜ばす事は出来たのですが、なにぶんページの殆どが知っている情報だったので僕にとって「ワクワクする」では無かったんです。 ずっと最新情報取り入れたVim出ないかなと思ってまし... http://mattn.kaoriya.net/software/vim/20110810203558.htm Vim テクニックバイブルを書いた時にも何冊か Vimを紹介しましたが、これまで「Vim を現場で使う事にスコープを当てた」というのは存在しませんでした。 あくまでリファレンスでしかなく、なぜ Vim はこんな動作なのか、編集する方法には複数の方法が

    Big Sky :: 「実践Vim 〜思考のスピードで編集しよう〜」書評
  • Review Board - It's a bright day for code review!

    It's a bright day for code review! Still on pull requests? See why organizations upgrade to Review Board: Code review, document review, and image review, all in one place Your code and data stays private, secure, and in your control (Review Board won't mine your data for AI training or other purposes) Works with what you use today (such as Git, Mercurial, Perforce, ClearCase, Cliosoft SOS, or Azur

    Review Board - It's a bright day for code review!
  • そして世界へ - 品評 - Note Anytime : 404 Blog Not Found

    2012年09月26日17:00 カテゴリ書評/画評/品評iTech そして世界へ - 品評 - Note Anytime 記者会見が終わったようなのでこちらでも紹介。 7notesで日のiOSユーザーを驚かせたmetamojiは、これで世界のモバイルOSユーザーを制覇するのか。 Note Anytimeのすごさを、言葉で表すのは難しい。難しいので、まずは以下をご覧頂きたい。英語であるが、言葉はいらない。 Note Anytime Note Anytime Introduction Video from Note Anytime on Vimeo. そしてiPadをお持ちの方は、すぐに試してほしい。異論は認めない。なにしろ、無料なのだから。 Note Anytime – Write stylish notes, mash up handwritten text or typed text

    そして世界へ - 品評 - Note Anytime : 404 Blog Not Found
  • 1