質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.94%

非同期処理で取得したデータをクラスのプロパティに入れたいです

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 446

score 469

質問させてください。
下のようなコードを作成しまして、C クラスには外部から json データを取得するメソッドがあり、new A()を実行したときに、そこで取得したデータを C クラスのプロパティに格納したいと思っております。
しかし下のコードだとfecth()が原因のためか中身が空の状態になってしまいます。

class A {
  constructor() {
    this.b = new B();
  }
}

class B {
  constructor() {
    this.c = new C();
  }
}

class C {
  constructor() {
    this.data = this.fetchData(); // ← ここのプロパティに取得したデータを入れたいです
    console.log(this.data); // undefined
  }

  // json からデータを取得
  fetchData() {
    fetch('sample.json')
      .then(res => {
        return res.json();
      })
  }
}

const a = new A();


そこで下のように非同期処理が終わるのを待つためにawaitを使おうと思ったのですが、コンストラクターにはasyncが使えないらしく、エラーになってしまいます。

async constructor() {
  this.data = await this.fetchData();
}


こちらのページを参考に init() を作ってみてもうまくいきませんでした・・・

class C {
  constructor() {
    this.init();
  }

  async init() {
    this.data = await this.fetchData();
    console.log(this.data);
  }

  fetchData() {
    fetch('sample.json')
      .then(res => {
        return res.json();
      })
  }
}


もし解決方法をご存じの方がいらっしゃいましたらご教授いただけると助かります。
よろしくお願いいたします。

追記

gentaro様からのアドバイスをもとに修正しました。

class A {
  constructor(data) {
    this.b = new B(data);
  }

  start() {
    this.b.active();
  }
}

class B {
  constructor(data) {
    this.c = new C(data);
  }

  active() {
    this.c.hoge();
  }
}

class C {
  constructor(data) {
    this.data = data;
    console.log(this.data);
  }

  hoge() {
    // このメソッドで取得したデータを使います
  }
}

// json からデータを取得
fetch('sample.json')
  .then(res => {
    return res.json();
  })
  .then(data => {
    // Aクラスのインスタンスを生成
    const a = new A(data);

    // ゲームをスタート
    a.start();
  });
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • gentaro

    2020/09/22 14:19

    シンプルに非同期で取得したデータをコンストラクタで渡すだけじゃダメなの?
    コンストラクタそのものを非同期にするとか、無駄にややこしくしてる気がしますが。

    キャンセル

  • takopo

    2020/09/22 15:28

    ありがとうございます。
    すみません、データをコンストラクタで渡すというのはAクラスの中で fetchData() を実行するということでしょうか…理解不足ですみません

    キャンセル

  • gentaro

    2020/09/22 15:36

    いや、コンストラクタは普通に引数で値をもらうだけ、という意味で。
    そのクラスを利用する側がfetchして、取得したデータをコンストラクタの引数に渡せばいいだけで、いま定義したいクラスの内部に非同期とかのややこしいコードを入れる必要あるの?という疑問です。

    キャンセル

  • takopo

    2020/09/22 16:11 編集

    ご返答ありがとうございます。
    実はややこしくなると思い書いてなかったのですが、Aクラスの中に start() というメソッドがありまして、そのメソッドを実行したらゲームがスタートするというものを作っています。
    いただいたアドバイスをもとに修正したコードを追記しました。こちらで取得したデータをCクラスのプロパティに入れることはできたのですが、Cクラスのメソッド内でしか使わないプロパティを外から取得しないといけないのでしょうか。Cクラス内で取得から保持まで完結できれば、まとまっていて良さそうなのですがどうでしょうか…

    キャンセル

回答 2

checkベストアンサー

+1

fetch をclassブロックのメソッドの中で使う場合、アロー関数(this は宣言したスコープになる)を使い、適切にコールバック内でプロパティに値を格納するだけです。

{"sample":"SAMPLE"}
/* omitted  class A{}, class B{} */

class C {
  constructor() {
    this.fetchData();
    console.log(this.data); // undefined
  }

  // json からデータを取得
  fetchData() {
    fetch('sample.json')
      .then( res => res.json() )
      .then( json => {
        this.data = json
      })
  }
}

const a = new A();
console.log( a.b.c );
/*
// fetch によるsample.json 読み込み完了後:
C {}
    data: 
        sample: "SAMPLE" <---- data プロパティに入っています。
        __proto__: Object
    __proto__: Object
 */
console.log( a.b.c.data )
// => {sample: "SAMPLE"}


追記)

fetch の遅延を考える

/* omitted */
  fetchData() {
    fetch('sample.json')
      .then(res => res.json())
      .then( json => {
        this.data = json;
        if( "function" === typeof this.onload ) {
          this.onload(this);
        }
      })
  }
/* omitted */
const a = new A();
a.b.c.onload = function(c){
  alert(JSON.stringify(c.data)); 
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/09/24 20:35

    ご回答ありがとうございます。
    すみません、教えていただいたコードを試してみたのですが、最後の console.log( a.b.c.data ) というところが undefined になってしまいます…どこか見落としているところがあるのでしょうか。。

    キャンセル

  • 2020/09/24 20:59

    > どこか見落としているところ
    非同期ですので、class C の メソッドで this.data = json の処理には遅延があります。
    コメント「fetch によるsample.json 読み込み完了後:」は a.b.c.data も同様ですね。

    キャンセル

  • 2020/09/25 10:36

    遅くなってすみません。たびたびのご返信ありがとうございます。
    this.data = json の処理は const a = new A(); よりも後になってしまうのが原因だったのですね。
    非同期完了を待つ処理まで書いていただいてありがとうございます。
    こちらで無事に解決でき助かりました。大変勉強になりました。

    キャンセル

+1

こんなのはどうでしょうか?
コンストラクタはどうしても非同期にできないので、ファクトリーを作るか、作った後から初期化する必要がありますね。

<!DOCTYPE html>
<html lang="ja">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Document</title>
</head>
<body>
    <script>
        class C {
            async init() {
                this.data = await fetch("john.json").then(a => a.json());
                return this;
            }
        }

        (async () => {
            const c = await new C().init();
            console.log(c.data.name);
        })();
    </script>
</body>
</html>
{ "name": "John" }

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/09/25 10:37

    ご回答ありがとうございます。
    インスタンス作成後に初期化するにはこのようにしたらよかったのですね。無事にプロパティに入れることができました。
    今回お二方に素晴らしいご回答をいただけてとても感謝しております。ベストアンサーは迷ったのですが、最初にご回答いただいたAkitoshiManabe様にさせていただきたいと思います。Zuishin様申し訳ございません。
    今回は本当にありがとうございました!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.94%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る