タグ

ブックマーク / qiita.com/matsuken314 (1)

  • フリーランス歴20年の強強エンジニアからのガチコードレビュー集 - Qiita

    はじめに こんにちは、まつけんです。 早いもので、Webエンジニアになって、10ヶ月経とうとしています。 先月末、僕の職場に参画していたフリーランス歴20年の強強エンジニアCさんが卒業されました。(以降Cさんと称します) Cさんには、いつも迅速かつ丁寧なレビューをしていただいてました。 たまに補助で僕のプルリクにコミットを積んでもらうことなどもあり、お世話になった記憶が大半です。 今回はそんなCさんから受けたコードレビューから、今後どう改善していくのかアウトプットして学びを深めたいため、こちらの記事を書きました。 ペアプロしている時の参考になったこともおまけで書いてます。 ※こちらの記事に出てくるコードに関しては全てRubyです。実務で学んだことなので、出てくるコードは全てフィクションです。(実際のサービスのコードではないです) レビュー1: migrationファイルを追加する時「db:

    フリーランス歴20年の強強エンジニアからのガチコードレビュー集 - Qiita
    tkmkg8m
    tkmkg8m 2023/07/16
    include? は本当にそう発言してたんならCさんの理解も怪しいな。Rubyには String#include? があり Array#include? とは関係ない。「#== より緩いから意図せず分岐が真となる(ので #include? は不適切)」という指摘なら妥当。
  • 1