コードレビューを支える『褒め文化』 - エムスリーテックブログ
コードレビュー、好きですか？ エンジニアリンググループの山口です。 クラウド電子カルテ「エムスリーデジカル」を開発しています。 今回は、チームに根ざしている『褒め文化』についてお話しします。 ※この記事は、エムスリー Advent Calendar 2018 13日目の記事です。 『褒め文化』とは 簡単に言えば、コードレビューで褒める文化です。 コメントに対してコメントしている様子 とても簡単です。 とても簡単なのですが、前職(SIer)ではこういった経験が全く無かったため*1、join直後は（良い意味で）驚いたのが印象に残っています。 とにかく褒める けっこう安易に安直に褒められますし、褒めます。 アカウント名は一部加工 思ったことを素直にコメントにしてしまいます。 褒め文化の効用 ここからは「※個人の感想です」になってしまいますが、こうした褒め文化は、レビュア・レビュイどちらの立場でも
コードレビューにおけるレビュアー側のアンチパターン
tl;drコードレビューが上手く回って無くてチームが疲弊して辛かったよレビュアーの言い方を変えるだけで大体解決するよ立場とかで例外を許さず、みんながレビューしてレビューされると良いよはじめにあるプロジェクトでGitHubのPRベースでのコードレビューを導入をしました。いかんせんチーム開発が初めてレベルの新人さんが多く、何かと苦労しました。特にレビュイーに対して不効率な指摘はそのまま指示の不明確さに繋がり、チーム全体の開発生産性を下げるので、レビュアーはレビュイー以上に気を使う必要があると感じました。下手をすると、レビュイーのメンタルが弱って闇堕ちするので、チームメンバーの最も大人な人がメンタルケアしたりします。大人な人は大体がリーダー格なので、その人の時間が奪われると何かと開発現場が疲弊しちゃいますね。コードレビューってそんなに難しいものだっけと思ったりもしますが、反省の意味も込めて実際に
【的確】報ステ出演・木村草太氏の森友解説がまっとうすぎると話題に！ネット「田〇の代わりにワイドショーに出て欲しい」
#森友学園 #報道ステーション 木村草太氏の解説、非常に分かりやすく明快です。 （抜粋）「事案の解明が進まないのは政府側が記録無くしたから。籠池側の証言を崩すには人格攻撃せざるを得なくなってる。…（中略）…疑惑が解明出来なければ財務大臣の責任。」 pic.twitter.com/ItEBHAOHWS — ⓢⓐⓘⓣⓞ (@kentaro_s1980) 2017年3月24日 スポンサーリンク 以下ネットの反応。 #報ステ 木村草太さん「解明が進まないのは政府側が記録がないと言っているから。籠池氏の証言を崩すためには、嘘ばかり言う人として人格攻撃をせざるを得ない。仮に籠池氏が潔白でないとしても、国家権力が一私人を人格攻撃するのは常軌を逸した対応。問い正すべきは認可を出した官僚、政治の側」 pic.twitter.com/NwxGcg5o9E — Tad (@CybershotTad) 2017年
プログラムにコメント書かない文化もあるよって話｜NZ MoyaSystem
以下の記事を読んで。 530000micro.hatenablog.com 僕が勤めている会社では、原則、プログラムにコメントを書かないのがルールです。 人生で初めてプログラムに触れてからこのかた、プログラムには必ずコメントを書けと指導されて来ましたし、自分自身も、後輩たちにちゃんとコメント書けよと言い聞かせてきました。そんなわけで、最初に全然コメントのないソースコードの山を見たときは、正直「ゲッ、なんじゃこりゃ……」と面食らったのは確かです。 ところが、「なぜうちのプログラムにはコメントがないのか？」と同僚に尋ねてみると、実に納得の行く回答が返ってきたのでした。 なぜコメントが必要なプログラムを書くのか？ 同僚いわく、「コメントが無くても読めるようなプログラムを書け」という思想が根底にあるのだそう。 適切に関数や変数が命名され、スコープがきちんと管理され、ロジックの流れが整理されているコ
チケット管理のアンチパターンとベストプラクティス
主語がでかいタイトルですが、自分なりに考えようという目的です。 自分の思考の整理。 何か意見があればガンガンコメントなりツイートなりして欲しい。 前提として、 チケット管理システムは２つぐらいしか使ったことがないので、そちらに偏ってしまうかも。 ちなみにRedmineとBacklog。 （今がRedmineだから多分それに偏る気が） アンチパターン アンチパターンというのかは分からないけど 今まで困ったこと、やりにくいと感じたこと、ダメだこりゃ、と思ったこと。 ・運用フローが複雑 フローが複雑だと途端に人は面倒になる。なるべく抜け道を探し秩序が乱れる。 抜け道が見つかると割れ窓理論とかいうやつで、どんどんぐちゃぐちゃになる。 誰もフローに乗ってくれない状態になる。 ・使われていない項目がある なぜあるのかわからない項目。重要そうだけど埋めなくても良い項目。 誰かが埋めてくれるだろう、他の人
最強の靴紐の結び方「イアン結び（6ステップ）」を覚えたぞ！ドクターマーチンの靴紐がほどける問題も解決だ！ - HHS
ドクターマーチンの靴紐ってしっかり結んだつもりでも、いつの間にかユルユルになってしまうんですよ。これが悩みであまり履いていなかったんですがこれからは履きまくれます！ ほどけない紐の結び方をマスターしました！今回ほどけにくい靴紐の結び方を調べていてたどり着いたのが「イアン結び」という結び方です！ イアン結び（6ステップ） ステップ1＆2 まずは左の紐が上、右の紐が下になるように普通に結びます。次に「左の紐（黄色）は手前側にひもを垂らし、右の紐（青）は奥にひもを垂らすように輪っかを作ります」 ステップ3＆4 左の輪に右奥の紐を通します。右の輪には左の手前の紐を通します。ここが分かりにくいんですが最初に貼った動画を何度か見れば大丈夫だと思います。 ステップ5＆6 あとは通した紐に持ち替えて左右にギュッと引っ張れば完成！ これだけで本当にほどけないの？って思いましたが…ほどけないんですよ！簡単だし
GoFの23のデザインパターンを，Javaで活用するための一覧表 （パターンごとの要約コメント付き） - 主に言語とシステム開発に関して
GoFデザインパターンの一覧表と，活用のためのコメント，および入門者が独学するためのリンク集（サンプルコード付き）。 入門者の独学を支援するために，このページのURLを提示して熟読させ，各パターンを短時間で効率よく学んでもらう。 デザインパターンはプログラマの常識だ。 Java使いかどうかは問わない。 にも関わらず，入門書を買ったまま，途中で挫折する人が多い。 挫折の原因は，パターンの数が23もあって，多いからだろう。 全パターンをすんなり覚えてもらうためには，各パターンごとに 「要するにこういう目的のパターンなんだ。」 「10文字で表現すると，パターンの意味はこうなんだ。」 という要点・本質を，短いコメントで伝えれば助けになるだろう。 こういった学習を通して，Java言語の「設計思想」も併せて感じ取ってゆけるはず。 全パターンの一覧表（要約コメント付き） 全パターンについて，10文字以内
又吉直樹さん出演『情熱大陸』での、又吉さんおすすめ本とご購入本をまとめてみた - 積読書店員のつくりかた
「芥川賞芸人」という前代未聞の称号が誕生した先週末。 今回の受賞については、議論もあるところですが「古舘伊知郎氏が意味深な発言 又吉直樹の芥川賞受賞に違和感？ - ライブドアニュース」という報道も逆に話題を集中する効果を果たしているのは間違いありません。古館氏にはいろいろ言いたいこともあるが、それはグッとこらえよう*1。 さて、この又吉さんの受賞は、さらに東山彰良さん『流（りゅう）』にも波及している様相で、通常の直木賞タイトルよりも売れ行き良好なのであります。 肝心の『火花』の売れ行きは想像以上の動きを呈しております。ミリオン確定しておりますが、矢継ぎ早に重版してもなくなってしまうのではないかと思えるレベル。 （記事）「芥川賞・又吉直樹さんの「火花」、１００万部突破：朝日新聞デジタル」 ところで、ダブル受賞であるにも関わらず、「もうひとりの受賞者」と呼ばれる形となった羽田先生については、i
IT屋必見！ 『コマンドプロンプト』のストレスが少し減る小技集 - ゆとりずむ
こんにちは。らくからちゃです。 今日も、東京も東京証券所も雨模様ですね。昨日よりかは、多少小雨になってきたようですが、遠くの方に台風が見えるのが気になります。最大瞬間風速50メートル以上と、大変強力な台風なので、お庭においてあるあれやこれも、ポートフォリオも吹き飛ばされないようにご警戒ください。 さて、自己紹介のところでも書かせて頂いていますが、なんちゃってコンサルタントをやらせて頂いております。ただ、システムのインストールやら、アドオン開発のテストなどなどで、見よう見まねでエンジニアっぽいことをやっている時も御座います。（到底本職の人には敵いませんが・・・） まあお客様がご利用している端末がWindowsであることが殆どですので、システムをメンテナンスする際にもWindowsのコンソール機能である『コマンドプロンプト』を使うことになります。このコマンドプロンプト、ちょっとした小技で随分い
あるシステム屋さんが平均残業時間一桁を実現した方法 - ゆとりずむ
こんにちは。 ここしばらく、システムトラブルの対応で午前帰りが続き、疲れてきてしまいました・・・。直接、トラブルの原因になった訳では有りませんが、エンジニアさんも巻き込んでしまい、もう少し上手く回す方法はなかったのかと、自分の未熟さを反省中です。 さて残業といえば、先生は大変そうですね。ただでさえ、ひとりで何十人もの生徒をみないといけない上、ほぼ無償ボランティアの部活顧問まで行い、その上で親に押しかけられたら溜まったもんじゃ有りませんよね。横浜市で、先制の『ノー残業デー』を設定するそうですが、多少なりとも状況が改善することを期待してやみません。 ただ、個人的にはこの『ノー残業デー』という制度がしっくり来ません。だって、『ノー残業デー』って、その日以外は残業することが前提なワケですよね？更に、こんなニュースも有ります。 正社員と同じ等級制度や人事制度を用いるため、基本給も同じ水準だ。賞与は正
意外と知られていない、はてブの神機能 - CGMプランナーの思考
はてブの機能を組み合わせるとこんなことが出来る。 まぁまずは見てほしい。 http://bukuma-diver.com/ ブックマークをWeb上に記録したり、Webページにコメントを付けたり出来るサービスは世の中にたくさんある。日本においては、はてなブックマーク、いわゆる「はてブ」が最王手最大手である。 そのため、ややIT系に偏っているものの、はてブのホットエントリーをチェックすればネットでの旬のページをだいたいチェックできる。 さらにはてブの拡張機能を入れると、閲覧しているページのブックマーク数が表示されるため、そのページがどれくらいブックマークされているかで人気度を把握することが出来る。それだけでなく、拡張機能のボタンをクリックすれば閲覧中のページに寄せられたコメントを確認することも出来る。 そしていつしか、「ブックマーク数が多いほど良いページ」となんとなく思うようになり、「良い」と
コードに対してコメントを書くと実装に関するコメントになる - きしだのHatena
おととい、渋谷JVMというイベントがあって登壇させてもらったんですが、そのあとビール飲んでるときに、ぼくが「コード書く前にコメントだけ書くのいいよね」と言ったあとの返答としてきょんくん(kyon_mm)が言った言葉。 全体としては 「コード先に書いてそのコードに対してテストを書くと実装に対するテストになるし、コードを先に書いてそのコードに対してコメントを書くと実装に対するコメントになる」 という感じ。 ここに至るまでの話もおもしろかったんだけど、ここでは、コメントについて書いてみます。 まず、実装に対するコメントってどういうのかというと、こういうの。 id = findId(name); if(id == -1){ // idが-1だったとき登録 register(name); } いやそれはコード見ればわかるから、ってやつですね。 これは、こうやるとより適切です。 id = findId
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
http://post.simplie.jp/posts/34
http://harold-spm.com/programmer-kyoutuuten/
20150327 古館VS古賀