javascript - jQuery / Javascript コード チェック、未定義でない場合

Question

このコードは良いですか？

var wlocation = $(this).closest('.myclass').find('li a').attr('href');
if (wlocation.prop !== undefined) { window.location = wlocation; }

または私はするべきですか

var wlocation = $(this).closest('.myclass').find('li a').attr('href');
if (wlocation.prop !== "undefined") { window.location = wlocation; }

Answer 1

私はこれが好き：

if (wlocation !== undefined)

しかし、2番目の方法を好む場合は、投稿したとおりではありません。それはそのようになります：

if (typeof wlocation  !== "undefined")

Answer 2

私は一般的に省略版が好きです：

if (!!wlocation) { window.location = wlocation; }

Answer 3

$.fn.attr(attributeName)は、属性値を文字列として返すかundefined、属性が存在しない場合に返します。

""とundefinedはどちらもJavaScriptでは偽の (ブール値に強制すると false に評価される) 値であるため、この場合、次のようにチェックを記述します。

if (wlocation) { ... }