【本題】Qiitaでコメントを頂いたゴールデンウィークは、個人開発を行って、その開発過程を毎日Qiitaに投稿していましたが、昨日投稿した記事にコメントを頂きました。動画付きでした!!(コメントが飛んできただけでもビックリなのに、この展開なので、正直ビビりまくっていました・・・)※追記(5/4):伊藤さんからのコメントを「マサカリ」と表現していましたが、「強い批判を指す場合に使用するのが適当」とご指摘頂いたので、文言修正しています。Qiita↓動画www.youtube.comご指摘は、私が記述したテストコードを関する内容です。私が書いたコードと、伊藤さんが修正して下さったコードを紹介します。'jnchitoの解説記事はもう古い!これからはこれがスタンダード。'jnchitoの解説記事はもう古い!これからはこれがスタンダード。'jnchitoの解説記事はもうオワコン!これからはこれがスタンダード。'jnchitoの解説記事はもうオワコン!これからはこれがスタンダード。endendidに依存しているまず、ご指摘があったのは、このコードからです。}こちらidをベタ打ちで指定していますが、テスト環境では、テストを実行する度に、新しいidが割り当てられる為、基本的に以前のidは使い回しません。その為、この様なid依存の記述をしてしまうと、別の実行環境ではテストが正常に動作しなくなり、再現性が無くなります。オブジェクトを生成する際には、原則id指定は避けたほうが良いという事です。下記の様に参照を渡して、外部キーを指定すれば、ベタ打ちをせずに済みます。}ドキュメントとしての読みづらいテストコードは、何もテストだけが目的ではなく、それ自体がドキュメントとしての役割を担うという事も意識する必要がある。いった意味では、I18nやname属性で対象のリンクやフォームを指定する私の方法だと、直ぐにイメージがしづらい欠点があった。頻繁に更新される項目であれば、I18nは有用だが、ある程度文言が固まっているのであれば、画面の表示そのものを記述した方が、理解しやすくなるというアドバイスを頂いた。'jnchitoさんの解説記事は最高!これからもこれがスタンダード。'投稿する'とはいえ、まだフロントの実装イメージが固まっていないので、ここの修正は後々に繰り越して、フロント実装時に変更しようと考えています。letとlet!が正しく使い分けられていないqiita.com宣伝お礼の意味も込めて、伊藤さんが書かれた書籍を紹介しておきます。YouTube《今日の学習進捗(3年以内に10000時間に向けて)》Qiitaで初めて技術的なアドバイスを頂いたが、コメントを送って頂いた方が伊藤さんという事もあり、非常に丁寧に訂正して下さったので、本当に理解が深まる良いきっかけになった。自分の間違いが白日の下に晒されるというのは非常に恥ずかしい事だけど、成長して行くには避けては通れない道だと、改めて痛感した。日)IBMのWatsonきっかけでエンジニアに憧れて28歳未経験でWebエンジニアへ転職。2018年12月から3年間で「10000時間」をプログラミングに捧げる計画を立て、学んだことをブログの投稿中。Activitiesryoutakuこのサイトについてプライバシーポリシ。