カプセル化

JavaScript でも Python でも '_' で始まるプロパティやメソッドは、そのファイルスコープ、あるいはオブジェクトのスコープだけで参照してください。
これはキーワードで明示的アクセス制限できない言語で紳士協定として導入された名前規則です。尊重してください。

— いけべ (@ikb) July 21, 2017

具体的にいうと this._hoge や self._foo() は良くて、 piyo._fuga や poyo._foo() はアウトです。
ちょっとわかりにくいかもしれませんが this._foo._bar もアウトです（_foo._bar がルールに抵触するため）。

— いけべ (@ikb) July 21, 2017

このルールを回避するために this._foo を foo に変名する、あるいは this.getFoo = function() { return this._foo; } のようなアクセッサーを追加する、ということも禁止です。

— いけべ (@ikb) July 21, 2017

オブジェクトの境界は、ある関連をもった複数のプロパティをまとめて管理するために使います。
オブジェクトの関数は、関連を持つ複数のプロパティが、同時に満たすべき制約条件を壊さないように導入するものです。

— いけべ (@ikb) July 21, 2017

プライベートなプロパティは、制約を満たし続けるよう管理するために使います。
これをひとつ取り出して変更できるアクセッサーを追加するということは、オブジェクトの管理責任放棄です。
プライベートなプロパティを、そのオブジェクトの外から変更することは、オブジェクトの管理責任侵害です。

— いけべ (@ikb) July 21, 2017

このアクセス制約は「カプセル化」として知られており、プログラムの品質担保に大きな役割を果たします。
知っていた人は改めて、知らなかった人はこれから、注意するようお願いします。

— いけべ (@ikb) July 21, 2017

さすがに同じレビューを繰り返すのに心が疲れ切ったので、あった…。