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

2017/08/26 18:29:49

magnoliak

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

2017/08/26 20:54:10

morioka

自分を振り返ると凹む

2017/08/26 21:25:50

mexxx

すげー初歩的な指摘の中に、すげー本質的な指摘が混在していてびっくりする。30歳未経験を採用した上でこんな指導をくれるとは、とてもいい職場で働いてそう。

2017/08/26 22:07:29

KentarouTakeda

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

2017/08/26 22:50:48

letsspeak

いい会社っぽい

2017/08/26 22:53:07

Hamukoro

適切な名前を出すことが出来ない場合、設計や抽象化が間違っていることが多い

2017/08/26 23:11:38

sometk

上司や目上・同僚で、見られるのが嫌で隠蔽体質な人がいて困ったりする。

2017/08/26 23:14:19

sassy_watson

こういうまとめをつくりたい

2017/08/26 23:58:09

knewton

すごくよい指摘だ。

2017/08/27 00:16:20

tiki0108

たまにガチで「isExist()」とかメソッド名付けてくる人に一度は読んでおいてほしい

2017/08/27 00:21:22

psfactory

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

2017/08/27 00:26:34

t-tane

勉強になる

2017/08/27 00:32:20

electrica666

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

2017/08/27 01:05:36

slkby

(羨ましさのあまり笑顔で誤魔化しているときの顔)

2017/08/27 01:17:15

honeybe

いい職場感ある。こういう指摘が出来る人間になりたい。

2017/08/27 01:36:08

uittie

勉強になります。

2017/08/27 01:49:35

odakaho

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

2017/08/27 02:12:35

hiro_curry

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

2017/08/27 02:29:26

bull2

コード

2017/08/27 02:33:28

kyogoku174

プログラマーを育てる人としての才能があるんだろうな。

2017/08/27 02:52:18

matarillo

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

2017/08/27 02:55:39

kibitaki

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

2017/08/27 02:58:07

Derabon

2017/08/27 03:16:23

tettekete37564

短絡評価が短絡されない言語もあるんだぜ？サービス・システム系だと起こり得ない条件全部まとめてフェイルセーフしないとダメだぜ？複雑な三項演算子は将来的に判定条件が不変である時以外では使うべきではないぜ

2017/08/27 04:33:20

kuronat

リーダブルコード100回読むより価値ある体験だこれ、すごい

2017/08/27 05:14:33

fnwa

いい職場に入りましたね。

2017/08/27 07:12:18

atori07

こんな指摘してもらえるなんて素敵やな

2017/08/27 07:29:27

headless_pasta

良いエンジニアさんだし、良い職場だなー。羨ましいレベル。

2017/08/27 07:44:17

motobitsrk

デスマーチ上等な会社なら指導するのも困難になるので、PMは思い切って優秀なプログラマーの負荷を下げる試みをやってみると良いだろう。そうすると、こうなる。

2017/08/27 08:22:02

uturi

初歩的な部分もあるけれども全体を見通しての指摘内容もあり、かなり良いまとめ。

2017/08/27 08:23:01

kuroaka1871

いい職場

2017/08/27 08:24:22

tydk27

これだけ良いレビューをしてくれるなんて、超絶優良企業且つ超優秀エンジニアがいるってことですよね。うらやましい。

2017/08/27 09:30:32

naga_sawa

『適切な名前を出すことが出来ない場合、設計や抽象化が間違っていることが多い』

2017/08/27 09:33:57

K2ICE

“適切な名前を出すことが出来ない場合、設計や抽象化が間違っていることが多い”

2017/08/27 09:37:54

enmtknt

実務経験半年でこれだけの指摘が腹落ちしているなら、これからめちゃくちゃ伸びそう。

2017/08/27 10:00:58

murasaki11

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

2017/08/27 10:17:01

ken3link

コツを聞かれたら、ここを紹介する。。。

2017/08/27 10:22:51

kkkklog

