タグ

ブックマーク / imagawa.hatenadiary.jp (2)

  • コードレビューのはなし - 今川館

    エンジニアをやっているとコードレビューをしたり受けたりすることが日常的ですが、レビューは人間関係に絡むのでお悩みの職場も多いのではないでしょうか。 そんな中、ぼやっと思ったことをここに書いておきます。 人間関係が浅いうちはレビューコメントにタグつけるといい レビューコメントにタグつける運用あるじゃないですか。 [MUST] ループの中で毎度このオブジェクトを初期化すると初期化コストがかさむのでループの外で一度だけ初期化するようにしてください。 [WANT] 要素の数が事前に判っているので、詰め替え先のスライスはmakeに長さを指定した方が省エネなので良いと思います。 [ASK] 好みの問題なんですけど、初期化した後にフィールドに代入するんじゃなくて、初期化のときに一緒に渡せば良いのではないかと思いました。 こういうの、いちいちつけるの面倒くさいと思うんですけど、レビューア・レビューイ*1の

    コードレビューのはなし - 今川館
    OKIIZO
    OKIIZO 2019/08/08
  • 二人の上司 - 今川館

    雑記 | 22:36大雨の中、友達が家を訪ねてきた。帰りに駅まで送る道を歩きながら、今まで自分が出会った二人の上司の話をしていた。最初の上司: すごい人・憧れる人・気のいい兄ちゃんタイプ最初に出会った上司は凄腕のプログラマーだった。コンピュータのおよそあらゆることに精通していた。WindowsのWordもExcelも使えないような新人の自分がコンピュータの操作方法が判らずわけのわからない質問の仕方をしても、的確に問題点を分析してすぐに解決してくれた。当時の自分はまったくの未経験からSQLJavaのプログラムを書き始めて、知らないこと・わからないことだらけだった。そして新人にありがちな、「何がわからないのか判らない」状態だった。質問の仕方も下手だった。すごい・憧れる・でもこわいこの人はすごい能力を持っていて憧れるけど、指導の仕方はぶっきらぼうで変なことをしたり間違えると怖い。でもいつも状況

    OKIIZO
    OKIIZO 2013/04/12
  • 1