jsで関数宣言→関数式の変換の仕方をご教授ください

質問

以下は、javascriptの構文の一部です。
構文の中で、function changeDiaType(DiaDay)と書くと、構文チェックサイトで、関数宣言→関数式への返還を求められました。
関数宣言→関数式への変換をお願いいたします。省略なしでお願いします。
function timeButtonInterfaceUpdate()
{
//引数がtrueなら、リアルタイムモードのインタフェースに
if(true)
{
realtimeUpdate = true;
document.getElementById("realTimeModeStatusDiv").innerHTML = "　リアルタイム表示中　";
//ドロップダウンリストの有効化
document.form1.updateIntervalSelect.disabled = false;
}
else
{
realtimeUpdate = false;
document.getElementById("realTimeModeStatusDiv").innerHTML = "リアルタイム表示にする";
//ドロップダウンリストの無効化
document.form1.updateIntervalSelect.disabled = true;

//もしタイマーが事前にかかっていたら、止める
clearInterval(updateIntervalTimer);
function changeDiaType(DiaDay)
{
if(DiaDay == 2)
{
document.getElementById("todayDiaDaySpan").innerHTML = "土曜ダイヤ";
document.getElementById("todayDiaDaySpan").style.backgroundColor = "#fcf";
DiaDay = 2;
}
else if(DiaDay == 1)
{
document.getElementById("todayDiaDaySpan").innerHTML = "土・休日ダイヤ";
document.getElementById("todayDiaDaySpan").style.backgroundColor = "#fcf";
DiaDay = 1;
}
else
{
document.getElementById("todayDiaDaySpan").innerHTML = "　平日ダイヤ　";
document.getElementById("todayDiaDaySpan").style.backgroundColor = "#9ff";
DiaDay = 0;
}
}}}

回答No.1

AsarKingChang · ベストアンサー

function timeButtonInterfaceUpdate()
{
function changeDiaType(DiaDay)

｛と｝の数が一致してないので、
timeButtonInterfaceUpdateの中に、changeDiaTypeがいると、
いわれてるのではないかと。

関数式というのは、変数名 = function ...
で始まる書き方ですが、そのエラー修正方法は無視していいです。
たまたま一番近いやり方の解決方法を書いてるつもりですが。
この場合、元々のシンタックスエラーなので、気にしなくていい！

次の問題
ifの中にブーリアン値を直接書いてるところがあります。
んま省略なしって言ってるので、コピペしますが

//引数がtrueなら、リアルタイムモードのインタフェースに
if(true)
引数が・・・と書いてますが、引数ではなく、ブーリアンを直接なので、if(true)文は無視され、単純に絶対に実行されます。
絶対に実行されるので、elseに来ることもあり得ません。

次の問題
判定後、判定値そのものを代入している。
一部は省略して書きますが。
if(DiaDay == 2)
{
DiaDay = 2;
}
２なら２を代入しており、意味のないコードです。
この場合、ifの最後はreturnのみでOKです。
要するに全体で１回しか実行されたくないのですから、
値を入れる必要はないのです。
最後の１回だけ、
DiaDay = 0;
があればOKですが。。。。。。これは
ローカル変数なので、その後changeDiaTypeに
ソースを追加しないのであれば、元々いらなかった処理です。
追加する予定があったとしても一番最後のelse {}
内にいれることで、やっぱりいらない処理となるはずなので、
不要ですね。

２つ目の問題の癖で１つ目の問題が起こっていると仮定すると
最初の問題は、
//引数がtrueなら、リアルタイムモードのインタフェースに
if(true)
ではなく、
if(realtimeUpdate)
こうだったという、推測が成り立ちます（あくまで推測です）
それと同時に、以後に続く、
realtimeUpdate = true;
realtimeUpdate = false;
の削除で、OKかなと思いますが。。

OKですかね？