JavaScriptでの"is already defined"（日時取得）

Question

下記コードを記述したところ「〜　is already defined」エラーが出てしまいました。
（一部moment.jsを使っています）

//サーバタイムを取得
$.ajax({
  type: 'GET'
}).done(function(data, status, xhr) {
  var serverDate = new Date(xhr.getResponseHeader('Date'));

//時刻データ
var now = moment(serverDate);
var year = now.year();
var month = now.month() + 1;
var date = now.date();
var hour = now.hours();
var minute = now.minutes();

//時計表示用：数字が10以下の場合は頭に0を足す
if(hour < 10) {
    var hour = "0" + hour
}
if(minute < 10) {
    var minute = "0" + minute
}
});

エラーが出たのは下から3行目「var minute = "0" + minute」行です。
（'minute' is already defined.、のエラーメッセージが出ます）
その上の「var hour = "0" + hour」部分はエラーになりません。

元々はサーバタイムではなく、シンプルに現在時刻を取得していたのですが
『//サーバタイムを取得』部分を足してからこのエラーが出るようになりました。

スコープがおかしいのでしょうか？
何らかヒントでも、ご教授いただければ幸いです。

Answer 1

if文の宣言子varは要りません。
エラー文を翻訳すると、
'minute' はすでに定義されています
です。
つまり、上で宣言されているので、
if文内で改めて宣言し直す必要がありません。

var hour = "0" + hour
がエラーが起きなかったのは、単純にそこを通ってないだけです。
（10時より前の時間だったら通るのでエラーになる）

//時計表示用：数字が10以下の場合は頭に0を足す
if(hour < 10) {
  hour = "0" + hour;
}
if(minute < 10) {
  minute = "0" + minute;
}

Comments

[spangl]

おっしゃる通りでした…エラーが起こらなかった理由についても納得です…。
大変お手数をお掛けしました。ありがとうございました。

[hentaiman]

いふのなかでばーしない方がいいですね

[miyabi_pudding]

エラー文は、なぜエラーなのかを書いている記載なので、
英語わからなければ、翻訳を使うと、
だいぶ理解できるようになると思います。

[miyabi_pudding]

hentaimanさん >
ですねー
関数スコープなので、
ifだと、スコープ閉じ込めできませんしね　汗
やっぱletですよ、ほんとに。

[spangl]

さきほど翻訳ツールもブックマークバーに追加しました…。
お恥ずかしい限りです。よい勉強になりました。

[spangl]

hentaimanさん >
実は通常のフロントエンドコーディングではなくMAツール管理画面でのコーディングでして、
環境上、constもletも使えないのです…。；
ただ非常に参考になりました。ありがとうございます。

Answer 2

var minute = now.minutes();

と上のほうで定義されているからでは。

    var hour = "0" + hour

がエラーにならないのは、たまたまその時点でそこを通らなかっただけのように思います。

Comments

[spangl]

おっしゃる通りでした…。ありがとうございます！