flg処理を変数と配列を使って行う

Question

flgの処理を関数を使って対応したいと考えております。
引数に数字を持たせて、配列から変数名を持ってきて変数に入れ、そのflgがfalseならばtrueにするというう処理を行いたいです

現状ですとflgが文章として認識されてしまいます。

$a_flg
$b_flg...
の変数でflg管理しています。

var $a_flg = false;
var $b_flg = false;
var $c_flg = false; 

function flgFunction(number){
	//各flgを割り出す
	var arr = ['$a_flg','$b_flg ','$c_flg']
	var this_flg = arr[number];

	//flgをtrueにする
	if(this_flg == false){
		this_flg = true;
	}
	return this_flg;
		
}

flgFunction(0);  //$a_flg

引数1を渡すと
$a_flg が戻り値になります。

true が戻り値になるのが理想です。

Comments

[m.ts10806]

タグにhtml,cssがありますがどのように関係するのでしょうか。コード内に$から始まる変数のような文字列がありますがこれは？稚拙などと言った自身を卑下する文言を入れるのではなく、説明できる範囲で全て説明してください。

[T.G]

webで使用しておりますので タグを挿入しました、コードには直接関係ありませんでした。$a_flgなどはflg処理で使用している関数名です

[m.ts10806]

コードと言うより要件に関係あるかどうかで判断してください。またこの内容だと要件自体も伝わりにくいように思います。質問を編集して要件をまとめてください。「flgが文章として認識されてしまいます」も結果どうなっているのかこれだけでは見えません。再現出来るコードを提示していただかないと回答しようがないです。

[x_x]

flgって何でしょう？ フラグ(flag: 旗)のことでしょうか？

[T.G]

ブーリアン型の変数です。わかりづらくてすいません

[x_x]

現状this_flgには文字列が入っているようですが、booleanにしたいのですか？

Answer 1

flags = {"$a" : true};
flags["$a"] = false;

とかってやってけば、

function flgFunction(number){
    //各flgを割り出す
    var arr = ['$a_flg','$b_flg ','$c_flg']
    var this_flg = flags[arr[number]];

    //flgをtrueにする
    if(this_flg == false){
        this_flg = true;
    }
    return this_flg;

}

で、できるとは思いますし、まあ動的変数で調べれば、色々とやり方は出てくるんじゃないかとは思いますが、その作りは最悪です。
何がやりたいかは知りませんが、関数が外の(グローバルの)$a_flgを直接参照しにいくようなのは、今のうちにやめたほうがいいと思います。

Comments

[T.G]

もしよろしければ、「関数が外の(グローバルの)$a_flgを直接参照しにいくようなのは、今のうちにやめたほうがいい」点について教えていただけないでしょうか？

[papinianus]

そのような変数は、どこでどう変更されているか分からないでしょう？
誰(関数や処理)がいつどこからsetして、また誰がいつどこからgetするか分からないような値はバグのもとです。
分かるのは最初のうちだけです。作りを見直したほうがいいと思います(最低でも変数のスコープをクラス内で閉じたほうがいいと思います)

[T.G]

ご親切にありがとうございます。勉強不足でした

Answer 2

関数を動的に宣言して利用したいというのでしたらちょっとコツが必要です。

• JS 関数を動的に呼び出す
• 文字列で関数を実行する
• 関数名を文字列で指定して動的に呼び出す機構（evalを使わず)
• teratail:javascriptで関数の動的呼び出し

Answer 3

まず例示のものだと$a_flgは返ってきません、$b_flgですね

if(this_flg == false)でチェックしてもfalseではないのでtrueは返りません
なにを根拠に$b_flgがfalseだと思ったのでしょうか？

拡大解釈してかりに$b_flgが「false」という文字だとしてもクォーテーションで
囲んでいるため文字列で判断され、this_flg == falseではヒットしません。

加えてfalseをチェックする演算子は「==」は絶対につかってはいけません
「===」を利用してください。（false以外0やnullなどヒットする場合があります）

Comments

[T.G]

完全に凡ミスです。
宣言を記述忘れました

[yambejp]

それでも
var arr = ['$a_flg','$b_flg ','$c_flg']
はNGです、少なくとも
var arr = [$a_flg,$b_flg ,$c_flg]
としてください

[yambejp]

よくよくかんがえればロジックとして
「もしfalseだったらtrueにして戻せ」って指示しかないつまり
「もしtrueならfalseにする」という考え方がないなら
戻ってくるのはtrueしかないですよね？
（もちろんfalseでもtrueでもないものかもしれませんが）

それなら何も変換せずにreturn true;と書くだけでしょう