タグ

2017年8月30日のブックマーク (3件)

  • コードレビューの極意。それは「自分のことは棚に上げる」こと!! - Qiita

    はじめに:コードを良くするためなら遠慮は不要 昨日Twitterに投稿した内容が思った以上に拡散されていたので、タイムラインに流れてしまわないようにQiitaにも書いておきます。 内容は上に書いてあるとおりです。 コードレビューはコードの問題点を指摘し、そのコードを良くすることが第一の目的です。 そのため、少しでもおかしいと思った部分は遠慮せずにどんどんツッコむ必要があります。 しかし、レビューする側が「これ、自分もあまりできてないんだよなあ」「お前もできてないじゃん!って言われたら返す言葉もないし・・・」などと思って遠慮してしまうと、コードを改善できるせっかくのチャンスが失われてしまいます。 「自分ができているかどうか」と「そのコードを改善すること」は、それぞれ別の問題です。 なので、レビューする人は自分のことを棚に上げてでも、コードの問題点を指摘する必要があります。 また、レビューされ

    コードレビューの極意。それは「自分のことは棚に上げる」こと!! - Qiita
    teckl
    teckl 2017/08/30
    > 「自分ができているかどうか」と「そのコードを改善すること」は、それぞれ別の問題です。遠慮してしまうと、コードを改善できるせっかくのチャンスが失われてしまいます。
  • 些末なコードレビュー - naoyaのはてなダイアリー

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

    些末なコードレビュー - naoyaのはてなダイアリー
    teckl
    teckl 2017/08/30
    ちょっと長めのコードレビューしていて、動作面など問題なさすぎたので寂しくて些末なコメントばかりしてしまって反省…
  • 北朝鮮から発射された弾道ミサイルが日本に飛来する可能性がある場合における全国瞬時警報システム(Jアラート)による情報伝達について - 内閣官房 国民保護ポータルサイト

    ホーム > 参考資料 > 武力攻撃やテロなどから身を守るために > 北朝鮮から弾道ミサイルが発射され、日に影響があり得る場合における全国瞬時警報システム(Jアラート)による情報伝達について 北朝鮮から発射された弾道ミサイルが日に飛来する可能性がある場合における全国瞬時警報システム(Jアラート)による情報伝達について 北朝鮮は過去に例を見ない頻度で弾道ミサイルを発射し、平成28年8月以降、弾道ミサイルの弾頭部分が日の排他的経済水域(EEZ)内に落下する事案も起こっています(※1)。 政府としては、いかなる事態にも対応することができるよう緊張感をもって必要な対応に万全を期しているところです。 北朝鮮から発射された弾道ミサイルが日に飛来する場合、弾道ミサイルは極めて短時間で日に飛来することが予想されます(※2)。 仮に、北朝鮮から発射された弾道ミサイルが日に飛来する可能性がある場合に

    北朝鮮から発射された弾道ミサイルが日本に飛来する可能性がある場合における全国瞬時警報システム(Jアラート)による情報伝達について - 内閣官房 国民保護ポータルサイト
    teckl
    teckl 2017/08/30
    Jアラートの国民保護ポータルサイト、SSL化されてないのね。そして「J」じゃなくて全角の「J」が正式なのかな