忍者ブログ

ひつ(じのひよこが)プログラミングします。
お仕事や趣味で困ったこととか、何度も「あれ?どうだったかしら」と調べたりしたこととか、作ったものとか、こどものこととかを書きます
★前は週末定期更新でしたが今は不定期更新です

2024/05    04« 1  2  3  4  5  6  7  8  9  10  11  12  13  14  15  16  17  18  19  20  21  22  23  24  25  26  27  28  29  30  31  »06

後輩のコードをレビューしての観点 - 4

後輩のコードをレビューしての観点 その1 その2 その3


使い慣れないものを使うときは複数のドキュメント、とくに信頼できるものを参照する

後輩がキーボード入力を受けてアクションを起こさせようとしていた。その際に KeyPress イベントの which プロパティを使って押されたキーを判定していた。しかし、JavaScript の KeyboardEvent (MDN) は charCode, keyCode, which が deprecated になっていたのである。ウェブで検索するとどうしても古い情報に当たることもあるので、出てきた情報が現代でも妥当か否かは気にかけた方がよさそうである。毎回毎回それをやるのも大変なので、使い慣れていないものや古い記事の情報を使う場合はより信頼できる情報源がないか探してみた方がよいかもしれない。

関数・変数名に and と入れたくなったら考え直せ

関数・変数は単一の機能だけを持たせるのが好ましい。すると、命名に and が入っているというのは危険な香りがする。何かそれ以上の話があるわけではないのだが、命名に and を入れたくなったら構造が間違えている可能性を疑った方がよい。

PR

コメント

ただいまコメントを受けつけておりません。

ブログ内検索

P R