関数で変数作成するにはどうすればいいのか

Question

###前提・実現したいこと
javascriptで
関数testに指定した引数１の変数に、引数２の値を代入したいのですがうまくいきません。

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

Uncaught ReferenceError: hello is not defined

###該当のソースコード

function test(name, value){ 
  eval('var ' + name + '=' + value + ';');
}
test('hoge', 'hello');
alert(hoge);

Answer 1

解決済みのようですが、Strict Mode で動作し、実装を選ばない(ブラウザでなくとも動作する)コードを紹介しておきます。

'use strict';
function defineGlobalVariable (name, value) {
  Function('return this')()[name] = value;
}

defineGlobalVariable('hoge', 'hello');
alert(hoge);

Object.defineProperty() を併用すると、descriptor を指定可能となり、更に汎用性が高くなります。

• Object.defineProperty() - JavaScript | MDN

Re: aiueo123 さん

Comments

[退会済みユーザー]

ご回答ありがとうございます

Answer 2

グローバルな変数は、windowオブジェクトのプロパティなので
こうすればいけるかもしれません

test('hoge', 'hello');
console.log(hoge);
function test(name, value){ 
  window[name]=value;
}

Comments

[退会済みユーザー]

ご回答ありがとうございます

Answer 3

これだと、var hoge = hello;になってしまいます。おそらく変数helloの値ではなくて、"hello"という文字列をセットしたいのでしょうから、valueを引用符で囲んでください。

Comments

[退会済みユーザー]

ご回答ありがとうございます。

Answer 4

ユーザ関数test() 内で定義された変数なのでそのスコープ外から参照できないだけでは。

追記：
varで定義するとそのスコープでしか使えない（参考URL：JavaScriptでは、varを使わずに定義した変数はグローバル変数として走査されます。）のでvarをはずしグローバル変数として定義し、valueを""で囲います
""で囲うのはご提示のエラー（「helloって変数はないよ」）対策です。

※Chromeで動作確認済み　alertボックスで「hello」と出力されます

function test(name, value){ 
  eval(name + '="' + value + '";');
}
test('hoge', 'hello');
alert(hoge);

Comments

[退会済みユーザー]

ご回答ありがとうございます。御指示いただいた方法で、意図した通りに動作できました。

Answer 5

こう

var hello = "hello";
function test(name, value){ 
  eval('var ' + name + '=' + value + ';');
}
test('hoge', 'hello');
alert(hoge);

またはこうかな？

function test(name, value){ 
  eval('var ' + name + '="' + value + '";');
}
test('hoge', 'hello');
alert(hoge);

Comments

[退会済みユーザー]

ご回答ありがとうございます

Answer 6

eval を使うのはお勧めしませんが、とりあえずエラーが起きない書き方を。

function test(name, value){ 
  eval('var ' + name + '="' + value + '";');
}
test('hoge', 'hello');

---

こういうことをしたいのでしょうか。

function test(name, value){ 
  return value;
}
var hoge = test('hello');

---

ブラウザ限定。

function test(name, value){ 
  if (window[name]) return;
  window[name]=value;
}
test('hoge', 'hello');

Comments

[退会済みユーザー]

ご回答ありがとうございます