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

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

ただいまの
回答率

90.33%

  • JavaScript

    17550questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Chrome extension

    126questions

    Chrome拡張機能

  • Firefox WebExtensions

    9questions

javascriptの変数の値を変えたい

解決済

回答 2

投稿 編集

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

nekojiro

score 35

前提・実現したいこと

youtubeを新しいウインドウで開くアドオンを作っています。ウインドウサイズを保存してその大きさのウインドウを開きたいんですがうまくできません。ウインドウは開けるんですがwidthとheightの値が変わらないので保存したサイズにならない。gettingWidthの中のwidthの値を変えてもbrowser.windows.createのwidthが変わらないので変えたいです。

該当のソースコード

function onGot(tabInfo) {
  const tabURL = tabInfo.url;
  const YouTubewatchURL = 'https://www.youtube.com/watch';
  if (tabInfo.url.match(YouTubewatchURL)) {
    browser.windows.onCreated.addListener(() => {
      newWindow(tabURL);
    });
    let width = 800;// 初期値
    let height = 450;// 初期値
    const gettingWidth = browser.storage.local.get('width');
    gettingWidth.then(item => {
      if (item.width) {
        width = item.width;
      }
      return width;
    });
    const gettingHeight = browser.storage.local.get('height');
    gettingHeight.then(item => {
      if (item.height) {
        height = item.height;
      }
      return height;
    });
    const popupURL = browser.extension.getURL('popup/popup.html');
    const creating = browser.windows.create({
      url: popupURL,
      type: 'panel',
      width: width,// ここの数値を変えたい
      height: height,// ここの数値を変えたい
    });
    creating.then();
  }
}

試したこと

thenやreturnのことを調べたけどさっぱりわからない。

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • toutou

    2017/03/12 15:18

    ウィンドウを開きたいのでしょうか?それともreturnの使い方が知りたいのでしょうか?

    キャンセル

  • nekojiro

    2017/03/12 15:53

    gettingWidthの中のwidthの値を変えてもbrowser.windows.createのwidthが変わらないので変えたいです。説明が下手でごめんなさい

    キャンセル

回答 2

checkベストアンサー

+1

width = item.width; や height = item.height; で値を入れてると思いますが、 then の中は非同期で実行されているので、

 const creating = browser.windows.create({
      url: popupURL,
      type: 'panel',
      width: width,// ここの数値を変えたい
      height: height,// ここの数値を変えたい
    });

コレが実行されるときにはまだ結果が入っていないのだと思います

試してはいないですがこれでいけないでしょうか?

async function onGot(tabInfo) {
  const tabURL = tabInfo.url;
  const YouTubewatchURL = 'https://www.youtube.com/watch';
  if (tabInfo.url.match(YouTubewatchURL)) {
    browser.windows.onCreated.addListener(() => {
      newWindow(tabURL);
    });
    let width = 800;// 初期値
    let height = 450;// 初期値
    const gettingWidth = await browser.storage.local.get('width');
    if(gettingWidth && gettingWidth.width){
      width = ~~gettingWidth.width
    }
    const gettingHeight = await browser.storage.local.get('height');
    if(gettingHeight && gettingHeight.height){
      height = ~~gettingHeight.height;
    }
    const popupURL = browser.extension.getURL('popup/popup.html');
    const creating = browser.windows.create({
      url: popupURL,
      type: 'panel',
      width: width,// ここの数値を変えたい
      height: height,// ここの数値を変えたい
    });
    creating.then();
  }
}

width/height の取得を await でまつようにしました。


Promise 版

function onGot(tabInfo) {
  const tabURL = tabInfo.url;
  const YouTubewatchURL = 'https://www.youtube.com/watch';
  if (tabInfo.url.match(YouTubewatchURL)) {
    browser.windows.onCreated.addListener(() => {
      newWindow(tabURL);
    });
    let width = 800;// 初期値
    let height = 450;// 初期値
    const gettingWidth = browser.storage.local.get('width');
    const gettingHeight = browser.storage.local.get('height');
    const popupURL = browser.extension.getURL('popup/popup.html');
    Promise.all([gettingWidth, gettingHeight]).then(results => {
      if (results[0] && results[0].width) {
        width = ~~results[0].width;
      }
      if (results[1] && results[1].height) {
        height = ~~results[1].height;
      }
      const creating = browser.windows.create({
        url: popupURL,
        type: 'panel',
        width: width,// ここの数値を変えたい
        height: height,// ここの数値を変えたい
      });
    })
  }
}

if 文抜けていたのを修正しました

