■「【初心者向け】第2回 クソコードを書かないためのテクニック4選」という動画の内容がひどい

https://www.youtube.com/watch?v=yhDLmGpjdms

これよりもっとひどい動画はごまんとあるが、ここまでタイトルで煽っている以上指摘するわ。

全体を通じて

• Ruby なのにインデントがスペース4つ（タブ？）
• Ruby なのに戻り値に return を付けている
• 3問目に突然現れるRubyライクの架空の言語（function - end の形で関数定義する言語で、かつ elsif と表記する言語を自分はしらない）

個別

• 1問目にて、引数としてto_iしたものを渡せばよいと言っているがあえてそうするメリットがない（インスタンスで渡せばいい）
• 2問目にて、Ruby なのに is とか has とかをメソッド名に使っている（それをしないための接尾語「?」なのに）
• 3問目にて、変更後は変更前と挙動が異なる（変更後は例外が発生することがありうる）
• 4問目、これが最悪で、まずメソッド名がget_first_url_parameters_keyとあるが、「first」ならば「url_parameter_key」である（複数形ではない）
• 4問目、頑張って変数名を修正したようだが、修正後のfirst_parameter_keyはkeyではなく、keyとvalueである（ただしハッシュではなく文字列）
• 4問目、そもそもロジックが間違っていて、url.split('?').first だとURLパラメータ以前の部分が取得される
• 4問目、「ライブラリを漁ってみたらもっとうまく書けるかも」と言っているが、URLパラメータの処理をするのに文字列操作を行っている時点でひどいし、それの修正の手段として標準ライブラリを使う方法を提示しないというのはひどい（これは初心者でもURIモジュール使うレベルだし、使うべき）

プロフィール見るとCTOを経て独立してプログラミングスクールの会社やっているっぽいけど、すごい時代だな。

晒しになっちゃったけど、他にも有名（と思われる）プログラミング系YouTuberが実際にコードを書いている場合でひどいのはザクザク見つけられるから、見つけてため息をつくといいと思います。

Permalink | 記事への反応(3) | 11:35

ツイートシェア