ウィンドウを開き、指定したウィンドウを閉じるプログラム URLを配列から引数へ変更したい

前提・実現したいこと

タイトルにもあるように配列からではなく、関数の引数からurlを得たいです。

発生している問題・エラーメッセージ

Uncaught SyntaxError: expected expression, got '}'

該当のソースコード

JavaScript
1<!DOCTYPE html>
2<html>
3 
4<head>
5  <meta charset="utf-8">
6  <title></title>
7</head>
8<script>
9 
10  // 変数を用意
11  var page = [];
12 
13  function createNewWindow (URL,n) {
14      // 別ウィンドウを開く
15      page[n] = window.open("URL","_blank","width=400, height=600");
16   
17  }
18 
19  function closeWindow (n){
20      page[n].close();
21  }
22   
23 
24</script>
25 
26<body>
27
28  <table border = "1">
29    <tr><th>ページ名</th><th>Open</th><th>Close</th></tr>
30    
31      <tr>     //amazonのページを開く
32	<td>Amazpn.co.jp</td><td><input type="button" value="Open" onclick="createNewWindow("https://www.amazon.co.jp",0)"></td>
33	<td><input type="button" value="Close" onclick="closeWindow(0);"></td>
34      </tr>
35      <tr>     //googleのページを開く
36	<td>Google</td><td><input type="button" value="Open" onclick="createNewWindow("https://www.google.co.jp" ,1);"></td>
37	<td><input type="button" value="Close" onclick="closeWindow(1);"></td>
38      </tr>
39	
40      
41
42    </table>
43</body>
44 
45</html>
46
47

JavaScript
1<!DOCTYPE html>
2<html>
3 
4<head>
5  <meta charset="utf-8">
6  <title></title>
7</head>
8<script>
9 
10  // 変数を用意
11  var page = [];
12  var array = ["https://www.amazon.co.jp","https://www.google.co.jp"];
13 
14  function createNewWindow (n) {
15      // 別ウィンドウを開く
16      page[n] = window.open(array[n],"_blank","width=400, height=600");
17   
18  }
19 
20  function closeWindow (n){
21      page[n].close();
22  }
23   
24 
25</script>
26 
27<body>
28
29  <table id = "d_table" border = "1">
30      <tr><th>ページ名</th><th>Open</th><th>Close</th></tr>
31      <tr>
32	<td>Amazpn.co.jp</td><td><input type="button" value="Open" onclick="createNewWindow(0);"></td>
33	<td><input type="button" value="Close" onclick="closeWindow(0);"></td>
34      </tr>
35      <tr>
36	<td>Google</td><td><input type="button" value="Open" onclick="createNewWindow(1);"></td>
37	<td><input type="button" value="Close" onclick="closeWindow(1);"></td>
38      </tr>
39
40    </table>
41</body>
42 
43</html>
44

試したこと

試したことに関してはコードの２個目のように配列arrayをつくり、そのなかにURLを入れておき、関数createnewwindowの引数を１つにすると目的は達成されました。しかし、コードの1つ目のように関数の引数にURLを入れてしまうと、動かなくなりました。コードを見た限り、onclick="createNewWindow"のURLの部分が怪しいなと思っていますが、解決方法がよくわかりません。アドバイスをいただきたいです。

行動規範の内容に同意します

回答1件

ベストアンサー

| <input type="button" value="Open" onclick="createNewWindow("https://www.amazon.co.jp",0)">

これだと、onclick=に続く文字列がcreateNewWindow(になってしまいますね。
"で始まった文字列がそこで閉じてしまっているからです。

この場合、"で囲った文字列の中では、"の代わりに'を使用できます。

HTML
1<input type="button" value="Open" onclick="createNewWindow('https://www.amazon.co.jp',0);">

あと、createWindowでは

| window.open("URL","_blank","width=400, height=600");

ではダメですね。これだと第1引数にURLという文字列を渡してしまっていますし、あと第3引数には空白が入っていてはいけません。

ここは

JavaScript
1window.open(URL,"_blank","width=400,height=600");

です。

投稿2021/06/06 13:24

itagagaki

総合スコア8402

199999

2021/06/06 14:18

解決いたしました!納得できました、ありがとうございました。

行動規範の内容に同意します

あなたの回答

tips

プレビュー

行動規範の内容に同意します

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

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

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