Идиоматически было бы описать методы в прототипе конструктора.
function Point(x, y) {
this.x = x;
this.y = y;
}
Point.prototype.getX = function () {
return this.x;
}
Point.prototype.getY = function () {
return this.y;
}
Или классом.
class Point {
constructor(x, y) {
this.x = x;
this.y = y;
}
getX() {
return this.x;
}
getY() {
return this.y;
}
}
Объявлять функции которые обращаются к this вне класса это излишнее усложнение для читающего и понимающего код. Плюс так нормальные люди не пишут. Нужно подсказать кодом о том чем будет этот самый this.
Остальное норм.