javascript での文字列変数の比較について

Question

javascript で

var old="old";
var new="new";

として

new の値は その時々で別処理で取得してきます

if (new===old) {
//何もしない
} else {
alert('update');
old = new;
}

として１度目は、当然 update が表示されますが、
その後 old に new 値を代入します

続いて 別処理で new の値に "new" が代入されると
本来は
何も処理が実行されないはずですが、
再び update が表示されてしまいます。

if の比較式を
new==old // =を２つ
や
new.equals(old) //こっちは機能しませんでした
とやってみてもダメでした

以前は new===old の記述で動作していたのですが
色々機能を追加しているうちに何度やっても、update が表示されるようになってしまいました

むしろ以前がたまたま運よく動いていたのでは？と思われますが、
確実に比較し、等しいときには実行しない、という処理を
行いたいので、
確実に判定する方法をご指導ください。

よろしくお願いします。

Answer 1

おやりになりたいことは検証していないのですが、そもそもnewは予約後なので変数名としては使えないのでは？

この後、コードを検証してみます

Comments

[Guu]

やはりnewは変数として扱えないので以下のコードで検証しましたが問題は無いですよ。

var old="old";
var new1="new";
if (new1===old) {
//何もしない
} else {
alert('update1');
old = new1;
}

if (new1===old) {
//何もしない
} else {
alert('update2');
old = new1;
}

[it_solution_lab]

new old は実際の記述では別の変数です。
変数の内容として、新旧という意味で書いただけで
実際には newは使っておらず、
ある変数　hensu に対して oldhensu とで比較していますが、
機能しません。

[Guu]

変数は適切ということなら、次の検証方法は問題が起きる再現できる最小のコードにすることです。
それでも原因が判らなければ、その最小のコードを提示してもらえれば、ここを見ている人たちが検証のお手伝いをしてくれると思います。
再現できるコードの提出がこの手の問題は一番片付きやすいです。

Answer 2

具体的なコードがないため何とも言えませんがもし値が正しく意図したとおりに代入されているならば正しく動作するはずでしょう。但し例外として考えられるのは関数とローカル変数によって問題が起きているということです。
ローカル変数に値を代入しており、関数を再度呼び出しているというようなことはありませんか？(再度呼び出しした際にローカル変数が初期化されているならば納得できますので)ご確認ください。

Comments

[it_solution_lab]

function 内での var 宣言（ローカル変数）はしておらず
グローバル変数でのお話になりますが、
自己解決いたしました
ありがとうございました。

Answer 3

Cf_cwdさんも書かれてますが、こんな話なのではないかなあ、と。

var oldhensu="old";
var hensu="new";

function test(hensu){
  if (hensu===oldhensu) {
    //何もしない
    var oldhensu; // ← あっ
  } else {
    alert('update');
    oldhensu = hensu;
  }
}

test(hensu);  // update
hensu="new";
test(hensu);  // update

javascriptの便利機能というかアンチパターンというか、とにかく仕様として「クロージャのどこかにvar宣言がされていると、その部分のコードが実行されていなくてもローカル変数扱い」というものがあります。
クロージャ部分のコードが長いとデバッグしてても気が付きにくいので、はまる時にははまるんですよねー。

↓参考
知らないと怖い「変数の巻き上げ」とは？｜もっこりJavaScript｜ANALOGIC（アナロジック）

Comments

[it_solution_lab]

function 内での var 宣言はしておらず
グローバル変数でのお話になりますが、
自己解決いたしました
ありがとうございました。

Answer 4

文字列の比較としては===を用いるのは正しいと思います。
であればoldに代入した値がおかしいのかもしれません。
下記のようにして条件判定の前にそれぞれの変数に入っている値と型を出力して確認してみてはいかがでしょうか？

alert(old)
alert(typeof(old));
alert(new)
alert(typeof(new));

Comments

[it_solution_lab]

new old は実際の記述では別の変数です。
変数の内容として、新旧という意味で書いただけで
実際には newは使っておらず、
ある変数　hensu に対して oldhensu とで比較していますが、
機能しません。

[it_solution_lab]

typeof を試しましたが
いずれの変数も string　となり違いは見当たりませんでした。
また、変数値の値の前後に、スペースなども入っていません。

Answer 5

基本的に変数の型は、string のはずでしたが、
取得側で、得られた値を 必ず toSting() で渡すようにしたら
解決しました。
どこかで、別の型がまじってしまっていたのかもしれません。

色々ありがとうございました。