現在のインスタンスを参照するthisキーワード、付ける？ or 付けない？

まりもさんの書き込み (2007-07-10 00:02) より:

メソッド内変数か、インスタンスフィールドか、静的フィールドか、常に区別しておくのが大切というのは確かにそうです。が、それは、ソースを読むたびに必ず強く頭に置かねばならないものでしょうか？

実際のところ、インスタンスフィールドと静的フィールドは同名のものが使えないので、
宣言するときに意識すればよく、使うときに混乱することはないですし。

thisとかは、確かに重要と言えば重要なんですが。
常に注釈としてつけるのは、帰って分かりにくいと思うのです。
細かいことまでいちいち全部書いてあるのがわかりやすいというものでもないでしょう。
雑多な情報が全部書いてあるとかえって見にくくなることもあります。

上で上がっている薬品のカラーリングの例で言うと、地味に背景色程度ですませていれば有用ですが、派手な飾りをべたべたと貼り付けて、肝心の薬品名が目立たなくなるくらいでうざい、という感じでしょう。

そもそも、一度に考えなくてはならないことを狭くするために、アクセスできる範囲を制限しているのに、アクセスする範囲のことを常に頭に置いておかなくてはならないというのでは、本末転倒なのではないでしょうか？

ということで。
thisはつけないほうが、可読性、ひいては保守性や拡張性が上がるものだと思っていました。

ここの議論を聞いていて、新人が混乱するというのとかは、一理あるかと思いましたけどね。

nagiseさんの書き込み (2007-07-09 14:17) より:
私はスコープの違いで似て非なる名前を付けたくはないのですね。
スコープといった（広義の）名前空間が違う場合に同名でも違うものとして認識するように
わざわざ言語が対応しているんだから、人間がそこを変える対応をするというのが腑に落ちない。
しかし、このあたりは感情的なものなのかもしれません。

まあ要するにどっちが正しいとかどっちの方が絶対的に合理的なんていうことはなくて、
対象とする開発者やそのレベル、人によってどの程度のやり方が合理的かは変わるという
ただそれだけのことを言いたかっただけです。

nagiseさんの書き込み (2007-07-09 18:03) より:
では、合理的ではない環境というのはどういう環境なのかを述べてください。

最初、
＞開発者の感じる、うざい、くどいって
＞結構重要だと思うのですよ。
と書いただけで、

＞心理的なストレスは確かに重要な考慮事項ではありますが、
＞誤認するほうがよほど問題なのでそこは理由あって譲りません。
＞
＞いえね、個人でプログラムしているんならなんでもいいんです。

という流れになってしまったので、まるで私が誤読よりもめんどくさいかどうか、
気分的なものの方が大事だと主張しているかのように言われてしまったので
（言われているように感じられたので）誤解されているようだ、と書きました。

もともとあれを書いたのは、表現はうざい、くどい、であっても、
そこにはプログラムの読みにくくなるという意味が含まれているのでは、
うざい、くどいという表現だけから、一概に軽視してよいものでもないのでは、
というようなことが言いたかったのです。

よねＫＥＮさんの書き込み (2007-07-10 10:34) より:
nagiseさんの「this」があった方が誤認がなくてよい、という意見は、
「インスタンス変数、ローカル変数などで同名を積極的に使いたい」
という前提に立ったものだと思っています。

つまり、コーディング規約においてどういうルールを採用するかで
この辺は意見が分かれるのが自然だと思います。

なのですが、その前提が抜けてやしませんかね？
(とりあえずインスタンス変数、ローカル変数しか話に出ていないようなので、
それらに絞ってコメントしています)

私の場合、基本的にm_XXXのようにインスタンス変数はそれとわかる命名をします。
(m_は単にアンダーバーを前/後ろにつけると適宜、その人に応じて読み替えてもらってもよいです)
この場合、thisは冗長なだけなんですね。

よねＫＥＮさんの書き込み (2007-07-10 10:34) より:

私の場合、基本的にm_XXXのようにインスタンス変数はそれとわかる命名をします。
(m_は単にアンダーバーを前/後ろにつけると適宜、その人に応じて読み替えてもらってもよいです)
この場合、thisは冗長なだけなんですね。

とおっしゃっているように、nagiseさんの「this」があった方が誤認がなくてよい、という意見は、「インスタンス変数、ローカル変数などで同名を積極的に使いたい」という前提に立ったものだと思っています。

私の場合、VB.netのことも頭にあるから特にインスタンス変数は区別するためにプレフィックスを付けたいのですよね。C#ではC#の流儀に合わせるべきとか意見はあるでしょうが、仕事上のルールなら数は少ない方がいいので、同じ.NET系では極力統一したいところです。もちろん言語仕様からくる制約なども違うので、それぞれを意識したカスタムルールとして分岐する部分がでてくると思います。