Semantik von Javascript-Code

Question

if(myVar = img.parent('a').length > 0){
    var Y = 1;
}else{
    var Y = 2;
}

Wenn ich diesen Code ausführe, nimmt myVar (das zum ersten Mal angekündigt wird) den Wert von img.parent('a').length > 0 und wird entweder false o true je nach Fall.

Erste Frage:
Ist dies eine korrekte Art der Definition meinVar ?

Zweite Frage:
Definiere ich Y zum zweiten Mal? Ist mein zweites 'var' Exzess?
d.h. sollte ich einfach schreiben Y = 2;