メソッドで値を定義する

Question

###前提・実現したいこと
javascriptでのオブジェクト指向について勉強中です。
クラスにそれぞれの変数名で値を保存する場所と、そこに値をセットし、出力するメソッド（メソッドの作成の仕方は一般的ではありませんが、それでも動くので問題はないはずです）を制作しました。しかし、実際にインスタンス化して、メソッドを実行すると値がセットされませんでした。
複数のインスタンスを用意する予定なので、逐一変数名を制作するのは嫌なのですが、どうすれば値がセットされるのでしょうか？
###該当のソースコード

SuperClass = function(){
//変数宣言と値の初期化
 this.x = 0;
 this.y = 0;
 this.w = 0;
 this.h = 0;
//メソッド
 this.log = function(x,y,w,h){
 this.x = x;
 this.y = y;
 this.w = w;
 this.h = h;
 console.log(""+this.x+""+this.y+""+this.w+""+this.w+"")　
　}
}
subclass = new SuperClass();
//consoleに出力されるのは0000になる。
subclass.log(10,10,10,10);

Comments

[defghi1977]

まず, あなたのコードはエラーを発しているので回答できません. また, 用語に混同が見られます. 上記の文脈では少なくとも「サブクラス」ではありません. 「インスタンス」のほうが適切です.

Answer 1

Rectangle = function() {
    // 変数宣言と値の初期化
    this.x = 0;
    this.y = 0;
    this.w = 0;
    this.h = 0;

    // インスタンス変数を出力する
    this.log = function(x, y, w, h) {
        this.x = x;
        this.y = y;
        this.w = w;
        this.h = h;
        console.log("" + this.x + this.y + this.w + this.h);
    }
}

rect = new Rectangle();
rect.log(2, 4, 6, 8);

オブジェクト指向以前に、まずコードの書き方の基本を学ぶ必要があります。
リファクタリングしたコードが上です。変更点を箇条書きで書きました。

• ネストに対応すべきカッコの数が違う（一個足りない）
• インデントがネストに対応していない（だから気づきにくい）
• 「SuperClass」という名前だが、実際にはスーパークラスではない
• サンプルコードは矩形を表現したと解釈し、「Rectangle」とより適切な命名をした
• 全角空白を取りのぞき、インデントをつけて整形した
• 他にも微妙な修正をほどこした

ES6だとこう書くのが良い、みたいな細かい点は今回省略しました。

Comments

[meron-pan]

無事動きました。ありがとうございます。

Answer 2

エラーが出ていませんか？

SuperClass = function(){
//変数宣言と値の初期化
 this.x = 0;
 this.y = 0;
 this.w = 0;
 this.h = 0;
//メソッド
 this.log = function(x,y,w,h){
 this.x = x;
 this.y = y;
 this.w = w;
 this.h = h;
 console.log(""+this.x+""+this.y+""+this.w+""+this.w+"")　/*← 全角空白*/
}/*← 閉じ括弧が足りない */

subclass = new SuperClass();
//consoleに出力されるのは0000になる。
subclass.log(10,10,10,10);

Comments

[meron-pan]

コードの件は質問するにあたって一から書いたのでそれのミスです。スイマセン