こんなコードレビューはダメ【VOICEVOX】【プログラミング】

Поділитися
Вставка
  • Опубліковано 29 лис 2024

КОМЕНТАРІ • 27

  • @revlaf4548
    @revlaf4548 Місяць тому +12

    落ち着いたタイミングで「では今までのレビューについてレビューしていきます」とか言うと楽しいことになる

  • @Nanashi774Mumei
    @Nanashi774Mumei Місяць тому +41

    どの業界でも「何でこんな事も分からないの?」は、後輩や部下を潰せる必殺技だと思う。現場によって常識や技術が違うITで言っちゃダメな言葉だと思ってる。

    • @penguin6241
      @penguin6241 Місяць тому +3

      ね。あんまりひどいとこっちもマネジメント方法否定しちゃうぞ❤

    • @ああ-l4g4s
      @ああ-l4g4s Місяць тому

      @@penguin6241
      これいう奴の心理がわからんのやが
      そういうやつからしたら、その業務は息したり歩いたりすることと同じレベルなのか?

    • @sukpajt
      @sukpajt Місяць тому

      場所によっては逆攻めして潰す

  • @中二ヤミナベ
    @中二ヤミナベ Місяць тому +3

    4:51 ちゃんと改善案出してるの好き

  • @ryoma0518
    @ryoma0518 Місяць тому +19

    全て私の上司に当てはまってて異常なんだと認識しました
    挙げ句の果てには「やっぱり君の仕様に戻そう」と言われました

    • @crawpi
      @crawpi 12 днів тому

      まぁ「全部自分でやれ」って新人教育がとんでもないのと同じように、「全部自分で考えて教えろ」って管理職に丸投げする会社もやばいんだよね。
      上司もわからんのよ(上司を擁護する意図ではなく、この視点持っとかないと将来の自分に返ってくる)

  • @xcvqwe3665
    @xcvqwe3665 Місяць тому +2

    勉強会と称して皆の前で公開処刑は意外とありそう且つやっちゃダメ度が高い

  • @福岡仁-s9n
    @福岡仁-s9n 14 днів тому +2

    「be動詞を付ける」となんて説明から、なぜhasError?

  • @penguin6241
    @penguin6241 Місяць тому +12

    コードをバカにするのは飯屋で飯をバカにするのと同じ。

  • @トモット-d9j
    @トモット-d9j Місяць тому +7

    ハードウェア部署の部署内講習でpython初歩やってたけど、50代の方たち半分もわかってなさそうだったから、経験ないこと出来ないのは当然なんだなって

  • @bambooooooooooooooooo
    @bambooooooooooooooooo Місяць тому +4

    こんな上司・先輩になりたいし、下で働きたい……

  • @fuemma--7122
    @fuemma--7122 Місяць тому +4

    豆腐メンタルなので、ホント動画最後のこんな感じで指摘して欲しいです😂

  • @kよろしく
    @kよろしく Місяць тому +6

    レビュー指摘しておいて改善案出さないなんてことあるのか??

    • @中二ヤミナベ
      @中二ヤミナベ Місяць тому +3

      意外と出し忘れるのよこれが

    • @user-gl9tx58qmdf7p
      @user-gl9tx58qmdf7p 29 днів тому +2

      あほな先輩は文句ばっかり言って改善案言わないんだよね

  • @荻野憲一-p7o
    @荻野憲一-p7o Місяць тому +1

    コーデング時間の短縮も「最適化」の一種なので、
    あまりにも簡単なプログラムにお作法を持ち込むのはナンセンスかもしれない。
    Ruby とか Python とかが流行ったのだって、そういうことやろ?

  • @とうふ-w9n1i
    @とうふ-w9n1i Місяць тому +8

    自分で書いた所なのに「ここAIにやらせてコピペしたでしょ?」って言われた
    本当にコピペしてる奴にはより効くのかもしれないけどわざわざクリティカル狙わないでほしい。

  • @岡本裕俊
    @岡本裕俊 Місяць тому +2

    レビュー以前に、「なんでこの仕様からこんなコード(ロジック)になるの?」的なコードが出てくる時はレビューにとても困る。。。
    (内心、「なんだ、このクソコードは?」って思う。。。)

  • @KisekI369
    @KisekI369 Місяць тому +4

    職場で「好みじゃない」なんて言葉が出てくるとは、学生気分の先輩なのかな?

    • @みこと-j2w
      @みこと-j2w 22 дні тому +1

      そう伝えるかどうかは別だがソフトに限らず開発やってたら趣味の範疇の領域はあるやろ
      この記述でも問題ないけど個人的にはちょっとわかりにくいと感じるが修正まではマストではないみたいな微妙なシーンなんて普通にでてくる。
      ダメなこと①は個人的には反対でむしろプロジェクトの誰もが同じコードがかけること自体は理想だろう。
      投稿主は1つのソースを長期にわたって人が入れ変わりながら保守するプロジェクトの経験がないのかもしれんが気持ち悪いと言っているその状況はある意味では保守性が最高なっている状態といえると思います。
      ただコーディング規約に落とし込めるようにちゃんとディスカッションするような形(好みと言っている部分の本質を突き詰める形)で進めなかったから投稿主がマイナスな印象になっちゃってるだけなのかなぁと思います。

    • @phono2
      @phono2 20 днів тому

      これだとxxxの状況の時に困るのでyyyの方が私は好みです、はあるかな。。
      例)
      if (hoge) {
      ⇨ 言語が変わったときに0とかの扱いで結果が変わることがあるので if (hoge === undefined)の方が好みです。
      とか。

  • @phono2
    @phono2 20 днів тому

    そもそもレビュー観点をきちんと発表していないレビュアー多すぎ問題。
    この観点表の必須の項目だけを満たしていればとりあえず通します、っていう感じにしないとレビューなんて思想の押し付けなのでね。その辺、答えがある、って無邪気に考えている傲慢な若手のレビュアー多いよね。

    • @hahahan6483
      @hahahan6483 4 дні тому

      レビュー観点必要なら自分で書けばええやろ😂

  • @yw3459
    @yw3459 25 днів тому

    書いた人を否定するのは良くないが、自分より給料もらってるプロダクトリーダーが過去に書いたコードが静的解析の警告全て無視してたから陰で見下してる