良いなー、プログラミングしたい

2017/08/27 10:26:26

kisara_icy

勉強になる

2017/08/27 10:32:16

sumithsonian

ブコメもとても好意的。真摯さっていつまでも必要なスキルだと思う。

2017/08/27 10:42:14

hkmaro

コードレビューしてもらえる会社うらやましい。

2017/08/27 10:47:42

lightcyan

素朴な疑問。この会社って日系？

2017/08/27 11:02:52

t_pastorale

勉強になった

2017/08/27 11:09:37

kirifue

的確で詳細なコードレビュー。それが当たり前のようにできる会社もスゴイ。 #開発 #プログラミング

2017/08/27 11:13:31

xyamashita

これはよい記事

2017/08/27 11:14:08

EnogunoCap

そこそこいいとこに入社している感じ

2017/08/27 11:14:37

cl-gaku

これだけ吸収できるなら素質ある

2017/08/27 11:17:20

nunulk

最近レビューする機会が増えたから、参考にしたい

2017/08/27 11:25:54

solidstatesociety

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

2017/08/27 11:26:44

t-cyrill

内容が的確

2017/08/27 11:34:21

mutinomuti

コード書く職種じゃないからブックマークしなかったけど、ちゃんと指導されてるのは感じた。応援しています(●´ω`●) というかブックマークが伸びててびっくりした

2017/08/27 11:50:07

chocolate0521

エンジニアしかもう仕事なさそうなわしだけど、何言ってるのかさっぱり分からんかったガクガクブルブル(´Д｀)めう

2017/08/27 12:13:10

marisatokinoko

半年でここまでってめちゃくちゃ素質あるのでは／キャメル、スネークの変換はコストとバグ生むリスクの方がどうやっても大きいかなあと思ってしまう

2017/08/27 12:18:06

laiso

めっちゃ学習能力高い人そう。PRになるから会社名出せばいいのにな

2017/08/27 12:30:42

hinashiki

コードレビューの参考、基準にできる良記事

2017/08/27 12:36:10

kikuchi1201

コードレビュー

2017/08/27 12:37:08

Shisama

自分のPRも振り返ってみよ

2017/08/27 12:47:35

takatama

よく言語化されてるなー

2017/08/27 13:02:12

seiunsky

いやーこれは良い会社っぽいなー

2017/08/27 13:22:09

nisshin-k

半年どころか五年以上やってても現場によっては学べない内容もある。指摘内容を振り返って取り出せてるのもすばらしい。やる気の無い自分みたいな人間は次の日には忘れてるからなぁ

2017/08/27 14:20:52

mint6626

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

2017/08/27 14:32:57

reitanigawa

勉強になる

2017/08/27 14:39:28

blp1526

“適切な名前を出すことが出来ない場合、設計や抽象化が間違っていることが多い”

2017/08/27 15:39:09

takuver4

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

2017/08/27 16:04:11

kitadon

メモ。

2017/08/27 16:24:33

t2wave

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

2017/08/27 16:52:48

igrep

ええ話や。。。

2017/08/27 18:34:34

peroon

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

2017/08/27 19:10:06

teruroom

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

2017/08/27 20:08:37

christine_signoret2090643

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

2017/08/27 20:44:54

EastHop

素晴らしい

2017/08/27 21:06:38

hamlet-r

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

2017/08/27 21:38:17

susanne

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

2017/08/27 23:02:46

fukken

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

2017/08/27 23:54:11

sylvan_l

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

2017/08/28 02:42:03

oakbow

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

2017/08/28 06:52:55

nasio7

メーモ

2017/08/28 09:26:45

yohskeey

べんきょうになる

2017/08/28 12:40:50

moccos_info

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

2017/08/28 20:28:55

t_f_m

あとで

2017/08/29 06:59:50

suquiya0

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

2017/08/29 19:55:01

muddydixon

たいへんよい

2017/08/30 11:06:06

ntstn

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