入社からの半年間でコードレビューで指摘されたことのまとめ - 30歳からのプログラミング

実務未経験でプログラマとして入社して半年以上が経った。 コードレビューで指摘されたことを備忘録としてまとめておく。 自分なりにまとめたものなので、レビュアーが言いたかったこととニュアンスや解釈がずれている可能性はある。 初歩的な内容ばかりで我ながらうんざりする。 せっかく優秀な同僚ばかりなのだからもっと高度なことを学びたいが、こういう初歩的なことが出来ないのが俺の現状なのだから、仕方ない。 そもそもPullRequestを送ったこともなかったわけだし。入社初日は、一人でPullRequestの出し方を練習していた。 それを考えればまあ、こんなものだろうか。 当たり前のことをちゃんと当たり前に出来るようになって、早く、次のステージに進みたい。 PullRequest(PR) PRのタイトルは分かりやすいものに。必要に応じてチケットの番号なども入れる。 コミットやPRは出来るだけ粒度を細かくす

[hiro_curry]

これが好き。「適切な名前を出すことが出来ない場合、設計や抽象化が間違っていることが多い」逆に語彙力が増えると、設計の幅が広がるということ。

[KentarouTakeda]

指摘は的確だし本人もそれを真摯に受け止めてる。素晴らしくそして羨ましい。初学者の頃にこの環境で学べる機会が用意されたことは本人にとっても周囲にとっても変え難い資産になると思う。いいもの読んだ。

[electrica666]

こんな真摯な姿勢、伸びるに決まっている。

[magnoliak]

凄く良い環境なのでは？ということと、ご本人が真摯に受け止めてて素晴らしいまとめ

[matarillo]

リーダブルコード読もう（会社に買ってもらえ）

[kibitaki]

いい職場かなと思ったが「～それが起こりえることなのかと混乱させてしまうので避けた方がいい」これだけで絶望に。Err側の管理してないの？こんなことで混乱する奴に合わせて保守性犠牲にできない。

[murasaki11]

これ、変更の1行1行にしかコメントしないタイプじゃなくて総括してコメントしてくれてる人がいそう。レビューの仕方って誰も教えてくれなくて変更にツッコミを入れるのがレビューだと思ってる人もいるから。。

[morioka]

自分を振り返ると凹む

[solidstatesociety]

これそもそもレビュー前にコーディングのガイドラインをまず立てられてなさそうなのがすごく心配。会社の不準備。

[odakaho]

変数名(とくにboolean)がおかしな人のコードは警戒します。変数に限らずクラスやメソッドの役割分担が怪しかったり、そのせいでコメント文が仕様の説明になってなかったりで保守性が最悪なことが多い。

[monokuma12]

入社したころこんな指摘受けたなぁと共感。

[ottonove]

@@, スキルアップ, 自己啓発

[Mukadon]

ためになった

[peyangu485]

ここまできちんと指摘してくれるのは羨ましい。コードレビューなんてされたことない……。

[minoru0707]

勉強になる

[inaba629]

こういうことを学べる環境って良いな

[ntstn]

参考になる。できてないこともちらほら。

[muddydixon]

たいへんよい

[suquiya0]

例外処理はいちおうやっておいたほうがいい…。ほかは参考になります。

[t_f_m]

あとで

[moccos_info]

よいリストだけど、こういった一般的な話だと、レビューで個別にチマチマ出すより普通に学習すべきなのではとも。

[yohskeey]

べんきょうになる

[seahouse]

参考

[nasio7]

メーモ

[oakbow]

半年間でこんだけ指摘してもらえるのも凄いな。一部はプロジェクトの取り決めにもよるかな？ってのがあるけど

[sylvan_l]

入社からの半年間でコードレビューで指摘されたことのまとめ

[fukken]

"処理の流れを分かりやすくする。頭のなかの変数を少なくする" クッソ分かる。プログラム言語の機能の大半はこのためだけにあると思ってる

[susanne]

チームごとにこういった感じで考え方がちゃんとまとまっていたらやりやすいだろうなぁってしみじみ

[hamlet-r]

後から、この記録を部署のみんなで共有、蓄積して、組織の力に出来るかが会社としての実力を問われるところかな。昔、話題になったプロセス改善モデルCMM、Level5の本質はそんなところだったと思う。

[EastHop]

素晴らしい

[christine_signoret2090643]

未経験半年でプログラマってすごいな

[teruroom]

優秀なプログラマなら割りとレビュー受けなくても自然に行ってる事が明文化されててなかなかいいと思う。

[peroon]

“データ構造を考えるときは、RDBの設計をイメージする”

[hiroomi]

”バグの対応は、以下のような手順で バグを再現させるテストコードを書く バグを修正する バグが再現しないことを確認する ”お疲れ様です。

[igrep]

ええ話や。。。

[t2wave]

“Object.assign()の仕様を理解し、基本的には第一引数には空のオブジェクトを渡すようにする”

[kitadon]

メモ。

[takuver4]

“起こりえない条件に対してコードを書くのは、それが起こりえることなのかと混乱させてしまうので避けた方がいい” 真面目にテストコードを書くと痛感する。

[reitanigawa]

勉強になる

[mint6626]

半年でレコードデビューできた話かと思った