Promise版追加しました

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/03/12 18:05

    awaitを付けてみたんですが動かなくなりました

    キャンセル

  • 2017/03/12 18:13

    どんなエラーがでていますか?

    キャンセル

  • 2017/03/12 18:18

    if文が抜けていたのを修正しました
    データが無い場合にエラーになっていたはずです

    キャンセル

  • 2017/03/12 18:29

    ESlintがawaitのところがParsing error: The keyword 'await' is reserved (Fatal)というエラーだと言ってます

    キャンセル

  • 2017/03/12 18:33

    新しい機能なのでESlintが未対応か使えるよう設定されていないのではないでしょうか
    ESlintでなくブラウザでエラーなく動くかたしかめてください

    拡張機能ならIEのような古いブラウザで動かないことを気にする必要はありませんし、タグにあるChromeとFirefoxの最新版ならawaitが使えるはずです

    キャンセル

  • 2017/03/12 18:34

    一番最初の行に書いてあるasyncって何ですか?

    キャンセル

  • 2017/03/12 18:36

    async関数の中で、awaitが使えるようになります
    https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Statements/async_function

    キャンセル

  • 2017/03/12 18:37

    SyntaxError: await is only valid in async functionsってエラーが出ました

    キャンセル

  • 2017/03/12 18:38

    async を書いていないということはないですか?
    エラーは「async 関数の中でしかawaitが使えません」というものです

    キャンセル

  • 2017/03/12 18:41

    asyncつけたらエラーがたくさん出ました
    18:39:08.460 Warning: attempting to write 4410 bytes to preference extensions.bootstrappedAddons. This is bad for general performance and memory usage. Such an amount of data should rather be written to an external file. This preference will not be sent to any content processes.
    18:39:08.469 Warning: attempting to write 5123 bytes to preference extensions.xpiState. This is bad for general performance and memory usage. Such an amount of data should rather be written to an external file. This preference will not be sent to any content processes.
    18:39:08.581 1489311548581 addons.webextension.YouTube_Popup_Panel@nekojirou.net WARN Please specify whether you want browser_style or not in your browser_action options.
    18:39:08.585 Warning: attempting to write 4138 bytes to preference browser.uiCustomization.state. This is bad for general performance and memory usage. Such an amount of data should rather be written to an external file. This preference will not be sent to any content processes.

    キャンセル

  • 2017/03/12 18:43

    新しいウインドウを開こうとするとError: Type error for parameter createData (Error processing width: Expected integer instead of "1280") for windows.create.って言われます

    キャンセル

  • 2017/03/12 18:50

    保存したデータに正しい数値が入っていないのかもしれません
    asyncが原因とは思えないので一応Promise版を書いてみました

    キャンセル

  • 2017/03/12 18:50

    こちらでも同じエラーになっていますか?

    キャンセル

  • 2017/03/12 18:51

    browser.windows.create
    ↑の直前で
    console.log(width, height)
    をしてNaNや極端に大きな値など入っていないか確認してみてください

    キャンセル

  • 2017/03/12 18:54

    > Expected integer instead of "1280"
    数値型じゃなくて文字列型がはいってるのではないですか?
    「~~width」などで数値型にキャストすればちゃんと動くかもしれません

    キャンセル

  • 2017/03/12 18:59

    console.log(width, height)書いてみたんですがさっきと同じようなエラーが出てコンソールは表示されませんでした。保存したあるのはこんなかんじです{"width":"1280","height":"720"}

    キャンセル

  • 2017/03/12 19:03

    やっぱり文字列になっているようです
    解答のコード更新してあるのでもう一度ためしてもらえますか?
    width と height に代入するところで数値型に変換しています

    キャンセル

  • 2017/03/12 19:09

    19:05:06.777 Warning: attempting to write 4410 bytes to preference extensions.bootstrappedAddons. This is bad for general performance and memory usage. Such an amount of data should rather be written to an external file. This preference will not be sent to any content processes.
    19:05:06.786 Warning: attempting to write 5123 bytes to preference extensions.xpiState. This is bad for general performance and memory usage. Such an amount of data should rather be written to an external file. This preference will not be sent to any content processes.
    19:05:06.912 1489313106912 addons.webextension.YouTube_Popup_Panel@nekojirou.net WARN Please specify whether you want browser_style or not in your browser_action options.
    19:05:06.917 Warning: attempting to write 4138 bytes to preference browser.uiCustomization.state. This is bad for general performance and memory usage. Such an amount of data should rather be written to an external file. This preference will not be sent to any content processes.
    またエラーがたくさん出ました

    キャンセル

  • 2017/03/12 19:22

    これらはFirefoxでstorageを使うと出ることのある警告のようです。
    manifest や全体のコードがわからないので対処方法はわかりませんが、エラーでなく警告なのでウィンドウを開くことには影響ないかと思います

    > Error: Type error for parameter createData (Error processing width: Expected integer instead of "1280") for windows.create.

    こっちのエラーはどうなったのでしょうか?

    キャンセル

  • 2017/03/12 19:29

    動きました保存してある数字を変えたてもしっかり変わりました。エラーが出てるからウインドウは開かないと思ってました。ありがとうございます。

    キャンセル

0

このコードってどこから持ってきたものでしょうか
何かしらの解説がのってると思うのですが

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/03/12 15:50

    僕が書きました

    キャンセル

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

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

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

  • JavaScript

    17550questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Chrome extension

    126questions

    Chrome拡張機能

  • Firefox WebExtensions

    9questions